Skip to content

Test: Refactor block waits#5784

Merged
kantai merged 8 commits intodevelopfrom
test/refactor-block-waits
Jan 31, 2025
Merged

Test: Refactor block waits#5784
kantai merged 8 commits intodevelopfrom
test/refactor-block-waits

Conversation

@kantai
Copy link
Copy Markdown
Contributor

@kantai kantai commented Jan 31, 2025

Description

Refactor the block waits in the nakamoto_integrations and signer tests. This should reduce a little bit of flakiness across the board.

@kantai kantai requested review from a team January 31, 2025 15:20
@kantai kantai requested a review from a team as a code owner January 31, 2025 15:20
@kantai kantai requested a review from a team January 31, 2025 15:20
@kantai kantai changed the base branch from master to develop January 31, 2025 15:20
hstove
hstove previously approved these changes Jan 31, 2025
Copy link
Copy Markdown
Contributor

@hstove hstove left a comment

Choose a reason for hiding this comment

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

Nice! This is a much more readable version, too.

…test retry sleep from 100ms to 500ms (less log spam in tests)
@kantai
Copy link
Copy Markdown
Contributor Author

kantai commented Jan 31, 2025

Just look at that glorious green check mark without any re-runs.

@kantai kantai requested review from hstove and jbencin January 31, 2025 20:48
Copy link
Copy Markdown
Contributor

@obycode obycode left a comment

Choose a reason for hiding this comment

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

👏

@kantai kantai enabled auto-merge January 31, 2025 22:10
@kantai kantai added this pull request to the merge queue Jan 31, 2025
@blockstack-devops
Copy link
Copy Markdown
Contributor

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@stacks-network stacks-network locked as resolved and limited conversation to collaborators Feb 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Flaky test: tests::signer::v0::multiple_miners_empty_sortition Flaky test: tests::nakamoto_integrations::test_tenure_extend_from_flashblocks

5 participants