Restore makefile so test_server does not depend on build_server

cyber-dojo · Oct 20, 2024 · 41810fc · 41810fc
1 parent 3689d8f
commit 41810fc
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 8 deletions.
diff --git a/sh/build_image.sh b/sh/build_image.sh
@@ -29,13 +29,7 @@ check_args()
       show_help
       exit 0
       ;;
-    'server')
-      if [ -n "${CI:-}" ] ; then
-        stderr "In CI workflow - use docker/build-push-action@v6 GitHub Action"
-        exit 42
-      fi
-      ;;
-    'client')
+    'server' | 'client')
       ;;
     '')
       show_help
@@ -52,7 +46,14 @@ check_args()
 build_image()
 {
   check_args "$@"
+
+  if [ -n "${CI:-}" ] ; then
+    stderr "In CI workflow - using previous docker/build-push-action@v6 GitHub Action"
+    exit 42
+  fi
+
   local -r TYPE="${1}"
+
   exit_non_zero_unless_installed docker
   export $(echo_versioner_env_vars)
   containers_down

diff --git a/sh/run_tests.sh b/sh/run_tests.sh
@@ -67,7 +67,7 @@ run_tests()
   exit_non_zero_unless_installed docker
   export SERVICE_NAME="${1}"
   # Don't do a build here, because in CI workflow, server image is built with GitHub Action
-  docker compose up --no-build --wait "${SERVICE_NAME}"
+  docker compose up --no-build --wait --wait-timeout=10 "${SERVICE_NAME}"
   exit_non_zero_unless_started_cleanly
   copy_in_saver_test_data