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

test: enable acceptance tests on emulator #70

Merged
merged 13 commits into from
Jun 8, 2021
Merged

test: enable acceptance tests on emulator #70

merged 13 commits into from
Jun 8, 2021

Conversation

olavloite
Copy link
Collaborator

@olavloite olavloite commented May 28, 2021

(This PR also includes the same changes as #68. It is advisable to first review #68, or otherwise only review this PR)

  • Adds more tests for migrations on the mock server.
  • Enables running the acceptance integration tests on the emulator. These finish in ~30-35 seconds. One of the acceptance tests is skipped, as it tests something that is not supported by Cloud Spanner (changing a nullable column to not-null).
  • Adds running the acceptance tests on Ruby versions 2.5, 2.6, 2.7 and 3.0 to GitHub Actions.

Updates #65
Closes #48 and #47

olavloite added 5 commits May 28, 2021 11:13
Adds a mock server and several end-to-end tests using the mock server.
Also fixes:
- Invalid function calls in migrations
- Unnecessary use of snapshots for INFORMATION_SCHEMA queries
- Unnecessary SELECT 1 query each time a connection is checked out of the pool
Enables running the acceptance integration tests on the emulator. These finish in ~20 seconds
instead of after several hours. All tests are green, except for one that is skipped, as
Cloud Spanner does not support changing a nullable column to a not-null column for a table
that already exists.

Updates #65
Enables running the acceptance integration tests on the emulator. These finish in ~20 seconds
instead of after several hours. All tests are green, except for one that is skipped, as
Cloud Spanner does not support changing a nullable column to a not-null column for a table
that already exists.

Updates #65
@product-auto-label product-auto-label bot added the api: spanner Issues related to the googleapis/ruby-spanner-activerecord API. label May 28, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label May 28, 2021
@olavloite olavloite marked this pull request as ready for review May 29, 2021 07:14
@olavloite olavloite requested a review from hengfengli May 29, 2021 07:14
@hengfengli hengfengli requested a review from xiangshen-dk June 3, 2021 10:40
Copy link
Collaborator

@xiangshen-dk xiangshen-dk left a comment

Choose a reason for hiding this comment

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

I have a few comments/questions. Let me know your thoughts.

Rakefile Show resolved Hide resolved
@olavloite olavloite requested a review from xiangshen-dk June 7, 2021 07:41
@olavloite olavloite merged commit 1b2d6b8 into googleapis:master Jun 8, 2021
@olavloite olavloite deleted the enable-acceptance-tests-on-emulator branch June 8, 2021 05:54
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/ruby-spanner-activerecord API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for Cloud Spanner Emulator
3 participants