ridgepole 0.9.5 → 0.9.6

data/spec/mysql/migrate/migrate_change_table_comment_spec.rb

@@ -1,72 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Ridgepole::Client#diff -> migrate' do
-  let(:actual_dsl) do
-    erbh(<<-ERB)
-      create_table "employees", force: :cascade, comment: "old comment" do |t|
-        t.date   "birth_date", null: false
-        t.string "first_name", limit: 14, null: false
-        t.string "last_name", limit: 16, null: false
-        t.string "gender", limit: 1, null: false
-        t.date   "hire_date", null: false
-      end
-    ERB
-  end
-
-  let(:expected_dsl) do
-    erbh(<<-ERB)
-      create_table "employees", force: :cascade, comment: "new comment" do |t|
-        t.date   "birth_date", null: false
-        t.string "first_name", limit: 14, null: false
-        t.string "last_name", limit: 16, null: false
-        t.string "gender", limit: 1, null: false
-        t.date   "hire_date", null: false
-      end
-    ERB
-  end
-
-  before { subject.diff(actual_dsl).migrate }
-
-  context 'when ignore_table_comment option is false' do
-    subject { client }
-
-    it {
-      expect(Ridgepole::Logger.instance).to receive(:warn).with(<<-MSG)
-[WARNING] Table option changes are ignored on `employees`.
-  from: {:comment=>"old comment"}
-    to: {:comment=>"new comment"}
-      MSG
-      delta = subject.diff(expected_dsl)
-      expect(delta.differ?).to be_falsey
-      expect(subject.dump).to match_ruby actual_dsl
-      delta.migrate
-      expect(subject.dump).to match_ruby actual_dsl
-    }
-  end
-
-  context 'when ignore_table_comment option is true' do
-    subject { client(ignore_table_comment: true) }
-
-    it {
-      expect(Ridgepole::Logger.instance).to_not receive(:warn)
-      delta = subject.diff(expected_dsl)
-      expect(delta.differ?).to be_falsey
-      expect(subject.dump).to match_ruby actual_dsl
-      delta.migrate
-      expect(subject.dump).to match_ruby actual_dsl
-    }
-  end
-
-  context 'when mysql_change_table_comment option is true' do
-    subject { client(mysql_change_table_comment: true) }
-
-    it {
-      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_ruby expected_dsl
-    }
-  end
-end

data/spec/mysql/migrate/migrate_change_table_option_spec.rb

@@ -1,83 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Ridgepole::Client#diff -> migrate' do
-  context 'when create_table options are different' do
-    let(:actual_dsl) do
-      erbh(<<-ERB)
-        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
-          t.string "last_name", limit: 16, null: false
-          t.string "gender", limit: 1, null: false
-          t.date   "hire_date", null: false
-        end
-      ERB
-    end
-
-    let(:expected_dsl) do
-      erbh(<<-ERB)
-        create_table "employees", primary_key: "emp_no2", force: :cascade do |t|
-          t.date   "birth_date", null: false
-          t.string "first_name", limit: 14, null: false
-          t.string "last_name", limit: 16, null: false
-          t.string "gender", limit: 1, null: false
-          t.date   "hire_date", null: false
-        end
-      ERB
-    end
-
-    before { subject.diff(actual_dsl).migrate }
-    subject { client }
-
-    it {
-      expect(Ridgepole::Logger.instance).to receive(:warn).with(<<-MSG)
-[WARNING] Table option changes are ignored on `employees`.
-  from: {:primary_key=>"emp_no"}
-    to: {:primary_key=>"emp_no2"}
-      MSG
-      delta = subject.diff(expected_dsl)
-      expect(delta.differ?).to be_falsey
-      expect(subject.dump).to match_ruby actual_dsl
-      delta.migrate
-      expect(subject.dump).to match_ruby actual_dsl
-    }
-  end
-
-  context 'when create_table options are different (ignore comment)' do
-    let(:actual_dsl) do
-      erbh(<<-ERB)
-        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
-          t.string "last_name", limit: 16, null: false
-          t.string "gender", limit: 1, null: false
-          t.date   "hire_date", null: false
-        end
-      ERB
-    end
-
-    let(:expected_dsl) do
-      erbh(<<-ERB)
-        create_table "employees", primary_key: "emp_no", comment: "my comment", force: :cascade do |t|
-          t.date   "birth_date", null: false
-          t.string "first_name", limit: 14, null: false
-          t.string "last_name", limit: 16, null: false
-          t.string "gender", limit: 1, null: false
-          t.date   "hire_date", null: false
-        end
-      ERB
-    end
-
-    before { subject.diff(actual_dsl).migrate }
-    subject { client(ignore_table_comment: true) }
-
-    it {
-      expect(Ridgepole::Logger.instance).to_not receive(:warn)
-      delta = subject.diff(expected_dsl)
-      expect(delta.differ?).to be_falsey
-      expect(subject.dump).to match_ruby actual_dsl
-      delta.migrate
-      expect(subject.dump).to match_ruby actual_dsl
-    }
-  end
-end

data/spec/mysql/migrate/migrate_check_relation_column_type_spec.rb

@@ -1,225 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Ridgepole::Client#diff -> migrate' do
-  context 'with warning' do
-    let(:actual_dsl) do
-      erbh(<<-ERB)
-        create_table "dept_manager", force: :cascade do |t|
-          t.string  "dept_no", limit: 4, null: false
-          t.date    "from_date", null: false
-          t.date    "to_date", null: false
-        end
-
-        create_table "employees", force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.date    "birth_date", null: false
-          t.string  "first_name", limit: 14, null: false
-          t.string  "last_name", limit: 16, null: false
-          t.string  "gender", limit: 1, null: false
-          t.date    "hire_date", null: false
-        end
-      ERB
-    end
-
-    let(:expected_dsl) do
-      erbh(<<-ERB)
-        create_table "dept_manager", force: :cascade do |t|
-          t.integer "employee_id"
-          t.string  "dept_no", limit: 4, null: false
-          t.date    "from_date", null: false
-          t.date    "to_date", null: false
-        end
-
-        create_table "employees", force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.date    "birth_date", null: false
-          t.string  "first_name", limit: 14, null: false
-          t.string  "last_name", limit: 16, null: false
-          t.string  "gender", limit: 1, null: false
-          t.date    "hire_date", null: false
-        end
-      ERB
-    end
-
-    before { subject.diff(actual_dsl).migrate }
-    subject { client(check_relation_type: 'bigint') }
-
-    it {
-      expect(Ridgepole::Logger.instance).to receive(:warn).with(<<-MSG)
-[WARNING] Relation column type is different.
-              employees.id: {:type=>:bigint}
-  dept_manager.employee_id: {:type=>:integer}
-      MSG
-
-      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_ruby expected_dsl
-    }
-  end
-
-  context 'with unsigned warning' do
-    let(:actual_dsl) do
-      erbh(<<-ERB)
-        create_table "dept_manager", force: :cascade do |t|
-          t.string  "dept_no", limit: 4, null: false
-          t.date    "from_date", null: false
-          t.date    "to_date", null: false
-        end
-
-        create_table "employees", <%= i cond(">= 6.1", { id: { type: :bigint, unsigned: true } }, { id: :bigint, unsigned: true }) %>, force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.date    "birth_date", null: false
-          t.string  "first_name", limit: 14, null: false
-          t.string  "last_name", limit: 16, null: false
-          t.string  "gender", limit: 1, null: false
-          t.date    "hire_date", null: false
-        end
-      ERB
-    end
-
-    let(:expected_dsl) do
-      erbh(<<-ERB)
-        create_table "dept_manager", force: :cascade do |t|
-          t.bigint "employee_id"
-          t.string  "dept_no", limit: 4, null: false
-          t.date    "from_date", null: false
-          t.date    "to_date", null: false
-        end
-
-        create_table "employees", <%= i cond(">= 6.1", { id: { type: :bigint, unsigned: true } }, { id: :bigint, unsigned: true }) %>, force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.date    "birth_date", null: false
-          t.string  "first_name", limit: 14, null: false
-          t.string  "last_name", limit: 16, null: false
-          t.string  "gender", limit: 1, null: false
-          t.date    "hire_date", null: false
-        end
-      ERB
-    end
-
-    before { subject.diff(actual_dsl).migrate }
-    subject { client(check_relation_type: 'bigint') }
-
-    it {
-      expect(Ridgepole::Logger.instance).to receive(:warn).with(<<-MSG)
-[WARNING] Relation column type is different.
-              employees.id: {:type=>:bigint, :unsigned=>true}
-  dept_manager.employee_id: {:type=>:bigint}
-      MSG
-
-      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_ruby expected_dsl
-    }
-  end
-
-  context 'without warning' do
-    let(:actual_dsl) do
-      erbh(<<-ERB)
-        create_table "dept_manager", force: :cascade do |t|
-          t.string  "dept_no", limit: 4, null: false
-          t.date    "from_date", null: false
-          t.date    "to_date", null: false
-        end
-
-        create_table "employees", force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.date    "birth_date", null: false
-          t.string  "first_name", limit: 14, null: false
-          t.string  "last_name", limit: 16, null: false
-          t.string  "gender", limit: 1, null: false
-          t.date    "hire_date", null: false
-        end
-      ERB
-    end
-
-    let(:expected_dsl) do
-      erbh(<<-ERB)
-        create_table "dept_manager", force: :cascade do |t|
-          t.bigint "employee_id"
-          t.string  "dept_no", limit: 4, null: false
-          t.date    "from_date", null: false
-          t.date    "to_date", null: false
-        end
-
-        create_table "employees", force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.date    "birth_date", null: false
-          t.string  "first_name", limit: 14, null: false
-          t.string  "last_name", limit: 16, null: false
-          t.string  "gender", limit: 1, null: false
-          t.date    "hire_date", null: false
-        end
-      ERB
-    end
-
-    before { subject.diff(actual_dsl).migrate }
-    subject { client(check_relation_type: 'bigint') }
-
-    it {
-      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_ruby expected_dsl
-    }
-  end
-
-  context 'with unsigned warning' do
-    let(:actual_dsl) do
-      erbh(<<-ERB)
-        create_table "dept_manager", force: :cascade do |t|
-          t.string  "dept_no", limit: 4, null: false
-          t.date    "from_date", null: false
-          t.date    "to_date", null: false
-        end
-
-        create_table "employees", <%= i cond(">= 6.1", { id: { type: :bigint, unsigned: true } }, { id: :bigint, unsigned: true }) %>, force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.date    "birth_date", null: false
-          t.string  "first_name", limit: 14, null: false
-          t.string  "last_name", limit: 16, null: false
-          t.string  "gender", limit: 1, null: false
-          t.date    "hire_date", null: false
-        end
-      ERB
-    end
-
-    let(:expected_dsl) do
-      erbh(<<-ERB)
-        create_table "dept_manager", force: :cascade do |t|
-          t.bigint "employee_id", unsigned: true
-          t.string  "dept_no", limit: 4, null: false
-          t.date    "from_date", null: false
-          t.date    "to_date", null: false
-        end
-
-        create_table "employees", <%= i cond(">= 6.1", { id: { type: :bigint, unsigned: true } }, { id: :bigint, unsigned: true }) %>, force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.date    "birth_date", null: false
-          t.string  "first_name", limit: 14, null: false
-          t.string  "last_name", limit: 16, null: false
-          t.string  "gender", limit: 1, null: false
-          t.date    "hire_date", null: false
-        end
-      ERB
-    end
-
-    before { subject.diff(actual_dsl).migrate }
-    subject { client(check_relation_type: 'bigint') }
-
-    it {
-      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_ruby expected_dsl
-    }
-  end
-end

data/spec/mysql/migrate/migrate_create_index2_spec.rb

@@ -1,203 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Ridgepole::Client#diff -> migrate' do
-  context 'when create index (use t.index)' do
-    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
-        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
-        end
-
-        create_table "dept_emp", id: false, force: :cascade do |t|
-          t.integer "emp_no", null: false
-          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"
-          t.index ["emp_no"], name: "emp_no"
-        end
-
-        create_table "dept_manager", id: false, force: :cascade do |t|
-          t.string  "dept_no", null: false
-          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"
-          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"
-        end
-
-        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
-          t.string "last_name", limit: 16, null: false
-          t.string "gender", limit: 1, null: false
-          t.date   "hire_date", null: false
-        end
-
-        create_table "salaries", id: false, force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.integer "salary", null: false
-          t.date    "from_date", null: false
-          t.date    "to_date", null: false
-          t.index ["emp_no"], name: "emp_no"
-        end
-
-        create_table "titles", id: false, force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.string  "title", limit: 50, null: false
-          t.date    "from_date", null: false
-          t.date    "to_date"
-          t.index ["emp_no"], name: "emp_no"
-        end
-      ERB
-    end
-
-    let(:actual_dsl) do
-      erbh(<<-ERB)
-        create_table "clubs", force: :cascade do |t|
-          t.string "name", default: "", null: false
-        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
-        end
-
-        create_table "dept_emp", id: false, force: :cascade do |t|
-          t.integer "emp_no", null: false
-          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"
-          t.index ["emp_no"], name: "emp_no"
-        end
-
-        create_table "dept_manager", id: false, force: :cascade do |t|
-          t.string  "dept_no", null: false
-          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"
-          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
-        end
-
-        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
-          t.string "last_name", limit: 16, null: false
-          t.string "gender", limit: 1, null: false
-          t.date   "hire_date", null: false
-        end
-
-        create_table "salaries", id: false, force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.integer "salary", null: false
-          t.date    "from_date", null: false
-          t.date    "to_date", null: false
-          t.index ["emp_no"], name: "emp_no"
-        end
-
-        create_table "titles", id: false, force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.string  "title", limit: 50, null: false
-          t.date    "from_date", null: false
-          t.date    "to_date"
-        end
-      ERB
-    end
-
-    let(:expected_dsl) do
-      erbh(<<-ERB)
-        create_table "clubs", force: :cascade do |t|
-          t.string "name", limit: 255, default: "", null: false
-          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
-        end
-
-        add_index "departments", ["dept_name"], name: "dept_name", unique: true
-
-        create_table "dept_emp", id: false, force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.string  "dept_no", null: false
-          t.date    "from_date", null: false
-          t.date    "to_date", null: false
-        end
-
-        add_index "dept_emp", ["dept_no"], name: "dept_no"
-        add_index "dept_emp", ["emp_no"], name: "emp_no"
-
-        create_table "dept_manager", id: false, force: :cascade do |t|
-          t.string  "dept_no", null: false
-          t.integer "emp_no", null: false
-          t.date    "from_date", null: false
-          t.date    "to_date", null: false
-        end
-
-        add_index "dept_manager", ["dept_no"], name: "dept_no"
-        add_index "dept_manager", ["emp_no"], name: "emp_no"
-
-        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"
-        end
-
-        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
-          t.string "last_name", limit: 16, null: false
-          t.string "gender", limit: 1, null: false
-          t.date   "hire_date", null: false
-        end
-
-        create_table "salaries", id: false, force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.integer "salary", null: false
-          t.date    "from_date", null: false
-          t.date    "to_date", null: false
-        end
-
-        add_index "salaries", ["emp_no"], name: "emp_no"
-
-        create_table "titles", id: false, force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.string  "title", limit: 50, null: false
-          t.date    "from_date", null: false
-          t.date    "to_date"
-          t.index ["emp_no"], name: "emp_no"
-        end
-      ERB
-    end
-
-    before { subject.diff(actual_dsl).migrate }
-    subject { client }
-
-    it {
-      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_ruby dsl
-    }
-  end
-end