Skip to content
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

fix AUT visit entity details page #14844

Merged
merged 5 commits into from
Jan 25, 2024
Merged

fix AUT visit entity details page #14844

merged 5 commits into from
Jan 25, 2024

Conversation

chirag-madlani
Copy link
Collaborator

Fixing issues with AUT
Getting 404 when trying to visit entity details page

Describe your changes:

Fixes

I worked on ... because ...

Type of change:

  • Bug fix
  • Improvement
  • New feature
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation

Checklist:

  • I have read the CONTRIBUTING document.
  • My PR title is Fixes <issue-number>: <short explanation>
  • I have commented on my code, particularly in hard-to-understand areas.
  • For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

Fixing issues with AUT 
Getting 404 when trying to visit entity details page
@github-actions github-actions bot added UI UI specific issues safe to test Add this label to run secure Github workflows on PRs labels Jan 24, 2024
Copy link
Contributor

github-actions bot commented Jan 24, 2024

Jest test Coverage

UI tests summary

Lines Statements Branches Functions
Coverage: 52%
52.77% (27084/51320) 35.22% (10820/30724) 33.67% (3162/9391)

Copy link

cypress bot commented Jan 24, 2024

12 failed tests on run #33045 ↗︎

12 614 25 30 Flakiness 0

Details:

fix postgres issue
Project: openmetadata Commit: f8ae273c3f
Status: Failed Duration: 35:16 💡
Started: Jan 25, 2024 5:47 AM Ended: Jan 25, 2024 6:23 AM
Failed  Service/ServiceIngestion.spec.ts • 3 failed tests • cypress-ci-psql-oss-7650312548-1

View Output Video

Test Artifacts
BigQuery Ingestion > Create & Ingest BigQuery service Screenshots Video
Redshift Ingestion > Add DBT ingestion Screenshots Video
Redshift Ingestion > Validate DBT is ingested properly Screenshots Video
Failed  Pages/Services.spec.ts • 1 failed test • cypress-ci-psql-oss-7650312548-1

View Output Video

Test Artifacts
Search Service page > Domain assign, update & delete Screenshots Video
Failed  Pages/Database.spec.ts • 1 failed test • cypress-ci-psql-oss-7650312548-1

View Output Video

Test Artifacts
Store Procedure page > Domain assign, update & delete Screenshots Video
Failed  Flow/Lineage.spec.js • 5 failed tests • cypress-ci-psql-oss-7650312548-1

View Output Video

Test Artifacts
Lineage verification > Lineage Remove Node between Topic Screenshots Video
Lineage verification > Lineage Remove Node between ML Model Screenshots Video
Lineage verification > Lineage Add Node for entity Container Screenshots Video
Lineage verification > Lineage Remove Node between Container Screenshots Video
Lineage verification > Add column lineage Screenshots Video
Failed  Pages/SearchIndexDetails.spec.js • 2 failed tests • cypress-ci-psql-oss-7650312548-1

View Output Video

Test Artifacts
Prerequisite for data steward role tests > Add data steward role to the user Screenshots Video
SearchIndexDetails page should work properly for data steward role > All permissible actions on search index details page should work properly Screenshots Video

Review all test suite changes for PR #14844 ↗︎

Copy link

Quality Gate Passed Quality Gate passed for 'open-metadata-ui'

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@chirag-madlani chirag-madlani merged commit ac4dc7f into main Jan 25, 2024
14 of 20 checks passed
@chirag-madlani chirag-madlani deleted the fix-aut branch January 25, 2024 06:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
e2e This will trigger e2e test workflows safe to test Add this label to run secure Github workflows on PRs UI UI specific issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants