Fix restore entity inherited fields specs#27432
Fix restore entity inherited fields specs#27432shrabantipaul-collate wants to merge 12 commits intomainfrom
Conversation
|
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
|
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
❌ Playwright Lint Check Failed — ESLint + Prettier + Organise ImportsThe following files have style issues that need to be fixed: Fix locally (fast — only for changed files in the branch): make ui-checkstyle-playwright-changedOr to fix all playwright files: make ui-checkstyle-playwright |
…hub.com/open-metadata/OpenMetadata into fix-restroreEntityInheritedFields-specs
Code Review ✅ ApprovedRestores correct inheritance for entity fields in specifications. No issues found. OptionsDisplay: compact → Showing less information. Comment with these commands to change:
Was this helpful? React with 👍 / 👎 | Gitar |
|
🟡 Playwright Results — all passed (29 flaky)✅ 3631 passed · ❌ 0 failed · 🟡 29 flaky · ⏭️ 89 skipped
🟡 29 flaky test(s) (passed on retry)
How to debug locally# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip # view trace |



Describe your changes:
saveRequestPromise captures the PATCH request initiated by our click (requests sent before registration are ignored)
saveRequest.response() waits for the response to that exact request object — no other PATCH can satisfy it, eliminating cross-test contamination entirely
Fixes
I worked on ... because ...
Type of change:
Checklist:
Fixes <issue-number>: <short explanation>