Skip to content

Commit

Permalink
Pass in testomat integration key, when not running unit tests. (#22294)
Browse files Browse the repository at this point in the history
* add option to log to xml

* set correct env variable for testomatio

* shorten test time

* Remove additional php unit param

* Change expression to only pass in if not unit test

* Temporarily use branch for test action

* Remove logic for deciding if api key should be passed in

* Only report to testomatio for UI tests when not pull request

* Only report to testomatio for UI tests when env variable is true

* Update matomo-tests.yml

---------

Co-authored-by: Stefan Giehl <[email protected]>
  • Loading branch information
caddoo and sgiehl committed Jun 17, 2024
1 parent 9964579 commit 2a71daf
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions .github/workflows/matomo-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,7 @@ jobs:
redis-service: true
artifacts-pass: ${{ secrets.ARTIFACTS_PASS }}
upload-artifacts: ${{ matrix.php == '7.2' }}
testomatio: ${{ secrets.TESTOMATIO_INTEGRATION }}
Javascript:
runs-on: ubuntu-20.04
timeout-minutes: 15
Expand Down
2 changes: 1 addition & 1 deletion tests/UI/config.dist.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ exports.chai = 'chai-1.9.0';
/**
* Mocha reporters to use (can be multiple delimited by a comma).
*/
if (process.env.TESTOMATIO) {
if (process.env.TESTOMATIO && process.env.SHOULD_SEND_TO_TESTOMATIO === 'true') {
exports.reporter = 'mocha-multi-reporters';
exports.reporterOptions = {
reporterEnabled: 'spec, @testomatio/reporter/lib/adapter/mocha.js',
Expand Down

0 comments on commit 2a71daf

Please sign in to comment.