Test: Add E2E tests for Assets Page#59990
Merged
vatsrahul1001 merged 5 commits intoapache:mainfrom Jan 14, 2026
Merged
Conversation
Contributor
|
@sarth-akvaish can you look at failing tests? |
Contributor
|
@sarth-akvaish Thanks for the PR! A few things to address:
Also maybe trigger multiple dags which uses assets and it will help in search test to filter
Let me know if you have questions |
Contributor
Author
Hi @vatsrahul1001, Thanks for the review. I’ll incorporate the suggested changes and update the PR. |
sarth-akvaish
force-pushed
the
test/Tests-for-Assets-Page
branch
from
49936ed to
f8f7d9a
Compare
Contributor
Author
|
Hi @vatsrahul1001 , currently in beforeAll I'm triggering a DAG, and checking its status in the run tab (using polling), Should I have to check the status of Dag run ? If yes, then current one is giving flaky test, can I test using rest api endpoints? |
sarth-akvaish
force-pushed
the
test/Tests-for-Assets-Page
branch
from
f8f7d9a to
5e55b8a
Compare
Contributor
Yes we can use RESTAPI and have added this in suggested change |
Contributor
|
@sarth-akvaish we are missing test for |
sarth-akvaish
force-pushed
the
test/Tests-for-Assets-Page
branch
from
5e55b8a to
2ccd01a
Compare
sarth-akvaish
force-pushed
the
test/Tests-for-Assets-Page
branch
from
2ccd01a to
b8c4c15
Compare
Contributor
|
Great work @sarth-akvaish merging the PR |
vatsrahul1001
approved these changes
Jan 14, 2026
ephraimbuddy
added a commit
to astronomer/airflow
that referenced
this pull request
Jan 14, 2026
junaiddshaukat
added a commit
to junaiddshaukat/airflow
that referenced
this pull request
Jan 15, 2026
- Rename AssetsPage to AssetDetailPage for clarity (coordinates with apache#59990) - Add beforeAll to trigger asset_produces_1 DAG before tests - Fix fragile selectors (use parent container + ARIA roles) - Refactor duplicate code into verifyStatSection helper - Remove unused assetsTable locator - Rename test to remove 'should' prefix - Change describe name to 'Asset Details Page'
ephraimbuddy
added a commit
to astronomer/airflow
that referenced
this pull request
Jan 15, 2026
ephraimbuddy
added a commit
to astronomer/airflow
that referenced
this pull request
Jan 15, 2026
ephraimbuddy
added a commit
that referenced
this pull request
Jan 15, 2026
* Update release command and guide Update some of the issues I noticed during the release * fixup! Test: Add E2E tests for Assets Page (#59990) * Apply suggestion from @ephraimbuddy * fixup! fixup! Test: Add E2E tests for Assets Page (#59990)
github-actions bot
pushed a commit
that referenced
this pull request
Jan 15, 2026
* Update release command and guide Update some of the issues I noticed during the release * fixup! Test: Add E2E tests for Assets Page (#59990) * Apply suggestion from @ephraimbuddy * fixup! fixup! Test: Add E2E tests for Assets Page (#59990) (cherry picked from commit 2de609e) Co-authored-by: Ephraim Anierobi <[email protected]>
github-actions bot
pushed a commit
to aws-mwaa/upstream-to-airflow
that referenced
this pull request
Jan 15, 2026
* Update release command and guide Update some of the issues I noticed during the release * fixup! Test: Add E2E tests for Assets Page (apache#59990) * Apply suggestion from @ephraimbuddy * fixup! fixup! Test: Add E2E tests for Assets Page (apache#59990) (cherry picked from commit 2de609e) Co-authored-by: Ephraim Anierobi <[email protected]>
potiuk
pushed a commit
that referenced
this pull request
Jan 15, 2026
* Update release command and guide Update some of the issues I noticed during the release * fixup! Test: Add E2E tests for Assets Page (#59990) * Apply suggestion from @ephraimbuddy * fixup! fixup! Test: Add E2E tests for Assets Page (#59990) (cherry picked from commit 2de609e) Co-authored-by: Ephraim Anierobi <[email protected]>
ephraimbuddy
added a commit
that referenced
this pull request
Jan 16, 2026
* Update release command and guide Update some of the issues I noticed during the release * fixup! Test: Add E2E tests for Assets Page (#59990) * Apply suggestion from @ephraimbuddy * fixup! fixup! Test: Add E2E tests for Assets Page (#59990) (cherry picked from commit 2de609e) Co-authored-by: Ephraim Anierobi <[email protected]>
jason810496
pushed a commit
to jason810496/airflow
that referenced
this pull request
Jan 22, 2026
Test: Add E2E tests for Assets Page (apache#59990) Co-authored-by: vatsrahul1001 <[email protected]> Co-authored-by: Rahul Vats <[email protected]>
jason810496
pushed a commit
to jason810496/airflow
that referenced
this pull request
Jan 22, 2026
* Update release command and guide Update some of the issues I noticed during the release * fixup! Test: Add E2E tests for Assets Page (apache#59990) * Apply suggestion from @ephraimbuddy * fixup! fixup! Test: Add E2E tests for Assets Page (apache#59990)
jhgoebbert
pushed a commit
to jhgoebbert/airflow_Owen-CH-Leung
that referenced
this pull request
Feb 8, 2026
Test: Add E2E tests for Assets Page (apache#59990) Co-authored-by: vatsrahul1001 <[email protected]> Co-authored-by: Rahul Vats <[email protected]>
jhgoebbert
pushed a commit
to jhgoebbert/airflow_Owen-CH-Leung
that referenced
this pull request
Feb 8, 2026
* Update release command and guide Update some of the issues I noticed during the release * fixup! Test: Add E2E tests for Assets Page (apache#59990) * Apply suggestion from @ephraimbuddy * fixup! fixup! Test: Add E2E tests for Assets Page (apache#59990)
choo121600
pushed a commit
to choo121600/airflow
that referenced
this pull request
Feb 22, 2026
Test: Add E2E tests for Assets Page (apache#59990) Co-authored-by: vatsrahul1001 <[email protected]> Co-authored-by: Rahul Vats <[email protected]>
choo121600
pushed a commit
to choo121600/airflow
that referenced
this pull request
Feb 22, 2026
* Update release command and guide Update some of the issues I noticed during the release * fixup! Test: Add E2E tests for Assets Page (apache#59990) * Apply suggestion from @ephraimbuddy * fixup! fixup! Test: Add E2E tests for Assets Page (apache#59990)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds the E2E tests for Assets Page.
The test now verfies:
Closes : #59929
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named
{pr_number}.significant.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.