ridgepole 0.6.5.beta3 → 0.6.5.beta4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9f55a300d2808032c178912f81d291ba094a867a
-  data.tar.gz: 20e173fc302bc246392f9efb96f4497c76154516
+  metadata.gz: 6eb5cd9b6da53e0af2afd188637a9196f5a59b6c
+  data.tar.gz: c0c798186e1cd37c7e1881278270d82f200020bd
 SHA512:
-  metadata.gz: 42b281cc0ebcb60117addc51ddba6b3b39b4358cc86fe804a5a184f2e96e4cff193e66d89b4dedc821e08d529e5ecf23874fa7ee4026b31533ad51f7d050d1c8
-  data.tar.gz: 4cdd724e376511e9cbbb82b269b4aecfc3657f3d6618744b90cb5c43bc8e5353e50cc2a9f1e5f2be85616fe72e92163f2fd7dd1728d0b8b42036ecec040e969b
+  metadata.gz: 34b3eca50c3aadbf464e4a23f3c5a16c1f8022709fd02df91992212b083eb3080ed50e06556d216405d462f2288948c617c21e1dfc15a2cea567d5d062692712
+  data.tar.gz: da081d87c9e157c515bfd81acd1363b3586bf532861f480ac627ba1e936e2b65d606b8751220ca2b1917282fc2ccd44f2507a5f1aa4581107f7481674f948abd

data/lib/ridgepole/dsl_parser.rb

@@ -184,6 +184,15 @@ class Ridgepole::DSLParser
         raise "Index `#{table_name}(#{idx})` already defined"
       end
 
+      if options[:length].is_a?(Numeric)
+        index_length = options[:length]
+        options[:length] = {}
+
+        column_name.each do |col|
+          options[:length][col] = index_length
+        end
+      end
+
       @__definition[table_name][:indices][idx] = {
         :column_name => column_name,
         :options => options,

data/lib/ridgepole/version.rb

@@ -1,3 +1,3 @@
 module Ridgepole
-  VERSION = '0.6.5.beta3'
+  VERSION = '0.6.5.beta4'
 end

data/spec/mysql/migrate/migrate_change_index7_spec.rb

@@ -0,0 +1,86 @@
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when change index (length is Numeric) / No update' do
+    let(:actual_dsl) {
+      erbh(<<-EOS)
+        create_table "employees", primary_key: "emp_no", <%= i unsigned(true) + {force: :cascade} %> do |t|
+          t.date   "birth_date",            null: false
+          t.string "first_name", limit: 14, null: false
+          t.string "last_name",  limit: 16, null: false
+          t.string "gender",     limit: 1,  null: false
+          t.date   "hire_date",             null: false
+        end
+
+        <%= add_index "employees", ["first_name", "last_name"], name: "idx_first_name_last_name", length: {"first_name"=>10, "last_name"=>10}, using: :btree %>
+      EOS
+    }
+
+    let(:expected_dsl) {
+      erbh(<<-EOS)
+        create_table "employees", primary_key: "emp_no", <%= i unsigned(true) + {force: :cascade} %> do |t|
+          t.date   "birth_date",            null: false
+          t.string "first_name", limit: 14, null: false
+          t.string "last_name",  limit: 16, null: false
+          t.string "gender",     limit: 1,  null: false
+          t.date   "hire_date",             null: false
+        end
+
+        <%= add_index "employees", ["first_name", "last_name"], name: "idx_first_name_last_name", length: 10, using: :btree %>
+      EOS
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_falsy
+      expect(subject.dump).to match_fuzzy actual_dsl
+    }
+  end
+
+  context 'when change index (length is Numeric) / Update' do
+    let(:actual_dsl) {
+      erbh(<<-EOS)
+        create_table "employees", primary_key: "emp_no", <%= i unsigned(true) + {force: :cascade} %> do |t|
+          t.date   "birth_date",            null: false
+          t.string "first_name", limit: 14, null: false
+          t.string "last_name",  limit: 16, null: false
+          t.string "gender",     limit: 1,  null: false
+          t.date   "hire_date",             null: false
+        end
+      EOS
+    }
+
+    let(:expected_dsl) {
+      erbh(<<-EOS)
+        create_table "employees", primary_key: "emp_no", <%= i unsigned(true) + {force: :cascade} %> do |t|
+          t.date   "birth_date",            null: false
+          t.string "first_name", limit: 14, null: false
+          t.string "last_name",  limit: 16, null: false
+          t.string "gender",     limit: 1,  null: false
+          t.date   "hire_date",             null: false
+        end
+
+        <%= add_index "employees", ["first_name", "last_name"], name: "idx_first_name_last_name", length: 10, using: :btree %>
+      EOS
+    }
+
+    let(:actual_dsl_plus_index) {
+      erbh(<<-EOS)
+        #{actual_dsl}
+        <%= add_index "employees", ["first_name", "last_name"], name: "idx_first_name_last_name", length: {"first_name"=>10, "last_name"=>10}, using: :btree %>
+      EOS
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to match_fuzzy actual_dsl
+      delta.migrate
+      expect(subject.dump).to match_fuzzy actual_dsl_plus_index
+    }
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ridgepole
 version: !ruby/object:Gem::Version
-  version: 0.6.5.beta3
+  version: 0.6.5.beta4
 platform: ruby
 authors:
 - Genki Sugawara
@@ -259,6 +259,7 @@ files:
 - spec/mysql/migrate/migrate_change_index4_spec.rb
 - spec/mysql/migrate/migrate_change_index5_spec.rb
 - spec/mysql/migrate/migrate_change_index6_spec.rb
+- spec/mysql/migrate/migrate_change_index7_spec.rb
 - spec/mysql/migrate/migrate_change_index_spec.rb
 - spec/mysql/migrate/migrate_change_table_option_spec.rb
 - spec/mysql/migrate/migrate_create_index2_spec.rb
@@ -384,6 +385,7 @@ test_files:
 - spec/mysql/migrate/migrate_change_index4_spec.rb
 - spec/mysql/migrate/migrate_change_index5_spec.rb
 - spec/mysql/migrate/migrate_change_index6_spec.rb
+- spec/mysql/migrate/migrate_change_index7_spec.rb
 - spec/mysql/migrate/migrate_change_index_spec.rb
 - spec/mysql/migrate/migrate_change_table_option_spec.rb
 - spec/mysql/migrate/migrate_create_index2_spec.rb