• sunbeam60@lemmy.one
    link
    fedilink
    arrow-up
    3
    arrow-down
    12
    ·
    11 months ago

    Yeah but even a single automated test would catch it and reject the PR. You just need a single test.

    • frezik@midwest.social
      link
      fedilink
      arrow-up
      16
      ·
      11 months ago

      No, you can’t assume that. The probability of hitting the condition each time is low. If there aren’t very many calls that hit this, it could easily slip through. Especially on 64-bit int platforms.

      • sunbeam60@lemmy.one
        link
        fedilink
        arrow-up
        2
        arrow-down
        1
        ·
        11 months ago

        Yes agree if you’re talking about unit tests. I’m thinking smoke tests, which is are the most common automated tests in games, where I’ve spent most of professional career. The amount of booleans checks that happen in a single frame I doubt the game wouldn’t crash within the first couple seconds.