ridgepole 0.8.4 → 0.8.9

data/spec/mysql/cli/ridgepole_spec.rb

@@ -17,6 +17,7 @@ describe 'ridgepole' do
       expect(out).to match_fuzzy <<-MSG
         -c, --config CONF_OR_FILE
         -E, --env ENVIRONMENT
+        -s, --spec-name SPEC_NAME
         -a, --apply
         -m, --merge
         -f, --file SCHEMAFILE

data/spec/mysql/collation/collation_spec.rb

@@ -138,6 +138,12 @@ describe 'Ridgepole::Client#diff -> migrate' do
           opts = ['--dump-without-table-options']
           out, status = run_ridgepole('--diff', "'#{JSON.dump(conn_spec)}'", f.path, *opts)
 
+          # v6.0.3 is the oldest version that doesn't produce any kwargs warnings with Ruby 2.7
+          if condition('< 6.0.3')
+            # https://github.com/jeremyevans/ruby-warning/blob/1.1.0/lib/warning.rb#L18
+            out = out.lines.grep_v(/: warning: (?:Using the last argument (?:for `.+' )?as keyword parameters is deprecated; maybe \*\* should be added to the call|Passing the keyword argument (?:for `.+' )?as the last hash parameter is deprecated|Splitting the last argument (?:for `.+' )?into positional and keyword parameters is deprecated|The called method (?:`.+' )?is defined here)\n\z/).join('')
+          end
+
           expect(out).to be_empty
           expect(status.success?).to be_truthy
         end

data/spec/mysql/diff/diff2_spec.rb

@@ -175,10 +175,10 @@ describe 'Ridgepole::Client.diff' do
       delta = subject.diff(actual_dsl, expected_dsl)
       expect(delta.differ?).to be_truthy
       expect(delta.script).to match_ruby erbh(<<-ERB)
-        change_column("employee_clubs", "club_id", :integer, <%= {:unsigned=>false, :null=>true, :default=>nil} + cond('>= 5.1', comment: nil) %>)
+        change_column("employee_clubs", "club_id", :integer, **<%= {:unsigned=>false, :null=>true, :default=>nil} + cond('>= 5.1', comment: nil) %>)
 
-        change_column("employees", "last_name", :string, <%= {:limit=>20, :default=>"XXX", :unsigned=>false} + cond('>= 5.1', comment: nil) %>)
-        change_column("employees", "gender", :string, <%= {:limit=>2, :null=>false, :default=>nil, :unsigned=>false} + cond('>= 5.1', comment: nil) %>)
+        change_column("employees", "last_name", :string, **<%= {:limit=>20, :default=>"XXX", :unsigned=>false} + cond('>= 5.1', comment: nil) %>)
+        change_column("employees", "gender", :string, **<%= {:limit=>2, :null=>false, :default=>nil, :unsigned=>false} + cond('>= 5.1', comment: nil) %>)
       ERB
     }
 

data/spec/mysql/diff/diff_spec.rb

@@ -149,10 +149,10 @@ describe 'Ridgepole::Client.diff' do
       delta = subject.diff(actual_dsl, expected_dsl)
       expect(delta.differ?).to be_truthy
       expect(delta.script).to match_ruby erbh(<<-ERB)
-        change_column("employee_clubs", "club_id", :integer, <%= {:unsigned=>false, :null=>true, :default=>nil} + cond('>= 5.1', comment: nil) %>)
+        change_column("employee_clubs", "club_id", :integer, **<%= {:unsigned=>false, :null=>true, :default=>nil} + cond('>= 5.1', comment: nil) %>)
 
-        change_column("employees", "last_name", :string, <%= {:limit=>20, :default=>"XXX", :unsigned=>false} + cond('>= 5.1', comment: nil) %>)
-        change_column("employees", "gender", :string, <%= {:limit=>2, :null=>false, :default=>nil, :unsigned=>false} + cond('>= 5.1', comment: nil) %>)
+        change_column("employees", "last_name", :string, **<%= {:limit=>20, :default=>"XXX", :unsigned=>false} + cond('>= 5.1', comment: nil) %>)
+        change_column("employees", "gender", :string, **<%= {:limit=>2, :null=>false, :default=>nil, :unsigned=>false} + cond('>= 5.1', comment: nil) %>)
       ERB
     }
   end

data/spec/mysql/fk/migrate_create_fk_spec.rb

@@ -36,7 +36,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
       expect(delta.script).to match_fuzzy <<-RUBY
-        add_foreign_key("child", "parent", {:name=>"child_ibfk_1"})
+        add_foreign_key("child", "parent", **{:name=>"child_ibfk_1"})
       RUBY
       delta.migrate
       expect(subject.dump).to match_ruby expected_dsl
@@ -144,7 +144,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
       expect(delta.script).to match_fuzzy <<-RUBY
-        add_foreign_key("child", "parent", {})
+        add_foreign_key("child", "parent", **{})
       RUBY
       delta.migrate
       expect(subject.dump).to match_ruby expected_dsl
@@ -188,13 +188,11 @@ describe 'Ridgepole::Client#diff -> migrate' do
     it {
       expect(Ridgepole::Logger.instance).to receive(:warn).with(<<-MSG).twice
 [WARNING] Table `child` has a foreign key on `parent_id` column, but doesn't have any indexes on the column.
-          Although an index will be added automatically by InnoDB, please add an index explicitly for your future operations.
+          Although an index will be added automatically by InnoDB, please add an index explicitly before the next operation.
       MSG
       subject.diff(dsl).migrate
 
-      expect do
-        subject.diff(dsl).migrate
-      end.to raise_error(/Mysql2::Error: Cannot drop index/)
+      expect(subject.diff(dsl).differ?).to be_truthy
     }
   end
 
@@ -219,7 +217,29 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(Ridgepole::Logger.instance).to_not receive(:warn)
       subject.diff(dsl).migrate
 
-      expect { subject.diff(dsl).migrate }.to_not raise_error
+      expect(subject.diff(dsl).differ?).to be_falsey
+    }
+  end
+
+  context 'when create fk on the primary key' do
+    let(:dsl) do
+      erbh(<<-ERB)
+        create_table "users", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|
+        end
+
+        create_table "icons", primary_key: "user_id", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|
+        end
+        add_foreign_key "icons", "users", name: "fk_icons_users"
+      ERB
+    end
+
+    subject { client(dump_without_table_options: false) }
+
+    it {
+      expect(Ridgepole::Logger.instance).to_not receive(:warn)
+      subject.diff(dsl).migrate
+
+      expect(subject.diff(dsl).differ?).to be_falsey
     }
   end
 end

data/spec/mysql/fk/migrate_drop_fk_spec.rb

@@ -50,7 +50,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_fuzzy sorted_actual_dsl
       expect(delta.script).to match_fuzzy <<-RUBY
-        remove_foreign_key("child", {:name=>"child_ibfk_1"})
+        remove_foreign_key("child", name: "child_ibfk_1")
       RUBY
       delta.migrate
       expect(subject.dump).to match_ruby expected_dsl

data/spec/mysql/fk/migrate_fk_with_column_spec.rb

@@ -44,8 +44,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
       expect(delta.script).to match_fuzzy <<-RUBY
-        add_foreign_key("direct_messages", "users", {:column=>"reciever_id"})
-        add_foreign_key("direct_messages", "users", {:column=>"sender_id"})
+        add_foreign_key("direct_messages", "users", **{:column=>"reciever_id"})
+        add_foreign_key("direct_messages", "users", **{:column=>"sender_id"})
       RUBY
       delta.migrate
       expect(subject.dump).to match_ruby expected_dsl

data/spec/mysql/migrate/migrate_add_column_spec.rb

@@ -145,13 +145,13 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
       expect(delta.script).to match_fuzzy erbh(<<-ERB)
-        change_table("employee_clubs", {:bulk=>true}) do |t|
-          t.column("any_col", :string, {:null=>false, :after=>"club_id", :limit=>255})
+        change_table("employee_clubs", bulk: true) do |t|
+          t.column("any_col", :string, **{:null=>false, :after=>"club_id", :limit=>255})
         end
 
-        change_table("employees", {:bulk => true}) do |t|
-          t.column("age", :integer, {:null=>false, :after=>"hire_date", :limit=>4})
-          t.column("updated_at", :date, {:after=>"age"})
+        change_table("employees", bulk: true) do |t|
+          t.column("age", :integer, **{:null=>false, :after=>"hire_date", :limit=>4})
+          t.column("updated_at", :date, **{:after=>"age"})
         end
       ERB
       delta.migrate

data/spec/mysql/migrate/migrate_change_column8_spec.rb

@@ -4,7 +4,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
   before { subject.diff(actual_dsl).migrate }
   subject { client(table_options: table_options, dump_without_table_options: dump_without_table_options) }
 
-  let(:warning_regexp) { /table options differ/ }
+  let(:warning_regexp) { /Table option changes are ignored/ }
   let(:dump_without_table_options) { false }
 
   let(:actual_dsl) do

data/spec/mysql/migrate/migrate_change_column_spec.rb

@@ -142,13 +142,13 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
       expect(delta.script).to match_fuzzy erbh(<<-ERB)
-        change_table("employee_clubs", {:bulk => true}) do |t|
-          t.change("club_id", :integer, <%= {:null=>true, :default=>nil, :unsigned=>false} + cond('>= 5.1',comment: nil) %>)
+        change_table("employee_clubs", bulk: true) do |t|
+          t.change("club_id", :integer, **<%= {:null=>true, :default=>nil, :unsigned=>false} + cond('>= 5.1',comment: nil) %>)
         end
 
-        change_table("employees", {:bulk => true}) do |t|
-          t.change("last_name", :string, <%= {:limit=>20, :default=>"XXX", :unsigned=>false} + cond('>= 5.1',comment: nil) %>)
-          t.change("gender", :string, <%= {:limit=>2, :null=>false, :default=>nil, :unsigned=>false} + cond('>= 5.1',comment: nil) %>)
+        change_table("employees", bulk: true) do |t|
+          t.change("last_name", :string, **<%= {:limit=>20, :default=>"XXX", :unsigned=>false} + cond('>= 5.1',comment: nil) %>)
+          t.change("gender", :string, **<%= {:limit=>2, :null=>false, :default=>nil, :unsigned=>false} + cond('>= 5.1',comment: nil) %>)
         end
       ERB
       delta.migrate

data/spec/mysql/migrate/migrate_change_index_spec.rb

@@ -204,19 +204,19 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
       expect(delta.script).to match_fuzzy <<-RUBY
-        change_table("dept_emp", {:bulk => true}) do |t|
-          t.remove_index({:name=>"emp_no"})
-          t.index(["from_date"], {:name=>"emp_no", :using=>:btree, :unique=>false})
+        change_table("dept_emp", bulk: true) do |t|
+          t.remove_index(name: "emp_no")
+          t.index(["from_date"], **{:name=>"emp_no", :using=>:btree, :unique=>false})
         end
 
-        change_table("dept_manager", {:bulk => true}) do |t|
-          t.remove_index({:name=>"emp_no"})
-          t.index(["from_date"], {:name=>"emp_no", :using=>:btree, :unique=>false})
+        change_table("dept_manager", bulk: true) do |t|
+          t.remove_index(name: "emp_no")
+          t.index(["from_date"], **{:name=>"emp_no", :using=>:btree, :unique=>false})
         end
 
-        change_table("salaries", {:bulk => true}) do |t|
-          t.remove_index({:name=>"emp_no"})
-          t.index(["from_date"], {:name=>"emp_no", :using=>:btree, :unique=>false})
+        change_table("salaries", bulk: true) do |t|
+          t.remove_index(name: "emp_no")
+          t.index(["from_date"], **{:name=>"emp_no", :using=>:btree, :unique=>false})
         end
       RUBY
 

data/spec/mysql/migrate/migrate_change_table_comment_spec.rb

@@ -32,7 +32,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
 
     it {
       expect(Ridgepole::Logger.instance).to receive(:warn).with(<<-MSG)
-[WARNING] No difference of schema configuration for table `employees` but table options differ.
+[WARNING] Table option changes are ignored on `employees`.
   from: {:comment=>"old comment"}
     to: {:comment=>"new comment"}
       MSG

data/spec/mysql/migrate/migrate_change_table_option_spec.rb

@@ -31,7 +31,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
 
     it {
       expect(Ridgepole::Logger.instance).to receive(:warn).with(<<-MSG)
-[WARNING] No difference of schema configuration for table `employees` but table options differ.
+[WARNING] Table option changes are ignored on `employees`.
   from: {:primary_key=>"emp_no"}
     to: {:primary_key=>"emp_no2"}
       MSG

data/spec/mysql/migrate/migrate_create_index_spec.rb

@@ -141,16 +141,16 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
       expect(delta.script).to match_fuzzy erbh(<<-ERB)
-        change_table("clubs", {:bulk => true}) do |t|
-          t.index(["name"], <%= {:name=>"idx_name", :unique=>true} + cond(5.0, using: :btree) %>)
+        change_table("clubs", bulk: true) do |t|
+          t.index(["name"], **<%= {:name=>"idx_name", :unique=>true} + cond(5.0, using: :btree) %>)
         end
 
-        change_table("employee_clubs", {:bulk => true}) do |t|
-          t.index(["emp_no", "club_id"], <%= {:name=>"idx_emp_no_club_id"} + cond(5.0, using: :btree) %>)
+        change_table("employee_clubs", bulk: true) do |t|
+          t.index(["emp_no", "club_id"], **<%= {:name=>"idx_emp_no_club_id"} + cond(5.0, using: :btree) %>)
         end
 
-        change_table("titles", {:bulk => true}) do |t|
-          t.index(["emp_no"], <%= {:name=>"emp_no"} + cond(5.0, using: :btree) %>)
+        change_table("titles", bulk: true) do |t|
+          t.index(["emp_no"], **<%= {:name=>"emp_no"} + cond(5.0, using: :btree) %>)
         end
       ERB
       delta.migrate

data/spec/mysql/migrate/migrate_create_table_with_index_spec.rb

@@ -25,13 +25,13 @@ describe 'Ridgepole::Client#diff -> migrate (with index)' do
       expect(delta.differ?).to be_truthy
 
       expect(delta.script).to match_fuzzy erbh(<<-ERB)
-        create_table("dept_emp", {:primary_key=>["emp_no", "dept_no"]}) do |t|
-          t.column("emp_no", :"integer", {:null=>false, :limit=>4})
-          t.column("dept_no", :"string", {:null=>false, :limit=>255})
-          t.column("from_date", :"date", {:null=>false})
-          t.column("to_date", :"date", {:null=>false})
-          t.index(["dept_no"], <%= {:name=>"dept_no"} + cond(5.0, using: :btree) %>)
-          t.index(["emp_no"], <%= {:name=>"emp_no"} + cond(5.0, using: :btree) %>)
+        create_table("dept_emp", **{:primary_key=>["emp_no", "dept_no"]}) do |t|
+          t.column("emp_no", :"integer", **{:null=>false, :limit=>4})
+          t.column("dept_no", :"string", **{:null=>false, :limit=>255})
+          t.column("from_date", :"date", **{:null=>false})
+          t.column("to_date", :"date", **{:null=>false})
+          t.index(["dept_no"], **<%= {:name=>"dept_no"} + cond(5.0, using: :btree) %>)
+          t.index(["emp_no"], **<%= {:name=>"emp_no"} + cond(5.0, using: :btree) %>)
         end
       ERB
 

data/spec/mysql/migrate/migrate_create_table_with_options_spec.rb

@@ -20,11 +20,11 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
 
       expect(delta.script).to match_fuzzy <<-RUBY
-        create_table("employee_clubs", {:options=>"ENGINE=MyISAM CHARSET=utf8"}) do |t|
-          t.column("emp_no", :"integer", {:null=>false, :unsigned=>true, :limit=>4})
-          t.column("club_id", :"integer", {:null=>false, :unsigned=>true, :limit=>4})
+        create_table("employee_clubs", **{:options=>"ENGINE=MyISAM CHARSET=utf8"}) do |t|
+          t.column("emp_no", :"integer", **{:null=>false, :unsigned=>true, :limit=>4})
+          t.column("club_id", :"integer", **{:null=>false, :unsigned=>true, :limit=>4})
         end
-        add_index("employee_clubs", ["emp_no", "club_id"], {:name=>"idx_emp_no_club_id", :using=>:btree})
+        add_index("employee_clubs", ["emp_no", "club_id"], **{:name=>"idx_emp_no_club_id", :using=>:btree})
       RUBY
     }
   end
@@ -48,11 +48,11 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
 
       expect(delta.script).to match_fuzzy <<-RUBY
-        create_table("employee_clubs", {:options=>"ENGINE=InnoDB CHARSET=utf8mb4"}) do |t|
-          t.column("emp_no", :"integer", {:null=>false, :unsigned=>true, :limit=>4})
-          t.column("club_id", :"integer", {:null=>false, :unsigned=>true, :limit=>4})
+        create_table("employee_clubs", **{:options=>"ENGINE=InnoDB CHARSET=utf8mb4"}) do |t|
+          t.column("emp_no", :"integer", **{:null=>false, :unsigned=>true, :limit=>4})
+          t.column("club_id", :"integer", **{:null=>false, :unsigned=>true, :limit=>4})
         end
-        add_index("employee_clubs", ["emp_no", "club_id"], {:name=>"idx_emp_no_club_id", :using=>:btree})
+        add_index("employee_clubs", ["emp_no", "club_id"], **{:name=>"idx_emp_no_club_id", :using=>:btree})
       RUBY
     }
   end

data/spec/mysql/migrate/migrate_drop_column_and_index_spec.rb

@@ -138,23 +138,23 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
       expect(delta.script).to match_fuzzy <<-RUBY
-change_table("dept_emp", {:bulk => true}) do |t|
+change_table("dept_emp", bulk: true) do |t|
   t.remove("emp_no")
   t.remove("from_date")
   t.remove("to_date")
 end
 
-change_table("dept_manager", {:bulk => true}) do |t|
+change_table("dept_manager", bulk: true) do |t|
   t.remove("emp_no")
   t.remove("from_date")
   t.remove("to_date")
 end
 
-change_table("employee_clubs", {:bulk => true}) do |t|
+change_table("employee_clubs", bulk: true) do |t|
   t.remove("club_id")
 end
 
-change_table("employees", {:bulk => true}) do |t|
+change_table("employees", bulk: true) do |t|
   t.remove("last_name")
   t.remove("gender")
   t.remove("hire_date")

data/spec/mysql/migrate/migrate_drop_column_spec.rb

@@ -135,17 +135,17 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
       expect(delta.script).to match_fuzzy <<-RUBY
-        change_table("dept_emp", {:bulk => true}) do |t|
+        change_table("dept_emp", bulk: true) do |t|
           t.remove("from_date")
           t.remove("to_date")
         end
 
-        change_table("dept_manager", {:bulk => true}) do |t|
+        change_table("dept_manager", bulk: true) do |t|
           t.remove("from_date")
           t.remove("to_date")
         end
 
-        change_table("employees", {:bulk => true}) do |t|
+        change_table("employees", bulk: true) do |t|
           t.remove("last_name")
           t.remove("gender")
           t.remove("hire_date")

data/spec/mysql/migrate/migrate_drop_index_spec.rb

@@ -141,16 +141,16 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
       expect(delta.script).to match_fuzzy <<-RUBY
-        change_table("clubs", {:bulk => true}) do |t|
-          t.remove_index({:name=>"idx_name"})
+        change_table("clubs", bulk: true) do |t|
+          t.remove_index(name: "idx_name")
         end
 
-        change_table("employee_clubs", {:bulk => true}) do |t|
-          t.remove_index({:name=>"idx_emp_no_club_id"})
+        change_table("employee_clubs", bulk: true) do |t|
+          t.remove_index(name: "idx_emp_no_club_id")
         end
 
-        change_table("titles", {:bulk => true}) do |t|
-          t.remove_index({:name=>"emp_no"})
+        change_table("titles", bulk: true) do |t|
+          t.remove_index(name: "emp_no")
         end
       RUBY
       delta.migrate

data/spec/mysql/migrate/migrate_rename_column_spec.rb

@@ -142,15 +142,15 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
       expect(delta.script).to match_fuzzy <<-RUBY
-        change_table("dept_emp", {:bulk => true}) do |t|
+        change_table("dept_emp", bulk: true) do |t|
           t.rename("from_date", "from_date2")
         end
 
-        change_table("dept_manager", {:bulk => true}) do |t|
+        change_table("dept_manager", bulk: true) do |t|
           t.rename("to_date", "to_date2")
         end
 
-        change_table("employees", {:bulk => true}) do |t|
+        change_table("employees", bulk: true) do |t|
           t.rename("gender", "gender2")
         end
       RUBY

data/spec/mysql/migrate/migrate_script_error_spec.rb

@@ -80,7 +80,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(subject.dump).to match_ruby actual_dsl
 
       errmsg = Regexp.new(Regexp.escape(<<-MSG.strip))
-        33: add_index("employee_clubs", ["emp_no", "Xclub_id"], {:name=>"idx_emp_no_club_id", :using=>:btree})
+        33: add_index("employee_clubs", ["emp_no", "Xclub_id"], **{:name=>"idx_emp_no_club_id", :using=>:btree})
       MSG
 
       expect do

data/spec/mysql/text_blob_types/text_blob_types_spec.rb

@@ -5,7 +5,7 @@ describe 'Ridgepole::Client (with new text/blob types)' do
     subject { client }
 
     it do
-      delta = subject.diff(<<-RUBY)
+      table_def = <<-RUBY
         create_table :foos, id: :unsigned_integer do |t|
           t.blob             :blob
           t.tinyblob         :tiny_blob
@@ -20,6 +20,7 @@ describe 'Ridgepole::Client (with new text/blob types)' do
           t.unsigned_integer :unsigned_integer
         end
       RUBY
+      delta = subject.diff(table_def)
 
       expect(delta.differ?).to be_truthy
       delta.migrate
@@ -39,6 +40,8 @@ describe 'Ridgepole::Client (with new text/blob types)' do
           t.integer "unsigned_integer", unsigned: true
         end
       ERB
+
+      expect(subject.diff(table_def).differ?).to be_falsey
     end
   end
 

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

@@ -36,7 +36,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
       expect(delta.script).to match_fuzzy <<-RUBY
-        add_foreign_key("child", "parent", {:name=>"fk_rails_e74ce85cbc"})
+        add_foreign_key("child", "parent", **{:name=>"fk_rails_e74ce85cbc"})
       RUBY
       delta.migrate
       expect(subject.dump).to match_ruby expected_dsl