Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rearrange test/recipes/95-test_*.t to use skip_all #3398

Closed

Conversation

levitte
Copy link
Member

@levitte levitte commented May 5, 2017

The conditions to skip these recipes entirely don't show in a
non-verbose test harness output. We prefer to know, so use skip_all,
as it is a little bit more verbose.

[extended tests]

Checklist
  • tests are added or updated

The conditions to skip these recipes entirely don't show in a
non-verbose test harness output.  We prefer to know, so use skip_all,
as it is a little bit more verbose.

[extended tests]
@levitte levitte added the branch: master Merge to master branch label May 5, 2017
@levitte
Copy link
Member Author

levitte commented May 6, 2017

Merged. b83ace3

@levitte levitte closed this May 6, 2017
levitte added a commit that referenced this pull request May 6, 2017
The conditions to skip these recipes entirely don't show in a
non-verbose test harness output.  We prefer to know, so use skip_all,
as it is a little bit more verbose.

[extended tests]

Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #3398)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch: master Merge to master branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants