Skip to content

Jest multi-project configuration#77894

Merged
tylersmalley merged 8 commits intoelastic:masterfrom
tylersmalley:jest-multi
Dec 2, 2020
Merged

Jest multi-project configuration#77894
tylersmalley merged 8 commits intoelastic:masterfrom
tylersmalley:jest-multi

Conversation

@tylersmalley
Copy link
Copy Markdown
Member

@tylersmalley tylersmalley commented Sep 18, 2020 •
edited
Loading

Closes #72569
Closes #75202
Closes #41129

Blocked on:

Overrides for tests which are incompatible with jest-circus added in 72d7e42. A PR is currently open (#82280) to resolve these incompatabilities but should not block this PR.

A CI check has been added to ensure that no new tests are added that are not covered by the main config.

Reviewing:

The commits have been split out to isolate the changes consisting of the auto-generated jest.config.js files in 7518751fffb0ad73c9649ffe42ef81824f875086.

@tylersmalley tylersmalley force-pushed the jest-multi branch 2 times, most recently from 908b73d to 07bce70 Compare September 23, 2020 04:47
@elastic elastic deleted a comment from kibanamachine Sep 23, 2020
@tylersmalley tylersmalley force-pushed the jest-multi branch 17 times, most recently from 8b38a6f to 827ce4b Compare September 28, 2020 14:52
@tylersmalley tylersmalley mentioned this pull request Sep 29, 2020
Merged
@tylersmalley tylersmalley force-pushed the jest-multi branch 6 times, most recently from 126dd31 to 1a491fc Compare October 6, 2020 21:12
@patrykkopycinski patrykkopycinski mentioned this pull request Nov 12, 2020
Closed
1 task
peteharverson
peteharverson approved these changes Dec 1, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ML and Transforms plugin changes LGTM

mikecote
mikecote approved these changes Dec 1, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@mikecote mikecote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alerting related changes LGTM

nreese
nreese approved these changes Dec 1, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maps changes LGTM
code review

flash1293
flash1293 approved these changes Dec 1, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@flash1293 flash1293 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

KibanaApp changes LGTM, code review

weltenwort
weltenwort reviewed Dec 1, 2020
View reviewed changes
Copy link
Copy Markdown
Member

@weltenwort weltenwort left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

infra plugin unit tests run individually and as part of the overall config.

But where do they show up in the CI test results? Is it intentional that they are not picked up by the test reporter anymore?

image

@tylersmalley
Defines test reporter on entry config
574793d 
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
cee-chen
cee-chen approved these changes Dec 1, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@cee-chen cee-chen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm so excited for this - we can hopefully finally bake in coverage reporting (and exceptions) into a shorter yarn jest command. Huge thanks from the Enterprise Search team!

nchaulet
nchaulet approved these changes Dec 1, 2020
View reviewed changes
Copy link
Copy Markdown
Member

@nchaulet nchaulet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ingest Manager/Fleet change LGTM

cjcenizal
cjcenizal approved these changes Dec 1, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@cjcenizal cjcenizal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ES UI plugins LGTM.

@tylersmalley
Fixes integration tests
ae87392 
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
spalger
spalger reviewed Dec 1, 2020
View reviewed changes
@tylersmalley
Review feedback
6296df8 
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
@tylersmalley
Copy link
Copy Markdown
Member Author

tylersmalley commented Dec 1, 2020

@weltenwort, good catch - I have resolved the issue of the test reporter.

weltenwort
weltenwort approved these changes Dec 2, 2020
View reviewed changes
Copy link
Copy Markdown
Member

@weltenwort weltenwort left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

infra plugin changes LGTM

sorenlouv
sorenlouv approved these changes Dec 2, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@sorenlouv sorenlouv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

apm changes lgtm

sorenlouv
sorenlouv reviewed Dec 2, 2020
View reviewed changes
x-pack/test/apm_api_integration/trial/tests/service_maps/__snapshots__/service_maps.snap
@@ -1,1995 +0,0 @@
// Jest Snapshot v1, https://goo.gl/fbAQLP

exports[`Service Maps with a trial license /api/apm/service-map when there is data returns service map elements filtering by environment not defined 1`] = `
Copy link
Copy Markdown
Contributor

@sorenlouv sorenlouv Dec 2, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dgieselaar Does this look okay to you? (removing all the snapshots)

Copy link
Copy Markdown
Member Author

@tylersmalley tylersmalley Dec 2, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jest integration tests donot currently run for X-Pack - I have a draft PR up showing that removing these does not effect the current CI jobs. #84783

pzl
pzl approved these changes Dec 2, 2020
View reviewed changes
Copy link
Copy Markdown
Member

@pzl pzl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Security / Endpoint LGTM

mistic
mistic approved these changes Dec 2, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@mistic mistic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tylersmalley
Copy link
Copy Markdown
Member Author

tylersmalley commented Dec 2, 2020 •
edited
Loading

@elasticmachine merge upstream

All teams affected by more than just adding a jest.config.js file to their plugin have approved this PR. Unless there are any objections, I will be merging this after the next successful CI run. Thanks all. There will be more follow-ups to come to improve Jest further now that we have per-project configurations.

@kibanamachine
Copy link
Copy Markdown
Contributor

kibanamachine commented Dec 2, 2020

💚 Build Succeeded

Metrics [docs]

Distributable file count

id before after diff
default 43196 43325 +129
oss 22576 22644 +68

History

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

@tylersmalley tylersmalley mentioned this pull request Dec 2, 2020
Merged
@trentm trentm mentioned this pull request Dec 2, 2020
Closed
@dmlemeshko dmlemeshko mentioned this pull request Dec 9, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nchaulet nchaulet nchaulet approved these changes

@nreese nreese nreese approved these changes

@flash1293 flash1293 flash1293 approved these changes

@weltenwort weltenwort weltenwort approved these changes

@azasypkin azasypkin azasypkin approved these changes

@pzl pzl pzl approved these changes

@andrewvc andrewvc andrewvc approved these changes

@mikecote mikecote mikecote approved these changes

@mistic mistic mistic approved these changes

@patrykkopycinski patrykkopycinski patrykkopycinski approved these changes

@peteharverson peteharverson peteharverson approved these changes

@vigneshshanmugam vigneshshanmugam Awaiting requested review from vigneshshanmugam

+7 more reviewers

@sorenlouv sorenlouv sorenlouv approved these changes

@spalger spalger spalger approved these changes

@cee-chen cee-chen cee-chen approved these changes

@cjcenizal cjcenizal cjcenizal approved these changes

@chrisronline chrisronline chrisronline approved these changes

@mshustov mshustov mshustov approved these changes

@myasonik myasonik myasonik approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Feature:Drilldowns Embeddable panel Drilldowns Feature:Embedding Embedding content via iFrame Feature:ExpressionLanguage Interpreter expression language (aka canvas pipeline) release_note:skip Skip the PR/issue when compiling release notes Team:APM - DEPRECATED Use Team:obs-ux-infra_services. Team:Fleet Team label for Observability Data Collection Fleet team Team:Operations Kibana-Operations Team Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v7.11.0 v8.0.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[test/dev] support a common jest entry Use Jest muti-project runner instead of a global config [test] run x-pack jest tests from root