Skip to content

Update Index Management Tests to Check 4 Tabs on the Home Page#113692

Merged
cuff-links merged 3 commits intoelastic:masterfrom
cuff-links:Update_Index_Management_CIT_For_Correct_Number_Of_Tabs
Oct 5, 2021
Merged

Update Index Management Tests to Check 4 Tabs on the Home Page#113692
cuff-links merged 3 commits intoelastic:masterfrom
cuff-links:Update_Index_Management_CIT_For_Correct_Number_Of_Tabs

Conversation

@cuff-links
Copy link
Contributor

Summary

This Pull Request is a follow up from #112010. It splits out the test to check for index management tabs. It was updated to do check all 4 tabs in index management from the previous 2.

@cuff-links cuff-links added Feature:Index Management Index and index templates UI test-jest-integration v8.0.0 Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t// release_note:skip Skip the PR/issue when compiling release notes auto-backport Deprecated - use backport:version if exact versions are needed v7.15.1 labels Oct 2, 2021
@cuff-links cuff-links requested a review from a team as a code owner October 2, 2021 02:52
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-stack-management (Team:Stack Management)

Copy link
Contributor

@yuliacech yuliacech left a comment

Choose a reason for hiding this comment

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

Thanks a lot for fixing those tests, @cuff-links! Code changes LGTM 👍

@cuff-links
Copy link
Contributor Author

@elaticmachine merge upstream

@cuff-links
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
indexManagement 525.5KB 525.6KB +81.0B

History

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

@cuff-links cuff-links merged commit 2dd01c0 into elastic:master Oct 5, 2021
@kibanamachine
Copy link
Contributor

The following labels were identified as gaps in your version labels and will be added automatically:

  • v7.16.0

If any of these should not be on your pull request, please manually remove them.

kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Oct 5, 2021
…ic#113692)

* Modified home page test to check for 4 tabs.

* Fixed lint issue.

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Oct 5, 2021
…ic#113692)

* Modified home page test to check for 4 tabs.

* Fixed lint issue.

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
7.x
7.15

The backport PRs will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Oct 5, 2021
…) (#113996)

* Modified home page test to check for 4 tabs.

* Fixed lint issue.

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: John Dorlus <silne.dorlus@elastic.co>
kibanamachine added a commit that referenced this pull request Oct 5, 2021
…) (#113995)

* Modified home page test to check for 4 tabs.

* Fixed lint issue.

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: John Dorlus <silne.dorlus@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Deprecated - use backport:version if exact versions are needed Feature:Index Management Index and index templates UI release_note:skip Skip the PR/issue when compiling release notes Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t// test-jest-integration v7.15.1 v7.16.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants