| ▲ | em-bee 7 hours ago | |
of course it is just an easy fix. it's the kind of solution that even someone like me could write who has no understanding of the code a all. (i am not trying to imply that the submitter of the PR doesn't understand the code, just that understanding it is unlikely to be necessary, thus the change bears no risk. but, the solution now hides the problem. if i wanted to get someone to solve the problem i'd set the new date in the near future until someone gets annoyed enough to fix it for real. and i have to ask, why is this a hardcoded date at all? why not "now plus one week"? | ||
| ▲ | db48x 4 hours ago | parent [-] | |
There’s a lot to be said for simplicity. The more logic you put into handling the dates correctly in the tests, the more likely you are to mess up the tests themselves. These tests were easy to write, easy to review, easy to verify, and served perfectly well for 10 years. But doing it right shouldn’t be all that hard. | ||