activerecord-spanner-adapter 1.5.1 → 1.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 73df0f7ee319edb7461b0fa60f8f8b611b38d798de2c0caf20a9c7d6796a7a85
-  data.tar.gz: 9dedd540069c5bc85d458008c20e48b9b550a75388a1f4d48b1b29b34bb689d4
+  metadata.gz: 931e6b17bf3e6d6b068c57ca879bc0dbb06933790101aa05415514c6448fd004
+  data.tar.gz: c4981e814899be433830e0380f69060dfa589ec39dfa7e831aad616e224030ec
 SHA512:
-  metadata.gz: faac93588f6e703f183a773caf553d61b64370439093ac27a7d1a4893f6d887c887fbf1be9082baf78bb81e2c23b24f45a1e4053672e386595ff95c0980a0c4b
-  data.tar.gz: 4d6061c3daf7760d3015a33b52a41354f6bafa21b594a0c2b44852ae928935e1b2e23421ce417de2e65eecdc56de0890d97d0321e07fd3a3c3c104cf56856c88
+  metadata.gz: a989bdd8195d98206802a05294b09a53fe76c0684fba781d22dc15aaa5a38106ee3f193555cbe83e5f65254e3b39d95ed1cfb6445ded245ad80683d9752fdfd2
+  data.tar.gz: 0dbe1209ccad8da313fafb110bbb4fb2c1ef9ff9209230e88673f031ad02c0c372a13422fd2a5756fb0bc97a2318c003433350e071f64bcef37ba5a7754e868e

data/.github/workflows/acceptance-tests-on-emulator.yaml

@@ -19,7 +19,7 @@ jobs:
       max-parallel: 4
       matrix:
         ruby: ["2.7", "3.0", "3.1", "3.2"]
-        ar: ["~> 6.0.6", "~> 6.1.7", "~> 7.0.4"]
+        ar: ["~> 6.0.6", "~> 6.1.7", "~> 7.0.4", "~> 7.1.0"]
         # Exclude combinations that are not supported.
         exclude:
           - ruby: "3.0"

data/.github/workflows/acceptance-tests-on-production.yaml

@@ -34,12 +34,14 @@ jobs:
       with:
         bundler-cache: true
         ruby-version: ${{ matrix.ruby }}
+    - name: Authenticate Google Cloud
+      uses: google-github-actions/auth@v2
+      with:
+        credentials_json: ${{ secrets.GCP_SA_KEY }}
     - name: Setup GCloud
-      uses: google-github-actions/setup-gcloud@v0
+      uses: google-github-actions/setup-gcloud@v2
       with:
         project_id: ${{ secrets.GCP_PROJECT_ID }}
-        service_account_key: ${{ secrets.GCP_SA_KEY }}
-        export_default_credentials: true
     - name: Install dependencies
       run: bundle install
     - name: Run acceptance tests on production

data/.github/workflows/ci.yaml

@@ -11,7 +11,7 @@ jobs:
       max-parallel: 4
       matrix:
         ruby: ["2.7", "3.0", "3.1", "3.2"]
-        ar: ["~> 6.0.6", "~> 6.1.7", "~> 7.0.4"]
+        ar: ["~> 6.0.6", "~> 6.1.7", "~> 7.0.4", "~> 7.1.0"]
         # Exclude combinations that are not supported.
         exclude:
           - ruby: "3.0"

data/.github/workflows/nightly-acceptance-tests-on-emulator.yaml

@@ -20,7 +20,7 @@ jobs:
       matrix:
         # Run acceptance tests all supported combinations of Ruby and ActiveRecord.
         ruby: [2.7, 3.0, 3.1, 3.2]
-        ar: [6.0.0, 6.0.1, 6.0.2.2, 6.0.3.7, 6.0.4, 6.1.3.2, 6.1.4.7, 6.1.5.1, 6.1.6.1, 7.0.2.4, 7.0.3.1, 7.0.4, 7.0.5, 7.0.6, 7.0.7]
+        ar: [6.0.0, 6.0.1, 6.0.2.2, 6.0.3.7, 6.0.4, 6.1.3.2, 6.1.4.7, 6.1.5.1, 6.1.6.1, 7.0.2.4, 7.0.3.1, 7.0.4, 7.0.5, 7.0.6, 7.0.7, 7.1.0, 7.1.1, 7.1.2]
         # Exclude combinations that are not supported.
         exclude:
           - ruby: 3.0

data/.github/workflows/nightly-acceptance-tests-on-production.yaml

@@ -20,12 +20,14 @@ jobs:
       with:
         bundler-cache: true
         ruby-version: ${{ matrix.ruby }}
+    - name: Authenticate Google Cloud
+      uses: google-github-actions/auth@v2
+      with:
+        credentials_json: ${{ secrets.GCP_SA_KEY }}
     - name: Setup GCloud
-      uses: google-github-actions/setup-gcloud@v0
+      uses: google-github-actions/setup-gcloud@v2
       with:
         project_id: ${{ secrets.GCP_PROJECT_ID }}
-        service_account_key: ${{ secrets.GCP_SA_KEY }}
-        export_default_credentials: true
     - name: Install dependencies
       run: bundle install
     - name: Run acceptance tests on production

data/.github/workflows/nightly-unit-tests.yaml

@@ -12,7 +12,7 @@ jobs:
       matrix:
         # Run unit tests all supported combinations of Ruby and ActiveRecord.
         ruby: [2.7, 3.0, 3.1, 3.2]
-        ar: [6.0.0, 6.0.1, 6.0.2.2, 6.0.3.7, 6.0.4, 6.1.3.2, 6.1.4.7, 6.1.5.1, 6.1.6.1, 7.0.2.4, 7.0.3.1, 7.0.4, 7.0.5]
+        ar: [6.0.0, 6.0.1, 6.0.2.2, 6.0.3.7, 6.0.4, 6.1.3.2, 6.1.4.7, 6.1.5.1, 6.1.6.1, 7.0.2.4, 7.0.3.1, 7.0.4, 7.0.5, 7.1.0, 7.1.1, 7.1.2]
         # Exclude combinations that are not supported.
         exclude:
           - ruby: 3.0

data/.github/workflows/release-please-label.yml

@@ -11,7 +11,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: ReleaseLabel
-        uses: actions/github-script@v6
+        uses: actions/github-script@v7
         with:
           github-token: ${{secrets.YOSHI_APPROVER_TOKEN}}
           script: |

data/.github/workflows/rubocop.yaml

@@ -18,7 +18,7 @@ jobs:
       with:
         ruby-version: '2.7'
     - name: cache gems
-      uses: actions/cache@v3
+      uses: actions/cache@v4
       with:
         path: vendor/bundle
         key: ${{ runner.os }}-rubocop-${{ hashFiles('**/Gemfile.lock') }}

data/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "1.5.1"
+  ".": "1.6.1"
 }

data/.rubocop.yml

@@ -41,6 +41,6 @@ Style/WordArray:
 Style/RegexpLiteral:
   Enabled: false
 Metrics/MethodLength:
-  Max: 40
+  Max: 60
 Metrics/BlockLength:
   Max: 30

data/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+### 1.6.1 (2024-02-05)
+
+#### Bug Fixes
+
+* _insert_record failed for other adapters ([#298](https://github.com/googleapis/ruby-spanner-activerecord/issues/298)) 
+
+### 1.6.0 (2023-12-20)
+
+#### Features
+
+* interleaved tables with built-in composite pk ([#282](https://github.com/googleapis/ruby-spanner-activerecord/issues/282)) 
+* support Query Logs ([#291](https://github.com/googleapis/ruby-spanner-activerecord/issues/291)) 
+* support Rails 7.1 ([#278](https://github.com/googleapis/ruby-spanner-activerecord/issues/278)) 
+
 ### 1.5.1 (2023-12-12)
 
 #### Bug Fixes

data/Gemfile

@@ -4,12 +4,17 @@ source "https://rubygems.org"
 gemspec
 
 gem "activerecord", ENV.fetch("AR_VERSION", "~> 6.1.6.1")
-gem "minitest", "~> 5.19.0"
+gem "minitest", "~> 5.20.0"
+gem "minitest-rg", "~> 5.3.0"
 gem "pry", "~> 0.13.0"
 gem "pry-byebug", "~> 3.9.0"
+# Add sqlite3 for testing for compatibility with other adapters.
+gem "sqlite3"
 
 # Required for samples and testing.
-gem "composite_primary_keys"
+install_if -> { ENV.fetch("AR_VERSION", "~> 6.1.6.1").dup.to_s.sub("~>", "").strip < "7.1.0" && !ENV["SKIP_COMPOSITE_PK"] } do
+  gem "composite_primary_keys"
+end
 
 # Required for samples
 gem "docker-api"

data/README.md

@@ -84,19 +84,19 @@ Some noteworthy examples in the snippets directory:
   for inserting, updating and deleting data in a Cloud Spanner database. Mutations can have a significant performance
   advantage compared to DML statements, but do not allow read-your-writes semantics during a transaction.
 - [array-data-type](examples/snippets/array-data-type): Shows how to work with `ARRAY` data types.
-- [interleaved-tables](examples/snippets/interleaved-tables): Shows how to work with [Interleaved Tables](https://cloud.google.com/spanner/docs/schema-and-data-model#create-interleaved-tables).
+- [interleaved-tables](examples/snippets/interleaved-tables-before-7.1): Shows how to work with [Interleaved Tables](https://cloud.google.com/spanner/docs/schema-and-data-model#create-interleaved-tables).
 
 ## Limitations
 
-Limitation|Comment|Resolution
----|---|---
-Interleaved tables require composite primary keys| Cloud Spanner requires composite primary keys for interleaved tables. See {file:examples/snippets/interleaved-tables/README.md this example} for an example on how to use interleaved tables with ActiveRecord |Use composite primary keys.
-Lack of sequential and auto-assigned IDs|Cloud Spanner doesn't autogenerate IDs and this integration instead creates UUID4 to avoid [hotspotting](https://cloud.google.com/spanner/docs/schema-design#uuid_primary_key) so you SHOULD NOT rely on IDs being sorted| UUID4s are automatically generated for primary keys.
-Table without Primary Key| Cloud Spanner support does not support tables without a primary key.| Always define a primary key for your table.
-Table names CANNOT have spaces within them whether back-ticked or not|Cloud Spanner DOES NOT support tables with spaces in them for example `Entity ID`|Ensure that your table names don't contain spaces.
-Table names CANNOT have punctuation marks and MUST contain valid UTF-8|Cloud Spanner DOES NOT support punctuation marks e.g. periods ".", question marks "?" in table names|Ensure that your table names don't contain punctuation marks.
-Index with fields length [add_index](https://apidock.com/rails/v5.2.3/ActiveRecord/ConnectionAdapters/SchemaStatements/add_index)|Cloud Spanner does not support index with fields length | Ensure that your database definition does not include index definitions with field lengths.
-Only GoogleSQL-dialect databases| Cloud Spanner supports both GoogleSQL- and PostgreSQL-dialect databases. This adapter only supports GoogleSQL-dialect databases. | Ensure that your database uses the GoogleSQL dialect.
+| Limitation                                                                                                                        | Comment                                                                                                                                                                                                                           | Resolution                                                                                  |
+|-----------------------------------------------------------------------------------------------------------------------------------|-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|---------------------------------------------------------------------------------------------|
+| Interleaved tables require composite primary keys                                                                                 | Cloud Spanner requires composite primary keys for interleaved tables. See {file:examples/snippets/interleaved-tables/README.md this example} for an example on how to use interleaved tables with ActiveRecord                        | Use composite primary keys.                                                                 |
+| Lack of sequential IDs                                                                                                            | Cloud Spanner uses either using bit-reversed sequences or UUID4 to generated primary keys to avoid [hotspotting](https://cloud.google.com/spanner/docs/schema-design#uuid_primary_key) so you SHOULD NOT rely on IDs being sorted | Use either UUID4s or bit-reversed sequences to automatically generate primary keys.         |
+| Table without Primary Key                                                                                                         | Cloud Spanner support does not support tables without a primary key.                                                                                                                                                              | Always define a primary key for your table.                                                 |
+| Table names CANNOT have spaces within them whether back-ticked or not                                                             | Cloud Spanner DOES NOT support tables with spaces in them for example `Entity ID`                                                                                                                                                 | Ensure that your table names don't contain spaces.                                          |
+| Table names CANNOT have punctuation marks and MUST contain valid UTF-8                                                            | Cloud Spanner DOES NOT support punctuation marks e.g. periods ".", question marks "?" in table names                                                                                                                              | Ensure that your table names don't contain punctuation marks.                               |
+| Index with fields length [add_index](https://apidock.com/rails/v5.2.3/ActiveRecord/ConnectionAdapters/SchemaStatements/add_index) | Cloud Spanner does not support index with fields length                                                                                                                                                                           | Ensure that your database definition does not include index definitions with field lengths. |
+| Only GoogleSQL-dialect databases                                                                                                  | Cloud Spanner supports both GoogleSQL- and PostgreSQL-dialect databases. This adapter only supports GoogleSQL-dialect databases.                                                                                                  | Ensure that your database uses the GoogleSQL dialect.                                       |
 
 ## Contributing
 

data/acceptance/cases/interleaved_associations/has_many_associations_using_interleaved_test.rb

@@ -6,6 +6,12 @@
 
 # frozen_string_literal: true
 
+# ActiveRecord 7.1 introduced native support for composite primary keys.
+# This deprecates the https://github.com/composite-primary-keys/composite_primary_keys gem that was previously used in
+# this library to support composite primary keys, which again are needed for interleaved tables. These tests use the
+# third-party composite primary key gem and are therefore not executed for Rails 7.1 and higher.
+return if ActiveRecord::gem_version >= Gem::Version.create('7.1.0')
+
 require "test_helper"
 require "models/singer"
 require "models/album"

data/acceptance/cases/migration/change_schema_test.rb

@@ -196,8 +196,14 @@ module ActiveRecord
             end
           end
         end
+        expected = if ActiveRecord::gem_version < Gem::Version.create('7.1.0')
+          "you can't redefine the primary key column 'id'. To define a custom primary key, pass { id: false } to create_table."
+        else
+          "you can't redefine the primary key column 'id' on 'testings'. To define a custom primary key, pass { id: false } to create_table."
+        end
+
 
-        assert_equal "you can't redefine the primary key column 'id'. To define a custom primary key, pass { id: false } to create_table.", error.message
+        assert_equal expected, error.message
       end
 
       def test_create_table_raises_when_redefining_custom_primary_key_column
@@ -209,7 +215,12 @@ module ActiveRecord
           end
         end
 
-        assert_equal "you can't redefine the primary key column 'testing_id'. To define a custom primary key, pass { id: false } to create_table.", error.message
+        expected = if ActiveRecord::gem_version < Gem::Version.create('7.1.0')
+          "you can't redefine the primary key column 'testing_id'. To define a custom primary key, pass { id: false } to create_table."
+        else
+          "you can't redefine the primary key column 'testing_id' on 'testings'. To define a custom primary key, pass { id: false } to create_table."
+        end
+        assert_equal expected, error.message
       end
 
       def test_create_table_raises_when_defining_existing_column
@@ -222,7 +233,12 @@ module ActiveRecord
           end
         end
 
-        assert_equal "you can't define an already defined column 'testing_column'.", error.message
+        expected = if ActiveRecord::gem_version < Gem::Version.create('7.1.0')
+          "you can't define an already defined column 'testing_column'."
+        else
+          "you can't define an already defined column 'testing_column' on 'testings'."
+        end
+        assert_equal expected, error.message
       end
 
       def test_create_table_with_timestamps_should_create_datetime_columns

data/acceptance/cases/migration/schema_dumper_test.rb

@@ -13,6 +13,10 @@ module ActiveRecord
     class IndexTest < SpannerAdapter::TestCase
       include SpannerAdapter::Migration::TestHelper
 
+      def is_7_1_or_higher?
+        ActiveRecord::gem_version >= Gem::Version.create('7.1.0')
+      end
+
       def test_dump_schema_contains_start_batch_ddl
         connection = ActiveRecord::Base.connection
         schema = StringIO.new
@@ -34,7 +38,12 @@ module ActiveRecord
         connection = ActiveRecord::Base.connection
         schema = StringIO.new
         ActiveRecord::SchemaDumper.dump connection, schema
-        assert schema.string.include?("create_table \"albums\", primary_key: \"albumid\"")
+        sql = schema.string
+        if is_7_1_or_higher?
+          assert schema.string.include?("create_table \"albums\", primary_key: [\"singerid\", \"albumid\"]"), sql
+        else
+          assert schema.string.include?("create_table \"albums\", primary_key: \"albumid\""), sql
+        end
       end
 
       def test_dump_schema_contains_interleaved_index

data/acceptance/cases/models/interleave_test.rb

@@ -6,6 +6,12 @@
 
 # frozen_string_literal: true
 
+# ActiveRecord 7.1 introduced native support for composite primary keys.
+# This deprecates the https://github.com/composite-primary-keys/composite_primary_keys gem that was previously used in
+# this library to support composite primary keys, which again are needed for interleaved tables. These tests use the
+# third-party composite primary key gem and are therefore not executed for Rails 7.1 and higher.
+return if ActiveRecord::gem_version >= Gem::Version.create('7.1.0')
+
 require "test_helper"
 require "test_helpers/with_separate_database"