declare_schema 1.3.1 → 1.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3d5e780d92cb93c57b69a6c7641f7dd20fc52114b0b25a2e47ad662039457290
-  data.tar.gz: a976df026f62e9a0512d71957898bddd077aa2ab8a63d63ddb8c114211df7060
+  metadata.gz: 474856e02a14224230a41cc4ef32234b8d20b69a647a30f810bce457c220261e
+  data.tar.gz: 13994a873d83d5bce533b71f36c3057a0a0caf8c7559f809091a7dc6627f5f7c
 SHA512:
-  metadata.gz: 9845c4bf47ac17b28b4b4b4eb6070cdaf6a66102bee5649af99ad8de8ed8a151eb76e5b49cbf9322997d7b1a62db3c3104eafc497ff7b81d7245825813c9bb48
-  data.tar.gz: 15eb4cc2bf1181bf8b2cd7152a203ceb6fc568920f0bf185a52dca4d5f65e1ead26ca29424c448834739b03fde489f74b7efe13ef01d9e643a6d0e4819691d52
+  metadata.gz: 7b298b54b826e8bb464c544ab2015ddca0a761cda03f03724598158617b89cef0b0f19521a103898e92157cad11d32e9feb71c1293a36bbc2c3a666c123d444f
+  data.tar.gz: 5e0c12a370ed83eeb31f398e124597a00dd7b5872d08b8c47e1eb2fb5842c6b295b251d2b58ff26619af8c92ca0342ac8bf585652c9822409dfab40798d0f086

data/.ruby-version

@@ -1 +1 @@
-3.2.1
+3.2.2

data/CHANGELOG.md

@@ -4,6 +4,10 @@ Inspired by [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 Note: this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.3.2] - 2024-01-12
+### Fixed
+- Fix bug in migrator when table option definitions differ
+
 ## [1.3.1] - 2023-10-12
 ### Fixed
 - Fix bug in default index name when table name + __ + column suffix exceeds `DeclareSchema.max_index_and_constraint_name_length`.

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    declare_schema (1.3.1)
+    declare_schema (1.3.2)
       rails (>= 5.0)
 
 GEM

data/lib/declare_schema/model/table_options_definition.rb

@@ -57,6 +57,9 @@ module DeclareSchema
         @key ||= [table_name, table_options].map(&:to_s)
       end
 
+      # Flatten out/join table options as a space-separated string
+      #
+      # @return [String] Table options as a string
       def settings
         @settings ||= table_options.map { |name, value| "#{TABLE_OPTIONS_TO_SQL_MAPPINGS[name]} #{value}" if value }.compact.join(" ")
       end

data/lib/declare_schema/schema_change/table_change.rb

@@ -5,8 +5,15 @@ require_relative 'base'
 module DeclareSchema
   module SchemaChange
     class TableChange < Base
+
+      # @param [String] table_name  The name of the table being changed
+      # @param [Hash]   old_options The old/existing table option definitions
+      # @param [Hash]   new_options The new/existing table option definitions
       def initialize(table_name, old_options, new_options)
-        @table_name = table_name
+        old_options.is_a?(Hash) or raise ArgumentError, "old_options must be a Hash but is: #{old_options.inspect}"
+        new_options.is_a?(Hash) or raise ArgumentError, "new_options must be a Hash but is: #{new_options.inspect}"
+
+        @table_name  = table_name
         @old_options = old_options
         @new_options = new_options
       end

data/lib/declare_schema/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DeclareSchema
-  VERSION = "1.3.1"
+  VERSION = "1.3.2"
 end

data/lib/generators/declare_schema/migration/migrator.rb

@@ -548,8 +548,8 @@ module Generators
           else
             [
               ::DeclareSchema::SchemaChange::TableChange.new(current_table_name,
-                                                             old_options_definition.settings,
-                                                             new_options_definition.settings)
+                                                             old_options_definition.table_options,
+                                                             new_options_definition.table_options)
             ]
           end
         end

data/spec/lib/declare_schema/schema_change/table_change_spec.rb

@@ -7,23 +7,52 @@ RSpec.describe DeclareSchema::SchemaChange::TableChange do
     load File.expand_path('../prepare_testapp.rb', __dir__)
   end
 
-  let(:table_name) { 'networks' }
-  let(:old_options) { { charset: 'utf8', collation: 'utf8_ci' } }
-  let(:new_options) { { charset: 'utf8mb4', collation: 'utf8mb4_bin' } }
-  subject { described_class.new(table_name, old_options, new_options) }
+  let(:table_name)     { 'networks' }
+  let(:old_options)    { { charset: 'utf8', collation: 'utf8_ci' } }
+  let(:new_options)    { { charset: 'utf8mb4', collation: 'utf8mb4_bin' } }
+  let(:options_string) { 'CHARACTER SET utf8mb4 COLLATE utf8mb4_bin' }
+  subject              { described_class.new(table_name, old_options, new_options) }
 
   describe '#up/down' do
-    describe '#up' do
-      it 'responds with command' do
-        statement = "ALTER TABLE #{ActiveRecord::Base.connection.quote_table_name(table_name)} CHARACTER SET utf8mb4 COLLATE utf8mb4_bin"
-        expect(subject.up).to eq("execute #{statement.inspect}\n")
+    context 'when Hashes are used to construct the TableChange' do
+      it 'does not raise' do
+        described_class.new(table_name, old_options, new_options)
+      end
+
+      describe '#up' do
+        it 'responds with command' do
+          statement = "ALTER TABLE #{ActiveRecord::Base.connection.quote_table_name(table_name)} CHARACTER SET utf8mb4 COLLATE utf8mb4_bin"
+          expect(subject.up).to eq("execute #{statement.inspect}\n")
+        end
+      end
+
+      describe '#down' do
+        it 'responds with command' do
+          statement = "ALTER TABLE #{ActiveRecord::Base.connection.quote_table_name(table_name)} CHARACTER SET utf8 COLLATE utf8_ci"
+          expect(subject.down).to eq("execute #{statement.inspect}\n")
+        end
       end
     end
 
-    describe '#down' do
-      it 'responds with command' do
-        statement = "ALTER TABLE #{ActiveRecord::Base.connection.quote_table_name(table_name)} CHARACTER SET utf8 COLLATE utf8_ci"
-        expect(subject.down).to eq("execute #{statement.inspect}\n")
+    context 'when Strings are used to construct the TableChange' do
+      context 'when old_options is a string' do
+        subject { described_class.new(table_name, options_string, new_options) }
+
+        it 'raises' do
+          expect {
+            subject
+          }.to raise_exception(ArgumentError, /old_options must be a Hash but is: "CHARACTER SET utf8mb4 COLLATE utf8mb4_bin"/)
+        end
+      end
+
+      context 'when new_options is a string' do
+        subject { described_class.new(table_name, old_options, options_string) }
+
+        it 'raises' do
+          expect {
+            subject
+          }.to raise_exception(ArgumentError, /new_options must be a Hash but is: "CHARACTER SET utf8mb4 COLLATE utf8mb4_bin"/)
+        end
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: declare_schema
 version: !ruby/object:Gem::Version
-  version: 1.3.1
+  version: 1.3.2
 platform: ruby
 authors:
 - Invoca Development adapted from hobo_fields by Tom Locke
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-10-12 00:00:00.000000000 Z
+date: 2024-01-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -141,7 +141,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 1.3.6
 requirements: []
-rubygems_version: 3.4.6
+rubygems_version: 3.4.10
 signing_key:
 specification_version: 4
 summary: Database schema declaration and migration generator for Rails