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

fix Failing test: X-Pack Saved Object Tagging Functional Tests.x-pack/test/saved_object_tagging/functional/tests/dashboard_integration·ts - saved objects tagging - functional tests dashboard integration creating allows to select tags for a new dashboard #160687

Merged
merged 2 commits into from
Jun 28, 2023

Conversation

nreese
Copy link
Contributor

@nreese nreese commented Jun 27, 2023

Closes #160583

flaky test runner https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/2513

Harden dashboard_page.enterDashboardTitleAndClickSave for retry loops so does not try to open save dialog when its already open from first save attempt failure.

nreese and others added 2 commits June 27, 2023 15:58
…/test/saved_object_tagging/functional/tests/dashboard_integration·ts - saved objects tagging - functional tests dashboard integration creating allows to select tags for a new dashboard
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 14 16 +2
securitySolution 413 417 +4
total +6

Total ESLint disabled count

id before after diff
enterpriseSearch 15 17 +2
securitySolution 492 496 +4
total +6

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

@nreese nreese marked this pull request as ready for review June 28, 2023 00:57
@nreese nreese added Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas release_note:skip Skip the PR/issue when compiling release notes v8.9.0 v8.10.0 labels Jun 28, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

Copy link
Contributor

@ThomThomson ThomThomson left a comment

Choose a reason for hiding this comment

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

Thank you for the detailed explanation in #160583 (comment)

These changes LGTM, it's often better to check with the dom to drive changes in behaviour than to take a boolean as an argument!

@nreese nreese merged commit ef7fda2 into elastic:main Jun 28, 2023
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jun 28, 2023
…/test/saved_object_tagging/functional/tests/dashboard_integration·ts - saved objects tagging - functional tests dashboard integration creating allows to select tags for a new dashboard (elastic#160687)

Closes elastic#160583

flaky test runner
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/2513

Harden `dashboard_page.enterDashboardTitleAndClickSave` for retry loops
so does not try to open save dialog when its already open from first
save attempt failure.

---------

Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit ef7fda2)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.9

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Jun 28, 2023
…x-pack/test/saved_object_tagging/functional/tests/dashboard_integration·ts - saved objects tagging - functional tests dashboard integration creating allows to select tags (#160782)

# Backport

This will backport the following commits from `main` to `8.9`:
- [fix Failing test: X-Pack Saved Object Tagging Functional
Tests.x-pack/test/saved_object_tagging/functional/tests/dashboard_integration·ts
- saved objects tagging - functional tests dashboard integration
creating allows to select tags for a new dashboard
(#160687)](#160687)

<!--- Backport version: 8.9.7 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Nathan
Reese","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-06-28T16:14:50Z","message":"fix
Failing test: X-Pack Saved Object Tagging Functional
Tests.x-pack/test/saved_object_tagging/functional/tests/dashboard_integration·ts
- saved objects tagging - functional tests dashboard integration
creating allows to select tags for a new dashboard (#160687)\n\nCloses
https://github.com/elastic/kibana/issues/160583\r\n\r\nflaky test
runner\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/2513\r\n\r\nHarden
`dashboard_page.enterDashboardTitleAndClickSave` for retry loops\r\nso
does not try to open save dialog when its already open from
first\r\nsave attempt failure.\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"ef7fda2d249f47a86b8e930b577b47990fb334ad","branchLabelMapping":{"^v8.10.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Presentation","release_note:skip","v8.9.0","v8.10.0"],"number":160687,"url":"https://github.com/elastic/kibana/pull/160687","mergeCommit":{"message":"fix
Failing test: X-Pack Saved Object Tagging Functional
Tests.x-pack/test/saved_object_tagging/functional/tests/dashboard_integration·ts
- saved objects tagging - functional tests dashboard integration
creating allows to select tags for a new dashboard (#160687)\n\nCloses
https://github.com/elastic/kibana/issues/160583\r\n\r\nflaky test
runner\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/2513\r\n\r\nHarden
`dashboard_page.enterDashboardTitleAndClickSave` for retry loops\r\nso
does not try to open save dialog when its already open from
first\r\nsave attempt failure.\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"ef7fda2d249f47a86b8e930b577b47990fb334ad"}},"sourceBranch":"main","suggestedTargetBranches":["8.9"],"targetPullRequestStates":[{"branch":"8.9","label":"v8.9.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.10.0","labelRegex":"^v8.10.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/160687","number":160687,"mergeCommit":{"message":"fix
Failing test: X-Pack Saved Object Tagging Functional
Tests.x-pack/test/saved_object_tagging/functional/tests/dashboard_integration·ts
- saved objects tagging - functional tests dashboard integration
creating allows to select tags for a new dashboard (#160687)\n\nCloses
https://github.com/elastic/kibana/issues/160583\r\n\r\nflaky test
runner\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/2513\r\n\r\nHarden
`dashboard_page.enterDashboardTitleAndClickSave` for retry loops\r\nso
does not try to open save dialog when its already open from
first\r\nsave attempt failure.\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"ef7fda2d249f47a86b8e930b577b47990fb334ad"}}]}]
BACKPORT-->

Co-authored-by: Nathan Reese <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas v8.9.0 v8.10.0
Projects
None yet
5 participants