Skip to content

test: untimely proposals must be decided after multiple rounds#2447

Merged
cason merged 7 commits intoglnro/adaptive-synchfrom
cason/2432-adaptive-test-fixed
Feb 27, 2024
Merged

test: untimely proposals must be decided after multiple rounds#2447
cason merged 7 commits intoglnro/adaptive-synchfrom
cason/2432-adaptive-test-fixed

Conversation

@cason
Copy link

@cason cason commented Feb 26, 2024

Adds a test unit to attest the proper operation, from consensus' point of view, of the changes introduced by #2431.

Addresses #2432.


PR checklist

  • Tests written/updated
  • Changelog entry added in .changelog (we use unclog to manage our changelog)
  • Updated relevant documentation (docs/ or spec/) and code comments
  • Title follows the Conventional Commits spec

@cason cason changed the base branch from feature/pbts to glnro/adaptive-synch February 26, 2024 16:02
@cason cason linked an issue Feb 26, 2024 that may be closed by this pull request
4 tasks
@cason cason marked this pull request as ready for review February 26, 2024 16:05
@cason cason requested a review from a team as a code owner February 26, 2024 16:05
@cason cason requested a review from a team February 26, 2024 16:05
@cason cason added pbts consensus testing related to unit testing in general labels Feb 26, 2024
Co-authored-by: lasaro <lasaro@informal.systems>
@cason cason merged commit 66bdb96 into glnro/adaptive-synch Feb 27, 2024
@cason cason deleted the cason/2432-adaptive-test-fixed branch February 27, 2024 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

consensus pbts testing related to unit testing in general

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

PBTS: Implement adaptive synchronous parameters for consensus

2 participants