-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Flaky(UI): Removed Flakyness from DataAssetRulesDisabled.spec.ts #25549
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
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! |
Rules 🎸 1 action takenGitar Rules
2 rules not applicable. Show all rules by commenting OptionsAuto-apply is off → Gitar will not commit updates to this branch. Comment with these commands to change:
Was this helpful? React with 👍 / 👎 | Gitar |
|



Describe your changes:
I worked on ... because ...
Type of change:
Checklist:
Fixes <issue-number>: <short explanation>Summary by Gitar
waitFor({ state: 'visible' })forowner-select-teams-search-barinDataAssetRulesDisabled.spec.tsassignGlossaryTermutility to await PATCH API responses before continuing test executionentityEndpointparameter toassignGlossaryTerminutils/entity.tsfor API synchronizationgetUserDisplayName()to returnnamewhendisplayNameis undefinedaddMultiOwnerfunctionfillTextInputDetailswith try-catch for more reliable element visibility detectionThis will update automatically on new commits.