Skip to content

initial commit datasource integration tests#11548

Merged
sejubar merged 9 commits into
developfrom
datasource_integration_tests
Oct 16, 2025
Merged

initial commit datasource integration tests#11548
sejubar merged 9 commits into
developfrom
datasource_integration_tests

Conversation

@sejubar

@sejubar sejubar commented Sep 10, 2025

Copy link
Copy Markdown
Contributor

Description

Please include a summary of the change and the issue it solves.

Fixes #issue_number

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

Verification Process

To ensure the changes are working as expected:

  • Test Location: Specify the URL or path for testing.
  • Verification Steps: Outline the steps or queries needed to validate the change. Include any data, configurations, or actions required to reproduce or see the new functionality.

Additional Media:

  • I have attached a brief loom video or screenshots showcasing the new functionality or change.

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.


@sejubar sejubar requested a review from hamishfagg September 10, 2025 12:39
@sejubar

sejubar commented Sep 10, 2025

Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

Comment thread tests/conftest.py Outdated
Comment thread tests/integration/handlers/conftest.py Outdated
Comment thread tests/conftest.py Outdated
Comment thread tests/integration/handlers/test_data_sources.py Outdated
Comment thread tests/integration/handlers/test_data_sources.py Outdated
Comment thread tests/integration/handlers/utils/helpers.py Outdated
hamishfagg
hamishfagg previously approved these changes Sep 15, 2025
Comment thread tests/integration/handlers/README.md
### Adding Custom Tests
To test specific queries, you can add a custom test configuration.

1. Create a JSON File: In tests/integration/handlers/configs/, create a new JSON file named after the handler (e.g., postgres.json).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should the this folder (../configs/) be added to git ignore?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, i was left to execute custome queries

Comment thread tests/integration/handlers/utils/helpers.py Outdated
ea-rus
ea-rus previously approved these changes Sep 16, 2025
@sejubar sejubar changed the base branch from main to develop October 16, 2025 13:06
@sejubar sejubar dismissed ea-rus’s stale review October 16, 2025 13:06

The base branch was changed.

@sejubar sejubar merged commit 0f366db into develop Oct 16, 2025
19 of 20 checks passed
@sejubar sejubar deleted the datasource_integration_tests branch October 16, 2025 16:05
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants