Skip to content

[Unit Testing] Configure react-testing-library queries to use Kibana's data-test-subj instead of default data-testid#59445

Merged
Dosant merged 14 commits intoelastic:masterfrom
Dosant:dev/configure-react-testing-library
Mar 10, 2020
Merged

[Unit Testing] Configure react-testing-library queries to use Kibana's data-test-subj instead of default data-testid#59445
Dosant merged 14 commits intoelastic:masterfrom
Dosant:dev/configure-react-testing-library

Conversation

@Dosant
Copy link
Copy Markdown
Contributor

@Dosant Dosant commented Mar 5, 2020 •
edited
Loading

Summary

I was working on a bunch of react components and noticed we don't have react-testing-library configured properly.

Originally I tried to add this configuration in this pr, but after moving components to x-pack realised that unrelated code changes have to be done to make it pass. So extracted to separate pr

  • This pr configures testing-library to use Kibana's data-test-subj instead of data-testid for it's assertions. Cleans up occasional places where testid was used or where subj workaround was introduced
  • There are couple problems with our current setup which I had to workaround right now. I think I discovered a larger problem which is contributing into jest unit tests flakiness: PLEASE SEE: [Unit Testing][Flaky] Jest suite and react-testing-library registration race condition #59469

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@Dosant Dosant mentioned this pull request Mar 5, 2020
Closed
@Dosant
Copy link
Copy Markdown
Contributor Author

Dosant commented Mar 6, 2020

@elasticmachine merge upstream

@Dosant Dosant added v8.0.0 v7.7.0 release_note:skip Skip the PR/issue when compiling release notes Feature:Unit Testing labels Mar 6, 2020
@Dosant Dosant marked this pull request as ready for review March 6, 2020 05:51
@Dosant Dosant requested a review from a team March 6, 2020 05:51
@Dosant Dosant requested review from a team as code owners March 6, 2020 05:51
@Dosant Dosant changed the title [wip] try configure react testing library [Unit Testing] Configure react-testing-library queries to use Kibana's data-test-subj instead of default data-testid Mar 6, 2020
@Dosant Dosant mentioned this pull request Mar 6, 2020
Merged
7 tasks
walterra
walterra approved these changes Mar 6, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@walterra walterra left a comment

Choose a reason for hiding this comment

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

Oh, that's a nice fix, thanks for doing this! ML changes LGTM 🎉

smith
smith approved these changes Mar 6, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@smith smith 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 look good. Thanks!

@Dosant
Copy link
Copy Markdown
Contributor Author

Dosant commented Mar 6, 2020

@elasticmachine merge upstream

@elasticmachine elasticmachine requested a review from a team as a code owner March 6, 2020 18:50
oatkiller
oatkiller approved these changes Mar 6, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@oatkiller oatkiller left a comment

Choose a reason for hiding this comment

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

Thanks!

mistic
mistic reviewed Mar 6, 2020
View reviewed changes
@Dosant Dosant requested a review from mistic March 9, 2020 12:03
streamich
streamich approved these changes Mar 9, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@streamich streamich left a comment

Choose a reason for hiding this comment

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

nice

mistic
mistic approved these changes Mar 10, 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

@Dosant
Copy link
Copy Markdown
Contributor Author

Dosant commented Mar 10, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

kibanamachine commented Mar 10, 2020

💚 Build Succeeded

History

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

@Dosant Dosant merged commit 8a3e9c6 into elastic:master Mar 10, 2020
Dosant added a commit to Dosant/kibana that referenced this pull request Mar 10, 2020
@Dosant
[Unit Testing] Configure react-testing-library queries to use Kibana'…
edd4751 
…s data-test-subj instead of default data-testid (elastic#59445)

# Conflicts:
#	x-pack/plugins/endpoint/public/applications/endpoint/view/managing/index.test.tsx
@Dosant Dosant mentioned this pull request Mar 10, 2020
Merged
Dosant added a commit that referenced this pull request Mar 10, 2020
@Dosant
[Unit Testing] Configure react-testing-library queries to use Kibana'…
958d733 
…s data-test-subj instead of default data-testid (#59445) (#59742)

# Conflicts:
#	x-pack/plugins/endpoint/public/applications/endpoint/view/managing/index.test.tsx
gmmorris added a commit to gmmorris/kibana that referenced this pull request Mar 10, 2020
@gmmorris
Merge branch 'master' into alerting/view-in-app
dd5ae27 
* master:
  Add a retry to dashboard test for a sometimes slow async operation (elastic#59600)
  [Endpoint] Sample data generator for endpoint app (elastic#58936)
  [Vis Editor] Refactoring metrics axes (elastic#59135)
  [DOCS] Changed Discover app to Discover (elastic#59769)
  [Metrics Alerts] Add support for search query and groupBy on alerts (elastic#59388)
  Enhancement - EUICodeEditor for Visualize JSON  (elastic#58679)
  [ML] Transforms: Data grid fixes. (elastic#59538)
  [SIEM] Fix and consolidate handling of error responses in the client (elastic#59438)
  [Maps] convert tooltip classes to typescript (elastic#59589)
  [ML] Functional tests - re-activate date_nanos test (elastic#59649)
  Move canvas to use NP Expressions service (elastic#58387)
  Update misc dependencies (elastic#59542)
  [Unit Testing] Configure react-testing-library queries to use Kibana's data-test-subj instead of default data-testid (elastic#59445)
  [Console] Remove unused code (elastic#59554)
  [Logs / Metrics UI] Link handling / stop page reloads (elastic#58478)
  Add SavedObject management section registration in core  (elastic#59291)
@Dosant Dosant mentioned this pull request Sep 29, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@smith smith smith approved these changes

@walterra walterra walterra approved these changes

@mistic mistic mistic approved these changes

+2 more reviewers

@oatkiller oatkiller oatkiller approved these changes

@streamich streamich streamich approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Feature:Unit Testing release_note:skip Skip the PR/issue when compiling release notes v7.7.0 v8.0.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@Dosant @kibanamachine @smith @oatkiller @walterra @mistic @streamich @elasticmachine