Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test-reporting bugfix #711

Merged
merged 4 commits into from
Apr 3, 2023
Merged

Test-reporting bugfix #711

merged 4 commits into from
Apr 3, 2023

Conversation

dominichofer
Copy link
Contributor

@dominichofer dominichofer commented Mar 31, 2023

Unit tests are listed in integration tests and system tests as well. Integration tests are listed in system tests too.
This PR fixes that.
Also fixes a variable name clash, that leads to strange outputs.

@dominichofer
Copy link
Contributor Author

launch jenkins icontools

@github-actions
Copy link
Contributor

github-actions bot commented Mar 31, 2023

PR Preview Action v1.3.0
Preview removed because the pull request was closed.
2023-04-03 12:44 UTC

@dominichofer dominichofer changed the title Test reporting bugfix Test-reporting bugfix Mar 31, 2023
@jenkins-apn
Copy link

balfrin

🟢 unit test
Test
🟢summary
🟢 integration test
Test
🟢icontools-spack_info
🟢icontools-spack_spec
🟢 system test
Test
🟢IconTools-install

@jenkins-apn
Copy link

tsa

🟢 unit test
Test
🟢summary
🟢 integration test
Test
🟢icontools-spack_info
🟢icontools-spack_spec
🟢 system test
Test
🟢IconTools-install

@jenkins-apn
Copy link

daint

🟢 unit test
Test
🟢summary
🟢 integration test
Test
🟢icontools-spack_info
🟢icontools-spack_spec
🟢 system test
Test
🟢IconTools-install

@dominichofer dominichofer requested a review from AnnikaLau March 31, 2023 17:30
@AnnikaLau
Copy link
Contributor

Thanks for noticing and fixing!

@dominichofer dominichofer merged commit 80f52a6 into main Apr 3, 2023
@dominichofer dominichofer deleted the test_reporting branch April 3, 2023 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants