Improve error messages in space-age exercise #68
Conversation
|
This PR touches files which potentially affect the outcome of the tests of an exercise. This will cause all students' solutions to affected exercises to be re-tested. If this PR does not affect the result of the test (or, for example, adds an edge case that is not worth rerunning all tests for), please add the following to the merge-commit message which will stops student's tests from re-running. Please copy-paste to avoid typos. For more information, refer to the documentation. If you are unsure whether to add the message or not, please ping |
pfertyk
force-pushed
the
space-age-improved-error-message
branch
4 times, most recently
from
a655858 to
5e8c37a
Compare
[no important files changed]
pfertyk
force-pushed
the
space-age-improved-error-message
branch
from
5e8c37a to
556438d
Compare
|
@BNAndras I think this will help with obscure error messages in space-age exercise ;) Could you please take a look? |
There was a problem hiding this comment.
Looks good. Do we need to round the expected values though? I think we can just round the calculated age. That should make it a bit clearer what's going on hopefully.
There was a problem hiding this comment.
The last example was raising an error, saying that the expected value was 0.35, and the actual one 0.35 ... 🤔 So, just to be sure, I decided to round all the values 😛 That took care of the error 😉
Fixes #67 .