-
-
Notifications
You must be signed in to change notification settings - Fork 331
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
chore: add unit test for getAttestationsForBlockElectra
#7464
Conversation
…ol.ts Co-authored-by: Nico Flaig <[email protected]>
43a9d66
to
89474c3
Compare
|
||
const blockAttestations = pool.getAttestationsForBlock(fork, forkchoiceStub, electraState); | ||
|
||
expect(blockAttestations.length).toBe(1); // Expect attestations from committee 0, 1, 2 and 3 to be aggregated into one |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## unstable #7464 +/- ##
============================================
- Coverage 50.48% 50.46% -0.02%
============================================
Files 602 602
Lines 40606 40606
Branches 2230 2225 -5
============================================
- Hits 20499 20493 -6
- Misses 20067 20073 +6
Partials 40 40 |
Performance Report✔️ no performance regression detected Full benchmark results
|
getAttestationsForBlockElectra
getAttestationsForBlockElectra
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Follow up on #7455 (comment) to add unit test of the changes introduced in #7455.