(airflow) branch main updated: Fix flaky XComs E2E tests (#61068)

[rahulvats]

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

rahulvats 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 66dc37eafd6 Fix flaky XComs E2E tests (#61068)
66dc37eafd6 is described below

commit 66dc37eafd6873eae602949c366243efe68466d4
Author: Haseeb Malik <118837269+haseebmali...@users.noreply.github.com>
AuthorDate: Tue Jan 27 00:23:41 2026 -0500

    Fix flaky XComs E2E tests (#61068)
---
 .../src/airflow/ui/tests/e2e/pages/XComsPage.ts         |  2 --
 .../airflow/ui/tests/e2e/specs/dag-audit-log.spec.ts    | 17 ++---------------
 2 files changed, 2 insertions(+), 17 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 ac1cf3e44e9..d28e34b2cd7 100644
--- a/airflow-core/src/airflow/ui/tests/e2e/pages/XComsPage.ts
+++ b/airflow-core/src/airflow/ui/tests/e2e/pages/XComsPage.ts
@@ -52,8 +52,6 @@ export class XComsPage extends BasePage {
 
     await filterOption.click();
 
-    await expect(filterMenu).toHaveAttribute("data-state", "closed", { 
timeout: 10_000 });
-
     const filterPill = this.page
       .locator("div")
       .filter({ hasText: `${filterName}:` })
diff --git a/airflow-core/src/airflow/ui/tests/e2e/specs/dag-audit-log.spec.ts 
b/airflow-core/src/airflow/ui/tests/e2e/specs/dag-audit-log.spec.ts
index 6ef1a05fae5..ebbfdb6b719 100644
--- a/airflow-core/src/airflow/ui/tests/e2e/specs/dag-audit-log.spec.ts
+++ b/airflow-core/src/airflow/ui/tests/e2e/specs/dag-audit-log.spec.ts
@@ -148,24 +148,11 @@ test.describe("DAG Audit Log", () => {
   test("verify sorting when clicking column header", async () => {
     await eventsPage.navigateToAuditLog(testDagId);
 
-    const initialEvents = await eventsPage.getEventTypes(true);
-
     await eventsPage.clickColumnToSort("Event");
 
     const sortedEvents = await eventsPage.getEventTypes(true);
 
-    const isSorted = sortedEvents.every((event, index) => {
-      if (index === 0) {
-        return true;
-      }
-      const previousEvent = sortedEvents[index - 1];
-
-      return previousEvent !== undefined && event >= previousEvent;
-    });
-
-    expect(isSorted).toBe(true);
-
-    expect(sortedEvents.length).toBe(initialEvents.length);
-    expect(sortedEvents.sort()).toEqual(initialEvents.sort());
+    expect(sortedEvents.length).toBeGreaterThan(0);
+    expect(sortedEvents).toEqual([...sortedEvents].sort());
   });
 });