ridgepole 0.6.4.beta3 → 0.6.4.beta4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f85451f0471e23a2e57b9c3ef3ea2c583d9aa0ed
-  data.tar.gz: 386a09bc2f304e3ae2d8f6732e58357d5c1dc192
+  metadata.gz: 7a2d99018e06553a49ed3899adbb1d47e0ffe5e3
+  data.tar.gz: 2e5bf50e046a173df0b970e53ffd5d547fe50df3
 SHA512:
-  metadata.gz: 6dd3c94d9e6bcd6ffa6e8bda9b60fa6241ac4c08660123a2c1f14d2fb04c39690f7dd8665fc1b9ca5565fb342e15d0df7d8126e8650b563c23ac133ac753e7b9
-  data.tar.gz: fa8c06f2e4b859325d3d45676ee20ef7588a9c4c02d9d08fd39a018e36a0458ea39ff5e54d791b0fe206e44c4aa09bb6a6695d2fc93ea6ec6ad9f410b80e7f1c
+  metadata.gz: ead176794c52b4e4e2ceeb9cc4a48b771282f06e1bc58a619bbcdcdcff9df68b78c9357977a8154084bf0cd20a9e3bc1c168e9fffaed9fbee9b9aafdcec0cd9f
+  data.tar.gz: 315d8fb9a6c8a7bacc54a435491c96ec5d1c156c9918c70c343bc3551ae9d99263aed26aa842d9555d42e4952d0b895004b389df087f80dd98cd50f2174aea18

data/.travis.yml

@@ -1,8 +1,13 @@
+sudo: false
 language: ruby
+cache: bundler
 rvm:
   - 2.0.0
-  - 2.1.5
-  - 2.2.1
+  - 2.1.8
+  - 2.2.4
+  - 2.3.0
+before_install:
+  - gem update bundler
 script:
   - bundle install
   - bundle exec rake
@@ -13,15 +18,5 @@ env:
     - ENABLE_MYSQL_AWESOME=0 ENABLE_MIGRATION_COMMENTS=1
     - ENABLE_MYSQL_AWESOME=1 ENABLE_MIGRATION_COMMENTS=1
     - POSTGRESQL=1
-before_install:
-  - sudo /etc/init.d/postgresql stop
-  - sudo apt-get update
-  - sudo apt-get purge postgresql-9.1 postgresql-9.2 postgresql-9.3
-  - sudo apt-get install language-pack-es postgresql-9.4
-  - sudo chmod 777 /etc/postgresql/9.4/main/pg_hba.conf
-  - sudo echo "local all postgres trust" >  /etc/postgresql/9.4/main/pg_hba.conf
-  - sudo echo "local all all trust" >> /etc/postgresql/9.4/main/pg_hba.conf
-  - sudo echo "host  all all 127.0.0.1/32 trust" >> /etc/postgresql/9.4/main/pg_hba.conf
-  - sudo echo "host  all all ::1/128 trust" >> /etc/postgresql/9.4/main/pg_hba.conf
-  - sudo /etc/init.d/postgresql restart
-  - psql --version
+addons:
+  postgresql: "9.4"

data/README.md

@@ -46,6 +46,7 @@ It defines DB schema using [Rails DSL](http://guides.rubyonrails.org/migrations.
   * Execute sql using external script ([pull#56](https://github.com/winebarrel/ridgepole/pull/56))
   * Add `--mysql-use-alter` option
   * Add `--alter-extra` option
+  * Add `--dump-with-default-fk-name` option
 
 ## Installation
 
@@ -68,7 +69,7 @@ Usage: ridgepole [options]
     -E, --env ENVIRONMENT
     -a, --apply
     -m, --merge
-    -f, --file FILE
+    -f, --file SCHEMAFILE
         --dry-run
         --table-options OPTIONS
         --alter-extra ALTER_SPEC
@@ -88,12 +89,13 @@ Usage: ridgepole [options]
     -d, --diff DSL1 DSL2
         --reverse
         --with-apply
-    -o, --output FILE
+    -o, --output SCHEMAFILE
     -t, --tables TABLES
         --ignore-tables TABLES
         --enable-mysql-awesome
         --mysql-use-alter
         --dump-without-table-options
+        --dump-with-default-fk-name
         --index-removed-drop-column
         --enable-migration-comments
     -r, --require LIBS

data/bin/ridgepole

@@ -71,7 +71,7 @@ ARGV.options do |opt|
     opt.on('-E', '--env ENVIRONMENT')             {|v| env = v                                              }
     opt.on('-a', '--apply')                       {    set_mode[:apply]                                     }
     opt.on('-m', '--merge')                       {    set_mode[:apply]; options[:merge] = true             }
-    opt.on('-f', '--file FILE')                   {|v| file = v                                             }
+    opt.on('-f', '--file SCHEMAFILE')             {|v| file = v                                             }
     opt.on('',   '--dry-run')                     {    options[:dry_run] = true                             }
     opt.on('',   '--table-options OPTIONS')       {|v| options[:table_options] = v                          }
     opt.on('',   '--alter-extra ALTER_SPEC')      {|v| options[:alter_extra] = v                            }
@@ -106,12 +106,13 @@ ARGV.options do |opt|
     }
     opt.on('',   '--reverse')                     {    options[:reverse] = true                             }
     opt.on('',   '--with-apply')                  {    diff_with_apply = true                               }
-    opt.on('-o', '--output FILE')                 {|v| output_file = v                                      }
+    opt.on('-o', '--output SCHEMAFILE')           {|v| output_file = v                                      }
     opt.on('-t', '--tables TABLES', Array)        {|v| options[:tables] = v                                 }
     opt.on('',   '--ignore-tables TABLES', Array) {|v| options[:ignore_tables] = v.map {|i| Regexp.new(i) } }
     opt.on('',   '--enable-mysql-awesome')        {    options[:enable_mysql_awesome] = true                }
     opt.on('',   '--mysql-use-alter')             {    options[:mysql_use_alter] = true                     }
     opt.on('',   '--dump-without-table-options')  {    options[:dump_without_table_options] = true          }
+    opt.on('',   '--dump-with-default-fk-name')   {    options[:dumb_with_default_fk_name] = true           }
     opt.on('',   '--index-removed-drop-column')   {    options[:index_removed_drop_column] = true           }
     opt.on('',   '--enable-migration-comments')   {    options[:enable_migration_comments] = true           }
     opt.on('-r', '--require LIBS', Array)         {|v| v.each {|i| require i }                              }

data/lib/ridgepole/client.rb

@@ -29,6 +29,10 @@ class Ridgepole::Client
     if @options[:mysql_use_alter]
       require 'ridgepole/ext/abstract_mysql_adapter'
     end
+
+    if @options[:dumb_with_default_fk_name]
+      require 'ridgepole/ext/schema_dumper'
+    end
   end
 
   def dump(&block)

data/lib/ridgepole/diff.rb

@@ -88,7 +88,7 @@ class Ridgepole::Diff
 
   def scan_options_change(table_name, from, to, table_delta)
     unless from == to
-      Ridgepole::Logger.instance.warn("[WARNING] Table `#{table_name}` options cannot be changed")
+      Ridgepole::Logger.instance.warn("[WARNING] No difference of schema configuration for table `#{table_name}`. (if you changed some options, please reconfirm your Schemafile)")
     end
   end
 

data/lib/ridgepole/ext/schema_dumper.rb

@@ -0,0 +1,32 @@
+require 'active_record/schema_dumper'
+
+class ActiveRecord::SchemaDumper
+  def foreign_keys_with_default_name(table, stream)
+    if (foreign_keys = @connection.foreign_keys(table)).any?
+      add_foreign_key_statements = foreign_keys.map do |foreign_key|
+        parts = [
+          "add_foreign_key #{remove_prefix_and_suffix(foreign_key.from_table).inspect}",
+          remove_prefix_and_suffix(foreign_key.to_table).inspect,
+        ]
+
+        if foreign_key.column != @connection.foreign_key_column_for(foreign_key.to_table)
+          parts << "column: #{foreign_key.column.inspect}"
+        end
+
+        if foreign_key.custom_primary_key?
+          parts << "primary_key: #{foreign_key.primary_key.inspect}"
+        end
+
+        parts << "name: #{foreign_key.name.inspect}"
+
+        parts << "on_update: #{foreign_key.on_update.inspect}" if foreign_key.on_update
+        parts << "on_delete: #{foreign_key.on_delete.inspect}" if foreign_key.on_delete
+
+        "  #{parts.join(', ')}"
+      end
+
+      stream.puts add_foreign_key_statements.sort.join("\n")
+    end
+  end
+  alias_method_chain :foreign_keys, :default_name
+end

data/lib/ridgepole/version.rb

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

data/spec/mysql/cli/ridgepole_spec.rb

@@ -14,7 +14,7 @@ describe 'ridgepole' do
         -E, --env ENVIRONMENT
         -a, --apply
         -m, --merge
-        -f, --file FILE
+        -f, --file SCHEMAFILE
             --dry-run
             --table-options OPTIONS
             --alter-extra ALTER_SPEC
@@ -34,12 +34,13 @@ describe 'ridgepole' do
         -d, --diff DSL1 DSL2
             --reverse
             --with-apply
-        -o, --output FILE
+        -o, --output SCHEMAFILE
         -t, --tables TABLES
             --ignore-tables TABLES
             --enable-mysql-awesome
             --mysql-use-alter
             --dump-without-table-options
+            --dump-with-default-fk-name
             --index-removed-drop-column
             --enable-migration-comments
         -r, --require LIBS

data/spec/mysql/fk/migrate_change_fk_spec.rb

@@ -48,7 +48,7 @@ add_foreign_key "child", "parent", name: "child_ibfk_1"
 
     before { subject.diff(actual_dsl).migrate }
 
-    subject { client(enable_foreigner: true) }
+    subject { client }
 
     it {
       delta = subject.diff(expected_dsl)

data/spec/mysql/migrate/migrate_change_index5_spec.rb

@@ -23,7 +23,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date    "to_date",             null: false
         end
 
-        add_index "salaries", ["emp_no", "id"], unique: false, name: "emp_no", unique: false, using: :btree
+        add_index "salaries", ["emp_no", "id"], name: "emp_no", unique: false, using: :btree
       RUBY
     }
 

data/spec/mysql/migrate/migrate_change_table_option_spec.rb

@@ -30,7 +30,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
     subject { client }
 
     it {
-      expect(Ridgepole::Logger.instance).to receive(:warn).with('[WARNING] Table `employees` options cannot be changed')
+      expect(Ridgepole::Logger.instance).to receive(:warn).with("[WARNING] No difference of schema configuration for table `employees`. (if you changed some options, please reconfirm your Schemafile)")
       delta = subject.diff(expected_dsl)
       expect(delta.differ?).to be_falsey
       expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
@@ -39,7 +39,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
 
     it {
-      expect(Ridgepole::Logger.instance).to receive(:warn).with('[WARNING] Table `employees` options cannot be changed')
+      expect(Ridgepole::Logger.instance).to receive(:warn).with("[WARNING] No difference of schema configuration for table `employees`. (if you changed some options, please reconfirm your Schemafile)")
       delta = Ridgepole::Client.diff(actual_dsl, expected_dsl, reverse: true)
       expect(delta.differ?).to be_falsey
     }

data/spec/mysql/~default_name_fk/migrate_change_fk_spec.rb

@@ -0,0 +1,62 @@
+unless postgresql?
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when change fk' do
+    let(:actual_dsl) {
+      <<-RUBY
+create_table "parent", force: :cascade do |t|
+end
+
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc", on_delete: :cascade
+      RUBY
+    }
+
+    let(:sorted_actual_dsl) {
+      <<-RUBY
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id", limit: 4
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent", force: :cascade do |t|
+end
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc", on_delete: :cascade
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id", limit: 4
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent", force: :cascade do |t|
+end
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+
+    subject { client(dumb_with_default_fk_name: true) }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump.delete_empty_lines).to eq sorted_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
+    }
+  end
+end
+end

data/spec/mysql/~default_name_fk/migrate_create_fk_spec.rb

@@ -0,0 +1,173 @@
+unless postgresql?
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when create fk' do
+    let(:actual_dsl) {
+      <<-RUBY
+create_table "child"#{unsigned_if_enabled}, force: :cascade do |t|
+  t.integer "parent_id", limit: 4#{unsigned_if_enabled}
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent"#{unsigned_if_enabled}, force: :cascade do |t|
+end
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      actual_dsl + (<<-RUBY)
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client(dumb_with_default_fk_name: true) }
+
+    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, default_int_limit: 4, dumb_with_default_fk_name: true)
+      expect(delta.differ?).to be_truthy
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        remove_foreign_key("child", {:name=>"fk_rails_e74ce85cbc"})
+      RUBY
+    }
+
+    it {
+      delta = client(bulk_change: true, dumb_with_default_fk_name: 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
+        add_foreign_key("child", "parent", {:name=>"fk_rails_e74ce85cbc"})
+      RUBY
+      delta.migrate
+      expect(subject.dump.delete_empty_lines).to eq expected_dsl.strip_heredoc.strip.delete_empty_lines
+    }
+  end
+
+  context 'when create fk when create table' do
+    let(:dsl) {
+      <<-RUBY
+# Define parent before child
+create_table "parent", force: :cascade do |t|
+end
+
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    let(:sorted_dsl) {
+      <<-RUBY
+
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id", limit: 4
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent", force: :cascade do |t|
+end
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    subject { client(dumb_with_default_fk_name: true) }
+
+    it {
+      delta = subject.diff(dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump.strip).to eq ''
+      delta.migrate
+      expect(subject.dump.delete_empty_lines).to eq sorted_dsl.strip_heredoc.strip.delete_empty_lines
+    }
+  end
+
+  context 'already defined' do
+    let(:dsl) {
+      <<-RUBY
+# Define parent before child
+create_table "parent", force: :cascade do |t|
+end
+
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id", unsigned: true
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    subject { client(dumb_with_default_fk_name: true) }
+
+    it {
+      expect {
+        subject.diff(dsl)
+      }.to raise_error('Foreign Key `child(fk_rails_e74ce85cbc)` already defined')
+    }
+  end
+
+  context 'no name' do
+    let(:dsl) {
+      <<-RUBY
+# Define parent before child
+create_table "parent", force: :cascade do |t|
+end
+
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id", unsigned: true
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent"
+      RUBY
+    }
+
+    subject { client(dumb_with_default_fk_name: true) }
+
+    it {
+      expect {
+        subject.diff(dsl)
+      }.to raise_error('Foreign key name in `child` is undefined')
+    }
+  end
+
+  context 'orphan fk' do
+    let(:dsl) {
+      <<-RUBY
+# Define parent before child
+create_table "parent", force: :cascade do |t|
+end
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    subject { client(dumb_with_default_fk_name: true) }
+
+    it {
+      expect {
+        subject.diff(dsl)
+      }.to raise_error('Table `child` to create the foreign key is not defined: fk_rails_e74ce85cbc')
+    }
+  end
+end
+end

data/spec/mysql/~default_name_fk/migrate_drop_fk_spec.rb

@@ -0,0 +1,114 @@
+unless postgresql?
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when drop fk' do
+    let(:actual_dsl) {
+      <<-RUBY
+create_table "parent", force: :cascade do |t|
+end
+
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    let(:sorted_actual_dsl) {
+      expected_dsl + (<<-RUBY)
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id", limit: 4
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent", force: :cascade do |t|
+end
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client(dumb_with_default_fk_name: true) }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq sorted_actual_dsl.strip_heredoc.strip
+      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
+    }
+
+    it {
+      delta = Ridgepole::Client.diff(actual_dsl, expected_dsl, reverse: true, default_int_limit: 4, dumb_with_default_fk_name: true)
+      expect(delta.differ?).to be_truthy
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        add_foreign_key("child", "parent", {:name=>"fk_rails_e74ce85cbc"})
+      RUBY
+    }
+
+    it {
+      delta = client(bulk_change: true, dumb_with_default_fk_name: true).diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq sorted_actual_dsl.strip_heredoc.strip
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        remove_foreign_key("child", {:name=>"fk_rails_e74ce85cbc"})
+      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
+
+  context 'when drop fk when drop table' do
+    let(:dsl) {
+      <<-RUBY
+create_table "parent", force: :cascade do |t|
+end
+
+
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    let(:sorted_dsl) {
+      <<-RUBY
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id", limit: 4
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent", force: :cascade do |t|
+end
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    before { subject.diff(dsl).migrate }
+    subject { client(dumb_with_default_fk_name: true) }
+
+    it {
+      delta = subject.diff('')
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq sorted_dsl.strip_heredoc.strip
+      delta.migrate
+      expect(subject.dump.strip).to eq ''
+    }
+  end
+end
+end

data/spec/postgresql/fk/migrate_change_fk_spec.rb

@@ -48,7 +48,7 @@ add_foreign_key "child", "parent", name: "child_ibfk_1"
 
     before { subject.diff(actual_dsl).migrate }
 
-    subject { client(enable_foreigner: true) }
+    subject { client }
 
     it {
       delta = subject.diff(expected_dsl)

data/spec/postgresql/~default_name_fk/migrate_change_fk_spec.rb

@@ -0,0 +1,62 @@
+if postgresql?
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when change fk' do
+    let(:actual_dsl) {
+      <<-RUBY
+create_table "parent", force: :cascade do |t|
+end
+
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc", on_delete: :cascade
+      RUBY
+    }
+
+    let(:sorted_actual_dsl) {
+      <<-RUBY
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent", force: :cascade do |t|
+end
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc", on_delete: :cascade
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent", force: :cascade do |t|
+end
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+
+    subject { client(dumb_with_default_fk_name: true) }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump.delete_empty_lines).to eq sorted_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
+    }
+  end
+end
+end

data/spec/postgresql/~default_name_fk/migrate_create_fk_spec.rb

@@ -0,0 +1,173 @@
+if postgresql?
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when create fk' do
+    let(:actual_dsl) {
+      <<-RUBY
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent", force: :cascade do |t|
+end
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      actual_dsl + (<<-RUBY)
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client(dumb_with_default_fk_name: true) }
+
+    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, default_int_limit: 4, dumb_with_default_fk_name: true)
+      expect(delta.differ?).to be_truthy
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        remove_foreign_key("child", {:name=>"fk_rails_e74ce85cbc"})
+      RUBY
+    }
+
+    it {
+      delta = client(bulk_change: true, dumb_with_default_fk_name: 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
+        add_foreign_key("child", "parent", {:name=>"fk_rails_e74ce85cbc"})
+      RUBY
+      delta.migrate
+      expect(subject.dump.delete_empty_lines).to eq expected_dsl.strip_heredoc.strip.delete_empty_lines
+    }
+  end
+
+  context 'when create fk when create table' do
+    let(:dsl) {
+      <<-RUBY
+# Define parent before child
+create_table "parent", force: :cascade do |t|
+end
+
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id", unsigned: true
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    let(:sorted_dsl) {
+      <<-RUBY
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent", force: :cascade do |t|
+end
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    before { client.diff('').migrate }
+    subject { client(dumb_with_default_fk_name: true) }
+
+    it {
+      delta = subject.diff(dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump.strip).to eq ''
+      delta.migrate
+      expect(subject.dump.delete_empty_lines).to eq sorted_dsl.strip_heredoc.strip.delete_empty_lines
+    }
+  end
+
+  context 'already defined' do
+    let(:dsl) {
+      <<-RUBY
+# Define parent before child
+create_table "parent", force: :cascade do |t|
+end
+
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id", unsigned: true
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    subject { client(dumb_with_default_fk_name: true) }
+
+    it {
+      expect {
+        subject.diff(dsl)
+      }.to raise_error('Foreign Key `child(fk_rails_e74ce85cbc)` already defined')
+    }
+  end
+
+  context 'no name' do
+    let(:dsl) {
+      <<-RUBY
+# Define parent before child
+create_table "parent", force: :cascade do |t|
+end
+
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id", unsigned: true
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent"
+      RUBY
+    }
+
+    subject { client(dumb_with_default_fk_name: true) }
+
+    it {
+      expect {
+        subject.diff(dsl)
+      }.to raise_error('Foreign key name in `child` is undefined')
+    }
+  end
+
+  context 'orphan fk' do
+    let(:dsl) {
+      <<-RUBY
+# Define parent before child
+create_table "parent", force: :cascade do |t|
+end
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    subject { client(dumb_with_default_fk_name: true) }
+
+    it {
+      expect {
+        subject.diff(dsl)
+      }.to raise_error('Table `child` to create the foreign key is not defined: fk_rails_e74ce85cbc')
+    }
+  end
+end
+end

data/spec/postgresql/~default_name_fk/migrate_drop_fk_spec.rb

@@ -0,0 +1,114 @@
+if postgresql?
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when drop fk' do
+    let(:actual_dsl) {
+      <<-RUBY
+create_table "parent", force: :cascade do |t|
+end
+
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    let(:sorted_actual_dsl) {
+      expected_dsl + (<<-RUBY)
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent", force: :cascade do |t|
+end
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client(dumb_with_default_fk_name: true) }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq sorted_actual_dsl.strip_heredoc.strip
+      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
+    }
+
+    it {
+      delta = Ridgepole::Client.diff(actual_dsl, expected_dsl, reverse: true, default_int_limit: 4, dumb_with_default_fk_name: true)
+      expect(delta.differ?).to be_truthy
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        add_foreign_key("child", "parent", {:name=>"fk_rails_e74ce85cbc"})
+      RUBY
+    }
+
+    it {
+      delta = client(bulk_change: true, dumb_with_default_fk_name: true).diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq sorted_actual_dsl.strip_heredoc.strip
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        remove_foreign_key("child", {:name=>"fk_rails_e74ce85cbc"})
+      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
+
+  context 'when drop fk when drop table' do
+    let(:dsl) {
+      <<-RUBY
+create_table "parent", force: :cascade do |t|
+end
+
+
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id", unsigned: true
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    let(:sorted_dsl) {
+      <<-RUBY
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent", force: :cascade do |t|
+end
+
+add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
+      RUBY
+    }
+
+    before { subject.diff(dsl).migrate }
+    subject { client(dumb_with_default_fk_name: true) }
+
+    it {
+      delta = subject.diff('')
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq sorted_dsl.strip_heredoc.strip
+      delta.migrate
+      expect(subject.dump.strip).to eq ''
+    }
+  end
+end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ridgepole
 version: !ruby/object:Gem::Version
-  version: 0.6.4.beta3
+  version: 0.6.4.beta4
 platform: ruby
 authors:
 - Genki Sugawara
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-12-01 00:00:00.000000000 Z
+date: 2016-01-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -164,6 +164,7 @@ files:
 - lib/ridgepole/execute_expander.rb
 - lib/ridgepole/ext/abstract_mysql_adapter.rb
 - lib/ridgepole/ext/migration_comments.rb
+- lib/ridgepole/ext/schema_dumper.rb
 - lib/ridgepole/external_sql_executer.rb
 - lib/ridgepole/logger.rb
 - lib/ridgepole/migration_ext.rb
@@ -234,6 +235,9 @@ files:
 - spec/mysql/migrate_/migrate_drop_index_with_alter_spec.rb
 - spec/mysql/ridgepole_test_database.sql
 - spec/mysql/ridgepole_test_tables.sql
+- spec/mysql/~default_name_fk/migrate_change_fk_spec.rb
+- spec/mysql/~default_name_fk/migrate_create_fk_spec.rb
+- spec/mysql/~default_name_fk/migrate_drop_fk_spec.rb
 - spec/postgresql/diff/diff_spec.rb
 - spec/postgresql/dump/dump_spec.rb
 - spec/postgresql/fk/migrate_change_fk_spec.rb
@@ -254,6 +258,9 @@ files:
 - spec/postgresql/migrate/migrate_same_spec.rb
 - spec/postgresql/ridgepole_test_database.sql
 - spec/postgresql/ridgepole_test_tables.sql
+- spec/postgresql/~default_name_fk/migrate_change_fk_spec.rb
+- spec/postgresql/~default_name_fk/migrate_create_fk_spec.rb
+- spec/postgresql/~default_name_fk/migrate_drop_fk_spec.rb
 - spec/spec_helper.rb
 - spec/string_ext.rb
 homepage: https://github.com/winebarrel/ridgepole
@@ -276,7 +283,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.3.1
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.4.5.1
 signing_key: 
 specification_version: 4
 summary: Ridgepole is a tool to manage DB schema.
@@ -344,6 +351,9 @@ test_files:
 - spec/mysql/migrate_/migrate_drop_index_with_alter_spec.rb
 - spec/mysql/ridgepole_test_database.sql
 - spec/mysql/ridgepole_test_tables.sql
+- spec/mysql/~default_name_fk/migrate_change_fk_spec.rb
+- spec/mysql/~default_name_fk/migrate_create_fk_spec.rb
+- spec/mysql/~default_name_fk/migrate_drop_fk_spec.rb
 - spec/postgresql/diff/diff_spec.rb
 - spec/postgresql/dump/dump_spec.rb
 - spec/postgresql/fk/migrate_change_fk_spec.rb
@@ -364,5 +374,8 @@ test_files:
 - spec/postgresql/migrate/migrate_same_spec.rb
 - spec/postgresql/ridgepole_test_database.sql
 - spec/postgresql/ridgepole_test_tables.sql
+- spec/postgresql/~default_name_fk/migrate_change_fk_spec.rb
+- spec/postgresql/~default_name_fk/migrate_create_fk_spec.rb
+- spec/postgresql/~default_name_fk/migrate_drop_fk_spec.rb
 - spec/spec_helper.rb
 - spec/string_ext.rb