Replace hardcoded deploymentId values in airnode-node tests (#1653)

api3dao · Apr 10, 2023 · 0a16769 · 0a16769
1 parent 4502238
commit 0a16769
Show file tree

Hide file tree

Showing 5 changed files with 25 additions and 6 deletions.
diff --git a/.changeset/silly-actors-think.md b/.changeset/silly-actors-think.md
@@ -0,0 +1,5 @@
+---
+'@api3/airnode-node': patch
+---
+
+Replace hardcoded deploymentId values in tests
diff --git a/packages/airnode-node/src/handlers/start-coordinator.test.ts b/packages/airnode-node/src/handlers/start-coordinator.test.ts
@@ -32,6 +32,9 @@ import * as fixtures from '../../test/fixtures';
 import * as calls from '../coordinator/calls';
 import { buildAggregatedRegularApiCall, buildConfig } from '../../test/fixtures';
 import { BLOCK_COUNT_HISTORY_LIMIT } from '../constants';
+import { DEPLOYMENT_ID_LENGTH } from '../workers';
+
+const deploymentIdRegex = RegExp(`local[0-9a-f]{${DEPLOYMENT_ID_LENGTH}}`);
 
 describe('startCoordinator', () => {
   jest.setTimeout(30_000);
@@ -172,7 +175,12 @@ describe('startCoordinator', () => {
     await startCoordinator(config, coordinatorId);
 
     // cached requests should not trigger an API call
-    expect(callApisSpy).toHaveBeenCalledWith([], expect.objectContaining({ deploymentId: 'localdd59d6d0' }));
+    expect(callApisSpy).toHaveBeenCalledWith(
+      [],
+      expect.objectContaining({
+        deploymentId: expect.stringMatching(deploymentIdRegex),
+      })
+    );
 
     expect(blockWithTransactionsSpy).toHaveBeenCalled();
     // API call was submitted

diff --git a/packages/airnode-node/src/providers/actions.test.ts b/packages/airnode-node/src/providers/actions.test.ts
@@ -31,6 +31,9 @@ import * as providers from './actions';
 import * as fixtures from '../../test/fixtures';
 import { GroupedRequests } from '../types';
 import { ChainConfig } from '../config';
+import { DEPLOYMENT_ID_LENGTH } from '../workers';
+
+const deploymentIdRegex = RegExp(`local[0-9a-f]{${DEPLOYMENT_ID_LENGTH}}`);
 
 const chainProviderName1 = 'Pocket Ethereum Mainnet';
 const chainProviderName3 = 'Infura Sepolia';
@@ -124,7 +127,7 @@ describe('initialize', () => {
           },
           settings: {
             airnodeAddress: '0xA30CA71Ba54E83127214D3271aEA8F5D6bD4Dace',
-            deploymentId: 'localdd59d6d0',
+            deploymentId: expect.stringMatching(deploymentIdRegex),
             authorizers: {
               requesterEndpointAuthorizers: [ethers.constants.AddressZero],
               crossChainRequesterAuthorizers: [],
@@ -179,7 +182,7 @@ describe('initialize', () => {
           },
           settings: {
             airnodeAddress: '0xA30CA71Ba54E83127214D3271aEA8F5D6bD4Dace',
-            deploymentId: 'localdd59d6d0',
+            deploymentId: expect.stringMatching(deploymentIdRegex),
             authorizers: {
               requesterEndpointAuthorizers: [ethers.constants.AddressZero],
               crossChainRequesterAuthorizers: [],

diff --git a/packages/airnode-node/src/providers/state.test.ts b/packages/airnode-node/src/providers/state.test.ts
@@ -4,6 +4,9 @@ import * as fixtures from '../../test/fixtures';
 import { BLOCK_MIN_CONFIRMATIONS } from '../constants';
 import { EVMProviderState, ProviderState } from '../types';
 import { ChainConfig } from '../config';
+import { DEPLOYMENT_ID_LENGTH } from '../workers';
+
+const deploymentIdRegex = RegExp(`local[0-9a-f]{${DEPLOYMENT_ID_LENGTH}}`);
 
 describe('create', () => {
   fixtures.setEnvVariables({ AIRNODE_WALLET_PRIVATE_KEY: fixtures.getAirnodeWalletPrivateKey() });
@@ -54,7 +57,7 @@ describe('create', () => {
       },
       settings: {
         airnodeAddress: '0xA30CA71Ba54E83127214D3271aEA8F5D6bD4Dace',
-        deploymentId: 'localdd59d6d0',
+        deploymentId: expect.stringMatching(deploymentIdRegex),
         authorizers: {
           requesterEndpointAuthorizers: [ethers.constants.AddressZero],
           crossChainRequesterAuthorizers: [],
@@ -150,7 +153,7 @@ describe('create', () => {
       },
       settings: {
         airnodeAddress: '0xA30CA71Ba54E83127214D3271aEA8F5D6bD4Dace',
-        deploymentId: 'localdd59d6d0',
+        deploymentId: expect.stringMatching(deploymentIdRegex),
         authorizers: {
           requesterEndpointAuthorizers: [ethers.constants.AddressZero],
           crossChainRequesterAuthorizers: [],

diff --git a/packages/airnode-node/src/workers/index.ts b/packages/airnode-node/src/workers/index.ts
@@ -5,7 +5,7 @@ import * as localHandlers from './local-handlers';
 import { WorkerParameters, WorkerResponse } from '../types';
 import { LocalOrCloudProvider } from '../config';
 
-const DEPLOYMENT_ID_LENGTH = 8;
+export const DEPLOYMENT_ID_LENGTH = 8;
 
 export function spawn(params: WorkerParameters): Promise<WorkerResponse> {
   switch (params.cloudProvider.type) {