Skip to content

Commit 6d24737

Browse files
anuj-kumaryclaude
andcommitted
test(e2e): drop DomainWidgetAssetCount spec — coverage already in DomainDataProductsWidgets
DomainWidgetAssetCount.spec.ts requires persona+widget-assignment setup to guarantee KnowledgePanel.Domains is visible on the home page before checking counts. Without that setup the poll always returns null and the test fails. DomainDataProductsWidgets.spec.ts already covers the exact same code paths ("Domain asset count should update when assets are added/removed") with the correct setup. Remove the redundant spec and drop its impact-map entry. The widget cache-invalidation fix (queryClient.invalidateQueries) and the useMemo dep fix (assetsCounts) remain in the PR — they are exercised by DomainDataProductsWidgets.spec.ts. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent dbe6a45 commit 6d24737

2 files changed

Lines changed: 0 additions & 131 deletions

File tree

.github/playwright/impact-map.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -296,7 +296,6 @@
296296
],
297297
"projects": ["chromium", "Basic"],
298298
"specs": [
299-
"playwright/e2e/Features/LandingPageWidgets/DomainWidgetAssetCount.spec.ts",
300299
"playwright/e2e/Features/LandingPageWidgets/DomainDataProductsWidgets.spec.ts"
301300
]
302301
},

openmetadata-ui/src/main/resources/ui/playwright/e2e/Features/LandingPageWidgets/DomainWidgetAssetCount.spec.ts

Lines changed: 0 additions & 130 deletions
This file was deleted.

0 commit comments

Comments
 (0)