Skip to content

Conversation

@Yashh56
Copy link
Member

@Yashh56 Yashh56 commented Jan 22, 2026

This pull request introduces a comprehensive overhaul of the testing infrastructure for database connectors, focusing on improved reliability, realistic test environments, and better configuration management. The changes include new Docker Compose and GitHub Actions workflows for spinning up real PostgreSQL, MySQL, and MariaDB services, standardized test data seeding, and updates to test cases to use environment-driven configuration.

Testing Infrastructure Improvements

  • Added .github/workflows/test.yml to automate tests with real PostgreSQL, MySQL, and MariaDB services using GitHub Actions, including service health checks and database seeding steps.
  • Introduced bridge/docker-compose.test.yml to allow local testing with Docker containers for all supported databases, including health checks and automatic seeding.

Database Seeding

  • Added seed scripts for PostgreSQL (bridge/scripts/seed-test-db.sql), MySQL (bridge/scripts/seed-mysql.sql), and MariaDB (bridge/scripts/seed-mariadb.sql) to ensure consistent tables and sample data for integration tests. [1] [2] [3]

Test Configuration and Reliability

  • Updated tests in bridge/__tests__/connectors/mysql.test.ts and bridge/__tests__/connectors/mariadb.test.ts to use environment variables for database names and SSL settings, improving flexibility and accuracy. [1] [2] [3] [4] [5]
  • Updated PostgreSQL tests to use a truly invalid hostname for failure scenarios and made the long-running query cancellation test skipped by default due to flakiness with local containers. [1] [2] [3] [4]

Environment Management

  • Changed bridge/jest.env.js to load environment variables from .env instead of .env.test to align with new configuration practices.

@Yashh56 Yashh56 merged commit f80d227 into feat/mariaDB Jan 22, 2026
Yashh56 added a commit that referenced this pull request Jan 22, 2026
* feat: MariaDB Connector Implementation

* feat: implementated MariaDB Connector

* feat: implemented test cases for mariaDB and fixed other cases

* fix: removed the unnecessary console.logs & fixed the test cases

* feat: Implemented the test cases workflow for Pull Requests (#36)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants