From d6628752ada153c77c2f367e7bd22b66ef18e421 Mon Sep 17 00:00:00 2001 From: Shrinath Rao Date: Sat, 24 Oct 2026 05:49:38 +0000 Subject: [PATCH] test(e2e): don't wait for streamed search responses to finish (#15735) --- .../pages/dashboardPages/dashboard-panel-actions.js | 6 +++++- tests/ui-testing/pages/logsPages/logsPage.js | 5 ++++- 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/tests/ui-testing/pages/dashboardPages/dashboard-panel-actions.js b/tests/ui-testing/pages/dashboardPages/dashboard-panel-actions.js index de68c4f2bd..df5b9f2b7f 100644 --- a/tests/ui-testing/pages/dashboardPages/dashboard-panel-actions.js +++ b/tests/ui-testing/pages/dashboardPages/dashboard-panel-actions.js @@ -338,7 +338,11 @@ export default class DashboardactionPage { { timeout: 30000 } ); await this.applyDashboardBtn(); - await (await promqlResponse).finished(); + const response = await promqlResponse; + // A streamed (text/event-stream) response never reports finished; waitForChartToRender covers completion. + if (!(await response.headerValue("content-type"))?.includes("text/event-stream")) { + await response.finished(); + } await this.waitForChartToRender(); } diff --git a/tests/ui-testing/pages/logsPages/logsPage.js b/tests/ui-testing/pages/logsPages/logsPage.js index a9d79a6602..8e036f6de2 100644 --- a/tests/ui-testing/pages/logsPages/logsPage.js +++ b/tests/ui-testing/pages/logsPages/logsPage.js @@ -13382,7 +13382,10 @@ export class LogsPage { ), this.page.locator(this.searchBarRefreshButton).click(), ]); - await response.finished(); + // A streamed (text/event-stream) response never reports finished; the Run button wait covers completion. + if (!(await response.headerValue('content-type'))?.includes('text/event-stream')) { + await response.finished(); + } await this.waitForRunQueryButton(timeout); await expect(this.resultsTable()).toBeVisible({ timeout }); }