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

EF test fails on process_sync_aggregate #115

Open
syjn99 opened this issue Feb 21, 2025 · 0 comments
Open

EF test fails on process_sync_aggregate #115

syjn99 opened this issue Feb 21, 2025 · 0 comments
Labels

Comments

@syjn99
Copy link
Member

syjn99 commented Feb 21, 2025

Reproduction

  1. Add test for process_sync_aggregate at testing/ef-tests/tests/tests.rs:
test_operation!(
    sync_aggregate,
    SyncAggregate,
    "sync_aggregate",
    process_sync_aggregate
);
  1. Run test via make test. In my case assert_eq messed up my shell, so I used make test &> test.log

Result

test tests_process_sync_aggregate::test_operation ... FAILED

...

---- tests_process_sync_aggregate::test_operation stdout ----
Testing case: random_misc_balances_and_half_participation_with_duplicates
Testing case: invalid_signature_infinite_signature_with_single_participant
Testing case: random_only_one_participant_with_duplicates
Testing case: sync_committee_rewards_duplicate_committee_max_effective_balance_only_participate_second_one
Testing case: sync_committee_with_nonparticipating_withdrawable_member
Testing case: invalid_signature_no_participants
Testing case: sync_committee_rewards_duplicate_committee_max_effective_balance_only_participate_first_one
Testing case: random_high_participation_with_duplicates
Testing case: random_with_exits_with_duplicates
thread 'tests_process_sync_aggregate::test_operation' panicked at testing/ef-tests/tests/tests.rs:101:1:
assertion `left == right` failed: Post state mismatch in case random_with_exits_with_duplicates
...

Comment

Balance mismatch at index 0: actual=30999064780, expected=30999064790
Balance mismatch at index 1: actual=31997830296, expected=31997830306
Balance mismatch at index 2: actual=31997184677, expected=31997184667
Balance mismatch at index 3: actual=31997184677, expected=31997184667
Balance mismatch at index 4: actual=31997184677, expected=31997184667
Balance mismatch at index 5: actual=31997184677, expected=31997184667
Balance mismatch at index 6: actual=31997183349, expected=31997183359
Balance mismatch at index 7: actual=31999519537, expected=31999519527
Balance mismatch at index 8: actual=31997184677, expected=31997184667
Balance mismatch at index 9: actual=31999519537, expected=31999519527
...

Hacking with debugger, I found two interesting facts:

  1. This test only fails for random_with_exits_with_duplicates. (Link for consensus-specs)
  2. It differs about 10 gwei for every balance entry.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant