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

Update CI test runs to store test results #884

Closed
wants to merge 2 commits into from

Conversation

huntie
Copy link
Member

@huntie huntie commented Oct 31, 2022

Summary:
This enables storing of test data in our CircleCI runs. This will allow CircleCI to better attribute failed runs to specific test failures, provide flaky test insights, and more.
https://circleci.com/docs/collect-test-data/

Changes

  • Relocate CircleCI config file (legacy circle.yml file will not be discovered for new CircleCI projects in forks).
  • Add store_test_results step to test-node-14 job via the jest-junit reporter.
  • Set display name for yarn_install run step.

Changelog: [Internal]

Differential Revision: D40805194

Examples of improved output

CircleCI test insights Visibility of specific failed tests
image image
Based on #885 for a Windows-enabled branch

huntie and others added 2 commits October 31, 2022 03:50
Differential Revision: D40805195

fbshipit-source-id: 0d13531c74f3adf2c777f59a52fc9e8f0f6eaa94
Summary:
This enables storing of test data in our CircleCI runs. This will allow CircleCI to better attribute failed runs to specific test failures, provide flaky test insights, and more.
https://circleci.com/docs/collect-test-data/

**Changes**

- Relocate CircleCI config file (legacy `circle.yml` file will not be discovered for new CircleCI projects in forks).
- Add `store_test_results` step to `test-node-14` job via the `jest-junit` reporter.
- Set display name for `yarn_install` run step.

Changelog: [Internal]

Differential Revision: D40805194

fbshipit-source-id: 906ff34488fecbdb2f7c37a405814d308007cd2b
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Oct 31, 2022
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D40805194

@huntie huntie deleted the export-D40805194 branch November 2, 2022 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants