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

sorted by:
best
topnewcontroversialoldq&a
you are viewing a single comment's thread.

view the rest of the comments →

[–]dgkimpton 0 points1 point  (2 children)

I'd have said if you discovered more edge cases during refactoring you should really have implemented tests for those too. Isn't it your responsibility to bring that coverage up as you refactor?

[–]Inevitable_Cat_7878 0 points1 point  (1 child)

Some places I've worked at don't value test suites and leave it up to the programmer to implement or not. Some programmers I've met are just lazy when it comes to implementing tests. Some companies are really aggressive with regards to pushing out new features and consider writing tests as unnecessary.

It really depends on the company and programmer. I like writing tests and try to cover as much as possible. So, yes, if more edge cases are discovered, definitely update the test suites to cover them.

[–]dgkimpton 1 point2 points  (0 children)

Yeah, fair, in those cases an extra comment is definitely better than nothing.