Skip to content

Conversation

@sarth-akvaish
Copy link
Contributor

This PR adds E2E tests for Providers Page in the Admin Menu.

Closes : #60570


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@boring-cyborg boring-cyborg bot added the area:UI Related to UI/UX. For Frontend Developers. label Jan 17, 2026
@sarth-akvaish sarth-akvaish force-pushed the test/Admin-Providers-Page branch from c727eca to d7db207 Compare January 17, 2026 16:51
@vatsrahul1001
Copy link
Contributor

@sarth-akvaish Thanks for the PR I will review soon

@sarth-akvaish
Copy link
Contributor Author

Xcom Page tests are failing ig

@vatsrahul1001
Copy link
Contributor

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 () => {
Copy link
Contributor

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

Copy link
Contributor Author

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 () => {
Copy link
Contributor

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

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried using the same approach as used in DagsPage, but it is failing for pagination, browser tests are failing (they are running fine in the local using ui-mode), i guess need to add the scroll behaviour to find the next and prev locators.

image

@sarth-akvaish sarth-akvaish force-pushed the test/Admin-Providers-Page branch 2 times, most recently from 42b5c1e to 75c90f2 Compare January 26, 2026 16:41
@vatsrahul1001
Copy link
Contributor

@sarth-akvaish I see test failing. Can you check?

@sarth-akvaish
Copy link
Contributor Author

@sarth-akvaish I see test failing. Can you check?

Yes, I'm checking this

@sarth-akvaish sarth-akvaish force-pushed the test/Admin-Providers-Page branch 2 times, most recently from a0161ca to 29b6560 Compare January 29, 2026 17:09
@sarth-akvaish sarth-akvaish force-pushed the test/Admin-Providers-Page branch from 29b6560 to a741e3a Compare January 30, 2026 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:UI Related to UI/UX. For Frontend Developers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UI E2E Test || ADMIN-004: Verify Providers Page functionality

2 participants