Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: enhanced end-to-end tests #10

Merged
merged 20 commits into from
Oct 17, 2024
Merged
Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
7be28ac
feat: refactored happy case test, splitting into several reusable fun…
NikitaMasych Sep 9, 2024
1448ec3
fix: threshould requirement shall allow equality
NikitaMasych Sep 9, 2024
c17fd34
feat: added faulty party test
NikitaMasych Sep 9, 2024
c1a689e
fix: resolve linter issue with complex return type
NikitaMasych Sep 9, 2024
9807e83
fix: include own weight in threshold checks in update_state
NikitaMasych Sep 10, 2024
dd3e01c
chore: improve update state warning source with comments
NikitaMasych Sep 10, 2024
8c3e5b0
feat: added rate limiting and malicious party test
NikitaMasych Sep 10, 2024
e9be6d6
feat: refactored tests - moved ballot tests to integration, created s…
NikitaMasych Sep 10, 2024
58891cd
feat: refactored integration tests, better results analysis
NikitaMasych Sep 10, 2024
736e409
feat: added faulty leader test for clarity
NikitaMasych Sep 10, 2024
ad1f6fa
chore: update cargo.toml and readme
NikitaMasych Sep 10, 2024
67fd502
chore: fixed clippy check with --all-features
NikitaMasych Sep 10, 2024
2d36dee
feat: added check for own message receival, warnings
NikitaMasych Sep 10, 2024
fd0f116
feat: improve config stages timeouts; feature-enabling for integratio…
NikitaMasych Sep 10, 2024
9fca324
feat: added integration test for large amount of parties, threshold c…
NikitaMasych Sep 11, 2024
35808da
chore: flaky tests in rust ci
NikitaMasych Sep 11, 2024
84d1d5c
fix: threshold ceiling division
NikitaMasych Sep 11, 2024
3d99cc0
chore: testing with nextest in rust workflow
NikitaMasych Sep 11, 2024
b421020
fix: changed scheduling to instant
NikitaMasych Sep 11, 2024
4d70977
chore: added security considerations section to readme
NikitaMasych Sep 11, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
chore: improve update state warning source with comments
NikitaMasych committed Sep 10, 2024

Verified

This commit was signed with the committer’s verified signature. The key has expired.
phi-gamma Philipp Gesang
commit dd3e01c6028b179227b7e9378814ad5740d5a11d
4 changes: 3 additions & 1 deletion src/party.rs
Original file line number Diff line number Diff line change
@@ -327,7 +327,9 @@ impl<V: Value, VS: ValueSelector<V>> Party<V, VS> {
debug!("Party {} received {} from party {}", self.id, msg.routing.msg_type, msg.routing.sender);
if let Err(err) = self.update_state(&msg) {
// Shouldn't fail the party, since invalid message
// may be sent by anyone.
// may be sent by anyone. Furthermore, since in consensus
// we are relying on redundancy of parties, we actually may need
// less messages than from every party to transit to next status.
warn!("Failed to update state with {}, got error: {err}", msg.routing.msg_type)
}
}else if self.msg_in_receiver.is_closed(){