Flaky(UI): Glossary import export

[dhruvjsx]

Describe your changes:

I worked on ... because ...

Type of change:

• Bug fix
• Improvement
• New feature
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation

Checklist:

• I have read the CONTRIBUTING document.
• My PR title is Fixes <issue-number>: <short explanation>
• I have commented on my code, particularly in hard-to-understand areas.
• For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

---

Summary by Gitar

• Fixed flaky file upload:
  • Replaced getByTestId('upload-file-widget') with reliable [type="file"] selector
• Improved wait strategy:
  • Replaced arbitrary waitForTimeout(500) with state-based waits for upload widget to disappear
• Streamlined assertions:
  • Removed explicit timeout parameters, relying on Playwright's default wait behavior for better test stability

_{This will update automatically on new commits.}

---

[github-actions]

Jest test Coverage

UI tests summary

Lines	Statements	Branches	Functions
	65.84% (56032/85108)	45.18% (29334/64923)	47.95% (8848/18454)

[gitar-bot]

Code Review 👍 Approved with suggestions 0 resolved / 1 findings

Clean fix for flaky tests — replaces arbitrary timeouts with state-based waits and uses a more reliable file input selector. One minor suggestion to scope the [type="file"] selector for future robustness.

💡 Quality: Generic [type="file"] selector could match multiple inputs

📄 openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/GlossaryImportExport.spec.ts:160

The selector [type="file"] is used across all 6 occurrences to target the file input. If the page ever introduces a second <input type="file">, this selector would match multiple elements, potentially causing test failures or operating on the wrong element.

Consider using a more specific selector that scopes to the upload widget, e.g.:

await page.waitForSelector('[data-testid="upload-file-widget"] [type="file"]', { state: 'attached' });
await page.setInputFiles('[data-testid="upload-file-widget"] [type="file"]', ...);

This combines the reliability of [type="file"] (which avoids issues with the test-id not being directly on the input) with the specificity of the upload widget container. That said, if there's only ever one file input on this page, the current approach works fine — this is just a robustness suggestion.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change:

Auto-apply	Compact
gitar auto-apply:on	gitar display:verbose

_{Was this helpful? React with 👍 / 👎 | Gitar}

[sonarqubecloud]

Quality Gate passed for 'open-metadata-ui'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

Failed to cherry-pick changes to the 1.11.9 branch.
Please cherry-pick the changes manually.
You can find more details here.