Docker runtimes may be undefined (#795)

testcontainers · Jul 9, 2024 · 04a1aad · 04a1aad
1 parent 794ebc4
commit 04a1aad
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 12 deletions.
diff --git a/packages/modules/ollama/src/ollama-container.ts b/packages/modules/ollama/src/ollama-container.ts
@@ -1,10 +1,4 @@
-import {
-  AbstractStartedContainer,
-  GenericContainer,
-  getContainerRuntimeClient,
-  StartedTestContainer,
-  Wait,
-} from "testcontainers";
+import { AbstractStartedContainer, GenericContainer, getContainerRuntimeClient, Wait } from "testcontainers";
 
 export const OLLAMA_PORT = 11434;
 
@@ -34,10 +28,6 @@ export class OllamaContainer extends GenericContainer {
 }
 
 export class StartedOllamaContainer extends AbstractStartedContainer {
-  constructor(startedTestContainer: StartedTestContainer) {
-    super(startedTestContainer);
-  }
-
   public getPort(): number {
     return this.startedTestContainer.getMappedPort(OLLAMA_PORT);
   }

diff --git a/packages/testcontainers/src/container-runtime/clients/client.ts b/packages/testcontainers/src/container-runtime/clients/client.ts
@@ -122,7 +122,7 @@ async function initStrategy(strategy: ContainerRuntimeClientStrategy): Promise<C
     architecture: dockerodeInfo.Architecture,
     cpus: dockerodeInfo.NCPU,
     memory: dockerodeInfo.MemTotal,
-    runtimes: Object.keys(dockerodeInfo.Runtimes),
+    runtimes: dockerodeInfo.Runtimes ? Object.keys(dockerodeInfo.Runtimes) : [],
   };
 
   const composeInfo: ComposeInfo = composeClient.info;