fix(testcontainers): convert stop() timeouts to ms for v11 + drop stale tar-fs override

Author: nicktrn

internal-packages/testcontainers/src/utils.ts

@@ -99,7 +99,7 @@ export async function createRedisContainer({
   const [error] = await tryCatch(verifyRedisConnection(startedContainer));
 
   if (error) {
-    await startedContainer.stop({ timeout: 30 });
+    await startedContainer.stop({ timeout: 30_000 });
     throw new Error("verifyRedisConnection error", { cause: error });
   }
 
@@ -238,7 +238,7 @@ export async function useContainer<TContainer extends StartedTestContainer>(
     metadata.useDurationMs = useDurationMs;
   } finally {
     // WARNING: Testcontainers by default will not wait until the container has stopped. It will simply issue the stop command and return immediately.
-    // If you need to wait for the container to be stopped, you can provide a timeout. The unit of timeout option here is second
-    await logCleanup(name, container.stop({ timeout: 10 }), metadata);
+    // If you need to wait for the container to be stopped, you can provide a timeout. The unit of timeout option here is milliseconds (changed from seconds in testcontainers v11)
+    await logCleanup(name, container.stop({ timeout: 10_000 }), metadata);
   }
 }

package.json

@@ -89,7 +89,6 @@
       "@types/node": "20.14.14",
       "express@^4>body-parser": "1.20.3",
       "@remix-run/dev@2.17.4>tar-fs": "2.1.4",
-      "testcontainers@10.28.0>tar-fs": "3.1.1",
       "tar@>=7 <7.5.11": "^7.5.11",
       "form-data@^2": "2.5.4",
       "form-data@^3": "3.0.4",