-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Fix #19147 - Executable Test Suites #19221
Conversation
The Python checkstyle failed. Please run You can install the pre-commit hooks with |
Quality Gate passed for 'open-metadata-airflow-apis'Issues Measures |
Quality Gate passed for 'open-metadata-ui'Issues Measures |
Failed to cherry-pick changes to the 1.6.2 branch. |
* backend * format & tests * rename backend * migrations and ingestion * format & tests * format & tests * tests * format & tests * tests * updated ui side of changes * addressing comment * fixed failing unit test * fix test list * added e2e test, and fixed existing test --------- Co-authored-by: Shailesh Parmar <[email protected]>
Describe your changes:
Fixes #19147
executable
test suite, it's abasic
test suite. This helps us maintain the same structure and overall BE logic we currently have and ensures that all test cases have a "home".executable
->basic
andexecutableEntityReference
->basicEntityReference
in the Test Suite JSON SchematestSuites
field stored in the db, with a list of all the suites referencing the test. However, this is actually being fetched at runtime when the fields are being requested here. We are cleaning up this data.testSuites
were also stored in the version history of tests in entity_extension. This was raising issues when updating the test case result. Cleaning those up too/basic
endpoints to replace the/executable
endpoints. We're keeping the/executable
passing deprecation headers to keep things compatible, since this will be part of 1.6.2Type of change:
Checklist:
Fixes <issue-number>: <short explanation>