[trivial] tester.nim add more diagnostics #8631
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
getCurrentExceptionMsg()
was added in this block:to provide useful diagnostics when
IOError
occur; otherwise looking at logs is not helpful:example from a failed travis job: https://travis-ci.org/nim-lang/Nim/jobs/416117242
with this PR we get the useful line "cannot open: nimcache/tests/pragmas/tnoreturn.nim_0d61f8370cad1d412f80b84d143e1257/.tests.pragmas.tnoreturn.c" :
without this PR, all we get is the enigmatic "Failure: reCodeNotFound"
" failed: ", failed
was added in test result output, because it's easier for readers to see how many test failed without having to do some math