test: salvage, migrate, and fix unused unit and integration tests#12299
test: salvage, migrate, and fix unused unit and integration tests#12299sejubar wants to merge 4 commits into
Conversation
|
Automatic Review Skipped Too many files for automatic review. If you would still like a review, you can trigger one manually by commenting: |
There was a problem hiding this comment.
@ZoranPandovski, should be 'load' folder be kept? or moved to other repo if it has something useful?
There was a problem hiding this comment.
it might be reworked in separated pr, it covers resp api tests
I will create an issue for it
There was a problem hiding this comment.
maybe move all tests from tests/unused/unit/handler_tests and tests/unused/unit/ml_handlers/ to handler folders? for example this one into mindsdb/integrations/handlers/apache_doris_handler/tests/
if we don't want to run them we can keep their code at least
|
All contributors have signed the CLA ✍️ ✅ |
|
signed and agreed to the CLA |
7fb04c3 to
f9d79ae
Compare
Description
Revive/remove unused tests and clean up unit test excluding knowledge base tests
Fixes #issue_number
Type of change
(Please delete options that are not relevant)
Verification Process
To ensure the changes are working as expected:
Additional Media:
Checklist: