choo121600 commented on code in PR #66081:
URL: https://github.com/apache/airflow/pull/66081#discussion_r3162326618


##########
airflow-core/src/airflow/ui/tests/e2e/pages/DagsPage.ts:
##########
@@ -55,7 +55,7 @@ export class DagsPage extends BasePage {
     this.confirmButton = page.getByRole("dialog").getByRole("button", { name: 
"Trigger" });
 
     this.searchBox = page.getByRole("textbox", { name: /search/i });
-    this.searchInput = page.getByPlaceholder("Search DAGs");
+    this.searchInput = page.getByPlaceholder("Search Dags");

Review Comment:
   Playwright’s `getByPlaceholder` performs case-insensitive substring matching 
by default, so `"Search DAGs"` has been matching `"Search Dags"` and tests have 
been passing.
   
   This change does not affect behavior and is only to keep the UI text and 
tests consistent.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to