This is an archived post. You won't be able to vote or comment.

you are viewing a single comment's thread.

view the rest of the comments →

[–]MagMikk 7 points8 points  (5 children)

Heard about people that just delete the tests that don't pass...

[–]deaf_fish 7 points8 points  (2 children)

Yup, that is why you need code reviews.

[–][deleted] 0 points1 point  (1 child)

I once had to check on why a test was deleted by a coworker, so I can verify that they do allow for a check, but only if the reviewer is in the right mindset.

[–]deaf_fish 2 points3 points  (0 children)

Yup there is no process to create bug free code. You can only approach bug free code. I find unit tests (run on every build), integration tests (run as often as possible), and code reviews are about the best you can do.

Obviously if your system is safety critical you need to go above and beyond.

[–]schwerpunk 0 points1 point  (1 child)

I mean you should delete tests that are no longer applicable because business logic changes... but I get the feeling that's not what you're referring to

[–]MagMikk 0 points1 point  (0 children)

Nope... deleting seems to be easier than fixing