fix: make electra-fork passes lint and check-types (ChainSafe#6785)

fix lint and check-types
g11tech · Jun 19, 2024 · 0668076 · 0668076
1 parent 8d18fbd
commit 0668076
Show file tree

Hide file tree

Showing 7 changed files with 13 additions and 13 deletions.
diff --git a/packages/beacon-node/src/chain/validation/signatureSets/aggregateAndProof.ts b/packages/beacon-node/src/chain/validation/signatureSets/aggregateAndProof.ts
@@ -1,7 +1,7 @@
 import type {PublicKey} from "@chainsafe/bls/types";
 import {DOMAIN_AGGREGATE_AND_PROOF, ForkSeq} from "@lodestar/params";
 import {allForks, ssz} from "@lodestar/types";
-import {Epoch, phase0} from "@lodestar/types";
+import {Epoch} from "@lodestar/types";
 import {
   computeSigningRoot,
   computeStartSlotAtEpoch,

diff --git a/packages/beacon-node/test/sim/electra-interop.test.ts b/packages/beacon-node/test/sim/electra-interop.test.ts
@@ -219,11 +219,11 @@ describe("executionEngine / ExecutionEngineHttp", function () {
       }
     }
 
-    if (payload.depositRequests.length !== 1) {
-      throw Error(`Number of depositRequests mismatched. Expected: 1, actual: ${payload.depositRequests.length}`);
+    if (payload.depositReceipts.length !== 1) {
+      throw Error(`Number of depositRequests mismatched. Expected: 1, actual: ${payload.depositReceipts.length}`);
     }
 
-    const actualDepositRequest = payload.depositRequests[0];
+    const actualDepositRequest = payload.depositReceipts[0];
     assert.deepStrictEqual(
       actualDepositRequest,
       depositRequestB,

diff --git a/packages/fork-choice/test/perf/forkChoice/util.ts b/packages/fork-choice/test/perf/forkChoice/util.ts
@@ -41,7 +41,7 @@ export function initializeForkChoice(opts: Opts): ForkChoice {
     genesisSlot
   );
 
-  const balances = new Uint8Array(Array.from({length: opts.initialValidatorCount}, () => 32));
+  const balances = new Uint16Array(Array.from({length: opts.initialValidatorCount}, () => 32));
 
   const fcStore: IForkChoiceStore = {
     currentSlot: genesisSlot,

diff --git a/packages/fork-choice/test/unit/forkChoice/forkChoice.test.ts b/packages/fork-choice/test/unit/forkChoice/forkChoice.test.ts
@@ -52,16 +52,16 @@ describe("Forkchoice", function () {
     currentSlot: genesisSlot + 1,
     justified: {
       checkpoint: {epoch: genesisEpoch, root: fromHexString(finalizedRoot), rootHex: finalizedRoot},
-      balances: new Uint8Array([32]),
+      balances: new Uint16Array([32]),
       totalBalance: 32,
     },
     unrealizedJustified: {
       checkpoint: {epoch: genesisEpoch, root: fromHexString(finalizedRoot), rootHex: finalizedRoot},
-      balances: new Uint8Array([32]),
+      balances: new Uint16Array([32]),
     },
     finalizedCheckpoint: {epoch: genesisEpoch, root: fromHexString(finalizedRoot), rootHex: finalizedRoot},
     unrealizedFinalizedCheckpoint: {epoch: genesisEpoch, root: fromHexString(finalizedRoot), rootHex: finalizedRoot},
-    justifiedBalancesGetter: () => new Uint8Array([32]),
+    justifiedBalancesGetter: () => new Uint16Array([32]),
     equivocatingIndices: new Set(),
   };
 

diff --git a/packages/fork-choice/test/unit/forkChoice/getProposerHead.test.ts b/packages/fork-choice/test/unit/forkChoice/getProposerHead.test.ts
@@ -102,12 +102,12 @@ describe("Forkchoice / GetProposerHead", function () {
     currentSlot: genesisSlot + 1,
     justified: {
       checkpoint: {epoch: genesisEpoch, root: fromHexString(genesisBlock.blockRoot), rootHex: genesisBlock.blockRoot},
-      balances: new Uint8Array(Array(32).fill(150)),
+      balances: new Uint16Array(Array(32).fill(150)),
       totalBalance: 32 * 150,
     },
     unrealizedJustified: {
       checkpoint: {epoch: genesisEpoch, root: fromHexString(genesisBlock.blockRoot), rootHex: genesisBlock.blockRoot},
-      balances: new Uint8Array(Array(32).fill(150)),
+      balances: new Uint16Array(Array(32).fill(150)),
     },
     finalizedCheckpoint: {
       epoch: genesisEpoch,
@@ -119,7 +119,7 @@ describe("Forkchoice / GetProposerHead", function () {
       root: fromHexString(genesisBlock.blockRoot),
       rootHex: genesisBlock.blockRoot,
     },
-    justifiedBalancesGetter: () => new Uint8Array(Array(32).fill(150)),
+    justifiedBalancesGetter: () => new Uint16Array(Array(32).fill(150)),
     equivocatingIndices: new Set(),
   };
 

diff --git a/packages/fork-choice/test/unit/protoArray/computeDeltas.test.ts b/packages/fork-choice/test/unit/protoArray/computeDeltas.test.ts
@@ -253,7 +253,7 @@ describe("computeDeltas", () => {
       nextEpoch: 0,
     }));
 
-    const balances = new Uint8Array([firstBalance, secondBalance]);
+    const balances = new Uint16Array([firstBalance, secondBalance]);
     // 1st validator is part of an attester slashing
     const equivocatingIndices = new Set([0]);
     let deltas = computeDeltas(indices.size, votes, balances, balances, equivocatingIndices);

diff --git a/packages/state-transition/src/epoch/processRewardsAndPenalties.ts b/packages/state-transition/src/epoch/processRewardsAndPenalties.ts
@@ -28,7 +28,7 @@ export function processRewardsAndPenalties(
   const balances = state.balances.getAll();
 
   for (let i = 0, len = rewards.length; i < len; i++) {
-    const result = balances[i] + rewards[i] - penalties[i] - (slashingPenalties[i] ?? 0)
+    const result = balances[i] + rewards[i] - penalties[i] - (slashingPenalties[i] ?? 0);
     balances[i] = Math.max(result, 0);
   }