Skip to content
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

Integration test checkconstraints #983

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

akashthawaitcc
Copy link
Contributor

I added an integration test for the MySQL check constraints migration dump and regular flow.

taherkl and others added 7 commits January 2, 2025 10:44
…INTS (GoogleCloudPlatform#978) (#30)

* verification ap and dump flow changes

* fixed IT issue

* Check constraints verificartion api v2 (#24)

* handled function not found

* added unhandled error

* updated the error msg

---------



* fix IT issue

* comment addressed (#27)

* comment addressed
1. rename the functionNotFound
2. added condition to call verification api

* spell checked

---------



* refactor the DbDumpImpl struct (#28)

* refactor the DbDumpImpl struct

* remove the GenerateCheckConstrainstExprId method

---------



* fixed if condition

---------

Co-authored-by: Taher Lakdawala <[email protected]>
Co-authored-by: taherkl <[email protected]>
Co-authored-by: Vivek Yadav <[email protected]>
Co-authored-by: Vivek Yadav <[email protected]>
1. integration test added for check constraints.
@akashthawaitcc akashthawaitcc requested a review from a team as a code owner January 6, 2025 10:04
@akashthawaitcc akashthawaitcc requested review from asthamohta and VardhanThigle and removed request for a team January 6, 2025 10:04
@VivekY1098 VivekY1098 force-pushed the integration_test_checkconstraints branch from 6d436b4 to ed753a1 Compare January 8, 2025 11:41
* fixed the query

* fixed

* fixed the code

* fixed query

* updated the code

* updated the code

---------

Co-authored-by: Vivek Yadav <[email protected]>
Co-authored-by: Vivek Yadav <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants