-
Notifications
You must be signed in to change notification settings - Fork 16.4k
UI E2E tests for Providers Page #60705
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
base: main
Are you sure you want to change the base?
UI E2E tests for Providers Page #60705
Conversation
c727eca to
d7db207
Compare
|
@sarth-akvaish Thanks for the PR I will review soon |
|
Xcom Page tests are failing ig |
@haseebmalik18 looks xcom tests are failing I suggest you to open a PR to fix them |
| await providers.waitForLoad(); | ||
| }); | ||
|
|
||
| test("verify assets page heading", async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not assets should be providers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
| } | ||
| }); | ||
|
|
||
| test("verify pagination controls navigate between pages", async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sarth-akvaish lets see how pagination tests are done dor dagpage and others. Let's ensure we use same approach
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
42b5c1e to
75c90f2
Compare
|
@sarth-akvaish I see test failing. Can you check? |
Yes, I'm checking this |
a0161ca to
29b6560
Compare
29b6560 to
a741e3a
Compare

This PR adds E2E tests for Providers Page in the Admin Menu.
Closes : #60570
Was generative AI tooling used to co-author this PR?
{pr_number}.significant.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.