ci: don't run install as a first step before testing

camunda · Mar 31, 2022 · 464432b · 464432b
1 parent aaa72a4
commit 464432b
Showing 1 changed file with 7 additions and 11 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -27,7 +27,7 @@ jobs:
           maven-version: 3.8.5
       - run: mvn -T1C -B -DskipChecks -DskipTests package
       - run: docker build --build-arg DISTBALL=dist/target/camunda-zeebe-*.tar.gz --build-arg APP_ENV=dev -t camunda/zeebe:current-test .
-      - run: mvn -pl !:zeebe-elasticsearch-exporter -T2 -B -D skipUTs -D skipChecks -Dfailsafe.rerunFailingTestsCount=3 -Dflaky.test.reportDir=failsafe-reports -D junitThreadCount=12 -P parallel-tests,extract-flaky-tests verify
+      - run: mvn -pl !:zeebe-elasticsearch-exporter -T2 -B -D skipUTs -D skipChecks -D failsafe.rerunFailingTestsCount=3 -D flaky.test.reportDir=failsafe-reports -D junitThreadCount=12 -P parallel-tests,extract-flaky-tests verify
       - name: Archive Test Results
         uses: actions/upload-artifact@v2
         if: failure()
@@ -48,9 +48,9 @@ jobs:
       - uses: stCarolas/setup-maven@v4.3
         with:
           maven-version: 3.8.5
-      - run: mvn -T1C -B -DskipChecks -DskipTests package
+      - run: mvn -T1C -B -D skipChecks -D skipTests package
       - run: docker build --build-arg DISTBALL=dist/target/camunda-zeebe-*.tar.gz --build-arg APP_ENV=dev -t camunda/zeebe:current-test .
-      - run: mvn -pl :zeebe-elasticsearch-exporter -B -D skipUTs -D skipChecks -Dfailsafe.rerunFailingTestsCount=3 verify
+      - run: mvn -pl :zeebe-elasticsearch-exporter -B -D skipUTs -D skipChecks -D failsafe.rerunFailingTestsCount=3 verify
       - name: Archive Test Results
         uses: actions/upload-artifact@v2
         if: failure()
@@ -92,8 +92,7 @@ jobs:
           distribution: 'temurin'
           java-version: '17'
           cache: 'maven'
-      - run: mvn -T1C -B -D skipTests -D skipChecks install
-      - run: mvn -B -D skipITs -D skipChecks verify -pl :${{ matrix.project }}
+      - run: mvn -B -D skipITs -D skipChecks verify -am -pl :${{ matrix.project }}
       - name: Archive Test Results
         uses: actions/upload-artifact@v2
         if: failure()
@@ -103,7 +102,7 @@ jobs:
           retention-days: 7
   slow-unit-tests:
     name: Slow unit tests
-    runs-on: ["n1-standard-8-netssd-preempt"]
+    runs-on: "n1-standard-8-netssd-preempt"
     steps:
       - uses: actions/checkout@v3
       - uses: actions/setup-java@v3
@@ -114,8 +113,7 @@ jobs:
       - uses: stCarolas/setup-maven@v4.3
         with:
           maven-version: 3.8.5
-      - run: mvn -T1C -B -D skipTests -D skipChecks package
-      - run: mvn -B -D skipITs -D skipChecks -pl :zeebe-workflow-engine,:zeebe-logstreams verify
+      - run: mvn -B -D skipITs -D skipChecks -am -pl :zeebe-workflow-engine,:zeebe-logstreams verify
       - name: Archive Test Results
         uses: actions/upload-artifact@v2
         if: failure()
@@ -143,10 +141,8 @@ jobs:
           java-version: '17'
           distribution: 'temurin'
           cache: 'maven'
-      - name: Build relevant modules
-        run: mvn -B -am -pl qa/integration-tests install -DskipTests -DskipChecks "-Dmaven.javadoc.skip=true" -T1C
       - name: Run smoke test
-        run: mvn -B -pl qa/integration-tests verify -P smoke-test -DskipUTs -DskipChecks
+        run: mvn -B -P smoke-test -D skipUTs -D skipChecks -am -pl qa/integration-tests verify
       - name: Archive Test Results
         uses: actions/upload-artifact@v2
         if: failure()