Skip to content

Commit

Permalink
Remove artifact-test from CI
Browse files Browse the repository at this point in the history
Artifacts via public API are not available before whole workflow is finished. Therefore it's not possible to execute test this way
  • Loading branch information
Michal Dorner committed Feb 18, 2021
1 parent 8efb156 commit 3340176
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 18 deletions.
13 changes: 0 additions & 13 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,16 +33,3 @@ jobs:
name: JEST Tests
path: __tests__/__results__/*.xml
reporter: jest-junit

artifact-test:
name: Artifact test
runs-on: ubuntu-latest
needs: build-test
steps:
- uses: actions/checkout@v2
- uses: ./
with:
artifact: test-results
name: Artifact Report
path: '*.xml'
reporter: jest-junit
4 changes: 2 additions & 2 deletions dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion dist/index.js.map

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions src/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -122,12 +122,12 @@ class TestReporter {
core.setOutput('time', time)

if (this.failOnError && isFailed) {
core.setFailed(`Failed test has been found and 'fail-on-error' option is set to ${this.failOnError}`)
core.setFailed(`Failed test were found and 'fail-on-error' option is set to ${this.failOnError}`)
return
}

if (results.length === 0) {
core.setFailed(`No test results file has been processed`)
core.setFailed(`No test report files were found`)
return
}
}
Expand Down

0 comments on commit 3340176

Please sign in to comment.