ridgepole 0.9.5 → 0.9.6

data/spec/mysql/migrate/migrate_primary_key_spec.rb

@@ -1,104 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Ridgepole::Client#diff -> migrate' do
-  let(:actual_dsl) do
-    erbh(<<-ERB)
-      create_table "employees", <%= i cond(">= 6.1", { id: { type: :integer, unsigned: true } }, { id: :integer, unsigned: true }) %>, force: :cascade do |t|
-      end
-    ERB
-  end
-
-  before { subject.diff(actual_dsl).migrate }
-  subject { client(allow_pk_change: allow_pk_change) }
-
-  context 'when allow_pk_change option is false' do
-    let(:allow_pk_change) { false }
-
-    context 'with difference' do
-      let(:expected_dsl) do
-        erbh(<<-ERB)
-          create_table "employees", id: :bigint, unsigned: true, force: :cascade do |t|
-          end
-        ERB
-      end
-
-      it {
-        expect(Ridgepole::Logger.instance).to receive(:warn).with(erbh(<<-ERB))
-[WARNING] Primary key definition of `employees` differ but `allow_pk_change` option is false
-  from: <%= i cond(">= 6.1", "{:id=>{:type=>:integer, :unsigned=>true}}", "{:id=>:integer, :unsigned=>true}") %>
-    to: {:id=>:bigint, :unsigned=>true}
-        ERB
-
-        delta = subject.diff(expected_dsl)
-        expect(delta.differ?).to be_falsey
-        delta.migrate
-        expect(subject.dump).to match_ruby actual_dsl
-      }
-    end
-
-    context 'with no difference' do
-      let(:actual_dsl) do
-        erbh(<<-ERB)
-          create_table "employees", unsigned: true, force: :cascade do |t|
-          end
-        ERB
-      end
-      let(:expected_dsl) { actual_dsl }
-
-      it {
-        expect(Ridgepole::Logger.instance).to_not receive(:warn)
-
-        delta = subject.diff(expected_dsl)
-        expect(delta.differ?).to be_falsey
-      }
-    end
-  end
-
-  context 'when allow_pk_change option is true' do
-    let(:allow_pk_change) { true }
-    let(:expected_dsl) do
-      erbh(<<-ERB)
-        create_table "employees", <%= i cond(">= 6.1", { id: { type: :bigint, unsigned: true } }, { id: :bigint, unsigned: true }) %>, force: :cascade do |t|
-        end
-
-        create_table "salaries", force: :cascade do |t|
-          t.bigint "employee_id", null: false, unsigned: true
-          t.index ["employee_id"], name: "fk_salaries_employees"
-        end
-        add_foreign_key "salaries", "employees", name: "fk_salaries_employees"
-      ERB
-    end
-
-    it {
-      delta = subject.diff(expected_dsl)
-      expect(delta.differ?).to be_truthy
-      delta.migrate
-      expect(subject.dump).to match_ruby expected_dsl
-    }
-  end
-
-  context 'when deafult: nil' do
-    let(:allow_pk_change) { true }
-
-    let(:actual_dsl) do
-      erbh(<<-ERB)
-        create_table "foo", id: :integer, unsigned: true, default: nil, force: :cascade do |t|
-        end
-      ERB
-    end
-
-    let(:expected_dsl) do
-      erbh(<<-ERB)
-        create_table "foo", id: :integer, default: nil, force: :cascade do |t|
-        end
-      ERB
-    end
-
-    it {
-      delta = subject.diff(expected_dsl)
-      expect(delta.differ?).to be_truthy
-      delta.migrate
-      expect(subject.dump).to match_ruby expected_dsl
-    }
-  end
-end

data/spec/mysql/migrate/migrate_rename_column_spec.rb

@@ -1,184 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Ridgepole::Client#diff -> migrate' do
-  context 'when rename column' do
-    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
-        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(: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
-        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_date2", null: false, renamed_from: 'from_date'
-          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_date2", null: false, renamed_from: 'to_date'
-          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 "gender2", limit: 1, null: false, renamed_from: 'gender'
-          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
-
-    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 expected_dsl.gsub(/\s*,\s*renamed_from:.*$/, '')
-    }
-
-    it {
-      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 <<-RUBY
-        change_table("dept_emp", bulk: true) do |t|
-          t.rename("from_date", "from_date2")
-        end
-
-        change_table("dept_manager", bulk: true) do |t|
-          t.rename("to_date", "to_date2")
-        end
-
-        change_table("employees", bulk: true) do |t|
-          t.rename("gender", "gender2")
-        end
-      RUBY
-      delta.migrate
-      expect(subject.dump).to match_ruby expected_dsl.gsub(/\s*,\s*renamed_from:.*$/, '')
-    }
-  end
-
-  context 'when rename column (not found)' do
-    before { restore_tables }
-    subject { client }
-
-    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
-          t.string "last_name", limit: 16, null: false
-          t.string "gender2", limit: 1, null: false, renamed_from: 'age'
-          t.date   "hire_date", null: false
-        end
-      RUBY
-    end
-
-    it {
-      expect do
-        subject.diff(dsl)
-      end.to raise_error('Column `age` not found')
-    }
-  end
-end

data/spec/mysql/migrate/migrate_rename_table_spec.rb

@@ -1,241 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Ridgepole::Client#diff -> migrate' do
-  context 'when rename table' do
-    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
-        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(: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
-        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 "employees2", primary_key: "emp_no", force: :cascade, renamed_from: 'employees' 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 "titles2", id: false, force: :cascade, renamed_from: :titles 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 expected_dsl.gsub(/, renamed_from: ('employees'|:titles)/, '')
-    }
-  end
-
-  context 'when rename table (dry-run)' 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
-          t.index ["first_name"], name: "first_name"
-        end
-
-        create_table "employees3", 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
-          t.index ["first_name"], name: "first_name"
-        end
-      ERB
-    end
-
-    let(:expected_dsl) do
-      erbh(<<-ERB)
-        create_table "employees2", primary_key: "emp_no", force: :cascade, renamed_from: 'employees' 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
-          t.index ["first_name"], name: "first_name"
-        end
-
-        create_table "employees4", primary_key: "emp_no", force: :cascade, renamed_from: :employees3 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
-          t.index ["first_name"], name: "first_name"
-        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(noop: true)
-      expect(subject.dump).to match_ruby actual_dsl
-    }
-  end
-
-  context 'when rename table (not found)' do
-    before { restore_tables }
-    subject { client }
-
-    let(:dsl) do
-      erbh(<<-ERB)
-        create_table "employees2", primary_key: "emp_no", force: :cascade, renamed_from: 'not_employees' 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
-
-    it {
-      expect(Ridgepole::Logger.instance).to receive(:warn).with('[WARNING] The table `not_employees` to be renamed does not exist.')
-      subject.diff(dsl)
-    }
-  end
-
-  context 'when rename table (already exist)' do
-    before { subject.diff(dsl).migrate }
-    subject { client }
-
-    let(:dsl) do
-      erbh(<<-ERB)
-        create_table "employees2", primary_key: "emp_no", force: :cascade, renamed_from: 'employees' 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
-
-    it {
-      expect(Ridgepole::Logger.instance).to receive(:warn).with('[WARNING] The table `employees` has already been renamed to the table `employees2`.')
-      subject.diff(dsl)
-    }
-  end
-end