fix: add retry stratefy on fetch

FuelLabs · luizstacio · Feb 14, 2023 · Feb 8, 2023 · Feb 10, 2023 · Feb 10, 2023
commit 9ff57619b7483c47b35acd0b4cda33e0c415b52d
@@ -3,4 +3,4 @@
 "@fuel-ts/wallet": minor
 ---
 
-BRAKE CHANGE, update support to fuel-core v0.17.1
+BREAKING CHANGE, update support to fuel-core v0.17.1
diff --git a/.changeset/lemon-scissors-live.md b/.changeset/lemon-scissors-live.md
@@ -0,0 +1,2 @@
+---
+---
@@ -1,5 +1,5 @@
 services-run:
-	docker compose up -d
+	docker compose up -d --build
 
 services-clean:
 	docker ps --filter name=fuels-ts* -aq | xargs docker stop | xargs docker rm -f

diff --git a/jest.config.ts b/jest.config.ts
@@ -3,6 +3,7 @@ import type { Config } from '@jest/types';
 const config: Config.InitialOptions = {
   preset: 'ts-jest',
   testEnvironment: 'node',
+  bail: true,
   setupFiles: ['./jest.env.ts'],
   testPathIgnorePatterns: ['/node_modules/', '/dist/'],
   modulePathIgnorePatterns: ['/dist/'],

diff --git a/package.json b/package.json
@@ -13,7 +13,7 @@
     "build:watch": "turbo run build --parallel -- --watch",
     "ci:test": "./scripts/ci-test.sh",
     "test": "jest --no-cache --runInBand",
-    "test:watch": "jest --watchAll",
+    "test:watch": "jest --no-cache --watchAll",
     "lint": "run-s lint:check prettier:check",
     "lint:check": "eslint . --ext .ts",
     "lint:fix": "pnpm lint:check -- --fix",

diff --git a/packages/fuel-gauge/src/contract.test.ts b/packages/fuel-gauge/src/contract.test.ts
@@ -382,7 +382,7 @@ describe('Contract', () => {
     expect(JSON.stringify(value)).toEqual(JSON.stringify([bn(100), bn(200)]));
   });
 
-  it.only('Get transaction cost with gasPrice 1', async () => {
+  it('Get transaction cost with gasPrice 1', async () => {
     const contract = await setupContract();
     const invocationScope = contract
       .multiCall([

@@ -314,16 +314,11 @@ export default class Provider {
   ): Promise<CallResult> {
     const transactionRequest = transactionRequestify(transactionRequestLike);
     await this.addMissingVariables(transactionRequest);
-
-    const chainInfo = await this.operations.getChain();
-    console.log('Debug values on CI', JSON.stringify(chainInfo, null, 2));
-
     const encodedTransaction = hexlify(transactionRequest.toTransactionBytes());
     const { dryRun: gqlReceipts } = await this.operations.dryRun({
       encodedTransaction,
       utxoValidation: utxoValidation || false,
     });
-    console.log('Debug values on CI', JSON.stringify(gqlReceipts, null, 2));
     const receipts = gqlReceipts.map(processGqlReceipt);
     return {
       receipts,

@@ -112,10 +112,23 @@ export class TransactionResponse {
       await this.provider.operations.getTransactionWithReceipts({
         transactionId: this.id,
       });
+
+    // If transactions are not found retry until they are available
+    // TODO: Implement subscriptions to avoid polling and improve performance
     if (!transactionWithReceipts) {
-      throw new Error('No Transaction was received from the client.');
+      if (this.attempts > 10) {
+        throw new Error('No Transaction was received from the client.');
+      }
+      this.attempts += 1;
+      await sleep(
+        Math.min(STATUS_POLLING_INTERVAL_MIN_MS * this.attempts, STATUS_POLLING_INTERVAL_MAX_MS)
+      );
+      return this.fetch<TTransactionType>();
     }
 
+    // Clean attempts
+    this.attempts = 0;
+
     const transaction = new TransactionCoder().decode(
       arrayify(transactionWithReceipts.rawPayload),
       0

diff --git a/scripts/ci-test.sh b/scripts/ci-test.sh
@@ -6,10 +6,10 @@ echo $@
 
 # Run test
 if [[ "$*" == *"--coverage"* ]]; then
-    pnpm test /fuel-gauge/src/contract.test.ts
+    pnpm test $@
     TEST_RESULT=$?
 else
-    pnpm test /fuel-gauge/src/contract.test.ts
+    pnpm test
     TEST_RESULT=$?
 fi