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

Github Status reported Failure even though tests passed #211

Closed
1 task done
nsoojin opened this issue Jul 22, 2022 · 11 comments
Closed
1 task done

Github Status reported Failure even though tests passed #211

nsoojin opened this issue Jul 22, 2022 · 11 comments
Labels

Comments

@nsoojin
Copy link

nsoojin commented Jul 22, 2022

Troubleshooting

  • I've searched discuss.bitrise.io for possible solutions.
  • Which version of the step is effected? 4.3
  • Is the issue reproducible with the latest version? YES
  • Does the issue happen sporadically, or every time? EVERY TIME
  • Is the issue reproducible locally by following our local debug guide? NOT APPLICABLE

Useful information

Issue description

I have some flaky tests that sometimes fail. So I'm using "Retry on Failure" option for my test plan. However, Bitrise is reporting tests failed even though flaky tests passed on subsequent tries.

XCResult here shows that the overall test suite passed with 614 passed tests, 0 failed tests, and 2 mixed tests which are the flaky tests that passed on the second try. However, Bitrise's Test result marks them "failed", so the status for my PR failed.

Screen Shot 2022-07-22 at 4 04 12 PM

Expected Result

If retried tests passed, the overall test should be marked successful and Github Status should pass.

Bitrise info

 
  
@DamienBitrise
Copy link

The test report addon does seem to be reporting failures even tho the step passed. Thanks for the report, we will look into this issue.

@d4rkd3v1l
Copy link

We're facing the same issue, I guess. (step version 4.6)

When a test fails initially, but then passes on retry, the build is passed in Bitrise (as expected), but Bitrise Checks reports a failed status to GitHub.

@nsoojin
Copy link
Author

nsoojin commented Oct 5, 2022

@DamienBitrise Can you share us an update?

@bitrise-coresteps-bot
Copy link
Contributor

Hello there, I'm a bot. On behalf of the community I thank you for opening this issue.

To help our human contributors focus on the most relevant reports, I check up on old issues to see if they're still relevant.
This issue has had no activity for 90 days, so I marked it as stale.

The community would appreciate if you could check if the issue still persists. If it isn't, please close it.
If the issue persists, and you'd like to remove the stale label, you simply need to leave a comment. Your comment can be as simple as "still important to me".

If no comment left within 21 days, this issue will be closed.

@nsoojin
Copy link
Author

nsoojin commented Jan 4, 2023

still important to me

@bitrise-coresteps-bot
Copy link
Contributor

Hello there, I'm a bot. On behalf of the community I thank you for opening this issue.

To help our human contributors focus on the most relevant reports, I check up on old issues to see if they're still relevant.
This issue has had no activity for 90 days, so I marked it as stale.

The community would appreciate if you could check if the issue still persists. If it isn't, please close it.
If the issue persists, and you'd like to remove the stale label, you simply need to leave a comment. Your comment can be as simple as "still important to me".

If no comment left within 21 days, this issue will be closed.

@lo1tuma
Copy link

lo1tuma commented Apr 25, 2023

Looks like this is duplicate of #195. It seems like only the test report and the github status check is wrongly reported as failed, the bitrise step itself is passing. This is a huge problem for us, because we rely on the github status check which must pass before we allow pull requests being merged.

Is there any update on this bug?

@bitrise-coresteps-bot
Copy link
Contributor

Hello there, I'm a bot. On behalf of the community I thank you for opening this issue.

To help our human contributors focus on the most relevant reports, I check up on old issues to see if they're still relevant.
This issue has had no activity for 90 days, so I marked it as stale.

The community would appreciate if you could check if the issue still persists. If it isn't, please close it.
If the issue persists, and you'd like to remove the stale label, you simply need to leave a comment. Your comment can be as simple as "still important to me".

If no comment left within 21 days, this issue will be closed.

@lo1tuma
Copy link

lo1tuma commented Jul 25, 2023

This is still an issue.

@bitrise-coresteps-bot
Copy link
Contributor

Hello there, I'm a bot. On behalf of the community I thank you for opening this issue.

To help our human contributors focus on the most relevant reports, I check up on old issues to see if they're still relevant.
This issue has had no activity for 90 days, so I marked it as stale.

The community would appreciate if you could check if the issue still persists. If it isn't, please close it.
If the issue persists, and you'd like to remove the stale label, you simply need to leave a comment. Your comment can be as simple as "still important to me".

If no comment left within 21 days, this issue will be closed.

@bitrise-coresteps-bot
Copy link
Contributor

I'll close this issue as it doesn't seem to be relevant anymore.
We believe an old issue probably has a bunch of context that's no longer relevant, therefore, if the problem still persists, please open a new issue.

@bitrise-steplib bitrise-steplib locked and limited conversation to collaborators Nov 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

5 participants