Skip to content

improve test output#1185

Merged
MartinNowak merged 4 commits intodlang:stablefrom
MartinNowak:improve_test_output
Jul 4, 2017
Merged

improve test output#1185
MartinNowak merged 4 commits intodlang:stablefrom
MartinNowak:improve_test_output

Conversation

@MartinNowak
Copy link
Member

  • properly catch errors and log where they happened
  • reduce verbosity a bit
  • use common error flags across scripts

- also add trap ERR handler to report exact error location
@dlang-bot
Copy link
Collaborator

Thanks for your pull request, @MartinNowak!

@CyberShadow
Copy link
Member

Thanks!!!

@MartinNowak MartinNowak force-pushed the improve_test_output branch from 83da542 to 3317ad8 Compare July 3, 2017 18:50
- those skipped the ERR trap
- add die function to exit with line number and message
@MartinNowak MartinNowak force-pushed the improve_test_output branch from c26f556 to 6e9de34 Compare July 4, 2017 14:23
@MartinNowak
Copy link
Member Author

Finally manage to get this green, lots of dub search dub failures.
Wondering whether Travis-CI (hosted on Google Cloud) has any particular connectivity issues.
Can't reproduce a single failure with with ab, curl, or dub.

@MartinNowak MartinNowak merged commit f3255f7 into dlang:stable Jul 4, 2017
@MartinNowak MartinNowak deleted the improve_test_output branch July 4, 2017 15:54
@MartinNowak
Copy link
Member Author

Major improvement btw.
dub_log_failure
Hope we didn't accidentally mute any tests ;).

Might be interesting to capture verbose dub logs to show them on failure. But atm. it's all easily traced down to failed dub-registry requests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments