ridgepole 0.7.3.beta → 0.7.3.beta2

data/spec/mysql/migrate/migrate_drop_column_and_index2_spec.rb

@@ -1,7 +1,7 @@
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when drop column and index (2)' do
-    let(:actual_dsl) {
-      erbh(<<-EOS)
+    let(:actual_dsl) do
+      erbh(<<-ERB)
         create_table "clubs", force: :cascade do |t|
           t.string "name", default: "", null: false
           t.index ["name"], name: "idx_name", unique: true, <%= i cond(5.0, using: :btree) %>
@@ -59,11 +59,11 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date    "to_date"
           t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
         end
-      EOS
-    }
+      ERB
+    end
 
-    let(:expected_dsl) {
-      erbh(<<-EOS)
+    let(:expected_dsl) do
+      erbh(<<-ERB)
         create_table "clubs", force: :cascade do |t|
           t.string "name", default: "", null: false
           t.index ["name"], name: "idx_name", unique: true, <%= i cond(5.0, using: :btree) %>
@@ -113,8 +113,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date    "to_date"
           t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
         end
-      EOS
-    }
+      ERB
+    end
 
     before { subject.diff(actual_dsl).migrate }
     subject { client }
@@ -131,7 +131,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
       delta = subject.diff(expected_dsl)
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
-      delta.migrate(:noop => true)
+      delta.migrate(noop: true)
       expect(subject.dump).to match_ruby actual_dsl
     }
   end

data/spec/mysql/migrate/migrate_drop_column_and_index_spec.rb

@@ -1,7 +1,7 @@
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when drop column and index' do
-    let(:actual_dsl) {
-      erbh(<<-EOS)
+    let(:actual_dsl) do
+      erbh(<<-ERB)
         create_table "clubs", force: :cascade do |t|
           t.string "name", default: "", null: false
           t.index ["name"], name: "idx_name", unique: true, <%= i cond(5.0, using: :btree) %>
@@ -59,11 +59,11 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date    "to_date"
           t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
         end
-      EOS
-    }
+      ERB
+    end
 
-    let(:expected_dsl) {
-      erbh(<<-EOS)
+    let(:expected_dsl) do
+      erbh(<<-ERB)
         create_table "clubs", force: :cascade do |t|
           t.string "name", default: "", null: false
           t.index ["name"], name: "idx_name", unique: true, <%= i cond(5.0, using: :btree) %>
@@ -109,8 +109,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date    "to_date"
           t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
         end
-      EOS
-    }
+      ERB
+    end
 
     before { subject.diff(actual_dsl).migrate }
     subject { client }
@@ -127,15 +127,15 @@ describe 'Ridgepole::Client#diff -> migrate' do
       delta = subject.diff(expected_dsl)
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
-      delta.migrate(:noop => true)
+      delta.migrate(noop: true)
       expect(subject.dump).to match_ruby actual_dsl
     }
 
     it {
-      delta = client(:bulk_change => true).diff(expected_dsl)
+      delta = client(bulk_change: true).diff(expected_dsl)
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
-      expect(delta.script).to match_fuzzy <<-EOS
+      expect(delta.script).to match_fuzzy <<-RUBY
 change_table("dept_emp", {:bulk => true}) do |t|
   t.remove("emp_no")
   t.remove("from_date")
@@ -157,7 +157,7 @@ change_table("employees", {:bulk => true}) do |t|
   t.remove("gender")
   t.remove("hire_date")
 end
-      EOS
+      RUBY
       delta.migrate
       expect(subject.dump).to match_ruby expected_dsl
     }

data/spec/mysql/migrate/migrate_drop_column_and_unique_index_spec.rb

@@ -1,22 +1,22 @@
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when drop column and unique index' do
-    let(:actual_dsl) {
-      erbh(<<-EOS)
+    let(:actual_dsl) do
+      erbh(<<-ERB)
         create_table "employees", primary_key: "emp_no", force: :cascade do |t|
           t.string "first_name", limit: 14, null: false
           t.string "last_name", limit: 16, null: false
           t.index ["first_name", "last_name"], name: "first_name_last_name", unique: true, <%= i cond(5.0, using: :btree) %>
         end
-      EOS
-    }
+      ERB
+    end
 
-    let(:expected_dsl) {
-      erbh(<<-EOS)
+    let(:expected_dsl) do
+      erbh(<<-ERB)
         create_table "employees", primary_key: "emp_no", force: :cascade do |t|
           t.string "first_name", limit: 14, null: false
         end
-      EOS
-    }
+      ERB
+    end
 
     before do
       subject.diff(actual_dsl).migrate
@@ -27,7 +27,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           ('Taro', 'Yamada'), ('Taro', 'Sato')
         ;
       SQL
-      )
+                                           )
     end
     subject { client }
 
@@ -43,7 +43,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
       delta = subject.diff(expected_dsl)
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
-      delta.migrate(:noop => true)
+      delta.migrate(noop: true)
       expect(subject.dump).to match_ruby actual_dsl
     }
   end

data/spec/mysql/migrate/migrate_drop_column_spec.rb

@@ -1,7 +1,7 @@
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when drop column' do
-    let(:actual_dsl) {
-      erbh(<<-EOS)
+    let(:actual_dsl) do
+      erbh(<<-ERB)
         create_table "clubs", force: :cascade do |t|
           t.string "name", default: "", null: false
           t.index ["name"], name: "idx_name", unique: true, <%= i cond(5.0, using: :btree) %>
@@ -59,11 +59,11 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date    "to_date"
           t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
         end
-      EOS
-    }
+      ERB
+    end
 
-    let(:expected_dsl) {
-      erbh(<<-EOS)
+    let(:expected_dsl) do
+      erbh(<<-ERB)
         create_table "clubs", force: :cascade do |t|
           t.string "name", default: "", null: false
           t.index ["name"], name: "idx_name", unique: true, <%= i cond(5.0, using: :btree) %>
@@ -114,8 +114,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date    "to_date"
           t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
         end
-      EOS
-    }
+      ERB
+    end
 
     before { subject.diff(actual_dsl).migrate }
     subject { client }
@@ -129,10 +129,10 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
 
     it {
-      delta = client(:bulk_change => true).diff(expected_dsl)
+      delta = client(bulk_change: true).diff(expected_dsl)
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
-      expect(delta.script).to match_fuzzy <<-EOS
+      expect(delta.script).to match_fuzzy <<-RUBY
         change_table("dept_emp", {:bulk => true}) do |t|
           t.remove("from_date")
           t.remove("to_date")
@@ -148,7 +148,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.remove("gender")
           t.remove("hire_date")
         end
-      EOS
+      RUBY
       delta.migrate
       expect(subject.dump).to match_ruby expected_dsl
     }

data/spec/mysql/migrate/migrate_drop_index_spec.rb

@@ -1,7 +1,7 @@
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when drop index' do
-    let(:dsl) {
-      erbh(<<-EOS)
+    let(:dsl) do
+      erbh(<<-ERB)
         create_table "clubs", force: :cascade do |t|
           t.string "name", default: "", null: false
           t.index ["name"], name: "idx_name", unique: true, <%= i cond(5.0, using: :btree) %>
@@ -59,13 +59,13 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date    "to_date"
           t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
         end
-      EOS
-    }
+      ERB
+    end
 
     let(:actual_dsl) { dsl }
 
-    let(:expected_dsl) {
-      erbh(<<-EOS)
+    let(:expected_dsl) do
+      erbh(<<-ERB)
         create_table "clubs", force: :cascade do |t|
           t.string "name", default: "", null: false
         end
@@ -120,8 +120,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date    "from_date", null: false
           t.date    "to_date"
         end
-      EOS
-    }
+      ERB
+    end
 
     before { subject.diff(actual_dsl).migrate }
     subject { client }
@@ -135,10 +135,10 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
 
     it {
-      delta = client(:bulk_change => true).diff(expected_dsl)
+      delta = client(bulk_change: true).diff(expected_dsl)
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
-      expect(delta.script).to match_fuzzy <<-EOS
+      expect(delta.script).to match_fuzzy <<-RUBY
         change_table("clubs", {:bulk => true}) do |t|
           t.remove_index({:name=>"idx_name"})
         end
@@ -150,7 +150,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
         change_table("titles", {:bulk => true}) do |t|
           t.remove_index({:name=>"emp_no"})
         end
-      EOS
+      RUBY
       delta.migrate
       expect(subject.dump).to match_ruby expected_dsl
     }

data/spec/mysql/migrate/migrate_drop_table_spec.rb

@@ -1,7 +1,7 @@
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when drop table' do
-    let(:dsl) {
-      erbh(<<-EOS)
+    let(:dsl) do
+      erbh(<<-ERB)
         create_table "clubs", force: :cascade do |t|
           t.string "name", default: "", null: false
           t.index ["name"], name: "idx_name", unique: true, <%= i cond(5.0, using: :btree) %>
@@ -59,13 +59,13 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date    "to_date"
           t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
         end
-      EOS
-    }
+      ERB
+    end
 
     let(:actual_dsl) { dsl }
 
-    let(:expected_dsl) {
-      erbh(<<-EOS)
+    let(:expected_dsl) do
+      erbh(<<-ERB)
         create_table "departments", primary_key: "dept_no", force: :cascade do |t|
           t.string "dept_name", limit: 40, null: false
           t.index ["dept_name"], name: "dept_name", unique: true, <%= i cond(5.0, using: :btree) %>
@@ -104,8 +104,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date    "to_date"
           t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
         end
-      EOS
-    }
+      ERB
+    end
 
     before { subject.diff(actual_dsl).migrate }
     subject { client }

data/spec/mysql/migrate/migrate_duplicate_index_spec.rb

@@ -1,7 +1,7 @@
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when index already defined' do
-    let(:dsl) {
-      <<-EOS
+    let(:dsl) do
+      <<-RUBY
         create_table "salaries", id: false, force: :cascade do |t|
           t.integer "emp_no", null: false
           t.integer "salary", null: false
@@ -11,15 +11,15 @@ describe 'Ridgepole::Client#diff -> migrate' do
 
         add_index "salaries", ["emp_no"], name: "emp_no", using: :btree
         add_index "salaries", ["emp_no"], name: "emp_no", using: :btree
-      EOS
-    }
+      RUBY
+    end
 
     subject { client }
 
     it {
-      expect {
+      expect do
         subject.diff(dsl)
-      }.to raise_error('Index `salaries(emp_no)` already defined')
+      end.to raise_error('Index `salaries(emp_no)` already defined')
     }
   end
 end

data/spec/mysql/migrate/migrate_duplicate_table_spec.rb

@@ -1,7 +1,7 @@
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when table already defined' do
-    let(:dsl) {
-      <<-EOS
+    let(:dsl) do
+      <<-RUBY
         create_table "employees", primary_key: "emp_no", force: :cascade do |t|
           t.date    "birth_date", null: false
           t.string  "first_name", limit: 14, null: false
@@ -21,15 +21,15 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "age", unsigned: true, null: false
           t.date    "updated_at"
         end
-      EOS
-    }
+      RUBY
+    end
 
     subject { client }
 
     it {
-      expect {
+      expect do
         subject.diff(dsl)
-      }.to raise_error('Table `employees` already defined')
+      end.to raise_error('Table `employees` already defined')
     }
   end
 end

data/spec/mysql/migrate/migrate_empty_spec.rb

@@ -1,8 +1,8 @@
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when database is empty' do
     let(:actual_dsl) { '' }
-    let(:expected_dsl) {
-      erbh(<<-EOS)
+    let(:expected_dsl) do
+      erbh(<<-ERB)
         create_table "clubs", force: :cascade do |t|
           t.string "name", default: "", null: false
           t.index ["name"], name: "idx_name", unique: true, <%= i cond(5.0, using: :btree) %>
@@ -60,8 +60,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date    "to_date"
           t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
         end
-      EOS
-    }
+      ERB
+    end
 
     subject { client }
 

data/spec/mysql/migrate/migrate_execute_spec.rb

@@ -1,7 +1,7 @@
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when execute' do
-    let(:dsl) {
-      erbh(<<-EOS)
+    let(:dsl) do
+      erbh(<<-ERB)
         create_table "authors", <%= i cond('>= 5.1',id: :integer) + {force: :cascade} %> do |t|
           t.string "name", null: false
         end
@@ -11,11 +11,11 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "author_id", null: false
           t.index ["author_id"], name: "idx_author_id", <%= i cond(5.0, using: :btree) %>
         end
-      EOS
-    }
+      ERB
+    end
 
-    let(:dsl_with_execute) {
-      erbh(<<-EOS)
+    let(:dsl_with_execute) do
+      erbh(<<-ERB)
         create_table "authors", <%= i cond('>= 5.1',id: :integer) + {force: :cascade} %> do |t|
           t.string "name", null: false
         end
@@ -29,8 +29,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
         execute("ALTER TABLE books ADD CONSTRAINT fk_author FOREIGN KEY (author_id) REFERENCES authors (id)") do |c|
           c.raw_connection.query("SELECT 1 FROM information_schema.key_column_usage WHERE TABLE_SCHEMA = '<%= TEST_SCHEMA %>' AND CONSTRAINT_NAME = 'fk_author' LIMIT 1").each.length.zero?
         end
-      EOS
-    }
+      ERB
+    end
 
     before { subject.diff(dsl).migrate }
     subject { client }
@@ -40,7 +40,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby dsl
 
-      expect(show_create_table(:books)).to match_fuzzy erbh(<<-EOS)
+      expect(show_create_table(:books)).to match_fuzzy erbh(<<-ERB)
         CREATE TABLE `books` (
           `id` int(11) NOT NULL AUTO_INCREMENT,
           `title` varchar(255) NOT NULL,
@@ -48,15 +48,15 @@ describe 'Ridgepole::Client#diff -> migrate' do
           PRIMARY KEY (`id`),
           KEY `idx_author_id` (`author_id`) <%= cond(5.0, 'USING BTREE') %>
         ) ENGINE=InnoDB DEFAULT CHARSET=utf8
-      EOS
+      ERB
 
       delta.migrate
 
-      expect(subject.dump).to match_fuzzy (dsl + (<<-EOS))
+      expect(subject.dump).to match_fuzzy dsl + <<-RUBY
         add_foreign_key "books", "authors", name: "fk_author"
-      EOS
+      RUBY
 
-      expect(show_create_table(:books)).to match_fuzzy erbh(<<-EOS)
+      expect(show_create_table(:books)).to match_fuzzy erbh(<<-ERB)
         CREATE TABLE `books` (
           `id` int(11) NOT NULL AUTO_INCREMENT,
           `title` varchar(255) NOT NULL,
@@ -65,13 +65,13 @@ describe 'Ridgepole::Client#diff -> migrate' do
           KEY `idx_author_id` (`author_id`) <%= cond(5.0, 'USING BTREE') %>,
           CONSTRAINT `fk_author` FOREIGN KEY (`author_id`) REFERENCES `authors` (`id`)
         ) ENGINE=InnoDB DEFAULT CHARSET=utf8
-      EOS
+      ERB
     }
   end
 
   context 'when not execute' do
-    let(:dsl) {
-      erbh(<<-EOS)
+    let(:dsl) do
+      erbh(<<-ERB)
         create_table "authors", <%= i cond('>= 5.1',id: :integer) + {force: :cascade} %> do |t|
           t.string "name", null: false
         end
@@ -82,11 +82,11 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.index ["author_id"], name: "idx_author_id", <%= i cond(5.0, using: :btree) %>
         end
         add_foreign_key "books", "authors", name: "fk_author"
-      EOS
-    }
+      ERB
+    end
 
-    let(:dsl_with_execute) {
-      erbh(<<-EOS)
+    let(:dsl_with_execute) do
+      erbh(<<-ERB)
         create_table "authors", <%= i cond('>= 5.1',id: :integer) + {force: :cascade} %> do |t|
           t.string "name", null: false
         end
@@ -102,8 +102,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
         end
 
         add_foreign_key "books", "authors", name: "fk_author"
-      EOS
-    }
+      ERB
+    end
 
     before { subject.diff(dsl_with_execute).migrate }
     subject { client }
@@ -113,7 +113,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby dsl
 
-      expect(show_create_table(:books)).to match_fuzzy erbh(<<-EOS)
+      expect(show_create_table(:books)).to match_fuzzy erbh(<<-ERB)
         CREATE TABLE `books` (
           `id` int(11) NOT NULL AUTO_INCREMENT,
           `title` varchar(255) NOT NULL,
@@ -122,13 +122,13 @@ describe 'Ridgepole::Client#diff -> migrate' do
           KEY `idx_author_id` (`author_id`) <%= cond(5.0, 'USING BTREE') %>,
           CONSTRAINT `fk_author` FOREIGN KEY (`author_id`) REFERENCES `authors` (`id`)
         ) ENGINE=InnoDB DEFAULT CHARSET=utf8
-      EOS
+      ERB
 
-      migrated, out = delta.migrate
+      migrated, _out = delta.migrate
       expect(migrated).to be_falsey
       expect(subject.dump).to match_ruby dsl
 
-      expect(show_create_table(:books)).to match_fuzzy erbh(<<-EOS)
+      expect(show_create_table(:books)).to match_fuzzy erbh(<<-ERB)
         CREATE TABLE `books` (
           `id` int(11) NOT NULL AUTO_INCREMENT,
           `title` varchar(255) NOT NULL,
@@ -137,13 +137,13 @@ describe 'Ridgepole::Client#diff -> migrate' do
           KEY `idx_author_id` (`author_id`) <%= cond(5.0, 'USING BTREE') %>,
           CONSTRAINT `fk_author` FOREIGN KEY (`author_id`) REFERENCES `authors` (`id`)
         ) ENGINE=InnoDB DEFAULT CHARSET=utf8
-      EOS
+      ERB
     }
   end
 
   context 'when execute (noop)' do
-    let(:dsl) {
-      erbh(<<-EOS)
+    let(:dsl) do
+      erbh(<<-ERB)
         create_table "authors", <%= i cond('>= 5.1',id: :integer) + {force: :cascade} %> do |t|
           t.string "name", null: false
         end
@@ -153,11 +153,11 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "author_id", null: false
           t.index ["author_id"], name: "idx_author_id", <%= i cond(5.0, using: :btree) %>
         end
-      EOS
-    }
+      ERB
+    end
 
-    let(:dsl_with_execute) {
-      erbh(<<-EOS)
+    let(:dsl_with_execute) do
+      erbh(<<-ERB)
         create_table "authors", <%= i cond('>= 5.1',id: :integer) + {force: :cascade} %> do |t|
           t.string "name", null: false
         end
@@ -171,8 +171,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
         execute("ALTER TABLE books ADD CONSTRAINT fk_author FOREIGN KEY (author_id) REFERENCES authors (id)") do |c|
           c.raw_connection.query("SELECT 1 FROM information_schema.key_column_usage WHERE TABLE_SCHEMA = '<%= TEST_SCHEMA %>' AND CONSTRAINT_NAME = 'fk_author' LIMIT 1").each.length.zero?
         end
-      EOS
-    }
+      ERB
+    end
 
     before { subject.diff(dsl).migrate }
     subject { client }
@@ -182,7 +182,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby dsl
 
-      expect(show_create_table(:books)).to match_fuzzy erbh(<<-EOS)
+      expect(show_create_table(:books)).to match_fuzzy erbh(<<-ERB)
         CREATE TABLE `books` (
           `id` int(11) NOT NULL AUTO_INCREMENT,
           `title` varchar(255) NOT NULL,
@@ -190,15 +190,15 @@ describe 'Ridgepole::Client#diff -> migrate' do
           PRIMARY KEY (`id`),
           KEY `idx_author_id` (`author_id`) <%= cond(5.0, 'USING BTREE') %>
         ) ENGINE=InnoDB DEFAULT CHARSET=utf8
-      EOS
+      ERB
 
-      migrated, sql = delta.migrate(:noop => true)
+      migrated, sql = delta.migrate(noop: true)
       expect(migrated).to be_truthy
       expect(subject.dump).to match_ruby dsl
 
-      expect(sql).to match_fuzzy "ALTER TABLE books ADD CONSTRAINT fk_author FOREIGN KEY (author_id) REFERENCES authors (id)"
+      expect(sql).to match_fuzzy 'ALTER TABLE books ADD CONSTRAINT fk_author FOREIGN KEY (author_id) REFERENCES authors (id)'
 
-      expect(show_create_table(:books)).to match_fuzzy erbh(<<-EOS)
+      expect(show_create_table(:books)).to match_fuzzy erbh(<<-ERB)
         CREATE TABLE `books` (
           `id` int(11) NOT NULL AUTO_INCREMENT,
           `title` varchar(255) NOT NULL,
@@ -206,13 +206,13 @@ describe 'Ridgepole::Client#diff -> migrate' do
           PRIMARY KEY (`id`),
           KEY `idx_author_id` (`author_id`) <%= cond(5.0, 'USING BTREE') %>
         ) ENGINE=InnoDB DEFAULT CHARSET=utf8
-      EOS
+      ERB
     }
   end
 
   context 'when not execute (noop)' do
-    let(:dsl) {
-      erbh(<<-EOS)
+    let(:dsl) do
+      erbh(<<-ERB)
         create_table "authors", <%= i cond('>= 5.1',id: :integer) + {force: :cascade} %> do |t|
           t.string "name", null: false
         end
@@ -223,11 +223,11 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.index ["author_id"], name: "idx_author_id", <%= i cond(5.0, using: :btree) %>
         end
         add_foreign_key "books", "authors", name: "fk_author"
-      EOS
-    }
+      ERB
+    end
 
-    let(:dsl_with_execute) {
-      erbh(<<-EOS)
+    let(:dsl_with_execute) do
+      erbh(<<-ERB)
         create_table "authors", <%= i cond('>= 5.1',id: :integer) + {force: :cascade} %> do |t|
           t.string "name", null: false
         end
@@ -243,8 +243,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
         end
 
         add_foreign_key "books", "authors", name: "fk_author"
-      EOS
-    }
+      ERB
+    end
 
     before { subject.diff(dsl_with_execute).migrate }
     subject { client }
@@ -254,7 +254,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby dsl
 
-      expect(show_create_table(:books)).to match_fuzzy erbh(<<-EOS)
+      expect(show_create_table(:books)).to match_fuzzy erbh(<<-ERB)
         CREATE TABLE `books` (
           `id` int(11) NOT NULL AUTO_INCREMENT,
           `title` varchar(255) NOT NULL,
@@ -263,15 +263,15 @@ describe 'Ridgepole::Client#diff -> migrate' do
           KEY `idx_author_id` (`author_id`) <%= cond(5.0, 'USING BTREE') %>,
           CONSTRAINT `fk_author` FOREIGN KEY (`author_id`) REFERENCES `authors` (`id`)
         ) ENGINE=InnoDB DEFAULT CHARSET=utf8
-      EOS
+      ERB
 
-      migrated, sql = delta.migrate(:noop => true)
+      migrated, sql = delta.migrate(noop: true)
       expect(migrated).to be_falsey
       expect(subject.dump).to match_ruby dsl
 
-      expect(sql).to match_fuzzy ""
+      expect(sql).to match_fuzzy ''
 
-      expect(show_create_table(:books)).to match_fuzzy erbh(<<-EOS)
+      expect(show_create_table(:books)).to match_fuzzy erbh(<<-ERB)
         CREATE TABLE `books` (
           `id` int(11) NOT NULL AUTO_INCREMENT,
           `title` varchar(255) NOT NULL,
@@ -280,7 +280,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           KEY `idx_author_id` (`author_id`) <%= cond(5.0, 'USING BTREE') %>,
           CONSTRAINT `fk_author` FOREIGN KEY (`author_id`) REFERENCES `authors` (`id`)
         ) ENGINE=InnoDB DEFAULT CHARSET=utf8
-      EOS
+      ERB
     }
   end
 end