14
karma
1y

Bugfix: 1 line of code

Apply bugfix within tests: 60 lines of code

Comments
  • 6
    We had a rule to add the missing tests if you touch a piece of untested code. Holy f that escalated often in terms of line count
  • 1
    @retoor this is kinda the point of testing, but in smaller codebases it's often possible to read the existing tests and extend those to catch the bug.

    Or to move most of the non-expect code into before and after hooks such that you only need to update those.
  • 0
    I imagine this can happen when that one line does more than one thing. Sounds like it's responsible for 6 (assuming 10 lines per test per responsibility)!
Add Comment