ridgepole 0.9.5 → 0.9.6

data/spec/postgresql/fk/migrate_create_fk_spec.rb

@@ -1,176 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Ridgepole::Client#diff -> migrate' do
-  context 'when create fk' do
-    let(:actual_dsl) do
-      erbh(<<-ERB)
-        create_table "child", force: :cascade do |t|
-          t.integer "parent_id"
-          t.index ["parent_id"], name: "par_id"
-        end
-
-        create_table "parent", force: :cascade do |t|
-        end
-      ERB
-    end
-
-    let(:expected_dsl) do
-      actual_dsl + <<-RUBY
-        add_foreign_key "child", "parent", name: "child_ibfk_1"
-      RUBY
-    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
-    }
-
-    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
-        add_foreign_key("child", "parent", **{:name=>"child_ibfk_1"})
-      RUBY
-      delta.migrate
-      expect(subject.dump).to match_ruby expected_dsl
-    }
-  end
-
-  context 'when create fk when create table' do
-    let(:dsl) do
-      erbh(<<-ERB)
-        # Define parent before child
-        create_table "parent", force: :cascade do |t|
-        end
-
-        create_table "child", force: :cascade do |t|
-          t.integer "parent_id", unsigned: true
-          t.index ["parent_id"], name: "par_id"
-        end
-
-        add_foreign_key "child", "parent", name: "child_ibfk_1"
-      ERB
-    end
-
-    let(:sorted_dsl) do
-      erbh(<<-ERB)
-        create_table "child", force: :cascade do |t|
-          t.integer "parent_id"
-          t.index ["parent_id"], name: "par_id"
-        end
-
-        create_table "parent", force: :cascade do |t|
-        end
-
-        add_foreign_key "child", "parent", name: "child_ibfk_1"
-      ERB
-    end
-
-    before { client.diff('').migrate }
-    subject { client }
-
-    it {
-      delta = subject.diff(dsl)
-      expect(delta.differ?).to be_truthy
-      expect(subject.dump).to match_fuzzy ''
-      delta.migrate
-      expect(subject.dump).to match_fuzzy sorted_dsl
-    }
-  end
-
-  context 'already defined' do
-    let(:dsl) do
-      erbh(<<-ERB)
-        # Define parent before child
-        create_table "parent", force: :cascade do |t|
-        end
-
-        create_table "child", force: :cascade do |t|
-          t.integer "parent_id", unsigned: true
-          t.index ["parent_id"], name: "par_id"
-        end
-
-        add_foreign_key "child", "parent", name: "child_ibfk_1"
-
-        add_foreign_key "child", "parent", name: "child_ibfk_1"
-      ERB
-    end
-
-    subject { client }
-
-    it {
-      expect do
-        subject.diff(dsl)
-      end.to raise_error('Foreign Key `child(child_ibfk_1)` already defined')
-    }
-  end
-
-  context 'when create fk without name' do
-    let(:actual_dsl) do
-      erbh(<<-ERB)
-        create_table "child", force: :cascade do |t|
-          t.integer "parent_id"
-          t.index ["parent_id"], name: "par_id"
-        end
-
-        create_table "parent", force: :cascade do |t|
-        end
-      ERB
-    end
-
-    let(:expected_dsl) do
-      actual_dsl + <<-RUBY
-        add_foreign_key "child", "parent"
-      RUBY
-    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
-    }
-
-    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
-        add_foreign_key("child", "parent", **{})
-      RUBY
-      delta.migrate
-      expect(subject.dump).to match_ruby expected_dsl
-    }
-  end
-
-  context 'orphan fk' do
-    let(:dsl) do
-      erbh(<<-ERB)
-        # Define parent before child
-        create_table "parent", force: :cascade do |t|
-        end
-
-        add_foreign_key "child", "parent", name: "child_ibfk_1"
-      ERB
-    end
-
-    subject { client }
-
-    it {
-      expect do
-        subject.diff(dsl)
-      end.to raise_error('Table `child` to create the foreign key is not defined: child_ibfk_1')
-    }
-  end
-end

data/spec/postgresql/fk/migrate_drop_fk_spec.rb

@@ -1,197 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Ridgepole::Client#diff -> migrate' do
-  context 'when drop fk' do
-    let(:actual_dsl) do
-      erbh(<<-ERB)
-        create_table "parent", force: :cascade do |t|
-        end
-
-        create_table "child", force: :cascade do |t|
-          t.integer "parent_id"
-          t.index ["parent_id"], name: "par_id"
-        end
-
-        add_foreign_key "child", "parent", name: "child_ibfk_1"
-      ERB
-    end
-
-    let(:sorted_actual_dsl) do
-      expected_dsl + <<-RUBY
-        add_foreign_key "child", "parent", name: "child_ibfk_1"
-      RUBY
-    end
-
-    let(:expected_dsl) do
-      erbh(<<-ERB)
-        create_table "child", force: :cascade do |t|
-          t.integer "parent_id"
-          t.index ["parent_id"], name: "par_id"
-        end
-
-        create_table "parent", force: :cascade do |t|
-        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_fuzzy sorted_actual_dsl
-      delta.migrate
-      expect(subject.dump).to match_ruby expected_dsl
-    }
-
-    it {
-      delta = client(bulk_change: true).diff(expected_dsl)
-      expect(delta.differ?).to be_truthy
-      expect(subject.dump).to match_fuzzy sorted_actual_dsl
-      expect(delta.script).to match_fuzzy <<-RUBY
-        remove_foreign_key("child", name: "child_ibfk_1")
-      RUBY
-      delta.migrate
-      expect(subject.dump).to match_ruby expected_dsl
-    }
-  end
-
-  context 'when drop fk when drop table' do
-    let(:dsl) do
-      erbh(<<-ERB)
-        create_table "parent", force: :cascade do |t|
-        end
-
-        create_table "child", force: :cascade do |t|
-          t.integer "parent_id", unsigned: true
-          t.index ["parent_id"], name: "par_id"
-        end
-
-        add_foreign_key "child", "parent", name: "child_ibfk_1"
-      ERB
-    end
-
-    let(:sorted_dsl) do
-      erbh(<<-ERB)
-        create_table "child", force: :cascade do |t|
-          t.integer "parent_id"
-          t.index ["parent_id"], name: "par_id"
-        end
-
-        create_table "parent", force: :cascade do |t|
-        end
-
-        add_foreign_key "child", "parent", name: "child_ibfk_1"
-      ERB
-    end
-
-    before { subject.diff(dsl).migrate }
-    subject { client }
-
-    it {
-      delta = subject.diff('')
-      expect(delta.differ?).to be_truthy
-      expect(subject.dump).to match_fuzzy sorted_dsl
-      delta.migrate
-      expect(subject.dump).to match_fuzzy ''
-    }
-  end
-
-  context 'when drop fk without name' do
-    let(:actual_dsl) do
-      erbh(<<-ERB)
-        create_table "parent", force: :cascade do |t|
-        end
-
-        create_table "child", force: :cascade do |t|
-          t.integer "parent_id"
-          t.index ["parent_id"], name: "par_id"
-        end
-
-        add_foreign_key "child", "parent"
-      ERB
-    end
-
-    let(:sorted_actual_dsl) do
-      expected_dsl + <<-RUBY
-        add_foreign_key "child", "parent"
-      RUBY
-    end
-
-    let(:expected_dsl) do
-      erbh(<<-ERB)
-        create_table "child", force: :cascade do |t|
-          t.integer "parent_id"
-          t.index ["parent_id"], name: "par_id"
-        end
-
-        create_table "parent", force: :cascade do |t|
-        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_fuzzy sorted_actual_dsl
-      delta.migrate
-      expect(subject.dump).to match_ruby expected_dsl
-    }
-
-    it {
-      delta = client(bulk_change: true).diff(expected_dsl)
-      expect(delta.differ?).to be_truthy
-      expect(subject.dump).to match_fuzzy sorted_actual_dsl
-      expect(delta.script).to match_fuzzy <<-RUBY
-        remove_foreign_key("child", "parent")
-      RUBY
-      delta.migrate
-      expect(subject.dump).to match_ruby expected_dsl
-    }
-  end
-
-  context 'when drop fk when drop table without name' do
-    let(:dsl) do
-      erbh(<<-ERB)
-        create_table "parent", force: :cascade do |t|
-        end
-
-        create_table "child", force: :cascade do |t|
-          t.integer "parent_id", unsigned: true
-          t.index ["parent_id"], name: "par_id"
-        end
-
-        add_foreign_key "child", "parent"
-      ERB
-    end
-
-    let(:sorted_dsl) do
-      erbh(<<-ERB)
-        create_table "child", force: :cascade do |t|
-          t.integer "parent_id"
-          t.index ["parent_id"], name: "par_id"
-        end
-
-        create_table "parent", force: :cascade do |t|
-        end
-
-        add_foreign_key "child", "parent"
-      ERB
-    end
-
-    before { subject.diff(dsl).migrate }
-    subject { client }
-
-    it {
-      delta = subject.diff('')
-      expect(delta.differ?).to be_truthy
-      expect(subject.dump).to match_fuzzy sorted_dsl
-      delta.migrate
-      expect(subject.dump).to match_fuzzy ''
-    }
-  end
-end

data/spec/postgresql/migrate/migrate_add_column_spec.rb

@@ -1,203 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Ridgepole::Client#diff -> migrate' do
-  context 'when add column' do
-    let(:actual_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
-
-        create_table "dept_emp", id: false, force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.string  "dept_no", limit: 4, null: false
-          t.date    "from_date", null: false
-          t.date    "to_date", null: false
-          t.index ["dept_no"], name: "idx_dept_emp_dept_no"
-          t.index ["emp_no"], name: "idx_dept_emp_emp_no"
-        end
-
-        create_table "dept_manager", id: false, force: :cascade do |t|
-          t.string  "dept_no", limit: 4, 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: "idx_dept_manager_dept_no"
-          t.index ["emp_no"], name: "idx_dept_manager_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_employee_clubs_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.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: "idx_salaries_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: "idx_titles_emp_no"
-        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
-
-        create_table "dept_emp", id: false, force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.string  "dept_no", limit: 4, null: false
-          t.date    "from_date", null: false
-          t.date    "to_date", null: false
-          t.index ["dept_no"], name: "idx_dept_emp_dept_no"
-          t.index ["emp_no"], name: "idx_dept_emp_emp_no"
-        end
-
-        create_table "dept_manager", id: false, force: :cascade do |t|
-          t.string  "dept_no", limit: 4, 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: "idx_dept_manager_dept_no"
-          t.index ["emp_no"], name: "idx_dept_manager_emp_no"
-        end
-
-        create_table "employee_clubs", force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.integer "club_id", null: false
-          t.string  "any_col", limit: 255, null: false
-          t.index ["emp_no", "club_id"], name: "idx_employee_clubs_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.date    "hire_date", null: false
-          t.integer "age", null: false
-          t.date    "updated_at"
-        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: "idx_salaries_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.serial    "title_no", null: false
-          t.bigserial "title_bigno", null: false
-          t.index ["emp_no"], name: "idx_titles_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
-    }
-
-    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("employee_clubs", bulk: true) do |t|
-          t.column("any_col", :string, **{:limit=>255, :null=>false})
-        end
-
-        change_table("employees", bulk: true) do |t|
-          t.column("age", :integer, **{:null=>false})
-          t.column("updated_at", :date, **{})
-        end
-
-        change_table("titles", bulk: true) do |t|
-          t.column("title_no", :serial, **{:null=>false})
-          t.column("title_bigno", :bigserial, **{:null=>false})
-        end
-      RUBY
-      delta.migrate
-      expect(subject.dump).to match_ruby expected_dsl
-    }
-  end
-
-  context 'when add column (int/noop)' do
-    let(:actual_dsl) do
-      erbh(<<-ERB)
-        create_table "dept_emp", id: false, force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.string  "dept_no", limit: 4, null: false
-          t.date    "from_date", null: false
-          t.date    "to_date", null: false
-        end
-      ERB
-    end
-
-    let(:expected_dsl) do
-      erbh(<<-ERB)
-        create_table "dept_emp", id: false, force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.integer "emp_no2", null: false
-          t.string  "dept_no", limit: 4, null: false
-          t.date    "from_date", null: false
-          t.date    "to_date", null: false
-        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
-      migrated, sql = delta.migrate(noop: true)
-      expect(migrated).to be_truthy
-      expect(subject.dump).to match_ruby actual_dsl
-
-      expect(sql).to match_fuzzy('ALTER TABLE "dept_emp" ADD "emp_no2" integer NOT NULL')
-    }
-  end
-end