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

[Search][FTR] Solution Nav #195327

Conversation

TattdCodeMonkey
Copy link
Contributor

@TattdCodeMonkey TattdCodeMonkey commented Oct 7, 2024

Summary

Adding functional_search suite with a set of test for the search solution navigation. But this suite will also grow to test search solution pages that do not require the enterprise search node.

Checklist

@TattdCodeMonkey TattdCodeMonkey added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 Team:Search backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Oct 7, 2024
@TattdCodeMonkey TattdCodeMonkey requested review from a team as code owners October 7, 2024 22:09
@TattdCodeMonkey TattdCodeMonkey linked an issue Oct 7, 2024 that may be closed by this pull request
@TattdCodeMonkey TattdCodeMonkey force-pushed the solution_sidenav/search-functional-tests branch from b5d108b to 1e90725 Compare October 8, 2024 13:42
Adding the function_search suite with a test for the new search solution
nav. But this suite will also be a place where we can test search
functionality not tied to the enterprise_search node.
@TattdCodeMonkey TattdCodeMonkey force-pushed the solution_sidenav/search-functional-tests branch from 1e90725 to d956117 Compare October 8, 2024 14:15
@TattdCodeMonkey TattdCodeMonkey requested a review from a team as a code owner October 8, 2024 14:15
Copy link
Contributor

@sebelga sebelga left a comment

Choose a reason for hiding this comment

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

LGTM!

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🎉 All tests passed! - kibana-flaky-test-suite-runner#7108

[✅] x-pack/test/functional_search/config.ts: 25/25 tests passed.

see run history

@elasticmachine
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
enterpriseSearch 2.6MB 2.6MB +162.0B
Unknown metric groups

ESLint disabled in files

id before after diff
@kbn/test-suites-xpack 24 25 +1

Total ESLint disabled count

id before after diff
@kbn/test-suites-xpack 746 747 +1

History

  • 💛 Build #240064 was flaky b5d108bcce0e28a9a40ab96b674feea0c1fa6513

Copy link
Member

@jbudz jbudz left a comment

Choose a reason for hiding this comment

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

.buildkite/ftr_platform_stateful_configs.yml

@TattdCodeMonkey TattdCodeMonkey merged commit 71fd96a into elastic:main Oct 9, 2024
21 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

@TattdCodeMonkey
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.x

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

Questions ?

Please refer to the Backport tool documentation

TattdCodeMonkey added a commit to TattdCodeMonkey/kibana that referenced this pull request Oct 9, 2024
## Summary

Adding `functional_search` suite with a set of test for the search
solution navigation. But this suite will also grow to test search
solution pages that do not require the enterprise search node.

### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [x] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed

(cherry picked from commit 71fd96a)

# Conflicts:
#	.github/CODEOWNERS
TattdCodeMonkey added a commit that referenced this pull request Oct 9, 2024
# Backport

This will backport the following commits from `main` to `8.x`:
- [[Search][FTR] Solution Nav
(#195327)](#195327)

<!--- Backport version: 8.9.8 -->

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

<!--BACKPORT [{"author":{"name":"Rodney
Norris","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-09T13:45:24Z","message":"[Search][FTR]
Solution Nav (#195327)\n\n## Summary\r\n\r\nAdding `functional_search`
suite with a set of test for the search\r\nsolution navigation. But this
suite will also grow to test search\r\nsolution pages that do not
require the enterprise search node.\r\n\r\n### Checklist\r\n\r\n- [x]
[Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common scenarios\r\n- [x] [Flaky
Test\r\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1)
was\r\nused on any tests
changed","sha":"71fd96ad8b8cd06ac3c30f099895f2723cd6f3bc","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Search","backport:prev-minor"],"number":195327,"url":"https://github.com/elastic/kibana/pull/195327","mergeCommit":{"message":"[Search][FTR]
Solution Nav (#195327)\n\n## Summary\r\n\r\nAdding `functional_search`
suite with a set of test for the search\r\nsolution navigation. But this
suite will also grow to test search\r\nsolution pages that do not
require the enterprise search node.\r\n\r\n### Checklist\r\n\r\n- [x]
[Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common scenarios\r\n- [x] [Flaky
Test\r\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1)
was\r\nused on any tests
changed","sha":"71fd96ad8b8cd06ac3c30f099895f2723cd6f3bc"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/195327","number":195327,"mergeCommit":{"message":"[Search][FTR]
Solution Nav (#195327)\n\n## Summary\r\n\r\nAdding `functional_search`
suite with a set of test for the search\r\nsolution navigation. But this
suite will also grow to test search\r\nsolution pages that do not
require the enterprise search node.\r\n\r\n### Checklist\r\n\r\n- [x]
[Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common scenarios\r\n- [x] [Flaky
Test\r\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1)
was\r\nused on any tests
changed","sha":"71fd96ad8b8cd06ac3c30f099895f2723cd6f3bc"}}]}]
BACKPORT-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes Team:Search v8.16.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Search stateful sidenav] Add functional tests
6 participants