ridgepole 0.9.0.beta → 0.9.3

data/spec/mysql/migrate/migrate_change_column3_spec.rb

@@ -99,7 +99,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.string "last_name", limit: 16, null: false
           t.string "gender", limit: 1, null: false
           t.date   "hire_date", null: false
-          t.<%= cond('>= 5.1','bigint', 'integer') %> "user_id"
+          t.bigint "user_id"
           t.index "user_id"
         end
       ERB
@@ -136,7 +136,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.string "last_name", limit: 16, null: false
           t.string "gender", limit: 1, null: false
           t.date   "hire_date", null: false
-          t.<%= cond('>= 5.1','bigint', 'integer') %> "user_id"
+          t.bigint "user_id"
           t.string "user_type"
           t.index ["user_type", "user_id"]
         end
@@ -169,7 +169,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
     let(:actual_dsl) do
       erbh(<<-ERB)
         create_table "employees", primary_key: "emp_no", force: :cascade do |t|
-          t.<%= cond('>= 5.1','bigint', 'integer') %> "user_id", unsigned: true
+          t.bigint "user_id", unsigned: true
           t.string "user_type"
           t.index ["user_type", "user_id"]
         end
@@ -227,8 +227,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.string "last_name", limit: 16, null: false
           t.string "gender", limit: 1, null: false
           t.date   "hire_date", null: false
-          t.<%= cond('>= 5.1','bigint', 'integer') %> "user_id"
-          t.index ["user_id"], name: "index_employees_on_user_id", <%= i cond(5.0, using: :btree) %>
+          t.bigint "user_id"
+          t.index ["user_id"], name: "index_employees_on_user_id"
         end
       ERB
     end
@@ -279,9 +279,9 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.string "last_name", limit: 16, null: false
           t.string "gender", limit: 1, null: false
           t.date   "hire_date", null: false
-          t.<%= cond('>= 5.1','bigint', 'integer') %> "user_id"
+          t.bigint "user_id"
           t.string "user_type"
-          t.index ["user_type", "user_id"], name: "index_employees_on_user_type_and_user_id", <%= i cond(5.0, using: :btree) %>
+          t.index ["user_type", "user_id"], name: "index_employees_on_user_type_and_user_id"
         end
       ERB
     end
@@ -310,7 +310,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.string "last_name", limit: 16, null: false
           t.string "gender", limit: 1, null: false
           t.date   "hire_date", null: false
-          t.<%= cond('>= 5.1','bigint', 'integer') %> "user_id"
+          t.bigint "user_id"
           t.index "user_id"
         end
 
@@ -381,8 +381,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.string "last_name", limit: 16, null: false
           t.string "gender", limit: 1, null: false
           t.date   "hire_date", null: false
-          t.<%= cond('>= 5.1','bigint', 'integer') %> "user_id"
-          t.index ["user_id"], name: "index_employees_on_user_id", <%= i cond(5.0, using: :btree) %>
+          t.bigint "user_id"
+          t.index ["user_id"], name: "index_employees_on_user_id"
         end
         create_table "users", force: :cascade do |t|
         end

data/spec/mysql/migrate/migrate_change_column5_spec.rb

@@ -12,7 +12,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date     "hire_date", null: false
           t.datetime "created_at", null: false
           t.datetime "updated_at", null: false
-          t.binary   "registered_name", <%= i cond(5.0, limit: 65535) %>
+          t.binary   "registered_name"
         end
       ERB
     end
@@ -70,7 +70,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date     "hire_date", null: false
           t.datetime "created_at", null: false
           t.datetime "updated_at", null: false
-          t.binary   "registered_name", <%= i cond(5.0, limit: 65535) %>
+          t.binary   "registered_name"
         end
       ERB
     end
@@ -98,7 +98,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date     "hire_date", null: false
           t.datetime "created_at", null: false
           t.datetime "updated_at", null: false
-          t.binary   "registered_name", <%= i cond(5.0, limit: 65535) %>
+          t.binary   "registered_name"
         end
       ERB
     end

data/spec/mysql/migrate/migrate_change_column6_spec.rb

@@ -12,7 +12,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date     "hire_date", null: false
           t.datetime "created_at", null: false
           t.datetime "updated_at", null: false
-          t.binary   "registered_name", <%= i cond(5.0, limit: 65535) %>
+          t.binary   "registered_name"
         end
       ERB
     end
@@ -45,7 +45,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
 
       expect(show_create_table_mysql('employees')).to match_fuzzy erbh(<<-ERB)
         CREATE TABLE `employees` (
-          `id` <%= cond('>= 5.1','bigint(20)', 'int(11)') %> NOT NULL AUTO_INCREMENT,
+          `id` bigint(20) NOT NULL AUTO_INCREMENT,
           `ext_column` varchar(255) NOT NULL,
           `birth_date` date NOT NULL,
           `first_name` varchar(14) NOT NULL,
@@ -72,7 +72,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date     "hire_date", null: false
           t.datetime "created_at", null: false
           t.datetime "updated_at", null: false
-          t.binary   "registered_name", <%= i cond(5.0, limit: 65535) %>
+          t.binary   "registered_name"
         end
       ERB
     end
@@ -105,7 +105,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
 
       expect(show_create_table_mysql('employees')).to match_fuzzy erbh(<<-ERB)
         CREATE TABLE `employees` (
-          `emp_id` <%= cond('>= 5.1','bigint(20)', 'int(11)') %> NOT NULL AUTO_INCREMENT,
+          `emp_id` bigint(20) NOT NULL AUTO_INCREMENT,
           `ext_column` varchar(255) NOT NULL,
           `birth_date` date NOT NULL,
           `first_name` varchar(14) NOT NULL,
@@ -132,7 +132,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date     "hire_date", null: false
           t.datetime "created_at", null: false
           t.datetime "updated_at", null: false
-          t.binary   "registered_name", <%= i cond(5.0, limit: 65535) %>
+          t.binary   "registered_name"
         end
       ERB
     end
@@ -190,7 +190,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.date     "hire_date", null: false
           t.datetime "created_at", null: false
           t.datetime "updated_at", null: false
-          t.binary   "registered_name", <%= i cond(5.0, limit: 65535) %>
+          t.binary   "registered_name"
         end
       ERB
     end
@@ -223,7 +223,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
 
       expect(show_create_table_mysql('employees')).to match_fuzzy erbh(<<-ERB)
         CREATE TABLE `employees` (
-          `id` <%= cond('>= 5.1','bigint(20)', 'int(11)') %> NOT NULL AUTO_INCREMENT,
+          `id` bigint(20) NOT NULL AUTO_INCREMENT,
           `ext_column` varchar(255) NOT NULL,
           `birth_date` date NOT NULL,
           `first_name` varchar(14) NOT NULL,

data/spec/mysql/migrate/migrate_change_column7_spec.rb

@@ -5,7 +5,11 @@ describe 'Ridgepole::Client#diff -> migrate' do
     let(:dsl) do
       erbh(<<-ERB)
         create_table "salaries", id: false, force: :cascade do |t|
-          t.integer "emp_no", limit: 8, null: false
+          t.integer "emp_no_int", limit: 4, null: false
+          t.integer "emp_no_bigint5", limit: 5, null: false
+          t.integer "emp_no_bigint6", limit: 6, null: false
+          t.integer "emp_no_bigint7", limit: 7, null: false
+          t.integer "emp_no_bigint8", limit: 8, null: false
           t.float   "salary", <%= i cond('< 5.2.0.beta2', limit: 24) %>, null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
@@ -17,7 +21,12 @@ describe 'Ridgepole::Client#diff -> migrate' do
     subject { client }
 
     it {
-      expect(subject.dump).to match_ruby dsl.sub(/t.integer "emp_no", limit: 8/, 't.bigint "emp_no"')
+      expect(subject.dump).to match_ruby dsl
+        .sub(/t.integer "emp_no_int", limit: 4/, 't.integer "emp_no_int"')
+        .sub(/t.integer "emp_no_bigint5", limit: 5/, 't.bigint "emp_no_bigint5"')
+        .sub(/t.integer "emp_no_bigint6", limit: 6/, 't.bigint "emp_no_bigint6"')
+        .sub(/t.integer "emp_no_bigint7", limit: 7/, 't.bigint "emp_no_bigint7"')
+        .sub(/t.integer "emp_no_bigint8", limit: 8/, 't.bigint "emp_no_bigint8"')
       delta = subject.diff(dsl)
       expect(delta.differ?).to be_falsey
     }

data/spec/mysql/migrate/migrate_change_column_default_spec.rb

@@ -175,7 +175,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
     let(:result_dsl) do
       erbh(<<-ERB)
         create_table "salaries", id: false, force: :cascade do |t|
-          t.integer "emp_no", <%= i cond(5.0, default: 0) + {null: false} %>
+          t.integer "emp_no", null: false
           t.float   "salary", <%= i cond('< 5.2.0.beta2', limit: 24) %>, null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
@@ -187,25 +187,13 @@ describe 'Ridgepole::Client#diff -> migrate' do
     subject { client }
 
     it {
-      if condition(5.0)
-        expect(Ridgepole::Logger.instance).to receive(:warn).with('[WARNING] Table `salaries`: `default: nil` is ignored when `null: false`. Please apply twice')
-      else
-        expect(Ridgepole::Logger.instance).to_not receive(:warn)
-      end
+      expect(Ridgepole::Logger.instance).to_not receive(:warn)
 
       delta = subject.diff(expected_dsl)
       expect(delta.differ?).to be_truthy
       expect(subject.dump).to match_ruby actual_dsl
       delta.migrate
       expect(subject.dump).to match_fuzzy result_dsl
-
-      if condition(5.0)
-        delta = subject.diff(expected_dsl)
-        expect(delta.differ?).to be_truthy
-        expect(subject.dump).to match_fuzzy result_dsl
-        delta.migrate
-        expect(subject.dump).to match_ruby expected_dsl
-      end
     }
   end
 end

data/spec/mysql/migrate/migrate_change_column_spec.rb

@@ -6,12 +6,12 @@ describe 'Ridgepole::Client#diff -> migrate' 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) %>
+          t.index ["name"], name: "idx_name", unique: true
         end
 
         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) %>
+          t.index ["dept_name"], name: "dept_name", unique: true
         end
 
         create_table "dept_emp", id: false, force: :cascade do |t|
@@ -19,8 +19,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.string  "dept_no", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index ["dept_no"], name: "dept_no", <%= i cond(5.0, using: :btree) %>
-          t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
+          t.index ["dept_no"], name: "dept_no"
+          t.index ["emp_no"], name: "emp_no"
         end
 
         create_table "dept_manager", id: false, force: :cascade do |t|
@@ -28,14 +28,14 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "emp_no", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index ["dept_no"], name: "dept_no", <%= i cond(5.0, using: :btree) %>
-          t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
+          t.index ["dept_no"], name: "dept_no"
+          t.index ["emp_no"], name: "emp_no"
         end
 
         create_table "employee_clubs", force: :cascade do |t|
           t.integer "emp_no", null: false
           t.integer "club_id", null: false
-          t.index ["emp_no", "club_id"], name: "idx_emp_no_club_id", <%= i cond(5.0, using: :btree) %>
+          t.index ["emp_no", "club_id"], name: "idx_emp_no_club_id"
         end
 
         create_table "employees", primary_key: "emp_no", force: :cascade do |t|
@@ -51,7 +51,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "salary", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
+          t.index ["emp_no"], name: "emp_no"
         end
 
         create_table "titles", id: false, force: :cascade do |t|
@@ -59,7 +59,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.string  "title", limit: 50, null: false
           t.date    "from_date", null: false
           t.date    "to_date"
-          t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
+          t.index ["emp_no"], name: "emp_no"
         end
       ERB
     end
@@ -68,12 +68,12 @@ describe 'Ridgepole::Client#diff -> migrate' 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) %>
+          t.index ["name"], name: "idx_name", unique: true
         end
 
         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) %>
+          t.index ["dept_name"], name: "dept_name", unique: true
         end
 
         create_table "dept_emp", id: false, force: :cascade do |t|
@@ -81,8 +81,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.string  "dept_no", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index ["dept_no"], name: "dept_no", <%= i cond(5.0, using: :btree) %>
-          t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
+          t.index ["dept_no"], name: "dept_no"
+          t.index ["emp_no"], name: "emp_no"
         end
 
         create_table "dept_manager", id: false, force: :cascade do |t|
@@ -90,14 +90,14 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "emp_no", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index ["dept_no"], name: "dept_no", <%= i cond(5.0, using: :btree) %>
-          t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
+          t.index ["dept_no"], name: "dept_no"
+          t.index ["emp_no"], name: "emp_no"
         end
 
         create_table "employee_clubs", force: :cascade do |t|
           t.integer "emp_no", null: false
           t.integer "club_id"
-          t.index ["emp_no", "club_id"], name: "idx_emp_no_club_id", <%= i cond(5.0, using: :btree) %>
+          t.index ["emp_no", "club_id"], name: "idx_emp_no_club_id"
         end
 
         create_table "employees", primary_key: "emp_no", force: :cascade do |t|
@@ -113,7 +113,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "salary", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
+          t.index ["emp_no"], name: "emp_no"
         end
 
         create_table "titles", id: false, force: :cascade do |t|
@@ -121,7 +121,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.string  "title", limit: 50, null: false
           t.date    "from_date", null: false
           t.date    "to_date"
-          t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
+          t.index ["emp_no"], name: "emp_no"
         end
       ERB
     end
@@ -143,12 +143,12 @@ describe 'Ridgepole::Client#diff -> migrate' do
       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) %>)
+          t.change("club_id", :integer, **{:null=>true, :default=>nil, :unsigned=>false, :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) %>)
+          t.change("last_name", :string, **{:limit=>20, :default=>"XXX", :unsigned=>false, :comment=>nil})
+          t.change("gender", :string, **{:limit=>2, :null=>false, :default=>nil, :unsigned=>false, :comment=>nil})
         end
       ERB
       delta.migrate

data/spec/mysql/migrate/migrate_change_index2_spec.rb

@@ -9,7 +9,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "salary", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
+          t.index ["emp_no"], name: "emp_no"
         end
       ERB
     end
@@ -21,7 +21,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "salary", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index "emp_no", name: "emp_no", <%= i cond(5.0, using: :btree) %>
+          t.index "emp_no", name: "emp_no"
         end
       ERB
     end
@@ -43,7 +43,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "salary", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
+          t.index ["emp_no"], name: "emp_no"
         end
       ERB
     end
@@ -55,7 +55,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "salary", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index "salary", name: "emp_no", <%= i cond(5.0, using: :btree) %>
+          t.index "salary", name: "emp_no"
         end
       ERB
     end
@@ -67,7 +67,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "salary", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index ["salary"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
+          t.index ["salary"], name: "emp_no"
         end
       ERB
     end

data/spec/mysql/migrate/migrate_change_index3_spec.rb

@@ -9,7 +9,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "salary", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
+          t.index ["emp_no"], name: "emp_no"
         end
       ERB
     end
@@ -43,7 +43,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "salary", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
+          t.index ["emp_no"], name: "emp_no"
         end
       ERB
     end
@@ -102,7 +102,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "salary", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
+          t.index ["emp_no"], name: "emp_no"
         end
       ERB
     end
@@ -114,7 +114,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "salary", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index ["emp_no"], <%= i cond(5.0, using: :btree) %>
+          t.index ["emp_no"]
         end
       ERB
     end
@@ -136,7 +136,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "salary", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
+          t.index ["emp_no"], name: "emp_no"
         end
       ERB
     end
@@ -148,7 +148,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "salary", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index ["salary"], <%= i cond(5.0, using: :btree) %>
+          t.index ["salary"]
         end
       ERB
     end
@@ -160,7 +160,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
           t.integer "salary", null: false
           t.date    "from_date", null: false
           t.date    "to_date", null: false
-          t.index ["salary"], name: "index_salaries_on_salary", <%= i cond(5.0, using: :btree) %>
+          t.index ["salary"], name: "index_salaries_on_salary"
         end
       ERB
     end