fix: decode result data error handling (#304)

# What ❔

Fix decode result error handling.

## Why ❔

Currently this error case is not properly handled and the retryable
contract retries requests even though this error is not retryable.

## Checklist

<!-- Check your PR fulfills the following items. -->
<!-- For draft PRs check the boxes as you complete them. -->

- [X] PR title corresponds to the body of PR (we generate changelog
entries from PRs).
- [X] Tests for the changes have been added / updated.

packages/data-fetcher/src/blockchain/retryableContract.spec.ts

@@ -98,7 +98,7 @@ describe("RetryableContract", () => {
       expect(result).toBe(functionResult);
     });
 
-    describe("when throws a permanent call exception function error", () => {
+    describe("when throws a permanent execution reverted error", () => {
       const callExceptionError = {
         code: 3,
         shortMessage: "execution reverted...",
@@ -125,6 +125,33 @@ describe("RetryableContract", () => {
       });
     });
 
+    describe("when throws a permanent could not decode result data error", () => {
+      const callExceptionError = {
+        code: "BAD_DATA",
+        shortMessage: "could not decode result data...",
+      };
+
+      beforeEach(() => {
+        (ethers.Contract as any as jest.Mock).mockReturnValue({
+          contractFn: async () => {
+            throw callExceptionError;
+          },
+        });
+
+        contract = new RetryableContract(tokenAddress, utils.IERC20, providerMock);
+      });
+
+      it("throws an error", async () => {
+        expect.assertions(1);
+
+        try {
+          await contract.contractFn();
+        } catch (e) {
+          expect(e).toBe(callExceptionError);
+        }
+      });
+    });
+
     describe("when throws an invalid argument function error", () => {
       const invalidArgumentError = {
         code: "INVALID_ARGUMENT",

packages/data-fetcher/src/blockchain/retryableContract.ts

@@ -21,7 +21,13 @@ const PERMANENT_ERRORS: ErrorCode[] = [
 
 const shouldRetry = (error: EthersError): boolean => {
   const isPermanentErrorCode = PERMANENT_ERRORS.find((errorCode) => isError(error, errorCode));
-  return !isPermanentErrorCode && !(error.code === 3 && error.shortMessage?.startsWith("execution reverted"));
+  return (
+    !isPermanentErrorCode &&
+    // example block mainnet 47752810
+    !(error.code === 3 && error.shortMessage?.startsWith("execution reverted")) &&
+    // example block mainnet 47819836
+    !(error.code === "BAD_DATA" && error.shortMessage?.startsWith("could not decode result data"))
+  );
 };
 
 const retryableFunctionCall = async (

packages/worker/src/blockchain/retryableContract.spec.ts

@@ -90,10 +90,37 @@ describe("RetryableContract", () => {
       expect(result).toBe(functionResult);
     });
 
-    describe("when throws a permanent call exception function error", () => {
+    describe("when throws a permanent execution reverted error", () => {
       const callExceptionError = {
         code: 3,
-        shortMessage: "execution reverted ....",
+        shortMessage: "execution reverted...",
+      };
+
+      beforeEach(() => {
+        (ethers.Contract as any as jest.Mock).mockReturnValue({
+          contractFn: async () => {
+            throw callExceptionError;
+          },
+        });
+
+        contract = new RetryableContract(tokenAddress, utils.IERC20, providerMock);
+      });
+
+      it("throws an error", async () => {
+        expect.assertions(1);
+
+        try {
+          await contract.contractFn();
+        } catch (e) {
+          expect(e).toBe(callExceptionError);
+        }
+      });
+    });
+
+    describe("when throws a permanent could not decode result data error", () => {
+      const callExceptionError = {
+        code: "BAD_DATA",
+        shortMessage: "could not decode result data...",
       };
 
       beforeEach(() => {

packages/worker/src/blockchain/retryableContract.ts

@@ -18,7 +18,13 @@ const PERMANENT_ERRORS: ErrorCode[] = [
 
 const shouldRetry = (error: EthersError): boolean => {
   const isPermanentErrorCode = PERMANENT_ERRORS.find((errorCode) => isError(error, errorCode));
-  return !isPermanentErrorCode && !(error.code === 3 && error.shortMessage?.startsWith("execution reverted"));
+  return (
+    !isPermanentErrorCode &&
+    // example block mainnet 47752810
+    !(error.code === 3 && error.shortMessage?.startsWith("execution reverted")) &&
+    // example block mainnet 47819836
+    !(error.code === "BAD_DATA" && error.shortMessage?.startsWith("could not decode result data"))
+  );
 };
 
 const retryableFunctionCall = async (