fix: unit tests

connext · May 17, 2024 · bb56191 · bb56191
1 parent be16d7d
commit bb56191
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 4 deletions.
diff --git a/packages/adapters/txservice/src/aggregator.ts b/packages/adapters/txservice/src/aggregator.ts
@@ -304,8 +304,6 @@ export class RpcProviderAggregator {
             synced: provider.synced,
             metrics: {
               reliability: provider.reliability,
-              latency: provider.latency,
-              cps: provider.cps,
               priority: provider.priority,
             },
           },

diff --git a/packages/adapters/txservice/test/aggregator.spec.ts b/packages/adapters/txservice/test/aggregator.spec.ts
@@ -82,6 +82,10 @@ describe("RpcProviderAggregator", () => {
     coreSyncProvider = Sinon.createStubInstance(SyncProvider);
     (coreSyncProvider as any)._syncedBlockNumber = 123;
     (coreSyncProvider as any).synced = true;
+    (coreSyncProvider as any).name = "-------fakeProvider------";
+    (coreSyncProvider as any).lag = 1;
+    (coreSyncProvider as any).reliability = 1.0;
+    (coreSyncProvider as any).priority = 0;
     (coreSyncProvider as any).url = "https://-------fakeProvider------";
     Sinon.stub(coreSyncProvider, "syncedBlockNumber").get(() => (coreSyncProvider as any)._syncedBlockNumber);
     (chainProvider as any).providers = [coreSyncProvider];
@@ -290,9 +294,24 @@ describe("RpcProviderAggregator", () => {
     it("should return error result if the signer readContract call throws", async () => {
       const testError = new Error("test error");
       signer.call.rejects(testError);
-
       // The error.context.error is the "test error" thrown by the signer.call.
-      await expect(chainProvider.readContract(TEST_READ_TX)).to.be.rejectedWith(TransactionReadError);
+      await expect(chainProvider.readContract(TEST_READ_TX)).to.rejected.then((error) => {
+        if (error instanceof TransactionReadError) {
+          expect(error.context.error).to.eq(testError);
+          expect(error.context.provider).to.deep.eq({
+            url: coreSyncProvider.name,
+            blockNumber: coreSyncProvider.syncedBlockNumber,
+            lag: coreSyncProvider.lag,
+            synced: coreSyncProvider.synced,
+            metrics: {
+              reliability: coreSyncProvider.reliability,
+              priority: coreSyncProvider.priority,
+            },
+          });
+        } else {
+          expect.fail("Error should be a TransactionReadError");
+        }
+      });
     });
 
     it("should execute with provider if no signer available", async () => {