-
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 |
22b9d5b to
24423a8
Compare
24423a8 to
8224502
Compare
|
Hi @vatsrahul1001 , updated the pagination tests Also are there any latest changes in the CI environment, beacuse while rebasing to latest main the Additional PRod checks were failing, when I rebased to some older commit all checks ran fine without errors, any thoughts or Am I missing something ? |

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.