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

feat(jest collector): add option to only post failures #43

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

irphilli
Copy link

Had a use case where it was only preferable to upload failures. i.e.: For non-default branch runs, we only really want to see the failure details and aren't too interested in passing tests.

So, adding an option for this to the jest collector!

@jasmine-q
Copy link
Collaborator

@irphilli thanks for the contribution, sounds great!
Could you please write a test covering that scenario in e2e/jest.test.js?

@irphilli irphilli force-pushed the feat/failures-only-option branch 2 times, most recently from 1d26a00 to d215084 Compare November 12, 2022 05:31
@irphilli
Copy link
Author

@irphilli thanks for the contribution, sounds great! Could you please write a test covering that scenario in e2e/jest.test.js?

@jasmine-q 👋 Added a test. Not sure what's going on with the tracer test, but it seems unrelated?

@irphilli
Copy link
Author

@jasmine-q The tests here look green after rebasing 🎉

@nprizal
Copy link
Contributor

nprizal commented Feb 15, 2024

Hi @irphilli, apologize for delayed response. I just triggered the test in the Buildkite pipeline, and it does green 💚.

I'm curious to know more about your use case of Test Analytics. Are you using Test Analytics mostly to capture failed tests in features branch to make it easier to know what to fix? Or have you been using other features such as flaky test detection? We'd appreciate it if you can give us feedbacks on how you and your team have been using our product.

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

3 participants