Skip to content

Fix failing MySQL API tets#11857

Merged
ZoranPandovski merged 5 commits into
developfrom
fileupload-test
Nov 7, 2025
Merged

Fix failing MySQL API tets#11857
ZoranPandovski merged 5 commits into
developfrom
fileupload-test

Conversation

@ZoranPandovski
Copy link
Copy Markdown
Member

@ZoranPandovski ZoranPandovski commented Nov 6, 2025

Description

This PR fixes the failing tests due to MySQL api returning column names in uppercase like "STATUS" and "NAME" but the tests checked for lowercase "status" and "name", causing all checks to fail and tests to loop until timeout instead of detecting when models finished training.
To avoid scenarios like this for now, I am normalizing the values in lowercase always.

Type of change

(Please delete options that are not relevant)

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ⚡ New feature (non-breaking change which adds functionality)
  • 📢 Breaking change (fix or feature that would cause existing functionality not to work as expected)
  • 📄 This change requires a documentation update

Checklist:

  • My code follows the style guidelines(PEP 8) of MindsDB.
  • I have appropriately commented on my code, especially in complex areas.
  • Necessary documentation updates are either made or tracked in issues.
  • Relevant unit and integration tests are updated or added.

@entelligence-ai-pr-reviews
Copy link
Copy Markdown
Contributor

🔒 Entelligence AI Vulnerability Scanner

No security vulnerabilities found!

Your code passed our comprehensive security analysis.


Signed-off-by: ZoranPandovski <zoran.pandovski@gmail.com>
@ZoranPandovski ZoranPandovski merged commit 127d9f2 into develop Nov 7, 2025
23 checks passed
@ZoranPandovski ZoranPandovski deleted the fileupload-test branch November 7, 2025 09:37
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants