Skip to content

Commit

Permalink
Refactor to IGNORE_TEST_FAILURES
Browse files Browse the repository at this point in the history
This is the same wording as used by Maven.
  • Loading branch information
SerhatG committed Apr 28, 2022
1 parent e504618 commit ceedb98
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 9 deletions.
6 changes: 3 additions & 3 deletions common/build-and-analyse-using-maven-and-sonar/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,9 @@ inputs:
ROOT_POM_DIRECTORY:
required: true
description: "The directory where the root POM can be found."
FAIL_ON_TEST_FAILURES:
IGNORE_TEST_FAILURES:
required: true
description: "Whether to fail step on test failures."
description: "Whether to ignore test failures."
SONAR_TOKEN:
required: false
description: "The token needed to push analyses to Sonar. Will skip this step if not provided."
Expand All @@ -25,5 +25,5 @@ runs:
shell: bash
run: |
SONAR_TOKEN=${{ inputs.SONAR_TOKEN }}
mvn --no-transfer-progress -f ${{ inputs.ROOT_POM_DIRECTORY }}/pom.xml -Dmaven.test.failure.ignore=${{ inputs.FAIL_ON_TEST_FAILURES }} -P ${SONAR_TOKEN:+sonar,}publish verify ${SONAR_TOKEN:+sonar:sonar}
mvn --no-transfer-progress -f ${{ inputs.ROOT_POM_DIRECTORY }}/pom.xml -Dmaven.test.failure.ignore=${{ inputs.IGNORE_TEST_FAILURES }} -P ${SONAR_TOKEN:+sonar,}publish verify ${SONAR_TOKEN:+sonar:sonar}
6 changes: 3 additions & 3 deletions common/build-using-maven/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,9 @@ inputs:
ROOT_POM_DIRECTORY:
required: true
description: "The directory where the root POM can be found."
FAIL_ON_TEST_FAILURES:
IGNORE_TEST_FAILURES:
required: true
description: "Whether to fail step on test failures."
description: "Whether to ignore test failures."

runs:
using: "composite"
Expand All @@ -19,5 +19,5 @@ runs:
env:
GITHUB_TOKEN: ${{ inputs.GITHUB_TOKEN }}
shell: bash
run: mvn --no-transfer-progress -f ${{ inputs.ROOT_POM_DIRECTORY }}/pom.xml -Dmaven.test.failure.ignore=${{ inputs.FAIL_ON_TEST_FAILURES }} -P publish verify
run: mvn --no-transfer-progress -f ${{ inputs.ROOT_POM_DIRECTORY }}/pom.xml -Dmaven.test.failure.ignore=${{ inputs.IGNORE_TEST_FAILURES }} -P publish verify

2 changes: 1 addition & 1 deletion events/pull_request-event-action/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,5 +28,5 @@ runs:
with:
GITHUB_TOKEN: ${{ inputs.GITHUB_TOKEN }}
ROOT_POM_DIRECTORY: ${{ inputs.ROOT_POM_DIRECTORY }}
FAIL_ON_TEST_FAILURES: false
IGNORE_TEST_FAILURES: true

2 changes: 1 addition & 1 deletion events/push-event-action/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ runs:
GITHUB_TOKEN: ${{ inputs.GITHUB_TOKEN }}
SONAR_TOKEN: ${{ inputs.SONAR_TOKEN }}
ROOT_POM_DIRECTORY: ${{ inputs.ROOT_POM_DIRECTORY }}
FAIL_ON_TEST_FAILURES: false
IGNORE_TEST_FAILURES: true

- id: get-project-version
uses: ./aerius-github-actions/common/get-maven-project-version
Expand Down
2 changes: 1 addition & 1 deletion events/release-event-action/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ runs:
GITHUB_TOKEN: ${{ inputs.GITHUB_TOKEN }}
SONAR_TOKEN: ${{ inputs.SONAR_TOKEN }}
ROOT_POM_DIRECTORY: ${{ inputs.ROOT_POM_DIRECTORY }}
FAIL_ON_TEST_FAILURES: true
IGNORE_TEST_FAILURES: false

- uses: ./aerius-github-actions/common/publish-maven-artifacts
with:
Expand Down

0 comments on commit ceedb98

Please sign in to comment.