(airflow) branch main updated: Add data-testid to XCom value for stable E2E selectors (#65944)

[bbovenzi]

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

bbovenzi 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 5bcb6aeb16e Add data-testid to XCom value for stable E2E selectors 
(#65944)
5bcb6aeb16e is described below

commit 5bcb6aeb16e6094fafe6294ccfb2d644a6e83598
Author: Yeonguk Choo <choo121...@gmail.com>
AuthorDate: Mon Apr 27 22:58:38 2026 +0900

    Add data-testid to XCom value for stable E2E selectors (#65944)
---
 airflow-core/src/airflow/ui/src/pages/XCom/XComEntry.tsx | 2 +-
 airflow-core/src/airflow/ui/tests/e2e/pages/XComsPage.ts | 8 +-------
 2 files changed, 2 insertions(+), 8 deletions(-)

diff --git a/airflow-core/src/airflow/ui/src/pages/XCom/XComEntry.tsx 
b/airflow-core/src/airflow/ui/src/pages/XCom/XComEntry.tsx
index a51d07808b4..ba014a60df6 100644
--- a/airflow-core/src/airflow/ui/src/pages/XCom/XComEntry.tsx
+++ b/airflow-core/src/airflow/ui/src/pages/XCom/XComEntry.tsx
@@ -87,7 +87,7 @@ export const XComEntry = ({ dagId, mapIndex, open = false, 
runId, taskId, xcomKe
       width={200} // TODO: Make Skeleton take style from column definition
     />
   ) : (
-    <HStack>
+    <HStack data-testid="xcom-value">
       {isObjectOrArray ? (
         <RenderedJsonField collapsed={!open} content={xcomValue as object} 
enableClipboard={false} />
       ) : (
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 a0e9b352389..5370ed4489c 100644
--- a/airflow-core/src/airflow/ui/tests/e2e/pages/XComsPage.ts
+++ b/airflow-core/src/airflow/ui/tests/e2e/pages/XComsPage.ts
@@ -161,12 +161,6 @@ export class XComsPage extends BasePage {
     const firstRow = this.tableRows.first();
 
     await expect(firstRow).toBeVisible({ timeout: 10_000 });
-
-    const valueCell = firstRow.locator("td").last();
-
-    await expect(valueCell).toBeVisible();
-    await expect(
-      
valueCell.getByRole("button").or(valueCell.locator("pre")).or(valueCell.locator("code")),
-    ).toBeVisible({ timeout: 10_000 });
+    await expect(firstRow.getByTestId("xcom-value")).toBeVisible({ timeout: 
10_000 });
   }
 }