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

[QA]Change to skipTourIfExists in console CCS tests instead of closeHelpifExists #199017

Merged
merged 3 commits into from
Nov 5, 2024

Conversation

marius-dr
Copy link
Member

@marius-dr marius-dr commented Nov 5, 2024

Summary

With the change from the ace console to monaco console, the UI has changed and the methods as well. We're now using skipTourIfExists instead of closeHelpIfExists.

@marius-dr marius-dr added test Team:QA Team label for QA Team release_note:skip Skip the PR/issue when compiling release notes test-stack-integration v9.0.0 v8.16.0 v8.17.0 v8.18.0 labels Nov 5, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-qa (Team:QA)

@marius-dr marius-dr added the backport:version Backport to applied version labels label Nov 5, 2024
@marius-dr marius-dr enabled auto-merge (squash) November 5, 2024 16:04
Copy link
Contributor

@bhavyarm bhavyarm left a comment

Choose a reason for hiding this comment

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

LGTM

@marius-dr marius-dr merged commit 2a9b42a into main Nov 5, 2024
22 checks passed
@marius-dr marius-dr deleted the marius-dr-patch-1 branch November 5, 2024 17:51
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.16, 8.18, 8.x

https://github.com/elastic/kibana/actions/runs/11689776869

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

cc @bhavyarm

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 5, 2024
…elpifExists (elastic#199017)

## Summary

With the change from the ace console to monaco console, the UI has
changed and the methods as well. We're now using `skipTourIfExists`
instead of `closeHelpIfExists`.

(cherry picked from commit 2a9b42a)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 5, 2024
…elpifExists (elastic#199017)

## Summary

With the change from the ace console to monaco console, the UI has
changed and the methods as well. We're now using `skipTourIfExists`
instead of `closeHelpIfExists`.

(cherry picked from commit 2a9b42a)
@kibanamachine
Copy link
Contributor

💔 Some backports could not be created

Status Branch Result
8.16
8.18 The branch "8.18" does not exist
8.x

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

Manual backport

To create the backport manually run:

node scripts/backport --pr 199017

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Nov 5, 2024
… closeHelpifExists (#199017) (#199034)

# Backport

This will backport the following commits from `main` to `8.16`:
- [[QA]Change to skipTourIfExists in console CCS tests instead of
closeHelpifExists
(#199017)](#199017)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT [{"author":{"name":"Marius
Dragomir","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-05T17:51:39Z","message":"[QA]Change
to skipTourIfExists in console CCS tests instead of closeHelpifExists
(#199017)\n\n## Summary\r\n\r\nWith the change from the ace console to
monaco console, the UI has\r\nchanged and the methods as well. We're now
using `skipTourIfExists`\r\ninstead of
`closeHelpIfExists`.","sha":"2a9b42a0dbcf6094854194e6e8775e0d4e7a5a18","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","Team:QA","release_note:skip","test-stack-integration","v9.0.0","v8.16.0","backport:version","v8.17.0","v8.18.0"],"title":"[QA]Change
to skipTourIfExists in console CCS tests instead of
closeHelpifExists","number":199017,"url":"https://github.com/elastic/kibana/pull/199017","mergeCommit":{"message":"[QA]Change
to skipTourIfExists in console CCS tests instead of closeHelpifExists
(#199017)\n\n## Summary\r\n\r\nWith the change from the ace console to
monaco console, the UI has\r\nchanged and the methods as well. We're now
using `skipTourIfExists`\r\ninstead of
`closeHelpIfExists`.","sha":"2a9b42a0dbcf6094854194e6e8775e0d4e7a5a18"}},"sourceBranch":"main","suggestedTargetBranches":["8.16","8.x","8.18"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/199017","number":199017,"mergeCommit":{"message":"[QA]Change
to skipTourIfExists in console CCS tests instead of closeHelpifExists
(#199017)\n\n## Summary\r\n\r\nWith the change from the ace console to
monaco console, the UI has\r\nchanged and the methods as well. We're now
using `skipTourIfExists`\r\ninstead of
`closeHelpIfExists`.","sha":"2a9b42a0dbcf6094854194e6e8775e0d4e7a5a18"}},{"branch":"8.16","label":"v8.16.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Marius Dragomir <[email protected]>
kibanamachine added a commit that referenced this pull request Nov 5, 2024
…closeHelpifExists (#199017) (#199035)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[QA]Change to skipTourIfExists in console CCS tests instead of
closeHelpifExists
(#199017)](#199017)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT [{"author":{"name":"Marius
Dragomir","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-05T17:51:39Z","message":"[QA]Change
to skipTourIfExists in console CCS tests instead of closeHelpifExists
(#199017)\n\n## Summary\r\n\r\nWith the change from the ace console to
monaco console, the UI has\r\nchanged and the methods as well. We're now
using `skipTourIfExists`\r\ninstead of
`closeHelpIfExists`.","sha":"2a9b42a0dbcf6094854194e6e8775e0d4e7a5a18","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","Team:QA","release_note:skip","test-stack-integration","v9.0.0","v8.16.0","backport:version","v8.17.0","v8.18.0"],"title":"[QA]Change
to skipTourIfExists in console CCS tests instead of
closeHelpifExists","number":199017,"url":"https://github.com/elastic/kibana/pull/199017","mergeCommit":{"message":"[QA]Change
to skipTourIfExists in console CCS tests instead of closeHelpifExists
(#199017)\n\n## Summary\r\n\r\nWith the change from the ace console to
monaco console, the UI has\r\nchanged and the methods as well. We're now
using `skipTourIfExists`\r\ninstead of
`closeHelpIfExists`.","sha":"2a9b42a0dbcf6094854194e6e8775e0d4e7a5a18"}},"sourceBranch":"main","suggestedTargetBranches":["8.16","8.x","8.18"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/199017","number":199017,"mergeCommit":{"message":"[QA]Change
to skipTourIfExists in console CCS tests instead of closeHelpifExists
(#199017)\n\n## Summary\r\n\r\nWith the change from the ace console to
monaco console, the UI has\r\nchanged and the methods as well. We're now
using `skipTourIfExists`\r\ninstead of
`closeHelpIfExists`.","sha":"2a9b42a0dbcf6094854194e6e8775e0d4e7a5a18"}},{"branch":"8.16","label":"v8.16.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Marius Dragomir <[email protected]>
mgadewoll pushed a commit to mgadewoll/kibana that referenced this pull request Nov 7, 2024
…elpifExists (elastic#199017)

## Summary

With the change from the ace console to monaco console, the UI has
changed and the methods as well. We're now using `skipTourIfExists`
instead of `closeHelpIfExists`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:QA Team label for QA Team test test-stack-integration v8.16.0 v8.17.0 v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants