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 07c3d0790c0 Replace Playwright locator anti-patterns with web-first
assertions in dag-tasks (#63445)
07c3d0790c0 is described below
commit 07c3d0790c0c959a194c7074616a23f3180c9b62
Author: Desel72 <[email protected]>
AuthorDate: Fri Mar 20 00:05:55 2026 -0500
Replace Playwright locator anti-patterns with web-first assertions in
dag-tasks (#63445)
Co-authored-by: Yeonguk Choo <[email protected]>
---
airflow-core/src/airflow/ui/tests/e2e/specs/dag-tasks.spec.ts | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/airflow-core/src/airflow/ui/tests/e2e/specs/dag-tasks.spec.ts
b/airflow-core/src/airflow/ui/tests/e2e/specs/dag-tasks.spec.ts
index 46927a6a129..6366439987c 100644
--- a/airflow-core/src/airflow/ui/tests/e2e/specs/dag-tasks.spec.ts
+++ b/airflow-core/src/airflow/ui/tests/e2e/specs/dag-tasks.spec.ts
@@ -46,7 +46,7 @@ test.describe("Dag Tasks Tab", () => {
const firstRow = dagPage.taskRows.first();
- await expect(firstRow.locator("a").first()).toBeVisible();
+ await expect(firstRow.getByRole("link").first()).toBeVisible();
await expect(firstRow).toContainText("BashOperator");
await expect(firstRow).toContainText("all_success");
});
@@ -56,7 +56,7 @@ test.describe("Dag Tasks Tab", () => {
await dagPage.navigateToDagTasks(testDagId);
- const firstTaskLink = dagPage.taskRows.first().locator("a").first();
+ const firstTaskLink = dagPage.taskRows.first().getByRole("link").first();
const taskName = await firstTaskLink.textContent();
if (taskName === null) {
@@ -160,7 +160,7 @@ test.describe("Dag Tasks Tab", () => {
await dagPage.navigateToDagTasks(testDagId);
const firstCard = dagPage.taskRows.first();
- const taskLink = firstCard.locator("a").first();
+ const taskLink = firstCard.getByRole("link").first();
await taskLink.click();
await expect(page).toHaveURL(new RegExp(`/dags/${testDagId}/tasks/.*`));