ridgepole 0.5.1 → 0.5.2.beta

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4ded281de053ba8c3ee49dde9acf4f1f5dbd0a46
-  data.tar.gz: b9dc9e5fd0ee3a92420793f4485ec11930da9cf6
+  metadata.gz: e89391bd7d67f71ab9d326e42226fffaa1c23962
+  data.tar.gz: a1966c6828572057af20625308e2cd53c0d1875e
 SHA512:
-  metadata.gz: a678c7f6b5744d4b4b9dd7fa1ed0c780a10b8e6299610929ec2f51b8f4a05bd964fd4d50ebfc619b456119aa3b51e94e1ac93d513019a6acb7e946b7f606ca0c
-  data.tar.gz: e600e2d2ce35962f73ac90f00c517f6528a0ccdde124e068e1dcea1588ae9705dcfe9f0b2e42f28f2e6148a4413150d6a54f558f2c1d415b57c73a56df1c04f7
+  metadata.gz: 33d6c97cf20903b6cbaaf4a2d1a15d6e4c49984011fd443a460cdff5b3a63af6cdf55ed2529674a62fbff468f678da4493bfe6994346ef31fe144396d08f34cf
+  data.tar.gz: 4dac5448a53e81c93847ae322e07940ec296627c8162afe62acdc406a4925992ca9d7e1e1414dec67c88494662c01704ca51d7d9af758eec305620c750ff2d02

data/.travis.yml

@@ -4,3 +4,7 @@ rvm:
 script:
   - bundle install
   - bundle exec rake
+env:
+  matrix:
+    - ENABLE_MYSQL_AWESOME=0
+    - ENABLE_MYSQL_AWESOME=1

data/README.md

@@ -23,6 +23,9 @@ It defines DB schema using [Rails DSL](http://guides.rubyonrails.org/migrations.
 * `>= 0.5.1`
   * Add `--enable-migration-comments` option ([migration_comments](https://github.com/pinnymz/migration_comments) is required)
   * Fix rails version `< 4.2.0`
+* `>= 0.5.2`
+  * Add `--enable-mysql-awesome` option ([activerecord-mysql-awesome](https://github.com/kamipo/activerecord-mysql-awesome) is required)
+  * It is not possible to enable both `--enable-migration-comments` and `--enable-mysql-awesome`
 
 ## Installation
 
@@ -55,6 +58,7 @@ Usage: ridgepole [options]
     -e, --export
         --split
         --split-with-dir
+        --without-table-options
     -d, --diff DSL1 DSL2
         --reverse
         --with-apply
@@ -65,7 +69,9 @@ Usage: ridgepole [options]
         --enable-mysql-pkdump
         --enable-foreigner
         --enable-migration-comments
+        --enable-mysql-awesome
         --normalize-mysql-float
+    -r, --require LIBS
         --log-file LOG_FILE
         --verbose
         --debug

data/bin/ridgepole

@@ -74,6 +74,7 @@ ARGV.options do |opt|
     opt.on('-e', '--export')                      {    set_mode[:export]                                    }
     opt.on('',   '--split')                       {|v| split = true                                         }
     opt.on('',   '--split-with-dir')              {|v| split = :with_dir                                    }
+    opt.on('',   '--without-table-options')       {    options[:without_table_options] = true               }
     opt.on('-d', '--diff DSL1 DSL2') {|diff_arg1|
       set_mode[:diff]
       diff_arg2 = ARGV.first
@@ -94,8 +95,10 @@ ARGV.options do |opt|
     opt.on('',   '--enable-mysql-unsigned')       {    options[:enable_mysql_unsigned] = true               }
     opt.on('',   '--enable-mysql-pkdump')         {    options[:enable_mysql_pkdump] = true                 }
     opt.on('',   '--enable-foreigner')            {    options[:enable_foreigner] = true                    }
-    opt.on('',   '--enable-migration-comments')   {    options[:migration_comments] = true                  }
+    opt.on('',   '--enable-migration-comments')   {    options[:enable_migration_comments] = true           }
+    opt.on('',   '--enable-mysql-awesome')        {    options[:enable_mysql_awesome] = true                }
     opt.on('',   '--normalize-mysql-float')       {    options[:normalize_mysql_float] = true               }
+    opt.on('-r'  , '--require LIBS', Array)       {|v| v.each {|i| require i }                              }
     opt.on(''  , '--log-file LOG_FILE')           {|v| options[:log_file] = v                               }
     opt.on(''  , '--verbose')                     {    Ridgepole::Logger.verbose = true                     }
     opt.on(''  , '--debug')                       {    options[:debug] = true                               }
@@ -118,6 +121,10 @@ ARGV.options do |opt|
 end
 
 begin
+  if options[:enable_migration_comments] and options[:enable_mysql_awesome]
+    raise "It is not possible to enable both `--enable-migration-comments` and `--enable-mysql-awesome`"
+  end
+
   logger = Ridgepole::Logger.instance
   logger.set_debug(options[:debug])
 

data/lib/ridgepole/client.rb

@@ -15,10 +15,15 @@ class Ridgepole::Client
       require 'activerecord-mysql-pkdump'
     end
 
-    if @options[:migration_comments]
+    if @options[:enable_migration_comments]
       require 'migration_comments'
     end
 
+    if @options[:enable_mysql_awesome]
+      require 'activerecord/mysql/awesome/base'
+      require 'ridgepole/ext/mysql_awesome.rb'
+    end
+
     if @options[:enable_foreigner]
       Ridgepole::ForeignKey.init
     end

data/lib/ridgepole/diff.rb

@@ -255,7 +255,7 @@ class Ridgepole::Diff
     end
 
     # XXX: MySQL only?
-    if @options[:enable_mysql_unsigned]
+    if @options[:enable_mysql_unsigned] or @options[:enable_mysql_awesome]
       opts[:unsigned] = false unless opts.has_key?(:unsigned)
     end
 

data/lib/ridgepole/dumper.rb

@@ -34,6 +34,12 @@ class Ridgepole::Dumper
       line !~ /\A#/ &&
       line !~ /\AActiveRecord::Schema\.define/ &&
       line !~ /\Aend/
+    }.map {|line|
+      if @options[:without_table_options] and line =~ /\A  create_table /
+        line.gsub(/, options: "[^"]*"/, '')
+      else
+        line
+      end
     }.join.strip_heredoc
 
     definitions = []

data/lib/ridgepole/ext/mysql_awesome.rb

@@ -0,0 +1,8 @@
+require 'active_record/connection_adapters/abstract/schema_definitions'
+
+# XXX: https://github.com/waka/activerecord-mysql-unsigned/blob/v0.3.1/lib/activerecord-mysql-unsigned/active_record/v3/connection_adapters/abstract/schema_definitions.rb#L14
+class ActiveRecord::ConnectionAdapters::TableDefinition
+  def primary_key(name, type = :primary_key, options = {})
+    column(name, type, options.merge(primary_key: true).reverse_merge(unsigned: true))
+  end
+end

data/lib/ridgepole/version.rb

@@ -1,3 +1,3 @@
 module Ridgepole
-  VERSION = '0.5.1'
+  VERSION = '0.5.2.beta'
 end

data/ridgepole.gemspec

@@ -28,4 +28,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'foreigner'
   spec.add_development_dependency 'activerecord-mysql-pkdump', '>= 0.1.0'
   spec.add_development_dependency 'migration_comments'
+  spec.add_development_dependency 'activerecord-mysql-awesome'
 end

data/spec/0_diff/dump_disable_unsigned_spec.rb

@@ -1,7 +1,7 @@
 describe 'Ridgepole::Client#dump' do
   context 'when there is a tables (disable unsigned)' do
     before { restore_tables }
-    subject { client(enable_mysql_unsigned: false) }
+    subject { client(enable_mysql_unsigned: false, enable_mysql_awesome: false) }
 
     it {
       expect(subject.dump).to eq <<-RUBY.strip_heredoc.strip

data/spec/bigint_pk/bigint_pkspec.rb

@@ -0,0 +1,24 @@
+if mysql_awesome_enabled?
+  describe 'Ridgepole::Client (with bigint pk)' do
+    let(:dsl) {
+      <<-RUBY
+        create_table "books", id: :primary_key, limit: 8, force: true do |t|
+          t.string   "title",      null: false
+          t.integer  "author_id",  null: false
+          t.datetime "created_at"
+          t.datetime "updated_at"
+        end
+      RUBY
+    }
+
+    context 'when dump with activerecord-mysql-pkdump' do
+      subject { client }
+
+      before { subject.diff(dsl).migrate }
+
+      it {
+        expect(show_create_table(:books)).to include '`id` bigint(20) unsigned NOT NULL AUTO_INCREMENT'
+      }
+    end
+  end
+end

data/spec/cli/ridgepole_spec.rb

@@ -23,6 +23,7 @@ describe 'ridgepole' do
         -e, --export
             --split
             --split-with-dir
+            --without-table-options
         -d, --diff DSL1 DSL2
             --reverse
             --with-apply
@@ -33,7 +34,9 @@ describe 'ridgepole' do
             --enable-mysql-pkdump
             --enable-foreigner
             --enable-migration-comments
+            --enable-mysql-awesome
             --normalize-mysql-float
+        -r, --require LIBS
             --log-file LOG_FILE
             --verbose
             --debug

data/spec/collation/collation_spec.rb

@@ -0,0 +1,132 @@
+if mysql_awesome_enabled?
+  describe 'Ridgepole::Client#diff -> migrate' do
+    context 'when change column (add collation)' do
+      let(:actual_dsl) {
+        <<-RUBY
+          create_table "employee_clubs", force: true do |t|
+            t.integer "emp_no",  null: false
+            t.integer "club_id", null: false, unsigned: true
+            t.string  "string",  null: false
+            t.text    "text",    null: false
+          end
+        RUBY
+      }
+
+      let(:expected_dsl) {
+        <<-RUBY
+          create_table "employee_clubs", force: true do |t|
+            t.integer "emp_no",  null: false
+            t.integer "club_id", null: false, unsigned: true
+            t.string  "string",  null: false,                 collation: "ascii_bin"
+            t.text    "text",    null: false,                 collation: "utf8mb4_bin"
+          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
+        delta.migrate
+        expect(subject.dump).to eq expected_dsl.strip_heredoc.strip
+      }
+    end
+
+    context 'when change column (delete collation)' do
+      let(:actual_dsl) {
+        <<-RUBY
+          create_table "employee_clubs", force: true do |t|
+            t.integer "emp_no",  null: false
+            t.integer "club_id", null: false, unsigned: true
+            t.string  "string",  null: false,                 collation: "ascii_bin"
+            t.text    "text",    null: false,                 collation: "utf8mb4_bin"
+          end
+        RUBY
+      }
+
+      let(:expected_dsl) {
+        <<-RUBY
+          create_table "employee_clubs", force: true do |t|
+            t.integer "emp_no",  null: false
+            t.integer "club_id", null: false, unsigned: true
+            t.string  "string",  null: false
+            t.text    "text",    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
+        delta.migrate
+        expect(subject.dump).to eq expected_dsl.strip_heredoc.strip
+      }
+    end
+
+    context 'when change column (change collation)' do
+      let(:actual_dsl) {
+        <<-RUBY
+          create_table "employee_clubs", force: true do |t|
+            t.integer "emp_no",  null: false
+            t.integer "club_id", null: false, unsigned: true
+            t.string  "string",  null: false,                 collation: "ascii_bin"
+            t.text    "text",    null: false,                 collation: "utf8mb4_bin"
+          end
+        RUBY
+      }
+
+      let(:expected_dsl) {
+        <<-RUBY
+          create_table "employee_clubs", force: true do |t|
+            t.integer "emp_no",  null: false
+            t.integer "club_id", null: false, unsigned: true
+            t.string  "string",  null: false,                 collation: "utf8mb4_bin"
+            t.text    "text",    null: false,                 collation: "ascii_bin"
+          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
+        delta.migrate
+        expect(subject.dump).to eq expected_dsl.strip_heredoc.strip
+      }
+    end
+
+    context 'when change column (no change collation)' do
+      let(:actual_dsl) {
+        <<-RUBY
+          create_table "employee_clubs", force: true do |t|
+            t.integer "emp_no",  null: false
+            t.integer "club_id", null: false, unsigned: true
+            t.string  "string",  null: false,                 collation: "ascii_bin"
+            t.text    "text",    null: false,                 collation: "utf8mb4_bin"
+          end
+        RUBY
+      }
+
+      before { subject.diff(actual_dsl).migrate }
+      subject { client }
+
+      it {
+        delta = subject.diff(actual_dsl)
+        expect(delta.differ?).to be_falsey
+        expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+        delta.migrate
+        expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      }
+    end
+  end
+end

data/spec/comment/comment_spec.rb

@@ -1,177 +1,179 @@
-describe 'Ridgepole::Client#diff -> migrate' do
-  context 'when change column (add comment)' do
-    let(:actual_dsl) {
-      <<-RUBY
-        create_table "employee_clubs", force: true do |t|
-          t.integer "emp_no",  null: false
-          t.integer "club_id", null: false, unsigned: true
-          t.string  "string",  null: false
-          t.text    "text",    null: false
-        end
-      RUBY
-    }
-
-    let(:expected_dsl) {
-      <<-RUBY
-        create_table "employee_clubs", force: true do |t|
-          t.integer "emp_no",  null: false,                 comment: "any comment"
-          t.integer "club_id", null: false, unsigned: true, comment: "any comment2"
-          t.string  "string",  null: false,                 comment: "any comment3"
-          t.text    "text",    null: false,                 comment: "any comment4"
-        end
-      RUBY
-    }
-
-    before { subject.diff(actual_dsl).migrate }
-    subject { client(migration_comments: true) }
-
-    it {
-      delta = subject.diff(expected_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
-    }
-  end
-
-  context 'when change column (delete comment)' do
-    let(:actual_dsl) {
-      <<-RUBY
-        create_table "employee_clubs", force: true do |t|
-          t.integer "emp_no",  null: false,                 comment: "any comment"
-          t.integer "club_id", null: false, unsigned: true, comment: "any comment2"
-          t.string  "string",  null: false,                 comment: "any comment3"
-          t.text    "text",    null: false,                 comment: "any comment4"
-        end
-      RUBY
-    }
-
-    let(:expected_dsl) {
-      <<-RUBY
-        create_table "employee_clubs", force: true do |t|
-          t.integer "emp_no",  null: false
-          t.integer "club_id", null: false, unsigned: true
-          t.string  "string",  null: false
-          t.text    "text",    null: false
-        end
-      RUBY
-    }
-
-    before { subject.diff(actual_dsl).migrate }
-    subject { client(migration_comments: true) }
-
-    it {
-      delta = subject.diff(expected_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
-    }
-  end
-
-  context 'when change column (change comment)' do
-    let(:actual_dsl) {
-      <<-RUBY
-        create_table "employee_clubs", force: true do |t|
-          t.integer "emp_no",  null: false,                 comment: "any comment"
-          t.integer "club_id", null: false, unsigned: true, comment: "any comment2"
-          t.string  "string",  null: false,                 comment: "any comment3"
-          t.text    "text",    null: false,                 comment: "any comment4"
-        end
-      RUBY
-    }
-
-    let(:expected_dsl) {
-      <<-RUBY
-        create_table "employee_clubs", force: true do |t|
-          t.integer "emp_no",  null: false,                 comment: "other comment"
-          t.integer "club_id", null: false, unsigned: true, comment: "other comment2"
-          t.string  "string",  null: false,                 comment: "other comment3"
-          t.text    "text",    null: false,                 comment: "other comment4"
-        end
-      RUBY
-    }
-
-    before { subject.diff(actual_dsl).migrate }
-    subject { client(migration_comments: true) }
-
-    it {
-      delta = subject.diff(expected_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
-    }
-  end
-
-  context 'when change column (no change comment)' do
-    let(:actual_dsl) {
-      <<-RUBY
-        create_table "employee_clubs", force: true do |t|
-          t.integer "emp_no",  null: false,                 comment: "any comment"
-          t.integer "club_id", null: false, unsigned: true, comment: "any comment2"
-          t.string  "string",  null: false,                 comment: "any comment3"
-          t.text    "text",    null: false,                 comment: "any comment4"
-        end
-      RUBY
-    }
-
-    before { subject.diff(actual_dsl).migrate }
-    subject { client(migration_comments: true) }
-
-    it {
-      delta = subject.diff(actual_dsl)
-      expect(delta.differ?).to be_falsey
-      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
-      delta.migrate
-      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
-    }
-  end
-
-  context 'when create table (with comment)' do
-    let(:expected_dsl) {
-      <<-RUBY
-        create_table "employee_clubs", force: true, comment: "table comment" do |t|
-          t.integer "emp_no",  null: false,                 comment: "other comment"
-          t.integer "club_id", null: false, unsigned: true, comment: "other comment2"
-          t.string  "string",  null: false,                 comment: "other comment3"
-          t.text    "text",    null: false,                 comment: "other comment4"
-        end
-      RUBY
-    }
-
-    subject { client(migration_comments: true) }
-
-    it {
-      delta = subject.diff(expected_dsl)
-      expect(delta.differ?).to be_truthy
-      expect(subject.dump.strip).to be_empty
-      delta.migrate
-      expect(subject.dump).to eq expected_dsl.strip_heredoc.strip
-    }
-  end
-
-  context 'when drop table (with comment)' do
-    let(:actual_dsl) {
-      <<-RUBY
-        create_table "employee_clubs", force: true, comment: "table comment" do |t|
-          t.integer "emp_no",  null: false,                 comment: "other comment"
-          t.integer "club_id", null: false, unsigned: true, comment: "other comment2"
-          t.string  "string",  null: false,                 comment: "other comment3"
-          t.text    "text",    null: false,                 comment: "other comment4"
-        end
-      RUBY
-    }
-
-    before { subject.diff(actual_dsl).migrate }
-    subject { client(migration_comments: true) }
-
-    it {
-      delta = subject.diff('')
-      expect(delta.differ?).to be_truthy
-      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
-      delta.migrate
-      expect(subject.dump).to be_empty
-    }
+unless mysql_awesome_enabled?
+  describe 'Ridgepole::Client#diff -> migrate' do
+    context 'when change column (add comment)' do
+      let(:actual_dsl) {
+        <<-RUBY
+          create_table "employee_clubs", force: true do |t|
+            t.integer "emp_no",  null: false
+            t.integer "club_id", null: false, unsigned: true
+            t.string  "string",  null: false
+            t.text    "text",    null: false
+          end
+        RUBY
+      }
+
+      let(:expected_dsl) {
+        <<-RUBY
+          create_table "employee_clubs", force: true do |t|
+            t.integer "emp_no",  null: false,                 comment: "any comment"
+            t.integer "club_id", null: false, unsigned: true, comment: "any comment2"
+            t.string  "string",  null: false,                 comment: "any comment3"
+            t.text    "text",    null: false,                 comment: "any comment4"
+          end
+        RUBY
+      }
+
+      before { subject.diff(actual_dsl).migrate }
+      subject { client(enable_migration_comments: true) }
+
+      it {
+        delta = subject.diff(expected_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
+      }
+    end
+
+    context 'when change column (delete comment)' do
+      let(:actual_dsl) {
+        <<-RUBY
+          create_table "employee_clubs", force: true do |t|
+            t.integer "emp_no",  null: false,                 comment: "any comment"
+            t.integer "club_id", null: false, unsigned: true, comment: "any comment2"
+            t.string  "string",  null: false,                 comment: "any comment3"
+            t.text    "text",    null: false,                 comment: "any comment4"
+          end
+        RUBY
+      }
+
+      let(:expected_dsl) {
+        <<-RUBY
+          create_table "employee_clubs", force: true do |t|
+            t.integer "emp_no",  null: false
+            t.integer "club_id", null: false, unsigned: true
+            t.string  "string",  null: false
+            t.text    "text",    null: false
+          end
+        RUBY
+      }
+
+      before { subject.diff(actual_dsl).migrate }
+      subject { client(enable_migration_comments: true) }
+
+      it {
+        delta = subject.diff(expected_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
+      }
+    end
+
+    context 'when change column (change comment)' do
+      let(:actual_dsl) {
+        <<-RUBY
+          create_table "employee_clubs", force: true do |t|
+            t.integer "emp_no",  null: false,                 comment: "any comment"
+            t.integer "club_id", null: false, unsigned: true, comment: "any comment2"
+            t.string  "string",  null: false,                 comment: "any comment3"
+            t.text    "text",    null: false,                 comment: "any comment4"
+          end
+        RUBY
+      }
+
+      let(:expected_dsl) {
+        <<-RUBY
+          create_table "employee_clubs", force: true do |t|
+            t.integer "emp_no",  null: false,                 comment: "other comment"
+            t.integer "club_id", null: false, unsigned: true, comment: "other comment2"
+            t.string  "string",  null: false,                 comment: "other comment3"
+            t.text    "text",    null: false,                 comment: "other comment4"
+          end
+        RUBY
+      }
+
+      before { subject.diff(actual_dsl).migrate }
+      subject { client(enable_migration_comments: true) }
+
+      it {
+        delta = subject.diff(expected_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
+      }
+    end
+
+    context 'when change column (no change comment)' do
+      let(:actual_dsl) {
+        <<-RUBY
+          create_table "employee_clubs", force: true do |t|
+            t.integer "emp_no",  null: false,                 comment: "any comment"
+            t.integer "club_id", null: false, unsigned: true, comment: "any comment2"
+            t.string  "string",  null: false,                 comment: "any comment3"
+            t.text    "text",    null: false,                 comment: "any comment4"
+          end
+        RUBY
+      }
+
+      before { subject.diff(actual_dsl).migrate }
+      subject { client(enable_migration_comments: true) }
+
+      it {
+        delta = subject.diff(actual_dsl)
+        expect(delta.differ?).to be_falsey
+        expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+        delta.migrate
+        expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      }
+    end
+
+    context 'when create table (with comment)' do
+      let(:expected_dsl) {
+        <<-RUBY
+          create_table "employee_clubs", force: true, comment: "table comment" do |t|
+            t.integer "emp_no",  null: false,                 comment: "other comment"
+            t.integer "club_id", null: false, unsigned: true, comment: "other comment2"
+            t.string  "string",  null: false,                 comment: "other comment3"
+            t.text    "text",    null: false,                 comment: "other comment4"
+          end
+        RUBY
+      }
+
+      subject { client(enable_migration_comments: true) }
+
+      it {
+        delta = subject.diff(expected_dsl)
+        expect(delta.differ?).to be_truthy
+        expect(subject.dump.strip).to be_empty
+        delta.migrate
+        expect(subject.dump).to eq expected_dsl.strip_heredoc.strip
+      }
+    end
+
+    context 'when drop table (with comment)' do
+      let(:actual_dsl) {
+        <<-RUBY
+          create_table "employee_clubs", force: true, comment: "table comment" do |t|
+            t.integer "emp_no",  null: false,                 comment: "other comment"
+            t.integer "club_id", null: false, unsigned: true, comment: "other comment2"
+            t.string  "string",  null: false,                 comment: "other comment3"
+            t.text    "text",    null: false,                 comment: "other comment4"
+          end
+        RUBY
+      }
+
+      before { subject.diff(actual_dsl).migrate }
+      subject { client(enable_migration_comments: true) }
+
+      it {
+        delta = subject.diff('')
+        expect(delta.differ?).to be_truthy
+        expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+        delta.migrate
+        expect(subject.dump).to be_empty
+      }
+    end
   end
 end

data/spec/dump/dump_class_method_spec.rb

@@ -3,8 +3,21 @@ describe 'Ridgepole::Client.dump' do
     before { restore_tables }
     subject { Ridgepole::Client }
 
+    let(:options) {
+      opts = {}
+
+      if mysql_awesome_enabled?
+        opts[:enable_mysql_awesome] = true
+        opts[:without_table_options] = true
+      else
+        opts[:enable_mysql_unsigned] = true
+      end
+
+      opts
+    }
+
     it {
-      expect(subject.dump(conn_spec, enable_mysql_unsigned: true)).to eq <<-RUBY.strip_heredoc.strip
+      expect(subject.dump(conn_spec, options)).to eq <<-RUBY.strip_heredoc.strip
         create_table "clubs", force: true do |t|
           t.string "name", default: "", null: false
         end

data/spec/spec_helper.rb

@@ -49,10 +49,18 @@ end
 def client(options = {}, config = {})
   config = conn_spec(config)
 
-  options = {
-    :enable_mysql_unsigned => true,
+  default_options = {
     :debug => !!ENV['DEBUG'],
-  }.merge(options)
+  }
+
+  if mysql_awesome_enabled?
+    default_options[:enable_mysql_awesome] = true
+    default_options[:without_table_options] = true
+  else
+    default_options[:enable_mysql_unsigned] = true
+  end
+
+  options = default_options.merge(options)
 
   Ridgepole::Client.new(config, options)
 end
@@ -129,3 +137,7 @@ def run_cli(options = {})
     Open3.capture2e(cmd)
   end
 end
+
+def mysql_awesome_enabled?
+  ENV['ENABLE_MYSQL_AWESOME'] == '1'
+end

data/spec/~pkdump/pkdump_spec.rb

@@ -27,7 +27,7 @@ describe 'Ridgepole::Client (with pkdump)' do
   end
 
   context 'when create with activerecord-mysql-pkdump' do
-    subject { client(enable_foreigner: true) }
+    subject { client(enable_mysql_pkdump: true) }
 
     it {
       delta = subject.diff(dsl)
@@ -39,7 +39,7 @@ describe 'Ridgepole::Client (with pkdump)' do
   end
 
   context 'update create with activerecord-mysql-pkdump' do
-    subject { client(enable_foreigner: true) }
+    subject { client(enable_mysql_pkdump: true) }
 
     before { subject.diff(dsl).migrate }
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ridgepole
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.5.2.beta
 platform: ruby
 authors:
 - Genki Sugawara
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-24 00:00:00.000000000 Z
+date: 2015-01-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -150,6 +150,20 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: activerecord-mysql-awesome
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Ridgepole is a tool to manage DB schema. It defines DB schema using Rails
   DSL, and updates DB schema according to DSL.
 email:
@@ -176,14 +190,17 @@ files:
 - lib/ridgepole/dumper.rb
 - lib/ridgepole/execute_expander.rb
 - lib/ridgepole/ext/foreign_key.rb
+- lib/ridgepole/ext/mysql_awesome.rb
 - lib/ridgepole/logger.rb
 - lib/ridgepole/migration_ext.rb
 - lib/ridgepole/schema_dumper_ext.rb
 - lib/ridgepole/version.rb
 - ridgepole.gemspec
 - spec/0_diff/dump_disable_unsigned_spec.rb
+- spec/bigint_pk/bigint_pkspec.rb
 - spec/cli/config_spec.rb
 - spec/cli/ridgepole_spec.rb
+- spec/collation/collation_spec.rb
 - spec/comment/comment_spec.rb
 - spec/diff/diff2_spec.rb
 - spec/diff/diff_spec.rb
@@ -250,9 +267,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - '>'
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.0.14
@@ -261,8 +278,10 @@ specification_version: 4
 summary: Ridgepole is a tool to manage DB schema.
 test_files:
 - spec/0_diff/dump_disable_unsigned_spec.rb
+- spec/bigint_pk/bigint_pkspec.rb
 - spec/cli/config_spec.rb
 - spec/cli/ridgepole_spec.rb
+- spec/collation/collation_spec.rb
 - spec/comment/comment_spec.rb
 - spec/diff/diff2_spec.rb
 - spec/diff/diff_spec.rb