Skip to content

fix: use correct error messages in tecdsa test asserts - #11020

Merged
nmattia merged 1 commit into
masterfrom
nm-tecdsa-assert
Aug 4, 2026
Merged

fix: use correct error messages in tecdsa test asserts#11020
nmattia merged 1 commit into
masterfrom
nm-tecdsa-assert

Conversation

@nmattia

@nmattia nmattia commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

#10995 caused the error messages returned by the IC to change slightly, and some long_tests (which do not run on PRs) started failing.

#10995 caused the error messages
returned by the IC to change slightly, and some `long_test`s (which do
not run on PRs) started failing.
@github-actions github-actions Bot added the fix label Aug 4, 2026
@nmattia

nmattia commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

The tests passed locally:

INFO: Build completed successfully, 4 total actions
//rs/tests/consensus/tecdsa:tecdsa_signature_fails_without_cycles_test (cached) PASSED in 303.9s
//rs/tests/consensus/tecdsa:tecdsa_signature_timeout_test       (cached) PASSED in 337.4s
//rs/tests/consensus/tecdsa:tecdsa_signature_life_cycle_test             PASSED in 647.5s

Executed 1 out of 3 tests: 3 tests pass.

@nmattia
nmattia marked this pull request as ready for review August 4, 2026 12:23
@nmattia
nmattia requested a review from a team as a code owner August 4, 2026 12:23
@nmattia
nmattia added this pull request to the merge queue Aug 4, 2026
Merged via the queue into master with commit 7c1ddb7 Aug 4, 2026
42 checks passed
@nmattia
nmattia deleted the nm-tecdsa-assert branch August 4, 2026 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants