Skip to content

Commit

Permalink
Cleaning up (#8683)
Browse files Browse the repository at this point in the history
  • Loading branch information
lucassaldanha authored Oct 8, 2024
1 parent eda8e93 commit 22421e8
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -128,13 +128,10 @@ private enum Operation {
"operations/withdrawal_request",
new OperationsTestExecutor<>(
"withdrawal_request.ssz_snappy", Operation.WITHDRAWAL_REQUEST))
// TODO re-enable consolidation tests (https://github.com/Consensys/teku/issues/8617)
.put(
"operations/consolidation_request",
// new OperationsTestExecutor<>(
// "consolidation_request.ssz_snappy",
// Operation.CONSOLIDATION_REQUEST))
TestExecutor.IGNORE_TESTS)
new OperationsTestExecutor<>(
"consolidation_request.ssz_snappy", Operation.CONSOLIDATION_REQUEST))
.build();

private final String dataFileName;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -167,9 +167,6 @@ public void processEffectiveBalanceUpdates(
final UInt64 hysteresisDownwardMultiplier = specConfig.getHysteresisDownwardMultiplier();
final UInt64 hysteresisQuotient = specConfig.getHysteresisQuotient();
final UInt64 effectiveBalanceIncrement = specConfig.getEffectiveBalanceIncrement();
// TODO-lucas Before we were using state.getValidators().size() but it can lead to an
// OutOfBoundsException when
// we have new validators being created but not added to ValidatorStatus.
for (int index = 0; index < statuses.size(); index++) {
final ValidatorStatus status = statuses.get(index);
final UInt64 balance = balances.getElement(index);
Expand Down

0 comments on commit 22421e8

Please sign in to comment.