Found a problem where when a date with a year less than 1000 was entered, (say 200) instead of getting 200AD, I got 200ADAD.
Looking at the code, the condition to add AD was duplicated. I think the second was meant to have been changed to handle the BC case. The specs for these cases were written incorrectly so that they always passed (didn't have an expect statement).
inukshuk
commented
6 years ago
Found a problem where when a date with a year less than 1000 was entered, (say 200) instead of getting 200AD, I got 200ADAD.
Looking at the code, the condition to add AD was duplicated. I think the second was meant to have been changed to handle the BC case. The specs for these cases were written incorrectly so that they always passed (didn't have an expect statement).