ridgepole 0.3.2 → 0.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: aa9b0a7dfc33d00e38afd82181c57d18100a9827
-  data.tar.gz: 6d1de2888d0d1881e7452dbfcd859b91291a8204
+  metadata.gz: f678cfdd6ac71010140d48e4d8174e93647de40f
+  data.tar.gz: 4463ea985e64b7eddf093cbee47c3973a2645a4b
 SHA512:
-  metadata.gz: a504fcec940a63f231904ce2f096bf5cc2c1b7793911e24e0a077b2561ffc9d990783b3feb65665cc1df61b5e46c09a0b08ed542fa001e11bc4b32c523951c4a
-  data.tar.gz: 04f49e0ca36179a1e5d495cda341cd05048379639083a3de96f1194deda7fbf093823295b6181758f730949ef2120fc68d557107cbf1690bbff9e2709ceffca8
+  metadata.gz: e02a89b0b5672a9d934c0dec3c550e05509e41a339a44f7c40169dc71d28733d63db2f64ac9cec55f008bea8253a31670fb8a44b93044c8b1f86dd7f107a3b33
+  data.tar.gz: ce27454f5a691e3dbcd87a57d2136e58ed6fbcde5af37febba2ec847768e2314bc80c8b3331e1659041fa2873150dd73aaf864134aa988250cc70d7acb9bf566

data/README.md

@@ -31,6 +31,7 @@ Usage: ridgepole [options]
     -f, --file FILE
         --dry-run
         --table-options OPTIONS
+        --bulk-change
     -e, --export
         --split
         --split-with-dir

data/bin/ridgepole

@@ -47,6 +47,10 @@ ARGV.options do |opt|
     opt.on('-f', '--file FILE')                   {|v| file = v                                             }
     opt.on('',   '--dry-run')                     {    options[:dry_run] = true                             }
     opt.on('',   '--table-options OPTIONS')       {|v| options[:table_options] = v                          }
+    opt.on('',   '--bulk-change') {
+      raise "Cannot use `bulk-change` in `merge`" if options[:merge]
+      options[:bulk_change] = true
+    }
     opt.on('-e', '--export')                      {    set_mode[:export]                                    }
     opt.on('',   '--split')                       {|v| split = true                                         }
     opt.on('',   '--split-with-dir')              {|v| split = :with_dir                                    }

data/lib/ridgepole/delta.rb

@@ -136,8 +136,12 @@ create_table(#{table_name.inspect}, #{options.inspect}) do |t|
 end
     EOS
 
-    indices.each do |index_name, index_attrs|
-      append_add_index(table_name, index_name, index_attrs, buf)
+    unless indices.empty?
+      append_change_table(table_name, buf) do
+        indices.each do |index_name, index_attrs|
+          append_add_index(table_name, index_name, index_attrs, buf)
+        end
+      end
     end
 
     buf.puts
@@ -160,11 +164,20 @@ drop_table(#{table_name.inspect})
   end
 
   def append_change(table_name, attrs, buf)
-    append_change_definition(table_name, attrs[:definition] || {}, buf)
-    append_change_indices(table_name, attrs[:indices] || {}, buf)
+    append_change_table(table_name, buf) do
+      append_change_definition(table_name, attrs[:definition] || {}, buf)
+      append_change_indices(table_name, attrs[:indices] || {}, buf)
+    end
+
     buf.puts
   end
 
+  def append_change_table(table_name, buf)
+    buf.puts "change_table(#{table_name.inspect}, {:bulk => true}) do |t|" if @options[:bulk_change]
+    yield
+    buf.puts 'end' if @options[:bulk_change]
+  end
+
   def append_change_definition(table_name, delta, buf)
     (delta[:add] || {}).each do |column_name, attrs|
       append_add_column(table_name, column_name, attrs, buf)
@@ -187,30 +200,54 @@ drop_table(#{table_name.inspect})
     type = attrs.fetch(:type)
     options = attrs[:options] || {}
 
-    buf.puts(<<-EOS)
+    if @options[:bulk_change]
+      buf.puts(<<-EOS)
+  t.column(#{column_name.inspect}, #{type.inspect}, #{options.inspect})
+      EOS
+    else
+      buf.puts(<<-EOS)
 add_column(#{table_name.inspect}, #{column_name.inspect}, #{type.inspect}, #{options.inspect})
-    EOS
+      EOS
+    end
   end
 
   def append_rename_column(table_name, to_column_name, from_column_name, buf)
-    buf.puts(<<-EOS)
+    if @options[:bulk_change]
+      buf.puts(<<-EOS)
+  t.rename(#{from_column_name.inspect}, #{to_column_name.inspect})
+      EOS
+    else
+      buf.puts(<<-EOS)
 rename_column(#{table_name.inspect}, #{from_column_name.inspect}, #{to_column_name.inspect})
-    EOS
+      EOS
+    end
   end
 
   def append_change_column(table_name, column_name, attrs, buf)
     type = attrs.fetch(:type)
     options = attrs[:options] || {}
 
-    buf.puts(<<-EOS)
+    if @options[:bulk_change]
+      buf.puts(<<-EOS)
+  t.change(#{column_name.inspect}, #{type.inspect}, #{options.inspect})
+      EOS
+    else
+      buf.puts(<<-EOS)
 change_column(#{table_name.inspect}, #{column_name.inspect}, #{type.inspect}, #{options.inspect})
-    EOS
+      EOS
+    end
   end
 
   def append_remove_column(table_name, column_name, attrs, buf)
-    buf.puts(<<-EOS)
+    if @options[:bulk_change]
+      buf.puts(<<-EOS)
+  t.remove(#{column_name.inspect})
+      EOS
+    else
+      buf.puts(<<-EOS)
 remove_column(#{table_name.inspect}, #{column_name.inspect})
-    EOS
+      EOS
+    end
   end
 
   def append_change_indices(table_name, delta, buf)
@@ -227,9 +264,15 @@ remove_column(#{table_name.inspect}, #{column_name.inspect})
     column_name = attrs.fetch(:column_name)
     options = attrs[:options] || {}
 
-    buf.puts(<<-EOS)
+    if @options[:bulk_change]
+      buf.puts(<<-EOS)
+  t.index(#{column_name.inspect}, #{options.inspect})
+      EOS
+    else
+      buf.puts(<<-EOS)
 add_index(#{table_name.inspect}, #{column_name.inspect}, #{options.inspect})
-    EOS
+      EOS
+    end
   end
 
   def append_remove_index(table_name, index_name, attrs, buf)
@@ -237,8 +280,14 @@ add_index(#{table_name.inspect}, #{column_name.inspect}, #{options.inspect})
     options = attrs[:options] || {}
     target = options[:name] ? {:name => options[:name]} : column_name
 
-    buf.puts(<<-EOS)
+    if @options[:bulk_change]
+      buf.puts(<<-EOS)
+  t.remove_index(#{target.inspect})
+      EOS
+    else
+      buf.puts(<<-EOS)
 remove_index(#{table_name.inspect}, #{target.inspect})
-    EOS
+      EOS
+    end
   end
 end

data/lib/ridgepole/version.rb

@@ -1,3 +1,3 @@
 module Ridgepole
-  VERSION = '0.3.2'
+  VERSION = '0.3.3'
 end

data/spec/migrate/migrate_add_column_spec.rb

@@ -162,5 +162,23 @@ describe 'Ridgepole::Client#diff -> migrate' do
         remove_column("employees", "updated_at")
       RUBY
     }
+
+    it {
+      delta = client(:bulk_change => true).diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        change_table("employee_clubs", {:bulk => true}) do |t|
+          t.column("any_col", :string, {:null=>false, :after=>"club_id"})
+        end
+
+        change_table("employees", {:bulk => true}) do |t|
+          t.column("age", :integer, {:unsigned=>true, :null=>false, :after=>"hire_date"})
+          t.column("updated_at", :date, {:after=>"age"})
+        end
+      RUBY
+      delta.migrate
+      expect(subject.dump).to eq expected_dsl.strip_heredoc.strip
+    }
   end
 end

data/spec/migrate/migrate_change_column_spec.rb

@@ -159,5 +159,23 @@ describe 'Ridgepole::Client#diff -> migrate' do
         change_column("employees", "gender", :string, {:limit=>1, :null=>false, :unsigned=>false})
       RUBY
     }
+
+    it {
+      delta = client(:bulk_change => true).diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        change_table("employee_clubs", {:bulk => true}) do |t|
+          t.change("club_id", :integer, {:unsigned=>false, :null=>true})
+        end
+
+        change_table("employees", {:bulk => true}) do |t|
+          t.change("last_name", :string, {:limit=>20, :default=>"XXX", :null=>false, :unsigned=>false})
+          t.change("gender", :string, {:limit=>2, :null=>false, :unsigned=>false})
+        end
+      RUBY
+      delta.migrate
+      expect(subject.dump).to eq expected_dsl.strip_heredoc.strip.gsub(/(\s*,\s*unsigned: false)?\s*,\s*null: true/, '')
+    }
   end
 end

data/spec/migrate/migrate_change_index_spec.rb

@@ -0,0 +1,263 @@
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when create index' do
+    let(:dsl) {
+      <<-RUBY
+        create_table "clubs", force: true do |t|
+          t.string "name", default: "", null: false
+        end
+
+        add_index "clubs", ["name"], name: "idx_name", unique: true, using: :btree
+
+        create_table "departments", primary_key: "dept_no", force: true do |t|
+          t.string "dept_name", limit: 40, null: false
+        end
+
+        add_index "departments", ["dept_name"], name: "dept_name", unique: true, using: :btree
+
+        create_table "dept_emp", id: false, force: true do |t|
+          t.integer "emp_no",              null: false
+          t.string  "dept_no",   limit: 4, null: false
+          t.date    "from_date",           null: false
+          t.date    "to_date",             null: false
+        end
+
+        add_index "dept_emp", ["dept_no"], name: "dept_no", using: :btree
+        add_index "dept_emp", ["emp_no"], name: "emp_no", using: :btree
+
+        create_table "dept_manager", id: false, force: true do |t|
+          t.string  "dept_no",   limit: 4, null: false
+          t.integer "emp_no",              null: false
+          t.date    "from_date",           null: false
+          t.date    "to_date",             null: false
+        end
+
+        add_index "dept_manager", ["dept_no"], name: "dept_no", using: :btree
+        add_index "dept_manager", ["emp_no"], name: "emp_no", using: :btree
+
+        create_table "employee_clubs", force: true do |t|
+          t.integer "emp_no",  unsigned: true, null: false
+          t.integer "club_id", unsigned: true, null: false
+        end
+
+        add_index "employee_clubs", ["emp_no", "club_id"], name: "idx_emp_no_club_id", using: :btree
+
+        create_table "employees", primary_key: "emp_no", force: true 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
+
+        create_table "salaries", id: false, force: true do |t|
+          t.integer "emp_no",    null: false
+          t.integer "salary",    null: false
+          t.date    "from_date", null: false
+          t.date    "to_date",   null: false
+        end
+
+        add_index "salaries", ["emp_no"], name: "emp_no", using: :btree
+
+        create_table "titles", id: false, force: true do |t|
+          t.integer "emp_no",               null: false
+          t.string  "title",     limit: 50, null: false
+          t.date    "from_date",            null: false
+          t.date    "to_date"
+        end
+
+        add_index "titles", ["emp_no"], name: "emp_no", using: :btree
+      RUBY
+    }
+
+    let(:actual_dsl) {
+      <<-RUBY
+        create_table "clubs", force: true do |t|
+          t.string "name", default: "", null: false
+        end
+
+        add_index "clubs", ["name"], name: "idx_name", unique: true, using: :btree
+
+        create_table "departments", primary_key: "dept_no", force: true do |t|
+          t.string "dept_name", limit: 40, null: false
+        end
+
+        add_index "departments", ["dept_name"], name: "dept_name", unique: true, using: :btree
+
+        create_table "dept_emp", id: false, force: true do |t|
+          t.integer "emp_no",              null: false
+          t.string  "dept_no",   limit: 4, null: false
+          t.date    "from_date",           null: false
+          t.date    "to_date",             null: false
+        end
+
+        add_index "dept_emp", ["dept_no"], name: "dept_no", using: :btree
+        add_index "dept_emp", ["emp_no"], name: "emp_no", using: :btree
+
+        create_table "dept_manager", id: false, force: true do |t|
+          t.string  "dept_no",   limit: 4, null: false
+          t.integer "emp_no",              null: false
+          t.date    "from_date",           null: false
+          t.date    "to_date",             null: false
+        end
+
+        add_index "dept_manager", ["dept_no"], name: "dept_no", using: :btree
+        add_index "dept_manager", ["emp_no"], name: "emp_no", using: :btree
+
+        create_table "employee_clubs", force: true do |t|
+          t.integer "emp_no",  unsigned: true, null: false
+          t.integer "club_id", unsigned: true, null: false
+        end
+
+        add_index "employee_clubs", ["emp_no", "club_id"], name: "idx_emp_no_club_id", using: :btree
+
+        create_table "employees", primary_key: "emp_no", force: true 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
+
+        create_table "salaries", id: false, force: true do |t|
+          t.integer "emp_no",    null: false
+          t.integer "salary",    null: false
+          t.date    "from_date", null: false
+          t.date    "to_date",   null: false
+        end
+
+        add_index "salaries", ["emp_no"], name: "emp_no", using: :btree
+
+        create_table "titles", id: false, force: true do |t|
+          t.integer "emp_no",               null: false
+          t.string  "title",     limit: 50, null: false
+          t.date    "from_date",            null: false
+          t.date    "to_date"
+        end
+
+        add_index "titles", ["emp_no"], name: "emp_no", using: :btree
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+        create_table "clubs", force: true do |t|
+          t.string "name", default: "", null: false
+        end
+
+        add_index "clubs", ["name"], name: "idx_name", unique: true, using: :btree
+
+        create_table "departments", primary_key: "dept_no", force: true do |t|
+          t.string "dept_name", limit: 40, null: false
+        end
+
+        add_index "departments", ["dept_name"], name: "dept_name", unique: true, using: :btree
+
+        create_table "dept_emp", id: false, force: true do |t|
+          t.integer "emp_no",              null: false
+          t.string  "dept_no",   limit: 4, null: false
+          t.date    "from_date",           null: false
+          t.date    "to_date",             null: false
+        end
+
+        add_index "dept_emp", ["dept_no"], name: "dept_no", using: :btree
+        add_index "dept_emp", ["from_date"], name: "emp_no", using: :btree
+
+        create_table "dept_manager", id: false, force: true do |t|
+          t.string  "dept_no",   limit: 4, null: false
+          t.integer "emp_no",              null: false
+          t.date    "from_date",           null: false
+          t.date    "to_date",             null: false
+        end
+
+        add_index "dept_manager", ["dept_no"], name: "dept_no", using: :btree
+        add_index "dept_manager", ["from_date"], name: "emp_no", using: :btree
+
+        create_table "employee_clubs", force: true do |t|
+          t.integer "emp_no",  unsigned: true, null: false
+          t.integer "club_id", unsigned: true, null: false
+        end
+
+        add_index "employee_clubs", ["emp_no", "club_id"], name: "idx_emp_no_club_id", using: :btree
+
+        create_table "employees", primary_key: "emp_no", force: true 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
+
+        create_table "salaries", id: false, force: true do |t|
+          t.integer "emp_no",    null: false
+          t.integer "salary",    null: false
+          t.date    "from_date", null: false
+          t.date    "to_date",   null: false
+        end
+
+        add_index "salaries", ["from_date"], name: "emp_no", using: :btree
+
+        create_table "titles", id: false, force: true do |t|
+          t.integer "emp_no",               null: false
+          t.string  "title",     limit: 50, null: false
+          t.date    "from_date",            null: false
+          t.date    "to_date"
+        end
+
+        add_index "titles", ["emp_no"], name: "emp_no", using: :btree
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump.delete_empty_lines).to eq actual_dsl.strip_heredoc.strip.delete_empty_lines
+      delta.migrate
+      expect(subject.dump.delete_empty_lines).to eq expected_dsl.strip_heredoc.strip.delete_empty_lines
+    }
+
+    it {
+      delta = Ridgepole::Client.diff(actual_dsl, expected_dsl, reverse: true)
+      expect(delta.differ?).to be_truthy
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        remove_index("dept_emp", {:name=>"emp_no"})
+        add_index("dept_emp", ["emp_no"], {:name=>"emp_no", :using=>:btree})
+
+        remove_index("dept_manager", {:name=>"emp_no"})
+        add_index("dept_manager", ["emp_no"], {:name=>"emp_no", :using=>:btree})
+
+        remove_index("salaries", {:name=>"emp_no"})
+        add_index("salaries", ["emp_no"], {:name=>"emp_no", :using=>:btree})
+      RUBY
+    }
+
+    it {
+      delta = client(:bulk_change => true).diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump.delete_empty_lines).to eq actual_dsl.strip_heredoc.strip.delete_empty_lines
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        change_table("dept_emp", {:bulk => true}) do |t|
+          t.remove_index({:name=>"emp_no"})
+          t.index(["from_date"], {:name=>"emp_no", :using=>:btree})
+        end
+
+        change_table("dept_manager", {:bulk => true}) do |t|
+          t.remove_index({:name=>"emp_no"})
+          t.index(["from_date"], {:name=>"emp_no", :using=>:btree})
+        end
+
+        change_table("salaries", {:bulk => true}) do |t|
+          t.remove_index({:name=>"emp_no"})
+          t.index(["from_date"], {:name=>"emp_no", :using=>:btree})
+        end
+      RUBY
+
+      # XXX: Can not add an index of the same name
+      expect {
+        delta.migrate
+      }.to raise_error(/Index name 'emp_no' on table 'dept_emp' already exists/)
+    }
+  end
+end

data/spec/migrate/migrate_create_index_spec.rb

@@ -99,5 +99,26 @@ describe 'Ridgepole::Client#diff -> migrate' do
        remove_index("titles", {:name=>"emp_no"})
       RUBY
     }
+
+    it {
+      delta = client(:bulk_change => true).diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump.delete_empty_lines).to eq actual_dsl.strip_heredoc.strip.delete_empty_lines
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        change_table("clubs", {:bulk => true}) do |t|
+          t.index(["name"], {:name=>"idx_name", :unique=>true, :using=>:btree})
+        end
+
+        change_table("employee_clubs", {:bulk => true}) do |t|
+          t.index(["emp_no", "club_id"], {:name=>"idx_emp_no_club_id", :using=>:btree})
+        end
+
+        change_table("titles", {:bulk => true}) do |t|
+          t.index(["emp_no"], {:name=>"emp_no", :using=>:btree})
+        end
+      RUBY
+      delta.migrate
+      expect(subject.dump.delete_empty_lines).to eq expected_dsl.strip_heredoc.strip.delete_empty_lines
+    }
   end
 end

data/spec/migrate/migrate_drop_column_and_index_spec.rb

@@ -144,5 +144,36 @@ describe 'Ridgepole::Client#diff -> migrate' do
       delta.migrate(:noop => true)
       expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
     }
+
+    it {
+      delta = client(:bulk_change => true).diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+change_table("dept_emp", {:bulk => true}) do |t|
+  t.remove("emp_no")
+  t.remove("from_date")
+  t.remove("to_date")
+end
+
+change_table("dept_manager", {:bulk => true}) do |t|
+  t.remove("emp_no")
+  t.remove("from_date")
+  t.remove("to_date")
+end
+
+change_table("employee_clubs", {:bulk => true}) do |t|
+  t.remove("club_id")
+end
+
+change_table("employees", {:bulk => true}) do |t|
+  t.remove("last_name")
+  t.remove("gender")
+  t.remove("hire_date")
+end
+      RUBY
+      delta.migrate
+      expect(subject.dump).to eq expected_dsl.strip_heredoc.strip
+    }
   end
 end

data/spec/migrate/migrate_drop_column_spec.rb

@@ -157,5 +157,30 @@ describe 'Ridgepole::Client#diff -> migrate' do
         add_column("employees", "hire_date", :date, {:null=>false, :after=>"gender"})
       RUBY
     }
+
+    it {
+      delta = client(:bulk_change => true).diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        change_table("dept_emp", {:bulk => true}) do |t|
+          t.remove("from_date")
+          t.remove("to_date")
+        end
+
+        change_table("dept_manager", {:bulk => true}) do |t|
+          t.remove("from_date")
+          t.remove("to_date")
+        end
+
+        change_table("employees", {:bulk => true}) do |t|
+          t.remove("last_name")
+          t.remove("gender")
+          t.remove("hire_date")
+        end
+      RUBY
+      delta.migrate
+      expect(subject.dump).to eq expected_dsl.strip_heredoc.strip
+    }
   end
 end

data/spec/migrate/migrate_drop_index_spec.rb

@@ -99,5 +99,26 @@ describe 'Ridgepole::Client#diff -> migrate' do
         add_index("titles", ["emp_no"], {:name=>"emp_no", :using=>:btree})
       RUBY
     }
+
+    it {
+      delta = client(:bulk_change => true).diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        change_table("clubs", {:bulk => true}) do |t|
+          t.remove_index({:name=>"idx_name"})
+        end
+
+        change_table("employee_clubs", {:bulk => true}) do |t|
+          t.remove_index({:name=>"idx_emp_no_club_id"})
+        end
+
+        change_table("titles", {:bulk => true}) do |t|
+          t.remove_index({:name=>"emp_no"})
+        end
+      RUBY
+      delta.migrate
+      expect(subject.dump.each_line.select {|i| i !~ /\A\Z/ }.join).to eq expected_dsl.strip_heredoc.strip.each_line.select {|i| i !~ /\A\Z/ }.join
+    }
   end
 end

data/spec/migrate/migrate_noop_spec.rb

@@ -100,5 +100,32 @@ describe 'Ridgepole::Client#diff -> migrate' do
         CREATE  INDEX `emp_no` USING btree ON `titles` (`emp_no`)
       SQL
     }
+
+    it {
+      delta = client(:bulk_change => true).diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      sql = delta.migrate(:noop => true)
+      expect(subject.dump).to eq actual_dsl
+
+      sql = sql.each_line.map {|i| i.strip }.join("\n")
+
+      expect(sql).to eq <<-SQL.strip_heredoc.strip
+        CREATE TABLE `clubs` (`id` int(10) unsigned DEFAULT NULL auto_increment PRIMARY KEY, `name` varchar(255) DEFAULT '' NOT NULL) ENGINE=InnoDB
+        ALTER TABLE `clubs` ADD UNIQUE INDEX idx_name (`name`)
+        CREATE TABLE `departments` (`dept_no` int(10) unsigned DEFAULT NULL auto_increment PRIMARY KEY, `dept_name` varchar(40) NOT NULL) ENGINE=InnoDB
+        ALTER TABLE `departments` ADD UNIQUE INDEX dept_name (`dept_name`)
+        CREATE TABLE `dept_emp` (`emp_no` int(10) NOT NULL, `dept_no` varchar(4) NOT NULL, `from_date` date NOT NULL, `to_date` date NOT NULL) ENGINE=InnoDB
+        ALTER TABLE `dept_emp` ADD  INDEX dept_no (`dept_no`), ADD  INDEX emp_no (`emp_no`)
+        CREATE TABLE `dept_manager` (`dept_no` varchar(4) NOT NULL, `emp_no` int(10) NOT NULL, `from_date` date NOT NULL, `to_date` date NOT NULL) ENGINE=InnoDB
+        ALTER TABLE `dept_manager` ADD  INDEX dept_no (`dept_no`), ADD  INDEX emp_no (`emp_no`)
+        CREATE TABLE `employee_clubs` (`id` int(10) unsigned DEFAULT NULL auto_increment PRIMARY KEY, `emp_no` int(10) unsigned NOT NULL, `club_id` int(10) unsigned NOT NULL) ENGINE=InnoDB
+        ALTER TABLE `employee_clubs` ADD  INDEX idx_emp_no_club_id (`emp_no`, `club_id`)
+        CREATE TABLE `employees` (`emp_no` int(10) unsigned DEFAULT NULL auto_increment PRIMARY KEY, `birth_date` date NOT NULL, `first_name` varchar(14) NOT NULL, `last_name` varchar(16) NOT NULL, `gender` varchar(1) NOT NULL, `hire_date` date NOT NULL) ENGINE=InnoDB
+        CREATE TABLE `salaries` (`emp_no` int(10) NOT NULL, `salary` int(10) NOT NULL, `from_date` date NOT NULL, `to_date` date NOT NULL) ENGINE=InnoDB
+        ALTER TABLE `salaries` ADD  INDEX emp_no (`emp_no`)
+        CREATE TABLE `titles` (`emp_no` int(10) NOT NULL, `title` varchar(50) NOT NULL, `from_date` date NOT NULL, `to_date` date) ENGINE=InnoDB
+        ALTER TABLE `titles` ADD  INDEX emp_no (`emp_no`)
+      SQL
+    }
   end
 end

data/spec/migrate/migrate_rename_column_spec.rb

@@ -160,6 +160,27 @@ describe 'Ridgepole::Client#diff -> migrate' do
        rename_column("employees", "gender2", "gender")
       RUBY
     }
+
+    it {
+      delta = client(:bulk_change => true).diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        change_table("dept_emp", {:bulk => true}) do |t|
+          t.rename("from_date", "from_date2")
+        end
+
+        change_table("dept_manager", {:bulk => true}) do |t|
+          t.rename("to_date", "to_date2")
+        end
+
+        change_table("employees", {:bulk => true}) do |t|
+          t.rename("gender", "gender2")
+        end
+      RUBY
+      delta.migrate
+      expect(subject.dump).to eq expected_dsl.strip_heredoc.strip.gsub(/\s*,\s*renamed_from:.*$/, '')
+    }
   end
 
   context 'when rename column (not found)' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ridgepole
 version: !ruby/object:Gem::Version
-  version: 0.3.2
+  version: 0.3.3
 platform: ruby
 authors:
 - Genki Sugawara
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-28 00:00:00.000000000 Z
+date: 2014-08-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -131,6 +131,7 @@ files:
 - spec/migrate/check_orphan_index_spec.rb
 - spec/migrate/migrate_add_column_spec.rb
 - spec/migrate/migrate_change_column_spec.rb
+- spec/migrate/migrate_change_index_spec.rb
 - spec/migrate/migrate_change_table_option_spec.rb
 - spec/migrate/migrate_create_index_spec.rb
 - spec/migrate/migrate_create_table_spec.rb
@@ -192,6 +193,7 @@ test_files:
 - spec/migrate/check_orphan_index_spec.rb
 - spec/migrate/migrate_add_column_spec.rb
 - spec/migrate/migrate_change_column_spec.rb
+- spec/migrate/migrate_change_index_spec.rb
 - spec/migrate/migrate_change_table_option_spec.rb
 - spec/migrate/migrate_create_index_spec.rb
 - spec/migrate/migrate_create_table_spec.rb