Unify CI under one workflow

HSLdevcom · May 29, 2024 · b4a06ae · b4a06ae
1 parent f61c916
commit b4a06ae
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/.github/workflows/cd.yml b/.github/workflows/cd.yml
@@ -1,4 +1,4 @@
-name: Push image to docker hub and run e2e tests
+name: Run Java tests, push image to docker hub and run e2e tests
 
 on:
   push:
@@ -8,7 +8,12 @@ on:
   workflow_dispatch:
 
 jobs:
+  run_java_tests:
+    name: Run Java tests
+    uses: ./.github/workflows/java-tests.yml
+
   push_to_registry:
+    needs: run_java_tests
     name: Build & Push docker image to dockerhub
     runs-on: ubuntu-22.04
 
@@ -58,7 +63,9 @@ jobs:
           docker push $IMAGE_NAME:latest
 
   run_e2e_tests:
-    needs: push_to_registry
+    needs:
+      - run_java_tests
+      - push_to_registry
     name: Run E2E tests
     runs-on: ubuntu-22.04
     steps:

diff --git a/.github/workflows/java-tests.yml b/.github/workflows/java-tests.yml
@@ -1,9 +1,8 @@
 name: Run java tests
 
 on:
-  push:
-  pull_request:
-    branches: [main]
+  # this workflow is only called by others, won't be executed on itself
+  workflow_call:
 
 jobs:
   run-tests: