You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "choo121600 (via GitHub)" <gi...@apache.org> on 2026/03/20 04:36:43 UTC

[I] E2E: Improve Playwright test patterns in specs/task-instances.spec.ts [airflow]

choo121600 opened a new issue, #63963:
URL: https://github.com/apache/airflow/issues/63963

   ### Description
   
   Improve Playwright test patterns in requiredAction.spec.ts and related page object to align with Playwright Best Practices.
   
   This PR focuses on improving test patterns only.
   It does not change test coverage or behavior.
   
   ### Checklist
   - [ ] page.waitForFunction() with DOM queries → locator-based waiting
   - [ ] page.waitForTimeout() → state-based waiting
   - [ ] waitForLoadState("networkidle") → wait for specific UI state
   - [ ] Manual assertions → web-first assertions
   - [ ] page.evaluate() for DOM manipulation → observe UI state instead
   - [ ] CSS :has-text() → user-facing locators
   - [ ] flaky check
   
   ### Scope
   This issue covers improvements in:
   - specs/task-instances.spec.ts
   - pages/TaskInstancePage.ts
   
   ### Use case/motivation
   
   _No response_
   
   ### Related issues
   
   Part of https://github.com/apache/airflow/issues/63036
   
   ### Are you willing to submit a PR?
   
   - [ ] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [x] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
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: commits-unsubscribe@airflow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] E2E: Improve Playwright test patterns in specs/task-instances.spec.ts [airflow]

Posted by "choo121600 (via GitHub)" <gi...@apache.org>.
choo121600 closed issue #63963: E2E: Improve Playwright test patterns in specs/task-instances.spec.ts
URL: https://github.com/apache/airflow/issues/63963


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] E2E: Improve Playwright test patterns in specs/task-instances.spec.ts [airflow]

Posted by "haseebmalik18 (via GitHub)" <gi...@apache.org>.
haseebmalik18 commented on issue #63963:
URL: https://github.com/apache/airflow/issues/63963#issuecomment-4095523310

   I'll work on this


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] E2E: Improve Playwright test patterns in specs/task-instances.spec.ts [airflow]

Posted by "choo121600 (via GitHub)" <gi...@apache.org>.
choo121600 commented on issue #63963:
URL: https://github.com/apache/airflow/issues/63963#issuecomment-4095602591

   Cool :)


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org