linxi507 commented on PR #63463: URL: https://github.com/apache/airflow/pull/63463#issuecomment-4088416052
I noticed a small improvement in `dag-audit-log.spec.ts`, specifically in the "verify audit log entries display valid data" test. Currently, the test extracts text using `textContent()` and then asserts non-empty values. This pattern can sometimes be less stable and bypasses Playwright’s built-in retry logic. It might be more robust to use locator-based assertions instead, for example: ```ts await expect(whenCell).toHaveText(/\S+/); await expect(eventCell).toHaveText(/\S+/); await expect(userCell).toHaveText(/\S+/); ``` This keeps the assertions within Playwright’s retry mechanism and could potentially help with some of the intermittent stability issues mentioned above. Happy to help implement this if it sounds reasonable 🙂 -- 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]
