ibrahima
commented
2 years ago Hmm, that's an interesting idea, though it's kind of subjective in my opinion. What if instead of having it hardcoded, we make it configurable by passing in an argument to the JSONTestRunner class, say failure_prefix="Test Failed:", and then anyone can override it to whatever they want? Could you implement it that way? Thanks!
isaacl
Test Failed, because it is two words, is harder to parse from the failure message itself. I think it is better to for the first word to be as descriptive as possible, so students can easily scan for the word "Fail" on the far left side.
Maybe it's subjective! Make it configurable.