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

An option added to keep only the latest test result for each test cas… #467

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

saim80
Copy link

@saim80 saim80 commented Dec 26, 2020

…e are kept. So, retry attempts will end up with one failure or pass result.

In our organization, we do not want to have repeated test results which made us confused via Jenkins JUnit Test Result page.

I added an option to keep only the latest result for each test case. So, this will keep us informed of either one pass or failure result only without duplicate test results when retry is attempted.

…e are kept. So, retry attempts will end up with one failure or pass result.
@ravimandala
Copy link
Contributor

Good to have, but my hesitation is we are adding one more flag.
@saim80 Instead can you do some post-processing over the Bluepill report to trim what you don't need?

@chenxiao0228
Copy link
Collaborator

@saim80 thanks for your contribution!

Come to think of it, we do not really need coverage data for the failed attempts, like you said it does not provide extra value but confusion. I think we are good to make it the default behavior and do not add the new flag.

A couple of things I'd like to ask though

  • can you figure out why one of the checks failed?
  • can you add a unit test case for the de-dup behavior?

Many thanks!

@saim80 saim80 requested a review from a team as a code owner January 15, 2022 03:17
@cccCody
Copy link

cccCody commented Feb 12, 2022

@chenxiao0228

"we are good to make it the default behavior"

Did this happen? Is this actually the default behavior now?

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.

None yet

5 participants