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

fix: correct test case for sql_flush for multiple delete table commands #629

Merged
merged 38 commits into from May 20, 2021

Conversation

vi3k6i5
Copy link
Contributor

@vi3k6i5 vi3k6i5 commented May 20, 2021

fix: corrected test case for sql_flush for multiple delete table commands.

Test case used to fail when RUNNING_SPANNER_BACKEND_TESTS was set because quote_name function works differently when that env variable is set.

fix: lint_setup_py was failing in Kokoro is now fixed (googleapis#607)
fix: Remove un necessary file from code base (googleapis#608)
feat: move migrations test modules to run against different emulator …
feat: update workflow files to uniformly distribute the test modules …
feat: update docs and nox file to compile it (googleapis#610)
@vi3k6i5 vi3k6i5 requested a review from a team as a code owner May 20, 2021 06:41
@product-auto-label product-auto-label bot added the api: spanner Issues related to the googleapis/python-spanner-django API. label May 20, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label May 20, 2021
@larkee larkee changed the title fix: corrected test case for sql_flush for multiple delete table commands fix: correct test case for sql_flush for multiple delete table commands May 20, 2021
@vi3k6i5 vi3k6i5 merged commit 3de1a81 into googleapis:master May 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: spanner Issues related to the googleapis/python-spanner-django API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants