ridgepole 0.6.6 → 0.7.0.alpha

data/spec/mysql/cli/ridgepole_spec.rb

@@ -7,7 +7,7 @@ describe 'ridgepole' do
   end
 
   context 'when help' do
-    it do
+    specify do
       out, status = run_cli(:args => ['-h'])
       out = out.gsub(/Usage: .*\n/, '')
 
@@ -55,7 +55,7 @@ describe 'ridgepole' do
   end
 
   context 'when export' do
-    it 'not split' do
+    specify 'not split' do
       out, status = run_cli(:args => ['-c', conf, '-e', conf, conf])
 
       expect(status.success?).to be_truthy
@@ -66,7 +66,7 @@ describe 'ridgepole' do
       EOS
     end
 
-    it 'not split with outfile' do
+    specify 'not split with outfile' do
       Tempfile.open("#{File.basename __FILE__}.#{$$}") do |f|
         out, status = run_cli(:args => ['-c', conf, '-e', '-o', f.path])
 
@@ -79,7 +79,7 @@ describe 'ridgepole' do
       end
     end
 
-    it 'not split with output stdout' do
+    specify 'not split with output stdout' do
       out, status = run_cli(:args => ['-c', conf, '-e', '-o', '-'])
 
       expect(status.success?).to be_truthy
@@ -90,7 +90,7 @@ describe 'ridgepole' do
       EOS
     end
 
-    it 'split' do
+    specify 'split' do
       out, status = run_cli(:args => ['-c', conf, '-e', '--split'])
 
       expect(status.success?).to be_truthy
@@ -102,7 +102,7 @@ describe 'ridgepole' do
       EOS
     end
 
-    it 'split with outdir' do
+    specify 'split with outdir' do
       Tempfile.open("#{File.basename __FILE__}.#{$$}") do |f|
         out, status = run_cli(:args => ['-c', conf, '-e', '--split', '-o', f.path, conf, conf])
 
@@ -118,7 +118,7 @@ describe 'ridgepole' do
   end
 
   context 'when apply' do
-    it 'apply' do
+    specify 'apply' do
       out, status = run_cli(:args => ['-c', conf, '-a'])
 
       expect(status.success?).to be_truthy
@@ -132,7 +132,7 @@ describe 'ridgepole' do
       EOS
     end
 
-    it 'apply with conf file' do
+    specify 'apply with conf file' do
       Tempfile.open(["#{File.basename __FILE__}.#{$$}", '.yml']) do |conf_file|
         conf_file.puts <<-EOS
           adapter: mysql2
@@ -154,7 +154,7 @@ describe 'ridgepole' do
       end
     end
 
-    it 'apply with conf file (production)' do
+    specify 'apply with conf file (production)' do
       Tempfile.open(["#{File.basename __FILE__}.#{$$}", '.yml']) do |conf_file|
         conf_file.puts <<-EOS
           development:
@@ -180,7 +180,7 @@ describe 'ridgepole' do
       end
     end
 
-    it 'dry-run' do
+    specify 'dry-run' do
       out, status = run_cli(:args => ['-c', conf, '-a', '--dry-run'])
 
       expect(status.success?).to be_truthy
@@ -196,7 +196,7 @@ describe 'ridgepole' do
     context 'when differ true' do
       let(:differ) { true }
 
-      it 'apply' do
+      specify 'apply' do
         out, status = run_cli(:args => ['-c', conf, '-a'])
 
         expect(status.success?).to be_truthy
@@ -209,7 +209,7 @@ describe 'ridgepole' do
         EOS
       end
 
-      it 'dry-run' do
+      specify 'dry-run' do
         out, status = run_cli(:args => ['-c', conf, '-a', '--dry-run'])
 
         expect(status.success?).to be_truthy
@@ -232,7 +232,7 @@ describe 'ridgepole' do
   end
 
   context 'when diff' do
-    it do
+    specify do
       out, status = run_cli(:args => ['-c', conf, '-d', conf, conf])
 
       expect(status.success?).to be_truthy
@@ -246,7 +246,7 @@ describe 'ridgepole' do
     context 'when differ true' do
       let(:differ) { true }
 
-      it do
+      specify do
         out, status = run_cli(:args => ['-c', conf, '-d', conf, conf])
 
         # Exit code 1 if there is a difference
@@ -269,7 +269,7 @@ describe 'ridgepole' do
     end
 
     context 'when config file' do
-      it '.yml' do
+      specify '.yml' do
         Tempfile.open(["#{File.basename __FILE__}.#{$$}", '.yml']) do |conf_file|
           conf_file.puts <<-EOS
             adapter: mysql2
@@ -289,7 +289,7 @@ describe 'ridgepole' do
         end
       end
 
-      it '.yml (file2)' do
+      specify '.yml (file2)' do
         Tempfile.open(["#{File.basename __FILE__}.#{$$}", '.yml']) do |conf_file|
           conf_file.puts <<-EOS
             adapter: mysql2
@@ -309,7 +309,7 @@ describe 'ridgepole' do
         end
       end
 
-      it '.yml (development)' do
+      specify '.yml (development)' do
         Tempfile.open(["#{File.basename __FILE__}.#{$$}", '.yml']) do |conf_file|
           conf_file.puts <<-EOS
             development:
@@ -333,7 +333,7 @@ describe 'ridgepole' do
         end
       end
 
-      it '.yml (production)' do
+      specify '.yml (production)' do
         Tempfile.open(["#{File.basename __FILE__}.#{$$}", '.yml']) do |conf_file|
           conf_file.puts <<-EOS
             development:
@@ -357,7 +357,7 @@ describe 'ridgepole' do
         end
       end
 
-      it '.yaml' do
+      specify '.yaml' do
         Tempfile.open(["#{File.basename __FILE__}.#{$$}", '.yaml']) do |conf_file|
           conf_file.puts <<-EOS
             adapter: mysql2
@@ -377,7 +377,7 @@ describe 'ridgepole' do
         end
       end
 
-      it '.rb' do
+      specify '.rb' do
         Tempfile.open(["#{File.basename __FILE__}.#{$$}", '.rb']) do |conf_file|
           conf_file.puts <<-EOS
             create_table :table do

data/spec/mysql/collation/collation_spec.rb

@@ -1,23 +1,23 @@
-describe 'Ridgepole::Client#diff -> migrate', condition: [:mysql_awesome_enabled, :activerecord_5] do
+describe 'Ridgepole::Client#diff -> migrate' do
   context 'when change column (add collation)' do
     let(:actual_dsl) {
       erbh(<<-EOS)
-        create_table "employee_clubs", unsigned: true, force: :cascade do |t|
-          t.integer "emp_no",  <%= i limit(4) + {null: false} %>
-          t.integer "club_id", <%= i limit(4) + {null: false, unsigned: true} %>
-          t.string  "string",  <%= i limit(255) + {null: false, collation: "ascii_bin"} %>
-          t.text    "text",    limit: 65535, null: false
+        create_table "employee_clubs", <%= i cond('5.1.', id: :bigint) %>, unsigned: true, force: :cascade 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",    <%= i cond('5.0.', limit: 65535) %>, null: false
         end
       EOS
     }
 
     let(:expected_dsl) {
       erbh(<<-EOS)
-        create_table "employee_clubs", unsigned: true, force: :cascade do |t|
-          t.integer "emp_no",  <%= i limit(4) + {null: false} %>
-          t.integer "club_id", <%= i limit(4) + {null: false, unsigned: true} %>
-          t.string  "string",  <%= i limit(255) + {null: false, collation: "ascii_bin"} %>
-          t.text    "text",    limit: 65535, null: false,                 collation: "utf8mb4_bin"
+        create_table "employee_clubs", <%= i cond('5.1.', id: :bigint) %>, unsigned: true, force: :cascade 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",    <%= i cond('5.0.', limit: 65535) %>, null: false,                 collation: "utf8mb4_bin"
         end
       EOS
     }
@@ -25,7 +25,7 @@ describe 'Ridgepole::Client#diff -> migrate', condition: [:mysql_awesome_enabled
     before { subject.diff(actual_dsl).migrate }
     subject { client }
 
-    it {
+    specify {
       delta = subject.diff(expected_dsl)
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_fuzzy actual_dsl
@@ -37,22 +37,22 @@ describe 'Ridgepole::Client#diff -> migrate', condition: [:mysql_awesome_enabled
   context 'when change column (delete collation)' do
     let(:actual_dsl) {
       erbh(<<-EOS)
-        create_table "employee_clubs", unsigned: true, force: :cascade do |t|
-          t.integer "emp_no",  <%= i limit(4) + {null: false} %>
-          t.integer "club_id", <%= i limit(4) + {null: false, unsigned: true} %>
-          t.string  "string",  <%= i limit(255) + {null: false, collation: "ascii_bin"} %>
-          t.text    "text",    limit: 65535, null: false,                 collation: "utf8mb4_bin"
+        create_table "employee_clubs", <%= i cond('5.1.', id: :bigint) %>, unsigned: true, force: :cascade 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",    <%= i cond('5.0.', limit: 65535) %>, null: false,                 collation: "utf8mb4_bin"
         end
       EOS
     }
 
     let(:expected_dsl) {
       erbh(<<-EOS)
-        create_table "employee_clubs", unsigned: true, force: :cascade do |t|
-          t.integer "emp_no",  <%= i limit(4) + {null: false} %>
-          t.integer "club_id", <%= i limit(4) + {null: false, unsigned: true} %>
-          t.string  "string",  <%= i limit(255) + {null: false, collation: "ascii_bin"} %>
-          t.text    "text",    limit: 65535, null: false
+        create_table "employee_clubs", <%= i cond('5.1.', id: :bigint) %>, unsigned: true, force: :cascade 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",    <%= i cond('5.0.', limit: 65535) %>, null: false
         end
       EOS
     }
@@ -60,7 +60,7 @@ describe 'Ridgepole::Client#diff -> migrate', condition: [:mysql_awesome_enabled
     before { subject.diff(actual_dsl).migrate }
     subject { client }
 
-    it {
+    specify {
       delta = subject.diff(expected_dsl)
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_fuzzy actual_dsl
@@ -72,22 +72,22 @@ describe 'Ridgepole::Client#diff -> migrate', condition: [:mysql_awesome_enabled
   context 'when change column (change collation)' do
     let(:actual_dsl) {
       erbh(<<-EOS)
-        create_table "employee_clubs", unsigned: true, force: :cascade do |t|
-          t.integer "emp_no",  <%= i limit(4) + {null: false} %>
-          t.integer "club_id", <%= i limit(4) + {null: false, unsigned: true} %>
-          t.string  "string",  <%= i limit(255) + {null: false, collation: "ascii_bin"} %>
-          t.text    "text",    limit: 65535, null: false,                 collation: "utf8mb4_bin"
+        create_table "employee_clubs", <%= i cond('5.1.', id: :bigint) %>, unsigned: true, force: :cascade 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",    <%= i cond('5.0.', limit: 65535) %>, null: false,                 collation: "utf8mb4_bin"
         end
       EOS
     }
 
     let(:expected_dsl) {
       erbh(<<-EOS)
-        create_table "employee_clubs", unsigned: true, force: :cascade do |t|
-          t.integer "emp_no",  <%= i limit(4) + {null: false} %>
-          t.integer "club_id", <%= i limit(4) + {null: false, unsigned: true} %>
+        create_table "employee_clubs", <%= i cond('5.1.', id: :bigint) %>, unsigned: true, force: :cascade do |t|
+          t.integer "emp_no",  null: false
+          t.integer "club_id", null: false, unsigned: true
           t.string  "string",  <%= i limit(255) + {null: false, collation: "utf8mb4_bin"} %>
-          t.text    "text",    limit: 65535, null: false,                 collation: "ascii_bin"
+          t.text    "text",    <%= i cond('5.0.', limit: 65535) %>, null: false,                 collation: "ascii_bin"
         end
       EOS
     }
@@ -95,7 +95,7 @@ describe 'Ridgepole::Client#diff -> migrate', condition: [:mysql_awesome_enabled
     before { subject.diff(actual_dsl).migrate }
     subject { client }
 
-    it {
+    specify {
       delta = subject.diff(expected_dsl)
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_fuzzy actual_dsl
@@ -107,11 +107,11 @@ describe 'Ridgepole::Client#diff -> migrate', condition: [:mysql_awesome_enabled
   context 'when change column (no change collation)' do
     let(:actual_dsl) {
       erbh(<<-EOS)
-        create_table "employee_clubs", unsigned: true, force: :cascade do |t|
-          t.integer "emp_no",  <%= i limit(4) + {null: false} %>
-          t.integer "club_id", <%= i limit(4) + {null: false, unsigned: true} %>
-          t.string  "string",  <%= i limit(255) + {null: false, collation: "ascii_bin"} %>
-          t.text    "text",    limit: 65535, null: false,                 collation: "utf8mb4_bin"
+        create_table "employee_clubs", <%= i cond('5.1.', id: :bigint) %>, unsigned: true, force: :cascade 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",    <%= i cond('5.0.', limit: 65535) %>, null: false,                 collation: "utf8mb4_bin"
         end
       EOS
     }
@@ -119,7 +119,7 @@ describe 'Ridgepole::Client#diff -> migrate', condition: [:mysql_awesome_enabled
     before { subject.diff(actual_dsl).migrate }
     subject { client }
 
-    it {
+    specify {
       delta = subject.diff(actual_dsl)
       expect(delta.differ?).to be_falsey
       expect(subject.dump).to match_fuzzy actual_dsl
@@ -128,13 +128,12 @@ describe 'Ridgepole::Client#diff -> migrate', condition: [:mysql_awesome_enabled
     }
 
     describe '#diff' do
-      it {
+      specify {
         Tempfile.open("#{File.basename __FILE__}.#{$$}") do |f|
           f.puts(actual_dsl)
           f.flush
 
           opts = ['--dump-without-table-options']
-          opts << '--enable-mysql-awesome' if condition(:mysql_awesome_enabled)
           out, status = run_ridgepole('--diff', "'#{JSON.dump(conn_spec)}'", f.path, *opts)
 
           expect(out).to be_empty

data/spec/mysql/comment/comment_spec.rb

@@ -1,23 +1,23 @@
-describe 'Ridgepole::Client#diff -> migrate', condition: [:activerecord_5] do
+describe 'Ridgepole::Client#diff -> migrate' do
   context 'when change column (add comment)' do
     let(:actual_dsl) {
-      <<-EOS
+      erbh(<<-EOS)
         create_table "employee_clubs", force: :cascade do |t|
           t.integer "emp_no",                null: false
           t.integer "club_id",               null: false
           t.string  "string",                null: false
-          t.text    "text",    limit: 65535, null: false
+          t.text    "text",    <%= i cond('5.0.', limit: 65535) %>, null: false
         end
       EOS
     }
 
     let(:expected_dsl) {
-      <<-EOS
+      erbh(<<-EOS)
         create_table "employee_clubs", force: :cascade do |t|
           t.integer "emp_no",                null: false, comment: "any comment"
           t.integer "club_id",               null: false, comment: "any comment2"
           t.string  "string",                null: false, comment: "any comment3"
-          t.text    "text",    limit: 65535, null: false, comment: "any comment4"
+          t.text    "text",    <%= i cond('5.0.', limit: 65535) %>, null: false, comment: "any comment4"
         end
       EOS
     }
@@ -36,23 +36,23 @@ describe 'Ridgepole::Client#diff -> migrate', condition: [:activerecord_5] do
 
   context 'when change column (delete comment)' do
     let(:actual_dsl) {
-      <<-EOS
+      erbh(<<-EOS)
         create_table "employee_clubs", force: :cascade do |t|
           t.integer "emp_no",                null: false, comment: "any comment"
           t.integer "club_id",               null: false, comment: "any comment2"
           t.string  "string",                null: false, comment: "any comment3"
-          t.text    "text",    limit: 65535, null: false, comment: "any comment4"
+          t.text    "text",    <%= i cond('5.0.', limit: 65535) %>, null: false, comment: "any comment4"
         end
       EOS
     }
 
     let(:expected_dsl) {
-      <<-EOS
+      erbh(<<-EOS)
         create_table "employee_clubs", force: :cascade do |t|
           t.integer "emp_no",                null: false
           t.integer "club_id",               null: false
           t.string  "string",                null: false
-          t.text    "text",    limit: 65535, null: false
+          t.text    "text",    <%= i cond('5.0.', limit: 65535) %>, null: false
         end
       EOS
     }
@@ -71,23 +71,23 @@ describe 'Ridgepole::Client#diff -> migrate', condition: [:activerecord_5] do
 
   context 'when change column (change comment)' do
     let(:actual_dsl) {
-      <<-EOS
+      erbh(<<-EOS)
         create_table "employee_clubs", force: :cascade do |t|
           t.integer "emp_no",                null: false, comment: "any comment"
           t.integer "club_id",               null: false, comment: "any comment2"
           t.string  "string",                null: false, comment: "any comment3"
-          t.text    "text",    limit: 65535, null: false, comment: "any comment4"
+          t.text    "text",    <%= i cond('5.0.', limit: 65535) %>, null: false, comment: "any comment4"
         end
       EOS
     }
 
     let(:expected_dsl) {
-      <<-EOS
+      erbh(<<-EOS)
         create_table "employee_clubs", force: :cascade do |t|
           t.integer "emp_no",                null: false, comment: "other comment"
           t.integer "club_id",               null: false, comment: "other comment2"
           t.string  "string",                null: false, comment: "other comment3"
-          t.text    "text",    limit: 65535, null: false, comment: "other comment4"
+          t.text    "text",    <%= i cond('5.0.', limit: 65535) %>, null: false, comment: "other comment4"
         end
       EOS
     }
@@ -106,12 +106,12 @@ describe 'Ridgepole::Client#diff -> migrate', condition: [:activerecord_5] do
 
   context 'when change column (no change comment)' do
     let(:actual_dsl) {
-      <<-EOS
+      erbh(<<-EOS)
         create_table "employee_clubs", force: :cascade do |t|
           t.integer "emp_no",                null: false, comment: "any comment"
           t.integer "club_id",               null: false, comment: "any comment2"
           t.string  "string",                null: false, comment: "any comment3"
-          t.text    "text",    limit: 65535, null: false, comment: "any comment4"
+          t.text    "text",    <%= i cond('5.0.', limit: 65535) %>, null: false, comment: "any comment4"
         end
       EOS
     }
@@ -130,12 +130,12 @@ describe 'Ridgepole::Client#diff -> migrate', condition: [:activerecord_5] do
 
   context 'when create table (with comment)' do
     let(:expected_dsl) {
-      <<-EOS
+      erbh(<<-EOS)
         create_table "employee_clubs", force: :cascade, comment: "table comment" do |t|
           t.integer "emp_no",                null: false, comment: "other comment"
           t.integer "club_id",               null: false, comment: "other comment2"
           t.string  "string",                null: false, comment: "other comment3"
-          t.text    "text",    limit: 65535, null: false, comment: "other comment4"
+          t.text    "text",    <%= i cond('5.0.', limit: 65535) %>, null: false, comment: "other comment4"
         end
       EOS
     }
@@ -153,12 +153,12 @@ describe 'Ridgepole::Client#diff -> migrate', condition: [:activerecord_5] do
 
   context 'when drop table (with comment)' do
     let(:actual_dsl) {
-      <<-EOS
+      erbh(<<-EOS)
         create_table "employee_clubs", force: :cascade, comment: "table comment" do |t|
           t.integer "emp_no",                null: false, comment: "other comment"
           t.integer "club_id",               null: false, comment: "other comment2"
           t.string  "string",                null: false, comment: "other comment3"
-          t.text    "text",    limit: 65535, null: false, comment: "other comment4"
+          t.text    "text",    <%= i cond('5.0.', limit: 65535) %>, null: false, comment: "other comment4"
         end
       EOS
     }