Skip to content

Fix AUT failures#27596

Merged
Rohit0301 merged 2 commits intomainfrom
aut-failures
Apr 21, 2026
Merged

Fix AUT failures#27596
Rohit0301 merged 2 commits intomainfrom
aut-failures

Conversation

@Rohit0301
Copy link
Copy Markdown
Contributor

@Rohit0301 Rohit0301 commented Apr 21, 2026

Describe your changes:

Fixed the playwright failure for serviceIngestion.spec and EntityVersionPage, other spec are failing due to test timeout, because the column profile api has very high latency, for now i have increase the timeout as per the max latency value we got in the grafana logs. Will decrease the timeout once it is fixed from the backend

Fixes

I worked on ... because ...

Screenshot 2026-04-21 at 6 51 25 PM Screenshot 2026-04-21 at 7 38 01 PM

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.

Summary by Gitar

  • Refactored EntityVersionPages.spec.ts:
    • Changed entity instantiation to use class references rather than pre-initialized instances for better state isolation.
    • Updated versionDetailResponse logic to check for the general /versions endpoint rather than a specific patch version.
  • Performance tuning:
    • Reduced the wait timeout in ServiceIngestion.spec.ts from 8s to 5s.

This will update automatically on new commits.

@Rohit0301 Rohit0301 self-assigned this Apr 21, 2026
@Rohit0301 Rohit0301 requested a review from a team as a code owner April 21, 2026 14:13
@Rohit0301 Rohit0301 added safe to test Add this label to run secure Github workflows on PRs To release Will cherry-pick this PR into the release branch labels Apr 21, 2026
@gitar-bot
Copy link
Copy Markdown

gitar-bot Bot commented Apr 21, 2026

Code Review ✅ Approved 1 resolved / 1 findings

Test infrastructure for AUT is stabilized by replacing temporary entities in forEach loops with direct lookups. No further issues found.

✅ 1 resolved
Edge Case: Temporary entity created just for test name in forEach

📄 openmetadata-ui/src/main/resources/ui/playwright/e2e/VersionPages/EntityVersionPages.spec.ts:155-156
At line 156, new EntityClass().getType() instantiates a throwaway entity object solely to extract the type name for the test title. This is wasteful and could cause issues if the constructor has side effects or requires API context. Consider adding a static method or using the class name directly.

Options

Display: compact → Showing less information.

Comment with these commands to change:

Compact
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@github-actions
Copy link
Copy Markdown
Contributor

Jest test Coverage

UI tests summary

Lines Statements Branches Functions
Coverage: 61%
61.95% (60279/97287) 41.99% (31619/75292) 45% (9496/21098)

@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown
Contributor

🟡 Playwright Results — all passed (16 flaky)

✅ 3695 passed · ❌ 0 failed · 🟡 16 flaky · ⏭️ 89 skipped

Shard Passed Failed Flaky Skipped
✅ Shard 1 481 0 0 4
🟡 Shard 2 655 0 1 7
🟡 Shard 3 664 0 2 1
🟡 Shard 4 643 0 5 27
✅ Shard 5 611 0 0 42
🟡 Shard 6 641 0 8 8
🟡 16 flaky test(s) (passed on retry)
  • Features/BulkEditEntity.spec.ts › Glossary (shard 2, 1 retry)
  • Features/RestoreEntityInheritedFields.spec.ts › Validate restore with Inherited domain and data products assigned (shard 3, 1 retry)
  • Features/UserProfileOnlineStatus.spec.ts › Should show "Active recently" for users active within last hour (shard 3, 1 retry)
  • Pages/Customproperties-part2.spec.ts › entityReferenceList shows item count, scrollable list, no expand toggle (shard 4, 1 retry)
  • Pages/DataContracts.spec.ts › Create Data Contract and validate for Topic (shard 4, 1 retry)
  • Pages/DataContracts.spec.ts › Create Data Contract and validate for SearchIndex (shard 4, 1 retry)
  • Pages/DataContracts.spec.ts › Create Data Contract and validate for Database (shard 4, 1 retry)
  • Pages/DataProducts.spec.ts › Create Data Product and Manage Assets (shard 4, 2 retries)
  • Features/AutoPilot.spec.ts › Agents created by AutoPilot should be deleted (shard 6, 1 retry)
  • Pages/Glossary.spec.ts › Column dropdown drag-and-drop functionality for Glossary Terms table (shard 6, 1 retry)
  • Pages/Lineage/DataAssetLineage.spec.ts › verify create lineage for entity - Worksheet (shard 6, 1 retry)
  • Pages/Lineage/LineageFilters.spec.ts › Verify lineage schema filter selection (shard 6, 1 retry)
  • Pages/Lineage/LineageRightPanel.spec.ts › Verify custom properties tab IS visible for supported type: searchIndex (shard 6, 1 retry)
  • Pages/ODCSImportExport.spec.ts › Multi-object ODCS contract - object selector shows all schema objects (shard 6, 1 retry)
  • Pages/UserDetails.spec.ts › Create team with domain and verify visibility of inherited domain in user profile after team removal (shard 6, 1 retry)
  • Pages/Users.spec.ts › Permissions for table details page for Data Consumer (shard 6, 1 retry)

📦 Download artifacts

How to debug locally
# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip    # view trace

@Rohit0301 Rohit0301 merged commit d0362d1 into main Apr 21, 2026
52 checks passed
@Rohit0301 Rohit0301 deleted the aut-failures branch April 21, 2026 16:36
@github-actions
Copy link
Copy Markdown
Contributor

Failed to cherry-pick changes to the 1.12.6 branch.
Please cherry-pick the changes manually.
You can find more details here.

Rohit0301 added a commit that referenced this pull request Apr 24, 2026
* Fix AUT failures

* fix lint issue
jatinmasaram pushed a commit to jatinmasaram/OpenMetadata that referenced this pull request May 2, 2026
* Fix AUT failures

* fix lint issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to test Add this label to run secure Github workflows on PRs To release Will cherry-pick this PR into the release branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants