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.

Sign up for GitHub

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

Jump to bottom

Fix BLS-aggregation integration test #96

Merged
merged 162 commits into from
Sep 2, 2024
Merged

Fix BLS-aggregation integration test #96

merged 162 commits into from
Sep 2, 2024

Conversation

TomasArrachea
Copy link
Collaborator

@TomasArrachea TomasArrachea commented Aug 30, 2024 •
edited
Loading

Currently, the integration tests are failing randomly, due to concurrency problems. In the tests, moved the OperatorInfoServiceInMemory::start_service to execute prior to operators registration.

supernovahs and others added 30 commits August 6, 2024 17:40
cids_multiplication
e12d3c6
wip
16e6439
latest
da6914e
bls crate working
9c50a07
bls crate and readme
0ab90dd
delete post.key file
88fb0c2
@supernovahs
Merge branch 'main' into supernovahs/bls
bfe95ca
nits , delete keystore crate which is replaced by new bls keystore re…
f579335 
…po in rust bls bn254
change to https for rust bls bn254 dependency , and some clippy
defdaf3
@supernovahs
Merge branch 'main' into supernovahs/bls
6781f1f
merge nits
52d6c85
fix : mapto curve function
df90ecf
complete sign message, verify message
902b14e
move verify message and map to curve to bn254, nits
30e22cc
@supernovahs
Merge branch 'main' into supernovahs/bls
2a982b6
update rust-bls-bn254 rev
7693f67
@TomasArrachea
add failing bls aggregator and chaincaller
1441538
@TomasArrachea
fix chaincaller pubkey addition
524d047
@TomasArrachea
refactor chaincaller and add bls utils
425b168
@TomasArrachea
WIP bls aggregation
924f983
@TomasArrachea
add affine point parse
417b142
@pablodeymo
wip fixes
804370c
@TomasArrachea
remove unused imports
750eb3c
@TomasArrachea
remove unused crate
cda4770
@TomasArrachea
change aggregation quorum into bls point
b188ace
@TomasArrachea
refactor verify signature
5ee156c
@TomasArrachea
refactor check stake threshold
dba5dc8
@TomasArrachea
format
e8cd3b8
@ricomateo
add TestOperator struct
98bcc5e
@ricomateo
add FakeAvsRegistryService struct
98e6dcf
ricomateo and others added 26 commits August 23, 2024 13:00
@ricomateo
add error handling
674e2d0
@TomasArrachea
add sleep to fix operators registration
f251924
@TomasArrachea
add test for 2 quorums 1 operator
783d8e8
@TomasArrachea
add test for 2 quorums 2 operators
618a9d5
@TomasArrachea
update anvil initialization scripts
e3f2283
@TomasArrachea
update logger msg
e1abc89
@TomasArrachea
ignore failing tests
df8c897
@TomasArrachea
remove comments and clean code
a233101
@TomasArrachea
Merge branch 'bls_aggregation' into bls_aggregation_integration_test
b2743d3
@TomasArrachea
fix merge
eb5c8ac
@TomasArrachea
remove anvil output when running tests
3865e03
@TomasArrachea
fix make pr
ab73c38
@TomasArrachea
remove prints
702276d
@TomasArrachea
remove extra test
8652e4b
@pablodeymo
improve error handling
7310e6c
@TomasArrachea
Merge branch 'main' into bls_aggregation_integration_test
b426902
@TomasArrachea
remove duplicate definitions
eae23be
@TomasArrachea
update make fireblocks-tests
1920cb4
@TomasArrachea
update comments
92146bc
@TomasArrachea
Merge branch 'main' into bls_aggregation_integration_test
615fad2
@TomasArrachea
Merge branch 'main' into bls_aggregation_integration_test
4b707d6
@TomasArrachea
Merge branch 'main' into bls_aggregation_integration_test
3d33c07
@TomasArrachea
update operator id in integration test
1ab3119
@TomasArrachea
Merge branch 'main' into bls_aggregation_integration_test
dcf06b4
@TomasArrachea
fix integration tests moving OperatorInfoServiceInMemory start
ec74703
@TomasArrachea
clippy
acaf0b5
@TomasArrachea TomasArrachea marked this pull request as ready for review August 30, 2024 18:53
@supernovahs supernovahs merged commit b906fe4 into Layr-Labs:main Sep 2, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dralves dralves dralves approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@TomasArrachea @dralves @supernovahs @pablodeymo @ricomateo