add check for error 127

Author: digital88

packages/testcontainers/src/wait-strategies/utils/port-check.test.ts

@@ -163,5 +163,20 @@ describe("PortCheck", () => {
         ],
       ]);
     });
+
+    it.for([126, 127])("should error log the distroless image when exit code is %i", async (code) => {
+      mockContainerExec.mockReturnValueOnce(Promise.resolve({ output: "ERROR 1", exitCode: code }));
+      mockContainerExec.mockReturnValueOnce(Promise.resolve({ output: "ERROR 2", exitCode: code }));
+      mockContainerExec.mockReturnValueOnce(Promise.resolve({ output: "ERROR 3", exitCode: code }));
+
+      await portCheck.isBound(8080);
+
+      expect(mockLogger.error.mock.calls).toEqual([
+        [
+          "The HostPortWaitStrategy will not work on a distroless image, use an alternate wait strategy",
+          { containerId: "containerId" },
+        ],
+      ]);
+    });
   });
 });

packages/testcontainers/src/wait-strategies/utils/port-check.ts

@@ -53,7 +53,10 @@ export class InternalPortCheck implements PortCheck {
     );
     const isBound = commandResults.some((result) => result.exitCode === 0);
 
-    const shellExists = commandResults.some((result) => result.exitCode !== 126);
+    // https://www.gnu.org/software/bash/manual/html_node/Exit-Status.html
+    // If a command is not found, the child process created to execute it returns a status of 127.
+    // If a command is found but is not executable, the return status is 126.
+    const shellExists = commandResults.some((result) => result.exitCode !== 126 && result.exitCode !== 127);
     if (!isBound && !shellExists && !this.isDistroless) {
       this.isDistroless = true;
       log.error(`The HostPortWaitStrategy will not work on a distroless image, use an alternate wait strategy`, {