Skip to content

[7.x] Consolidates Jest configuration files and scripts (#82671)#83362

Merged
tylersmalley merged 3 commits intoelastic:7.xfrom
tylersmalley:backport/7.x/pr-82671
Nov 13, 2020
Merged

[7.x] Consolidates Jest configuration files and scripts (#82671)#83362
tylersmalley merged 3 commits intoelastic:7.xfrom
tylersmalley:backport/7.x/pr-82671

Conversation

@tylersmalley
Copy link
Copy Markdown
Member

Backports the following commits to 7.x:

@tylersmalley tylersmalley added the backport This PR is a backport of another PR label Nov 13, 2020
@tylersmalley
Copy link
Copy Markdown
Member Author

Opened #83369 to backport changes to reduce conflicts.

Jest tests are currently organized into main configuration files (src/dev/jest/config.js and x-pack/dev-tools/jest/create_jest_config.js). Both of these are similar, but very slightly due to  previously being in separate repositories. This change consolidates the scripts referenced in those configs and moves them to the `@kbn/test` project.

OSS contained an alias for `test_utils`. Those aliases have been removed in favor of importing these utilities from `@kbn/test/jest`

Blocker to elastic#72569

Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
@tylersmalley tylersmalley reopened this Nov 13, 2020
@tylersmalley
Copy link
Copy Markdown
Member Author

@elasticmachine merge upstream

@tylersmalley
Copy link
Copy Markdown
Member Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

  • 💚 Build #87826 succeeded b2d16b8439ab5e9c762f6939bfa8da6d0b439e42
  • 💔 Build #87804 failed d33c615948acdcf8458aad3816b4b7d18647d238

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@tylersmalley tylersmalley merged commit 8c2976d into elastic:7.x Nov 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants