4 ms·
> "must not decrease the overall percentage of tested code" This rule has been a problem for me when deleting code. On my precious check we had an automated ch
by Fargren 3y ago
> "must not decrease the overall percentage of tested code"
This rule has been a problem for me when deleting code. On my precious check we had an automated checker that wouldn't let us merge (easily) if we broke this rule. We had mediocre coverage. Some parts of the code had lots of tests, some parts were totally uncovered. Many times I had to delete code in the tested parts, and the checker would complain that total coverage went down because I deleted a tested line without adding more tested lines.
But I agree it's a good rule in general, which is why we kept it despite the occasional hiccup.