Skip to content

mixing: Increment PRNG seed nonce for each test#3618

Merged
davecgh merged 1 commit intodecred:masterfrom
jrick:seednonce
Feb 18, 2026
Merged

mixing: Increment PRNG seed nonce for each test#3618
davecgh merged 1 commit intodecred:masterfrom
jrick:seednonce

Conversation

@jrick
Copy link
Copy Markdown
Member

@jrick jrick commented Feb 18, 2026

This avoids reusing the same initial seed for multiple tests when testing with -count greater than 1.

The -seed flag now requires providing the initial nonce in the string, so that the failing test can be reproduced with -count=1. If a test fails, the flag to reproduce is logged at the end of the output.

This avoids reusing the same initial seed for multiple tests when testing with -count greater than 1.

The -seed flag now requires providing the initial nonce in the string, so that
the failing test can be reproduced with -count=1.  If a test fails, the flag
to reproduce is logged at the end of the output.
Copy link
Copy Markdown
Member

@davecgh davecgh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made it optional in the code I'm working on which does similar based on our conversations about this, but no objections to it being required in these tests.

@davecgh davecgh added this to the 2.2.0 milestone Feb 18, 2026
@davecgh davecgh merged commit 9516143 into decred:master Feb 18, 2026
34 checks passed
@jrick jrick deleted the seednonce branch February 19, 2026 14:52
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.

2 participants