Skip to content

Conversation

v-tarasevich-blitz-brain
Copy link
Contributor

This PR adds cypress tests to cover flow with the new setting property Hide when Empty in structured properties

@github-actions github-actions bot added product PR or Issue related to the DataHub UI/UX smoke_test Contains changes related to smoke tests labels Sep 30, 2025
Copy link

codecov bot commented Sep 30, 2025

Codecov Report

❌ Patch coverage is 76.74419% with 10 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...chemy-components/components/Checkbox/components.ts 27.27% 8 Missing ⚠️
...edproperties/UpdateStructuredPropertyResolver.java 77.77% 0 Missing and 2 partials ⚠️

📢 Thoughts on this report? Let us know!

Copy link

codecov bot commented Sep 30, 2025

Bundle Report

Changes will increase total bundle size by 2.37kB (0.01%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
datahub-react-web-esm 28.58MB 2.37kB (0.01%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: datahub-react-web-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-*.js 2.37kB 18.92MB 0.01%

Files in assets/index-*.js:

  • ./src/app/entityV2/shared/sidebarSection/SidebarStructuredProperties.tsx → Total Size: 4.32kB

  • ./src/alchemy-components/components/Checkbox/components.ts → Total Size: 2.29kB

  • ./src/alchemy-components/components/Checkbox/Checkbox.tsx → Total Size: 2.31kB

  • ./src/app/analytics/event.ts → Total Size: 13.83kB

  • ./src/app/govern/structuredProperties/DisplayPreferences.tsx → Total Size: 6.41kB

  • ./src/app/govern/structuredProperties/StructuredPropsTable.tsx → Total Size: 9.39kB

  • ./src/app/govern/structuredProperties/ViewDisplayPreferences.tsx → Total Size: 3.38kB

  • ./src/app/govern/structuredProperties/cacheUtils.ts → Total Size: 2.92kB

  • ./src/app/govern/structuredProperties/useStructuredProp.ts → Total Size: 3.69kB

  • ./src/app/govern/structuredProperties/StructuredPropsDrawer.tsx → Total Size: 11.77kB

  • ./src/app/govern/structuredProperties/styledComponents.ts → Total Size: 9.21kB

@v-tarasevich-blitz-brain v-tarasevich-blitz-brain changed the title tests(structuredProperties): add cypress tests [WAIT FOR CI] tests(structuredProperties): add cypress tests Sep 30, 2025
@v-tarasevich-blitz-brain v-tarasevich-blitz-brain force-pushed the vt--ch-806-structured-properties--add-cypress-tests branch from c9b4636 to 150086b Compare October 1, 2025 08:35
@v-tarasevich-blitz-brain v-tarasevich-blitz-brain changed the title [WAIT FOR CI] tests(structuredProperties): add cypress tests tests(structuredProperties): add cypress tests Oct 1, 2025
@v-tarasevich-blitz-brain v-tarasevich-blitz-brain marked this pull request as ready for review October 1, 2025 09:46
@datahub-cyborg datahub-cyborg bot added the needs-review Label for PRs that need review from a maintainer. label Oct 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-review Label for PRs that need review from a maintainer. product PR or Issue related to the DataHub UI/UX smoke_test Contains changes related to smoke tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant