Skip to content

Comments

test: print OPENSSL_TEST_RAND_ORDER=x when a randomised test fails.#13672

Closed
paulidale wants to merge 2 commits intoopenssl:masterfrom
paulidale:test-rand-order
Closed

test: print OPENSSL_TEST_RAND_ORDER=x when a randomised test fails.#13672
paulidale wants to merge 2 commits intoopenssl:masterfrom
paulidale:test-rand-order

Conversation

@paulidale
Copy link
Contributor

@paulidale paulidale commented Dec 12, 2020

The previous message "random seed x" is a lot less descriptive.

  • documentation is added or updated
  • tests are added or updated

The previous message "random seed x" is a lot less descriptive.
@paulidale paulidale added the branch: master Applies to master branch label Dec 12, 2020
@paulidale paulidale self-assigned this Dec 12, 2020
@paulidale paulidale added the approval: review pending This pull request needs review by a committer label Dec 13, 2020
@t8m t8m added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Dec 14, 2020
@openssl-machine openssl-machine removed the approval: done This pull request has the required number of approvals label Dec 15, 2020
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

@openssl-machine openssl-machine added the approval: ready to merge The 24 hour grace period has passed, ready to merge label Dec 15, 2020
@paulidale
Copy link
Contributor Author

Merged to master. Thanks for the review.

@paulidale paulidale closed this Dec 15, 2020
openssl-machine pushed a commit that referenced this pull request Dec 15, 2020
The previous message "random seed x" is a lot less descriptive.

Reviewed-by: Tomas Mraz <[email protected]>
(Merged from #13672)
openssl-machine pushed a commit that referenced this pull request Dec 15, 2020
@paulidale paulidale deleted the test-rand-order branch December 15, 2020 10:04
@mspncp
Copy link
Contributor

mspncp commented Dec 17, 2020

Good idea to make it copy&pasteable.

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

Labels

approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Applies to master branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants