Fix (#27660): Re-enable Exasol cli-e2e-tests after fixing issues#27661
Fix (#27660): Re-enable Exasol cli-e2e-tests after fixing issues#27661ArBridgeman wants to merge 7 commits intoopen-metadata:mainfrom
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! |
|
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! |
|
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! |
🟡 Playwright Results — all passed (15 flaky)✅ 3960 passed · ❌ 0 failed · 🟡 15 flaky · ⏭️ 86 skipped
🟡 15 flaky test(s) (passed on retry)
How to debug locally# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip # view trace |
|
Hi @ulixius9 If it's possible, could I get feedback on this PR? |
Code Review ✅ ApprovedRe-enables Exasol cli-e2e-tests following previous stability fixes. No issues found. OptionsDisplay: compact → Showing less information. Comment with these commands to change:
Was this helpful? React with 👍 / 👎 | Gitar |
|


Describe your changes:
Fixes #27660
To re-enable the Exasol cli-e2e-tests, I:
ingestion/tests/cli_e2e/test_cli_exasol.py.github/workflows/py-cli-e2e-tests.ymlI worked on the tests because we would like to ensure that the Exasol connector continues to work.
Most of our tests are centralized in the
ingestion/tests/cli_e2e/test_cli_exasol.py, so it is critical that these are regular run, and if there are issues, it would be helpful if we were contacted.Type of change:
Checklist:
Fixes <issue-number>: <short explanation>Bug Fix
Summary by Gitar
# noqacomments acrossexasolingestion source and CLI tests to suppress linting errors after migration toruff.basedpyrightto~=1.39.0and addednoxto development dependencies insetup.py.This will update automatically on new commits.