• FizzyOrange@programming.dev
    link
    fedilink
    arrow-up
    8
    arrow-down
    1
    ·
    1 day ago

    The article said it pretty well:

    if your answer to any perceived failing in a person is “just try harder”, you are either woefully inexperienced or a just a dick

    That applies to writing impossibly comprehensive unit tests too.

    Though really for a filesystem they should really do silicon-style verification (which we’re calling Deterministic System Testing now).

    • kcuf2@lemmynsfw.com
      link
      fedilink
      arrow-up
      2
      arrow-down
      2
      ·
      1 day ago

      This is a pretty straightforward utility function that wouldn’t be that hard to test. It’s normal to have standards for code coverage as part of the review process, I don’t code in C but I’d be surprised if setting any of that up is actually that burdensome.

      I disagree with your reference of that quote being applicable here though. In fact, adding unit tests is the exact opposite: that quote is saying ‘if your answer is “make less mistakes” …’ Unit tests in general are an acceptance of the fact that we will fail to be perfect and we need to mitigate that with extra checks. The article already said it had two human reviewers, so they’re not opposed to extra process to help ensure quality, unit tests are just another (actually very cheap) extra step.