chore(testcontainers): increase testTimeout 300 secs to cater for mor…

…e complex playground tests (#2033) #### What this PR does / why we need it: As per title. Also bumped testcontainers to 9.1.1 and removed testcontainers in `apps/package.json`.
DeFiCh · Feb 5, 2023 · 2e22801 · 2e22801
1 parent 770cadd
commit 2e22801
Show file tree

Hide file tree

Showing 5 changed files with 24 additions and 79 deletions.
diff --git a/apps/package.json b/apps/package.json
@@ -78,7 +78,6 @@
     "@types/validator": "13.7.12",
     "source-map-support": "0.5.21",
     "supertest": "6.3.3",
-    "testcontainers": "8.16.0",
     "ts-loader": "9.4.2",
     "ts-node": "10.9.1",
     "tsconfig-paths": "4.1.0"

diff --git a/jest.config.js b/jest.config.js
@@ -11,7 +11,7 @@ module.exports = {
   },
   verbose: true,
   clearMocks: true,
-  testTimeout: 180000,
+  testTimeout: 300000,
   testPathIgnorePatterns: [
     '__sanity__'
   ],

diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/testcontainers/package.json b/packages/testcontainers/package.json
@@ -19,7 +19,7 @@
     "dockerode": "^3.3.4",
     "tar-fs": "^2.1.1",
     "uuid": "^9.0.0",
-    "testcontainers": "^9.0.1"
+    "testcontainers": "^9.1.1"
   },
   "peerDependencies": {
     "@types/tar-fs": "^2.0.1",

diff --git a/packages/testcontainers/src/containers/AppContainer/PlaygroundApiContainer.ts b/packages/testcontainers/src/containers/AppContainer/PlaygroundApiContainer.ts
@@ -66,6 +66,6 @@ export class StartedPlaygroundApiContainer extends AbstractStartedContainer {
       })
       const { data } = await response.json()
       return data.details.playground.status === 'up'
-    }, timeout, 200, 'waitForIndexedBlockHeight')
+    }, timeout, 200, 'waitForReady')
   }
 }