Thanks to visit codestin.com
Credit goes to github.com

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
@codecov
Copy link

codecov bot commented Sep 30, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@codecov
Copy link

codecov bot commented Sep 30, 2025

Bundle Report

Changes will increase total bundle size by 134 bytes (0.0%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
datahub-react-web-esm 28.58MB 134 bytes (0.0%) ⬆️

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 134 bytes 18.92MB 0.0%

Files in assets/index-*.js:

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

@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
@alwaysmeticulous
Copy link

alwaysmeticulous bot commented Oct 3, 2025

✅ Meticulous spotted 0 visual differences across 971 screens tested: view results.

Meticulous evaluated ~9 hours of user flows against your PR.

Expected differences? Click here. Last updated for commit f51fd11. This comment will update as new commits are pushed.

Copy link
Collaborator

@chriscollins3456 chriscollins3456 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this seems reasonable - will merge if CI is green!

@datahub-cyborg datahub-cyborg bot added pending-submitter-merge and removed needs-review Label for PRs that need review from a maintainer. labels Oct 3, 2025
@chriscollins3456 chriscollins3456 merged commit b4d6877 into master Oct 3, 2025
41 checks passed
@chriscollins3456 chriscollins3456 deleted the vt--ch-806-structured-properties--add-cypress-tests branch October 3, 2025 18:55
alplatonov pushed a commit to alplatonov/datahub that referenced this pull request Oct 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pending-submitter-merge 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.

3 participants