ridgepole 0.4.1 → 0.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 74af55adebfffcd9880e5a0fde3e2eee3fc2c75d
-  data.tar.gz: 1b3255a9a2c1b29b3980b32739a55b83100542c3
+  metadata.gz: a09898d122a3138706ad9e835a0e0941a56749be
+  data.tar.gz: e0bd6aa6ba825488ef7e53fcd68db3267dcde79a
 SHA512:
-  metadata.gz: 83ae32148265a558bb781c2eadf4cc3b988d2a4380fb66be2e7e13163b239e7ea1c3d1a4136f3d7149ede683338dc0fb2b0e8ced5c8d3b460fd4d9647bb10fe0
-  data.tar.gz: 9099196ec666a10f7da892c3af21302e654c0801516815e1a888eec86a034d649cfe4eb940cd8ffafa0cd457c08b63657a1cf05621a57793ee78773143a31a42
+  metadata.gz: ecf91300412d185e49794b4a894180d70747783b88a641856d3af20e35b668aa24201c4a5188cbeb68e2f1c83177dccf19eddafb66a4ad61f807e5f688ac569c
+  data.tar.gz: 9f83de31e8549e7956b79b0c3033208354a4e823901384406ca171295f33a5ea98f8c9f4f224af917a4854ae395d05a6112d81b00aac3267b88ac6266f41bfff

data/README.md

@@ -32,6 +32,7 @@ Usage: ridgepole [options]
         --dry-run
         --table-options OPTIONS
         --bulk-change
+        --default-int-limit LIMIT
         --pre-query QUERY
         --post-query QUERY
     -e, --export

data/bin/ridgepole

@@ -51,6 +51,9 @@ ARGV.options do |opt|
       raise "Cannot use `bulk-change` in `merge`" if options[:merge]
       options[:bulk_change] = true
     }
+    opt.on('',   '--default-int-limit LIMIT', Integer) {|v|
+      options[:default_int_limit] = v
+    }
     opt.on('',   '--pre-query QUERY')             {|v| options[:pre_query] = v                              }
     opt.on('',   '--post-query QUERY')            {|v| options[:post_query] = v                             }
     opt.on('-e', '--export')                      {    set_mode[:export]                                    }

data/lib/ridgepole/delta.rb

@@ -153,6 +153,10 @@ create_table(#{table_name.inspect}, #{options.inspect}) do |t|
       column_type = column_attrs.fetch(:type)
       column_options = column_attrs[:options] || {}
 
+      if @options[:default_int_limit] and column_type == :integer
+        column_options[:limit] ||= @options[:default_int_limit]
+      end
+
       buf.puts(<<-EOS)
   t.#{column_type}(#{column_name.inspect}, #{column_options.inspect})
       EOS
@@ -226,6 +230,10 @@ drop_table(#{table_name.inspect})
     type = attrs.fetch(:type)
     options = attrs[:options] || {}
 
+    if @options[:default_int_limit] and type == :integer
+      options[:limit] ||= @options[:default_int_limit]
+    end
+
     if @options[:bulk_change]
       buf.puts(<<-EOS)
   t.column(#{column_name.inspect}, #{type.inspect}, #{options.inspect})

data/lib/ridgepole/dsl_parser.rb

@@ -37,6 +37,22 @@ class Ridgepole::DSLParser
           column_names.each {|name| column(name, column_type, options) }
         end
       end
+
+      def timestamps(*args)
+        options = { :null => false }.merge(args.extract_options!)
+        column(:created_at, :datetime, options.dup)
+        column(:updated_at, :datetime, options.dup)
+      end
+
+      def references(*args)
+        options = args.extract_options!
+        polymorphic = options.delete(:polymorphic)
+        args.each do |col|
+          column("#{col}_id", :integer, options.dup)
+          column("#{col}_type", :string, polymorphic.is_a?(Hash) ? polymorphic : options.dup) unless polymorphic.nil?
+        end
+      end
+      alias :belongs_to :references
     end
 
     attr_reader :__definition

data/lib/ridgepole/version.rb

@@ -1,3 +1,3 @@
 module Ridgepole
-  VERSION = '0.4.1'
+  VERSION = '0.4.2'
 end

data/ridgepole.gemspec

@@ -19,7 +19,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
 
   spec.add_dependency 'activerecord'
-  spec.add_dependency 'activerecord-mysql-unsigned', '~> 0.1.3'
+  spec.add_dependency 'activerecord-mysql-unsigned', '>= 0.2.0'
   spec.add_development_dependency 'bundler'
   spec.add_development_dependency 'rake'
   spec.add_development_dependency 'rspec', '>= 3.0.0'

data/spec/migrate/migrate_add_column2_spec.rb

@@ -0,0 +1,118 @@
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when add column (int/noop) (1)' do
+    let(:actual_dsl) {
+      <<-RUBY
+        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
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+        create_table "dept_emp", id: false, force: true do |t|
+          t.integer "emp_no",              null: false
+          t.integer "emp_no2",             null: false
+          t.string  "dept_no",   limit: 4, null: false
+          t.date    "from_date",           null: false
+          t.date    "to_date",             null: false
+        end
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      sql = delta.migrate(:noop => true)
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+
+      sql = sql.each_line.map {|i| i.strip }.join("\n")
+      expect(sql).to eq("ALTER TABLE `dept_emp` ADD `emp_no2` int(4) NOT NULL AFTER `emp_no`")
+    }
+  end
+
+  context 'when add column (int/noop) (2)' do
+    let(:actual_dsl) {
+      <<-RUBY
+        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
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+        create_table "dept_emp", id: false, force: true do |t|
+          t.integer "emp_no",              null: false
+          t.integer "emp_no2",             null: false
+          t.string  "dept_no",   limit: 4, null: false
+          t.date    "from_date",           null: false
+          t.date    "to_date",             null: false
+        end
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client(:default_int_limit => 11) }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      sql = delta.migrate(:noop => true)
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+
+      sql = sql.each_line.map {|i| i.strip }.join("\n")
+      expect(sql).to eq("ALTER TABLE `dept_emp` ADD `emp_no2` int(11) NOT NULL AFTER `emp_no`")
+    }
+  end
+
+  context 'when add column (int/noop) (3)' do
+    let(:actual_dsl) {
+      <<-RUBY
+        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
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+        create_table "dept_emp", id: false, force: true do |t|
+          t.integer "emp_no",              null: false
+          t.integer "emp_no2",   limit: 4, null: false
+          t.string  "dept_no",   limit: 4, null: false
+          t.date    "from_date",           null: false
+          t.date    "to_date",             null: false
+        end
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client(:default_int_limit => 11) }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      sql = delta.migrate(:noop => true)
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+
+      sql = sql.each_line.map {|i| i.strip }.join("\n")
+      expect(sql).to eq("ALTER TABLE `dept_emp` ADD `emp_no2` int(4) NOT NULL AFTER `emp_no`")
+    }
+  end
+end

data/spec/migrate/migrate_change_column3_spec.rb

@@ -0,0 +1,272 @@
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when use timestamps (no change)' do
+    let(:actual_dsl) {
+      <<-RUBY
+        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
+          t.datetime "created_at",            null: false
+          t.datetime "updated_at",            null: false
+        end
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+        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
+          t.timestamps
+        end
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_falsey
+    }
+  end
+
+  context 'when use timestamps (change)' do
+    let(:actual_dsl) {
+      <<-RUBY
+        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
+      RUBY
+    }
+
+    let(:dsl) {
+      <<-RUBY
+        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
+          t.timestamps
+        end
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+        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
+          t.datetime "created_at",            null: false
+          t.datetime "updated_at",            null: false
+        end
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff(dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      delta.migrate
+      expect(subject.dump).to eq expected_dsl.strip_heredoc.strip.gsub(/(\s*,\s*unsigned: false)?\s*,\s*null: true/, '')
+    }
+  end
+
+  context 'when use references (no change)' do
+    let(:actual_dsl) {
+      <<-RUBY
+        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
+          t.integer "products_id"
+          t.integer "user_id"
+        end
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+        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
+          t.references :products, :user
+        end
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_falsey
+    }
+  end
+
+  context 'when use references with polymorphic (no change)' do
+    let(:actual_dsl) {
+      <<-RUBY
+        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
+          t.integer  "products_id"
+          t.string   "products_type"
+          t.integer  "user_id"
+          t.string   "user_type"
+        end
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+        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
+          t.references :products, :user, polymorphic: true
+        end
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_falsey
+    }
+  end
+
+  context 'when use references (change)' do
+    let(:actual_dsl) {
+      <<-RUBY
+        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
+      RUBY
+    }
+
+    let(:dsl) {
+      <<-RUBY
+        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
+          t.references :products, :user
+        end
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+        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
+          t.integer "products_id"
+          t.integer "user_id"
+        end
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff(dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      delta.migrate
+      expect(subject.dump).to eq expected_dsl.strip_heredoc.strip.gsub(/(\s*,\s*unsigned: false)?\s*,\s*null: true/, '')
+    }
+  end
+
+  context 'when use references with polymorphic (change)' do
+    let(:actual_dsl) {
+      <<-RUBY
+        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
+      RUBY
+    }
+
+    let(:dsl) {
+      <<-RUBY
+        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
+          t.references :products, :user, polymorphic: true
+        end
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+        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
+          t.integer "products_id"
+          t.string  "products_type"
+          t.integer "user_id"
+          t.string  "user_type"
+        end
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff(dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      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_noop_spec.rb

@@ -85,18 +85,18 @@ describe 'Ridgepole::Client#diff -> migrate' do
         CREATE UNIQUE INDEX `idx_name` USING btree ON `clubs` (`name`)
         CREATE TABLE `departments` (`dept_no` int(10) unsigned DEFAULT NULL auto_increment PRIMARY KEY, `dept_name` varchar(40) NOT NULL) ENGINE=InnoDB
         CREATE UNIQUE INDEX `dept_name` USING btree ON `departments` (`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
+        CREATE TABLE `dept_emp` (`emp_no` int(4) NOT NULL, `dept_no` varchar(4) NOT NULL, `from_date` date NOT NULL, `to_date` date NOT NULL) ENGINE=InnoDB
         CREATE  INDEX `dept_no` USING btree ON `dept_emp` (`dept_no`)
         CREATE  INDEX `emp_no` USING btree ON `dept_emp` (`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
+        CREATE TABLE `dept_manager` (`dept_no` varchar(4) NOT NULL, `emp_no` int(4) NOT NULL, `from_date` date NOT NULL, `to_date` date NOT NULL) ENGINE=InnoDB
         CREATE  INDEX `dept_no` USING btree ON `dept_manager` (`dept_no`)
         CREATE  INDEX `emp_no` USING btree ON `dept_manager` (`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
         CREATE  INDEX `idx_emp_no_club_id` USING btree ON `employee_clubs` (`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
+        CREATE TABLE `salaries` (`emp_no` int(4) NOT NULL, `salary` int(4) NOT NULL, `from_date` date NOT NULL, `to_date` date NOT NULL) ENGINE=InnoDB
         CREATE  INDEX `emp_no` USING btree ON `salaries` (`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
+        CREATE TABLE `titles` (`emp_no` int(4) NOT NULL, `title` varchar(50) NOT NULL, `from_date` date NOT NULL, `to_date` date) ENGINE=InnoDB
         CREATE  INDEX `emp_no` USING btree ON `titles` (`emp_no`)
       SQL
     }
@@ -114,18 +114,121 @@ describe 'Ridgepole::Client#diff -> migrate' do
         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
+        CREATE TABLE `dept_emp` (`emp_no` int(4) 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
+        CREATE TABLE `dept_manager` (`dept_no` varchar(4) NOT NULL, `emp_no` int(4) 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
+        CREATE TABLE `salaries` (`emp_no` int(4) NOT NULL, `salary` int(4) 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
+        CREATE TABLE `titles` (`emp_no` int(4) 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
+
+  context 'when no operation' do
+    let(:actual_dsl) { '' }
+    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", ["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",    limit: 4, 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
+    }
+
+    subject { client(:default_int_limit => 11) }
+
+    it {
+      delta = subject.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
+        CREATE UNIQUE INDEX `idx_name` USING btree ON `clubs` (`name`)
+        CREATE TABLE `departments` (`dept_no` int(10) unsigned DEFAULT NULL auto_increment PRIMARY KEY, `dept_name` varchar(40) NOT NULL) ENGINE=InnoDB
+        CREATE UNIQUE INDEX `dept_name` USING btree ON `departments` (`dept_name`)
+        CREATE TABLE `dept_emp` (`emp_no` int(11) NOT NULL, `dept_no` varchar(4) NOT NULL, `from_date` date NOT NULL, `to_date` date NOT NULL) ENGINE=InnoDB
+        CREATE  INDEX `dept_no` USING btree ON `dept_emp` (`dept_no`)
+        CREATE  INDEX `emp_no` USING btree ON `dept_emp` (`emp_no`)
+        CREATE TABLE `dept_manager` (`dept_no` varchar(4) NOT NULL, `emp_no` int(4) NOT NULL, `from_date` date NOT NULL, `to_date` date NOT NULL) ENGINE=InnoDB
+        CREATE  INDEX `dept_no` USING btree ON `dept_manager` (`dept_no`)
+        CREATE  INDEX `emp_no` USING btree ON `dept_manager` (`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
+        CREATE  INDEX `idx_emp_no_club_id` USING btree ON `employee_clubs` (`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(11) NOT NULL, `salary` int(11) NOT NULL, `from_date` date NOT NULL, `to_date` date NOT NULL) ENGINE=InnoDB
+        CREATE  INDEX `emp_no` USING btree ON `salaries` (`emp_no`)
+        CREATE TABLE `titles` (`emp_no` int(11) NOT NULL, `title` varchar(50) NOT NULL, `from_date` date NOT NULL, `to_date` date) ENGINE=InnoDB
+        CREATE  INDEX `emp_no` USING btree ON `titles` (`emp_no`)
+      SQL
+    }
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ridgepole
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.4.2
 platform: ruby
 authors:
 - Genki Sugawara
@@ -28,16 +28,16 @@ dependencies:
   name: activerecord-mysql-unsigned
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - '>='
       - !ruby/object:Gem::Version
-        version: 0.1.3
+        version: 0.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - '>='
       - !ruby/object:Gem::Version
-        version: 0.1.3
+        version: 0.2.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -129,8 +129,10 @@ files:
 - spec/dump/dump_some_tables_spec.rb
 - spec/dump/dump_spec.rb
 - spec/migrate/check_orphan_index_spec.rb
+- spec/migrate/migrate_add_column2_spec.rb
 - spec/migrate/migrate_add_column_spec.rb
 - spec/migrate/migrate_change_column2_spec.rb
+- spec/migrate/migrate_change_column3_spec.rb
 - spec/migrate/migrate_change_column_spec.rb
 - spec/migrate/migrate_change_index2_spec.rb
 - spec/migrate/migrate_change_index_spec.rb
@@ -194,8 +196,10 @@ test_files:
 - spec/dump/dump_some_tables_spec.rb
 - spec/dump/dump_spec.rb
 - spec/migrate/check_orphan_index_spec.rb
+- spec/migrate/migrate_add_column2_spec.rb
 - spec/migrate/migrate_add_column_spec.rb
 - spec/migrate/migrate_change_column2_spec.rb
+- spec/migrate/migrate_change_column3_spec.rb
 - spec/migrate/migrate_change_column_spec.rb
 - spec/migrate/migrate_change_index2_spec.rb
 - spec/migrate/migrate_change_index_spec.rb