Make sure we don't hide exceptions from waitUntilContainerStarted (#6167)

This adds the original exception as the cause. It also adds the
text "Wait strategy failed." to each exception message.

This addresses two issues:

1. The original exception from waitUntilContainerStarted was not
   being included as the cause in newly thrown exceptions in
   tryStart. The exceptions were not completely hidden because
   they were being logged, however, but the initial cause was
   less visible.

2. The exception messages focused on state that was found at the
   time the exception was received but did not mention that the
   waitUntilContainerStarted method threw an exception.

Because of these issues, unless you look at the source or see the
logs, it isn't clear that this was caused by an exception from
waitUntilContainerStarted. In systems that might highlight the
failure message and stack trace and not logs/console output (test
failure reports, CI systems, etc.), it makes it harder to debug
these issues.

Updated tests to check for the updated messages as well as the
exception from waitUntilContainerStarted.

Author: deejgregor

core/src/main/java/org/testcontainers/containers/GenericContainer.java

@@ -493,25 +493,31 @@ private void tryStart(Instant startedAt) {
                 }
 
                 if (inspectContainerResponse == null) {
-                    throw new IllegalStateException("Container is removed");
+                    throw new IllegalStateException("Wait strategy failed. Container is removed", e);
                 }
 
                 InspectContainerResponse.ContainerState state = inspectContainerResponse.getState();
                 if (Boolean.TRUE.equals(state.getDead())) {
-                    throw new IllegalStateException("Container is dead");
+                    throw new IllegalStateException("Wait strategy failed. Container is dead", e);
                 }
 
                 if (Boolean.TRUE.equals(state.getOOMKilled())) {
-                    throw new IllegalStateException("Container crashed with out-of-memory (OOMKilled)");
+                    throw new IllegalStateException(
+                        "Wait strategy failed. Container crashed with out-of-memory (OOMKilled)",
+                        e
+                    );
                 }
 
                 String error = state.getError();
                 if (!StringUtils.isBlank(error)) {
-                    throw new IllegalStateException("Container crashed: " + error);
+                    throw new IllegalStateException("Wait strategy failed. Container crashed: " + error, e);
                 }
 
                 if (!Boolean.TRUE.equals(state.getRunning())) {
-                    throw new IllegalStateException("Container exited with code " + state.getExitCode());
+                    throw new IllegalStateException(
+                        "Wait strategy failed. Container exited with code " + state.getExitCode(),
+                        e
+                    );
                 }
 
                 throw e;

core/src/test/java/org/testcontainers/containers/GenericContainerTest.java

@@ -55,7 +55,9 @@ public void shouldReportOOMAfterWait() {
                 })
                 .withCommand("sh", "-c", "A='0123456789'; for i in $(seq 0 32); do A=$A$A; done; sleep 10m")
         ) {
-            assertThatThrownBy(container::start).hasStackTraceContaining("Container crashed with out-of-memory");
+            assertThatThrownBy(container::start)
+                .hasStackTraceContaining("Wait strategy failed. Container crashed with out-of-memory (OOMKilled)")
+                .hasStackTraceContaining("Nope!");
         }
     }
 
@@ -67,7 +69,9 @@ public void shouldReportErrorAfterWait() {
                 .waitingFor(new WaitForExitedState(state -> state.getExitCode() > 0))
                 .withCommand("sh", "-c", "usleep 100; exit 123")
         ) {
-            assertThatThrownBy(container::start).hasStackTraceContaining("Container exited with code 123");
+            assertThatThrownBy(container::start)
+                .hasStackTraceContaining("Wait strategy failed. Container exited with code 123")
+                .hasStackTraceContaining("Nope!");
         }
     }
 
@@ -80,7 +84,9 @@ public void shouldCopyTransferableAsFile() {
                 .waitingFor(new WaitForExitedState(state -> state.getExitCodeLong() > 0))
                 .withCommand("sh", "-c", "grep -q test /tmp/test && exit 100")
         ) {
-            assertThatThrownBy(container::start).hasStackTraceContaining("Container exited with code 100");
+            assertThatThrownBy(container::start)
+                .hasStackTraceContaining("Wait strategy failed. Container exited with code 100")
+                .hasStackTraceContaining("Nope!");
         }
     }
 
@@ -93,7 +99,9 @@ public void shouldCopyTransferableAsFileWithFileMode() {
                 .waitingFor(new WaitForExitedState(state -> state.getExitCodeLong() > 0))
                 .withCommand("sh", "-c", "ls -ll /tmp | grep '\\-rwxrwxrwx\\|test' && exit 100")
         ) {
-            assertThatThrownBy(container::start).hasStackTraceContaining("Container exited with code 100");
+            assertThatThrownBy(container::start)
+                .hasStackTraceContaining("Wait strategy failed. Container exited with code 100")
+                .hasStackTraceContaining("Nope!");
         }
     }
 
@@ -107,7 +115,9 @@ public void shouldCopyTransferableAfterMountableFile() {
                 .waitingFor(new WaitForExitedState(state -> state.getExitCodeLong() > 0))
                 .withCommand("sh", "-c", "grep -q test /tmp/test && exit 100")
         ) {
-            assertThatThrownBy(container::start).hasStackTraceContaining("Container exited with code 100");
+            assertThatThrownBy(container::start)
+                .hasStackTraceContaining("Wait strategy failed. Container exited with code 100")
+                .hasStackTraceContaining("Nope!");
         }
     }