Skip to content

Commit

Permalink
[Build] Always collect test-results and don't fail on test-failures
Browse files Browse the repository at this point in the history
The 'Publish Unit Test Results' introduced with
eclipse-xtext#3144
will fail if test-failures are found.
  • Loading branch information
HannesWell committed Aug 11, 2024
1 parent 7a17f87 commit a0c94e4
Showing 1 changed file with 16 additions and 4 deletions.
20 changes: 16 additions & 4 deletions .github/workflows/maven.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,11 +50,17 @@ jobs:
- name: Build and test
uses: coactions/[email protected]
with:
run: ./mvnw clean verify -B -fae "-Dmaven.home=${{ env.MAVEN_WRAPPER_HOME }}" -PuseJenkinsSnapshots ${{ matrix.additional-maven-args }} -f org.eclipse.xtext.full.releng
run: >
./mvnw "-Dmaven.home=${{ env.MAVEN_WRAPPER_HOME }}"
clean verify
-f org.eclipse.xtext.full.releng
-B -fae -PuseJenkinsSnapshots
-Dmaven.test.failure.ignore=true
${{ matrix.additional-maven-args }}
- name: Archive Test Results
uses: actions/upload-artifact@v4
if: ${{ failure() || cancelled() }}
if: always()
with:
name: test-results-${{ runner.os }}
path: '**/*.tests/target/surefire-reports'
Expand Down Expand Up @@ -89,11 +95,17 @@ jobs:
run: echo "MAVEN_WRAPPER_HOME=$(./mvnw --version | grep "Maven home:" | cut -c 13-)" >> "$GITHUB_ENV"

- name: Build Maven artifacts
run: ./mvnw clean verify -B -fae "-Dmaven.home=${{ env.MAVEN_WRAPPER_HOME }}" -PuseJenkinsSnapshots -Pstrict-jdk-21 -f org.eclipse.xtext.maven.releng
run: >
./mvnw "-Dmaven.home=${{ env.MAVEN_WRAPPER_HOME }}"
clean verify
-f org.eclipse.xtext.maven.releng
-B -fae -PuseJenkinsSnapshots
-Dmaven.test.failure.ignore=true
-Pstrict-jdk-21
- name: Archive Test Results
uses: actions/upload-artifact@v4
if: ${{ failure() || cancelled() }}
if: always()
with:
name: test-results-${{ runner.os }}
path: '**/target/surefire-reports'
Expand Down

0 comments on commit a0c94e4

Please sign in to comment.