Skip to content

fix test result output on Windows #951

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

Merged
merged 4 commits into from
Oct 27, 2020

Conversation

beniwohli
Copy link
Contributor

Using call swallows the exit code of the test runner, making
Jenkins mark failed test runs as unstable instead of failed.

Using `call` swallows the exit code of the test runner, making
Jenkins mark failed test runs as unstable instead of failed.
@apmmachine
Copy link
Contributor

apmmachine commented Oct 27, 2020

💚 Flaky test report

Tests succeeded.

Test stats 🧪

Test Results
Failed 0
Passed 11121
Skipped 8409
Total 19530

@apmmachine
Copy link
Contributor

apmmachine commented Oct 27, 2020

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #951 updated]

  • Start Time: 2020-10-27T09:46:13.226+0000

  • Duration: 22 min 27 sec

Test stats 🧪

Test Results
Failed 0
Passed 11121
Skipped 8409
Total 19530

@beniwohli beniwohli merged commit 0f2f40a into elastic:master Oct 27, 2020
@beniwohli beniwohli deleted the fix-windows-test-failure-output branch October 27, 2020 10:12
beniwohli added a commit to beniwohli/apm-agent-python that referenced this pull request Sep 14, 2021
* fix test result output on Windows

Using `call` swallows the exit code of the test runner, making
Jenkins mark failed test runs as unstable instead of failed.

* exit if pytest fails

* set error code explicitly
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.

2 participants