Merge pull request #215 from vorburger/docker-logs

show docker logs in case of test failure

test.sh

@@ -15,13 +15,14 @@ function test_app() {
   local http_port="$(docker port ${container_id} ${port}|sed 's/0.0.0.0://')"
   local http_reply=$(curl --silent --show-error http://localhost:$http_port)
 
-  docker rm -f "$container_id"
-
   if [ "$http_reply" = 'hello, world' ]; then
     echo "APP TEST PASSED"
+    docker rm -f ${container_id}
     return 0
   else
     echo "APP TEST FAILED"
+    docker logs ${container_id}
+    docker rm -f ${container_id}
     return -123
   fi
 }
@@ -40,15 +41,16 @@ function test_metrics() {
   local metrics_port="$(docker port ${container_id} ${port}|sed 's/0.0.0.0://')"
   local metrics_reply=$(curl --silent --show-error http://localhost:$metrics_port/metrics)
 
-  docker rm -f "$container_id"
-
   case $metrics_reply in
     *"jvm_threads_current"*)
       echo "METRICS TEST PASSED"
+      docker rm -f ${container_id}
       return 0
       ;;
     *)
       echo "METRICS TEST FAILED"
+      docker logs ${container_id}
+      docker rm -f ${container_id}
       return -123
       ;;
   esac
@@ -75,7 +77,7 @@ function test_entrypoint() {
     echo "APP TEST FAILED (with entrypoint ${entrypoint})"
     docker logs ${container_id}
     docker rm -f ${container_id}
-  return -123
+    return -123
   fi
 }
 
@@ -137,18 +139,18 @@ function test_image() {
 
   test_container "${name}-binary-example"
 
+
   # ----------------------------------------------------------------------------------
   # Maven Wrapper
   # ----------------------------------------------------------------------------------
 
   s2i build --copy java/examples/maven-wrapper ${name} ${name}-maven-wrapper-example
   s2i build --copy java/examples/maven-wrapper ${name} ${name}-maven-wrapper-example --incremental
 
+
   # ----------------------------------------------------------------------------------
   # Entrypoint Binary
   # ----------------------------------------------------------------------------------
-  local dir=$1
-  local name=$2
 
   curl https://repo.spring.io/release/org/springframework/cloud/spring-cloud-deployer-spi-test-app/1.3.4.RELEASE/spring-cloud-deployer-spi-test-app-1.3.4.RELEASE-exec.jar \
        -o java/examples/binary/deployments/app.jar
@@ -159,7 +161,6 @@ function test_image() {
   test_entrypoint "${name}-entrypoint-binary-example" "java -jar /deployments/app.jar"  # works
   test_entrypoint "${name}-entrypoint-binary-example" /opt/run-java/run-java.sh         # will fail until https://github.com/fabric8io-images/run-java-sh/issues/75 is fixed
   test_entrypoint "${name}-entrypoint-binary-example" /usr/local/s2i/run                # will fail until https://github.com/fabric8io-images/run-java-sh/issues/75 is fixed
-
 }
 
 # ==================================================================================