Skip to content

ci(cloudsqlmysql): add integration tests #243

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

Merged
merged 4 commits into from
Jan 30, 2025
Merged

ci(cloudsqlmysql): add integration tests #243

merged 4 commits into from
Jan 30, 2025

Conversation

Yuan325
Copy link
Contributor

@Yuan325 Yuan325 commented Jan 28, 2025

Add integration test for CloudSQL for MySQL.

Added other integration tests' tag into .golangci.yaml, and fixing lint errors.

Moved getCloudSQLDialOpts to common_test.go since it is used across all three cloud sql integration tests.

@Yuan325 Yuan325 requested a review from a team as a code owner January 28, 2025 21:24
@Yuan325 Yuan325 force-pushed the cloudsql-mysql-int branch 4 times, most recently from bc7966e to 8d84254 Compare January 28, 2025 23:22
@Yuan325 Yuan325 force-pushed the cloudsql-mysql-int branch from 8d84254 to b6c4054 Compare January 29, 2025 21:56
@Yuan325 Yuan325 changed the base branch from main to mysql-tools January 29, 2025 21:56
@Yuan325 Yuan325 force-pushed the cloudsql-mysql-int branch from b6c4054 to 065cfd3 Compare January 29, 2025 21:59
Base automatically changed from mysql-tools to main January 29, 2025 23:09
@Yuan325 Yuan325 force-pushed the cloudsql-mysql-int branch from 065cfd3 to 13ad15a Compare January 30, 2025 00:56
@Yuan325 Yuan325 merged commit 8152a98 into main Jan 30, 2025
8 checks passed
@Yuan325 Yuan325 deleted the cloudsql-mysql-int branch January 30, 2025 01:03
jeffreyrubi pushed a commit to jeffreyrubi/genai-toolbox that referenced this pull request Jun 7, 2025
Add integration test for CloudSQL for MySQL.

Added other integration tests' tag into `.golangci.yaml`, and fixing
lint errors.

Moved getCloudSQLDialOpts to `common_test.go` since it is used across
all three cloud sql integration tests.
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