(airflow) branch main updated: Remove non-essential sorting xcom assertions in E2E tests (#65931)

[choo121600]

This is an automated email from the ASF dual-hosted git repository.

choo121600 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new fe2e1e2ee56 Remove non-essential sorting xcom assertions in E2E tests 
(#65931)
fe2e1e2ee56 is described below

commit fe2e1e2ee56df3df59f89679a2d72e54e96b1a82
Author: Yeonguk Choo <choo121...@gmail.com>
AuthorDate: Mon Apr 27 15:21:55 2026 +0900

    Remove non-essential sorting xcom assertions in E2E tests (#65931)
---
 .../src/airflow/ui/tests/e2e/pages/XComsPage.ts    | 26 ----------------------
 .../src/airflow/ui/tests/e2e/specs/xcoms.spec.ts   |  8 -------
 2 files changed, 34 deletions(-)

diff --git a/airflow-core/src/airflow/ui/tests/e2e/pages/XComsPage.ts 
b/airflow-core/src/airflow/ui/tests/e2e/pages/XComsPage.ts
index a18b4a2fd77..a0e9b352389 100644
--- a/airflow-core/src/airflow/ui/tests/e2e/pages/XComsPage.ts
+++ b/airflow-core/src/airflow/ui/tests/e2e/pages/XComsPage.ts
@@ -125,32 +125,6 @@ export class XComsPage extends BasePage {
     }
   }
 
-  public async verifySortByColumn(columnName: string): Promise<void> {
-    await this.navigate();
-
-    const columnHeader = this.xcomsTable.getByRole("columnheader", { name: 
columnName });
-
-    await expect(columnHeader).toBeVisible({ timeout: 10_000 });
-
-    const firstSortResponse = this.page.waitForResponse(
-      (response) => response.url().includes("xcomEntries") && response.ok(),
-      { timeout: 10_000 },
-    );
-
-    await columnHeader.click();
-    await firstSortResponse;
-    await expect(this.page).toHaveURL(/sorting/);
-
-    const secondSortResponse = this.page.waitForResponse(
-      (response) => response.url().includes("xcomEntries") && response.ok(),
-      { timeout: 10_000 },
-    );
-
-    await columnHeader.click();
-    await secondSortResponse;
-    await expect(this.page).toHaveURL(/sorting/);
-  }
-
   public async verifyXComDetailsDisplay(): Promise<void> {
     const firstRow = this.tableRows.first();
 
diff --git a/airflow-core/src/airflow/ui/tests/e2e/specs/xcoms.spec.ts 
b/airflow-core/src/airflow/ui/tests/e2e/specs/xcoms.spec.ts
index dc3281652dc..19fdb0bee77 100644
--- a/airflow-core/src/airflow/ui/tests/e2e/specs/xcoms.spec.ts
+++ b/airflow-core/src/airflow/ui/tests/e2e/specs/xcoms.spec.ts
@@ -57,12 +57,4 @@ test.describe("XComs Page", () => {
   test("verify filtering by DAG display name", async ({ xcomRunsData, 
xcomsPage }) => {
     await xcomsPage.verifyDagDisplayNameFiltering(xcomRunsData.dagId);
   });
-
-  test("verify sorting by key column", async ({ xcomsPage }) => {
-    await xcomsPage.verifySortByColumn("Key");
-  });
-
-  test("verify sorting by timestamp column", async ({ xcomsPage }) => {
-    await xcomsPage.verifySortByColumn("Timestamp");
-  });
 });