test: retry tests that flake on transient RPC connection drops (#1249)

Author: MoonBoi9001

packages/indexer-common/src/allocations/__tests__/tap.test.ts

@@ -25,6 +25,10 @@ import { hexlify, verifyTypedData } from 'ethers'
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 declare const __DATABASE__: any
 declare const __LOG_LEVEL__: never
+
+// This suite reaches a live RPC endpoint, whose transient connection drops (socket hang
+// up, ECONNRESET) would otherwise fail CI, so failed tests retry up to 2 times.
+jest.retryTimes(2, { logErrorsBeforeRetry: true })
 let logger: Logger
 let tapCollector: TapCollector
 let metrics: Metrics

packages/indexer-common/src/indexer-management/__tests__/resolvers/cost-models.test.ts

@@ -18,6 +18,10 @@ import { defineQueryFeeModels } from '../../../query-fees/models'
 declare const __DATABASE__: any
 declare const __LOG_LEVEL__: never
 
+// This suite reaches a live RPC endpoint, whose transient connection drops (socket hang
+// up, ECONNRESET) would otherwise fail CI, so failed tests retry up to 2 times.
+jest.retryTimes(2, { logErrorsBeforeRetry: true })
+
 const SET_COST_MODEL_MUTATION = gql`
   mutation setCostModel($costModel: CostModelInput!) {
     setCostModel(costModel: $costModel) {

packages/indexer-common/src/indexer-management/__tests__/resolvers/indexing-rules.test.ts

@@ -26,6 +26,10 @@ declare const __DATABASE__: any
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 declare const __LOG_LEVEL__: any
 
+// This suite reaches a live RPC endpoint, whose transient connection drops (socket hang
+// up, ECONNRESET) would otherwise fail CI, so failed tests retry up to 2 times.
+jest.retryTimes(2, { logErrorsBeforeRetry: true })
+
 const SET_INDEXING_RULE_MUTATION = gql`
   mutation setIndexingRule($rule: IndexingRuleInput!) {
     setIndexingRule(rule: $rule) {
@@ -258,7 +262,6 @@ describe('Indexing rules', () => {
       protocolNetwork: 'eip155:421614',
     }
 
-    // Write the original
     await expect(
       client.mutation(SET_INDEXING_RULE_MUTATION, { rule: originalInput }).toPromise(),
     ).resolves.toHaveProperty('data.setIndexingRule', original)
@@ -324,7 +327,6 @@ describe('Indexing rules', () => {
       protocolNetwork: 'eip155:421614',
     }
 
-    // Write the original
     await expect(
       client.mutation(SET_INDEXING_RULE_MUTATION, { rule: originalInput }).toPromise(),
     ).resolves.toHaveProperty('data.setIndexingRule', original)

packages/indexer-common/src/indexer-management/__tests__/resolvers/poi-disputes.test.ts

@@ -21,6 +21,10 @@ declare const __DATABASE__: any
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 declare const __LOG_LEVEL__: any
 
+// This suite reaches a live RPC endpoint, whose transient connection drops (socket hang
+// up, ECONNRESET) would otherwise fail CI, so failed tests retry up to 2 times.
+jest.retryTimes(2, { logErrorsBeforeRetry: true })
+
 const STORE_POI_DISPUTES_MUTATION = gql`
   mutation storeDisputes($disputes: [POIDisputeInput!]!) {
     storeDisputes(disputes: $disputes) {