-
Notifications
You must be signed in to change notification settings - Fork 16.4k
Add E2E test to verify XComs page displays XCom entries correctly #59358 #60620
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add E2E test to verify XComs page displays XCom entries correctly #59358 #60620
Conversation
fa17ae0 to
17c0ab0
Compare
|
Nice! Added few review comments |
17c0ab0 to
77eeb83
Compare
|
@vatsrahul1001 Made the changes |
9785def to
420bbd9
Compare
Thanks @haseebmalik18, this is great progress. I noticed webkit and firefox tests are flaky and needed retry https://github.com/apache/airflow/actions/runs/21125402571/job/60747088841?pr=60620. Can you look at it? |
e8235bf to
bbbe111
Compare
bbbe111 to
2d891d9
Compare
|
@vatsrahul1001 Should be good now |
|
@haseebmalik18 I see we have some flakiness in xcom filter test. https://github.com/apache/airflow/actions/runs/21274675447/job/61233478866?pr=60514. Can you look into this? |
Add comprehensive E2E tests for the XComs page (
/xcoms) to verify XCom entries display correctly.closes: #59358
Changes
Created
XComsPage.ts- Page Object Model for XComs page interactionsCreated
xcoms.spec.ts- E2E test specification with 8 test cases