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

REST: Add implementation of postStateValidators and postStateValidatorBalances. #5632

Merged
merged 7 commits into from
Nov 29, 2023

Conversation

cheatfate
Copy link
Contributor

This PR implements ethereum/beacon-APIs#367 and tests.

…validators and /eth/v1/beacon/states/{state_id}/validator_balances.

Add tests.
Copy link

github-actions bot commented Nov 28, 2023

Unit Test Results

         9 files  ±0    1 098 suites  ±0   27m 19s ⏱️ -5s
  3 953 tests +1    3 606 ✔️ +1  347 💤 ±0  0 ±0 
16 072 runs  +3  15 674 ✔️ +3  398 💤 ±0  0 ±0 

Results for commit efc9202. ± Comparison against base commit ab5343d.

♻️ This comment has been updated with latest results.

for item in statuses.get():
let value = decodeString(ValidatorFilter, item).valueOr:
reader.raiseUnexpectedValue($error)
if value * res != {}:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They are not identical but they are not unique, according to specification values should be unique, so there should be no intersection.
https://github.com/ethereum/beacon-APIs/pull/367/files#diff-cc9071f40d45ee8fad4e7513c3351001c1445d206d6726c65a0000421110b1c3R122

Fix unique check for validator identifiers.
@tersec tersec merged commit 26bcb70 into unstable Nov 29, 2023
11 checks passed
@tersec tersec deleted the post-validators branch November 29, 2023 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants