reverting changed loop details in retry

ChainSafe · May 27, 2022 · 1b34c47 · 1b34c47
1 parent 2252766
commit 1b34c47
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/packages/lodestar/src/util/retry.ts b/packages/lodestar/src/util/retry.ts
@@ -28,7 +28,7 @@ export async function retry<A>(fn: (attempt: number) => A | Promise<A>, opts?: I
   const shouldRetry = opts?.shouldRetry;
 
   let lastError: Error = Error("RetryError");
-  for (let i = 0; i < maxRetries; i++) {
+  for (let i = 1; i <= maxRetries; i++) {
     try {
       return await fn(i);
     } catch (e) {

diff --git a/packages/lodestar/test/unit/executionEngine/httpRetry.test.ts b/packages/lodestar/test/unit/executionEngine/httpRetry.test.ts
@@ -3,8 +3,7 @@ import chaiAsPromised from "chai-as-promised";
 import {fastify} from "fastify";
 import {AbortController} from "@chainsafe/abort-controller";
 import {fromHexString} from "@chainsafe/ssz";
-import {ExecutionEngineHttp} from "../../../src/executionEngine/http.js";
-import {defaultExecutionEngineHttpOpts} from "../../../lib/executionEngine/http.js";
+import {ExecutionEngineHttp, defaultExecutionEngineHttpOpts} from "../../../src/executionEngine/http.js";
 import {bytesToData, numToQuantity} from "../../../src/eth1/provider/utils.js";
 
 chai.use(chaiAsPromised);