Skip to content

[Search] Replace flaky FTR tests with Scout tests for Search Homepage#248381

Merged
saikatsarkar056 merged 28 commits intoelastic:mainfrom
saikatsarkar056:scout_for_homepage
Jan 15, 2026
Merged

[Search] Replace flaky FTR tests with Scout tests for Search Homepage#248381
saikatsarkar056 merged 28 commits intoelastic:mainfrom
saikatsarkar056:scout_for_homepage

Conversation

@saikatsarkar056
Copy link
Contributor

@saikatsarkar056 saikatsarkar056 commented Jan 9, 2026

Summary

This PR migrates the FTR (Functional Test Runner) tests for the Search Homepage to Scout tests.

Background

The existing FTR tests for Search Homepage have been experiencing flakiness issues (see #226572), with several tests being skipped. This migration to Scout provides a more stable and maintainable test infrastructure.

How to Run Search Homepage Scout Tests

Prerequisites

Start the appropriate server first:

For Stateful (ESS)

node scripts/scout.js start-server --stateful

For Serverless

node scripts/scout.js start-server --serverless=es

Run all Search Homepage tests with browser visible

npx playwright test --config x-pack/solutions/search/plugins/search_homepage/test/scout/ui/parallel.playwright.config.ts --project local --headed --grep "Search Homepage"

Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

  • Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
  • Documentation was added for features that require explanation or tutorials
  • Unit or functional tests were updated or added to match the most common scenarios
  • If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the docker list
  • This was checked for breaking HTTP API changes, and any breaking changes have been approved by the breaking-change committee. The release_note:breaking label should be applied in these situations.
  • Flaky Test Runner was used on any tests changed
  • The PR description includes the appropriate Release Notes section, and the correct release_note:* label is applied per the guidelines
  • Review the backport guidelines and apply applicable backport:* labels.

Identify risks

Does this PR introduce any risks? For example, consider risks like hard to test bugs, performance regression, potential of data loss.

Describe the risk, its severity, and mitigation for each identified risk. Invite stakeholders and evaluate how to proceed before merging.

- Create search_homepage.spec.ts with tests migrated from FTR
- Add embedded console methods to homepage page object
- Use uiSettings to disable V2 flag for testing V1 homepage
- Tests: embedded dev console, page load verification
@saikatsarkar056 saikatsarkar056 requested a review from a team as a code owner January 9, 2026 04:53
@saikatsarkar056 saikatsarkar056 marked this pull request as draft January 9, 2026 04:53
@saikatsarkar056 saikatsarkar056 changed the title Scout for homepage [Search] Replace flaky FTR tests with Scout tests for Search Homepage Jan 9, 2026
@saikatsarkar056 saikatsarkar056 self-assigned this Jan 9, 2026
@saikatsarkar056 saikatsarkar056 added backport:skip This PR does not require backporting v9.4.0 Team:Search release_note:skip Skip the PR/issue when compiling release notes labels Jan 9, 2026
@saikatsarkar056 saikatsarkar056 marked this pull request as ready for review January 9, 2026 18:49
@saikatsarkar056
Copy link
Contributor Author

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🟠 Some tests failed. - kibana-flaky-test-suite-runner#10390

[❌] x-pack/solutions/search/plugins/search_homepage/test/scout/ui/parallel.playwright.config.ts: 0/200 tests passed.

see run history

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🟠 Some tests failed. - kibana-flaky-test-suite-runner#10391

[❌] x-pack/solutions/search/plugins/search_homepage/test/scout/ui/parallel.playwright.config.ts: 0/200 tests passed.

see run history

@saikatsarkar056
Copy link
Contributor Author

Flaky test runner was failing due to an invalid CLI flag. This PR fixes the issue.

@saikatsarkar056
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

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

[✅] x-pack/solutions/search/plugins/search_homepage/test/scout/ui/parallel.playwright.config.ts: 15/15 tests passed.

see run history

@saikatsarkar056
Copy link
Contributor Author

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

There are no new commits on the base branch.


test.describe('Search Homepage (V1)', { tag: ['@ess', '@svlSearch'] }, () => {
// Disable the V2 homepage to test the old (V1) homepage
test.beforeAll(async ({ kbnClient, esClient }) => {
Copy link
Member

Choose a reason for hiding this comment

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

Hey @saikatsarkar056

We are not going to re-enable v1 homepage. I think the ticket was not ready 100% 😬 But what we should do is instead, take a look at the flows from skipped homepage tests and implement similar ones on the new homepage where it makes sense.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@efegurkan You are right. Let me modify this PR to cover only relevant changes needed for new homepage.

@saikatsarkar056
Copy link
Contributor Author

@efegurkan Could you please review the PR for another round? Thank you.

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

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

[✅] x-pack/solutions/search/plugins/search_homepage/test/scout/ui/parallel.playwright.config.ts: 15/15 tests passed.

see run history

@saikatsarkar056
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

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

[✅] x-pack/solutions/search/plugins/search_homepage/test/scout/ui/parallel.playwright.config.ts: 15/15 tests passed.

see run history

@saikatsarkar056 saikatsarkar056 enabled auto-merge (squash) January 14, 2026 15:00
@saikatsarkar056
Copy link
Contributor Author

/ci

auto-merge was automatically disabled January 14, 2026 15:21

Pull Request is not mergeable

@saikatsarkar056
Copy link
Contributor Author

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

expected head sha didn’t match current head ref.

@saikatsarkar056
Copy link
Contributor Author

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

cc @saikatsarkar056

@saikatsarkar056 saikatsarkar056 merged commit 36b6632 into elastic:main Jan 15, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Search v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants