Skip to content

Commit

Permalink
ci(tests): Fix test reports archives not generated on maven f (#29348) (
Browse files Browse the repository at this point in the history
#29349)

### Proposed Changes
* Force test archives to upload even when maven fails

To test
Will Test to success in PR, then will force a postman failure to force
generation of the archive and report, then fix the failure before
merging.

### Checklist
- [ ] Tests
- [ ] Translations
- [ ] Security Implications Contemplated (add notes if applicable)

### Additional Info
Related to #29348 (Fix test reports archives not generated on maven f).


### Screenshots
Original             |  Updated
:-------------------------:|:-------------------------:
** original screenshot **  |  ** updated screenshot **
  • Loading branch information
spbolton authored Jul 25, 2024
1 parent f67336f commit e10a52b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/actions/maven-job/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -291,7 +291,7 @@ runs:
- id: create-test-reports-artifact
name: Create Test Reports Artifact
uses: actions/upload-artifact@v4
if: ${{ inputs.generates-test-results == 'true' }}
if: always() && inputs.generates-test-results == 'true'
with:
name: "build-reports-test-${{ inputs.stage-name }}"
path: |
Expand Down

0 comments on commit e10a52b

Please sign in to comment.