• sunbeam60
      link
      fedilink
      arrow-up
      3
      ·
      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
        17
        ·
        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
          link
          fedilink
          arrow-up
          2
          ·
          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.