👍 as is

Some thoughts though:

• It is confusing that we have both meets_testing_requirements and met_testing_requirements and that they don't share code. I think met does the same things as meets except it tests for a comment also. Perhaps met could be refactored to call meets first, and then check for the comment?
• It would be nice to add a test that has an update the meets but hasn't yet met. Assert those things are true and false respectively. Add the comment. Then finally assert that both are true afterwards.

Okay, I think I addressed both of your concerns?

👍

Thanks :)