test: Fix Service Dashboards Scout wait (strict progressbar locator)

[jennypavlova]

Summary

• Replaced getByRole('progressbar') in DashboardsTab.waitForDashboardsToLoad with a wait for apmMainTemplateServiceAgentLoader to be hidden. The generic progressbar role matched multiple concurrent EUI loaders (chrome, environment filter, service title, etc.) and caused Playwright strict mode failures.
• Extended waitForTabLoad on the dashboards tab to wait for the same main template loader after the unified search bar is visible, matching MetricsTab / ServiceDetailsPage.goToPage and stabilizing navigation before the unlink test.

Testing

• Ran service_dashboards.spec.ts locally — passed. Expect serverless observability Scout (local-serverless-observability_complete) to validate after CI merges.

Cc flaky Scout/serverless monitoring: worth confirming stability once merged.

Closes #256350

Made with Cursor

[infra-vault-gh-plugin-prod]

Pinging @elastic/obs-presentation-team (Team:obs-presentation)

[macroscopeapp]

Approvability

Verdict: Would Approve

Test-only change that replaces a generic progressbar locator with a more specific test ID locator for improved test stability. No production code or runtime behavior is affected.

^{Macroscope would have approved this PR. A repo admin can enable approvability here.}

[macroscopeapp]

Catch flakiness early (recommended): run the flaky test runner against this PR before merging.

This PR changes wait/timing logic in a shared Scout page object (DashboardsTab), replacing a generic progressbar role check with a waitFor on a specific test ID. Since the fix addresses strict-mode timing issues, a 30-run pass would help confirm stability.

Trigger a run with the Flaky Test Runner UI or post this comment on the PR:

/flaky scoutConfig:x-pack/solutions/observability/plugins/apm/test/scout/ui/parallel.playwright.config.ts:30

^{Share feedback in the #appex-qa channel.}

^{Posted via Macroscope — Flaky Test Runner nudge}

[jennypavlova]

/flaky scoutConfig:x-pack/solutions/observability/plugins/apm/test/scout/ui/parallel.playwright.config.ts:30

[kibanamachine]

Flaky Test Runner

✅ Build triggered - kibana-flaky-test-suite-runner#12118

• x-pack/solutions/observability/plugins/apm/test/scout/ui/parallel.playwright.config.ts x30

[kibanamachine]

Flaky Test Runner Stats

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

[❌] x-pack/solutions/observability/plugins/apm/test/scout/ui/parallel.playwright.config.ts (--arch stateful --domain classic): 28/30 tests passed.
[✅] x-pack/solutions/observability/plugins/apm/test/scout/ui/parallel.playwright.config.ts (--arch serverless --domain observability_complete): 30/30 tests passed.

see run history

[jennypavlova]

/flaky scoutConfig:x-pack/solutions/observability/plugins/apm/test/scout/ui/parallel.playwright.config.ts:30

[kibanamachine]

Flaky Test Runner

✅ Build triggered - kibana-flaky-test-suite-runner#12130

• x-pack/solutions/observability/plugins/apm/test/scout/ui/parallel.playwright.config.ts x30

[sbelastic]

LGTM

[kibanamachine]

💛 Build succeeded, but was flaky

• Buildkite Build
• Commit: 3c1ab74

Failed CI Steps

• FTR Configs #153

Test Failures

• [job] [logs] FTR Configs #153 / Cloud Security Posture POST /internal/cloud_security_posture/graph Happy flows Pinning should pin an event by document id and pin related entities

Metrics [docs]

✅ unchanged

History

• 💛 Build #438316 was flaky 534eeef

cc @jennypavlova

[kibanamachine]

Flaky Test Runner Stats

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

[✅] x-pack/solutions/observability/plugins/apm/test/scout/ui/parallel.playwright.config.ts (--arch stateful --domain classic): 30/30 tests passed.
[✅] x-pack/solutions/observability/plugins/apm/test/scout/ui/parallel.playwright.config.ts (--arch serverless --domain observability_complete): 30/30 tests passed.

see run history