ridgepole 0.9.5 → 0.9.6

data/spec/mysql57/virtual/add_virtual_column_spec.rb

@@ -1,37 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Ridgepole::Client#diff -> migrate' do
-  context 'when add virtual column' do
-    let(:actual_dsl) do
-      <<-RUBY
-        create_table "books", force: :cascade do |t|
-          t.string  "title"
-          t.index ["title"], name: "index_books_on_title"
-        end
-      RUBY
-    end
-
-    let(:expected_dsl) do
-      <<-RUBY
-        create_table "books", force: :cascade do |t|
-          t.string   "title"
-          t.virtual  "upper_title", type: :string, as: "upper(`title`)"
-          t.virtual  "title_length", type: :integer, as: "length(`title`)", stored: true
-          t.index ["title"], name: "index_books_on_title"
-          t.index ["title_length"], name: "index_books_on_title_length"
-        end
-      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
-    }
-  end
-end

data/spec/mysql57/virtual/change_virtual_column_spec.rb

@@ -1,40 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Ridgepole::Client#diff -> migrate' do
-  context 'when change virtual column' do
-    let(:actual_dsl) do
-      <<-RUBY
-        create_table "books", force: :cascade do |t|
-          t.string   "title"
-          t.virtual "upper_title", type: :string, null: false, as: "upper(`title`)"
-          t.virtual "title_length", type: :integer, null: false, as: "length(`title`)", stored: true
-          t.index ["title"], name: "index_books_on_title"
-          t.index ["title_length"], name: "index_books_on_title_length"
-        end
-      RUBY
-    end
-
-    let(:expected_dsl) do
-      <<-RUBY
-        create_table "books", force: :cascade do |t|
-          t.string   "title"
-          t.virtual "upper_title", type: :string, null: false, as: "length(`title`)"
-          t.virtual "title_length", type: :integer, null: false, as: "upper(`title`)", stored: true
-          t.index ["title"], name: "index_books_on_title"
-          t.index ["title_length"], name: "index_books_on_title_length"
-        end
-      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
-    }
-  end
-end

data/spec/mysql57/virtual/drop_virtual_column_spec.rb

@@ -1,37 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Ridgepole::Client#diff -> migrate' do
-  context 'when drop virtual column' do
-    let(:actual_dsl) do
-      <<-RUBY
-        create_table "books", force: :cascade do |t|
-          t.string   "title"
-          t.virtual  "upper_title", type: :string, as: "upper(`title`)"
-          t.virtual  "title_length", type: :integer, as: "length(`title`)", stored: true
-          t.index ["title"], name: "index_books_on_title"
-          t.index ["title_length"], name: "index_books_on_title_length"
-        end
-      RUBY
-    end
-
-    let(:expected_dsl) do
-      <<-RUBY
-        create_table "books", force: :cascade do |t|
-          t.string  "title"
-          t.index ["title"], name: "index_books_on_title"
-        end
-      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
-    }
-  end
-end

data/spec/postgresql/diff/diff_spec.rb

@@ -1,216 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Ridgepole::Client.diff' do
-  before do
-    allow(Ridgepole::Diff).to receive(:postgresql?).and_return(true)
-  end
-
-  context 'when change column' do
-    let(:actual_dsl) do
-      <<-RUBY
-        create_table "clubs", force: :cascade do |t|
-          t.string "name", limit: 255, default: "", null: false
-        end
-
-        add_index "clubs", ["name"], name: "idx_name", unique: true, using: :btree
-
-        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: "idx_dept_name", unique: true, using: :btree
-
-        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
-
-        add_index "dept_emp", ["dept_no"], name: "idx_dept_emp_dept_no", using: :btree
-        add_index "dept_emp", ["emp_no"], name: "idx_dept_emp_emp_no", using: :btree
-
-        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
-        end
-
-        add_index "dept_manager", ["dept_no"], name: "idx_dept_manager_dept_no", using: :btree
-        add_index "dept_manager", ["emp_no"], name: "idx_dept_manager_emp_no", using: :btree
-
-        create_table "employee_clubs", force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.integer "club_id", null: false
-        end
-
-        add_index "employee_clubs", ["emp_no", "club_id"], name: "idx_employee_clubs_emp_no_club_id", using: :btree
-
-        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
-        end
-
-        add_index "salaries", ["emp_no"], name: "idx_salaries_emp_no", using: :btree
-
-        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
-
-        add_index "titles", ["emp_no"], name: "idx_titles_emp_no", using: :btree
-      RUBY
-    end
-
-    let(:expected_dsl) do
-      <<-RUBY
-        create_table "clubs", force: :cascade do |t|
-          t.string "name", limit: 255, default: "", null: false
-        end
-
-        add_index "clubs", ["name"], name: "idx_name", unique: true, using: :btree
-
-        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: "idx_dept_name", unique: true, using: :btree
-
-        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
-
-        add_index "dept_emp", ["dept_no"], name: "idx_dept_emp_dept_no", using: :btree
-        add_index "dept_emp", ["emp_no"], name: "idx_dept_emp_emp_no", using: :btree
-
-        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
-        end
-
-        add_index "dept_manager", ["dept_no"], name: "idx_dept_manager_dept_no", using: :btree
-        add_index "dept_manager", ["emp_no"], name: "idx_dept_manager_emp_no", using: :btree
-
-        create_table "employee_clubs", force: :cascade do |t|
-          t.integer "emp_no", null: false
-          t.integer "club_id", null: true
-        end
-
-        add_index "employee_clubs", ["emp_no", "club_id"], name: "idx_employee_clubs_emp_no_club_id", using: :btree
-
-        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: 20, default: "XXX", 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: "idx_salaries_emp_no", using: :btree
-
-        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
-
-        add_index "titles", ["emp_no"], name: "idx_titles_emp_no", using: :btree
-      RUBY
-    end
-
-    subject { Ridgepole::Client }
-
-    it {
-      delta = subject.diff(actual_dsl, expected_dsl)
-      expect(delta.differ?).to be_truthy
-      expect(delta.script).to match_fuzzy <<-RUBY
-        change_column("employee_clubs", "club_id", :integer, **{:null=>true, :default=>nil})
-
-        change_column("employees", "last_name", :string, **{:limit=>20, :default=>"XXX"})
-      RUBY
-    }
-  end
-
-  describe 'column position warning' do
-    subject { Ridgepole::Client }
-
-    context 'when adding a column to the last' do
-      let(:actual_dsl) { <<-RUBY }
-      create_table "users", force: :cascade do |t|
-        t.string "name", null: false
-      end
-      RUBY
-
-      let(:expected_dsl) { <<-RUBY }
-      create_table "users", force: :cascade do |t|
-        t.string "name", null: false
-        t.datetime "created_at", null: false
-        t.datetime "updated_at", null: false
-      end
-      RUBY
-
-      # XXX:
-      before { client }
-
-      it "doesn't warn anything" do
-        expect(Ridgepole::Logger.instance).to_not receive(:warn)
-        delta = subject.diff(actual_dsl, expected_dsl)
-        expect(delta).to be_differ
-        expect(delta.script).to_not include('after')
-      end
-    end
-
-    context 'when adding a column to the middle' do
-      let(:actual_dsl) { <<-RUBY }
-      create_table "users", force: :cascade do |t|
-        t.datetime "created_at", null: false
-      end
-      RUBY
-
-      let(:expected_dsl) { <<-RUBY }
-      create_table "users", force: :cascade do |t|
-        t.string "name", null: false
-        t.integer "age", null: false
-        t.datetime "created_at", null: false
-        t.datetime "updated_at", null: false
-      end
-      RUBY
-
-      # XXX:
-      before { client }
-
-      it 'warns position' do
-        expect(Ridgepole::Logger.instance).to receive(:warn).with(/PostgreSQL doesn't support adding a new column .* users\.name/)
-        expect(Ridgepole::Logger.instance).to receive(:warn).with(/PostgreSQL doesn't support adding a new column .* users\.age/)
-        expect(Ridgepole::Logger.instance).to_not receive(:warn)
-        delta = subject.diff(actual_dsl, expected_dsl)
-        expect(delta).to be_differ
-        expect(delta.script).to_not include('after')
-      end
-    end
-  end
-end

data/spec/postgresql/dump/dump_spec.rb

@@ -1,69 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Ridgepole::Client#dump' do
-  context 'when there is a tables' do
-    before { restore_tables }
-    subject { client }
-
-    it {
-      expect(subject.dump).to match_fuzzy erbh(<<-ERB)
-        create_table "clubs", id: :serial, 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", <%= i cond(">= 6.1", { id: { type: :string, limit: 4 } }, { id: :string, limit: 4 }) %>, force: :cascade do |t|
-          t.string "dept_name", limit: 40, null: false
-          t.index ["dept_name"], name: "idx_dept_name", unique: true
-        end
-
-        create_table "dept_emp", primary_key: ["emp_no", "dept_no"], 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", primary_key: ["emp_no", "dept_no"], 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", id: :serial, 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", id: :integer, default: nil, 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", primary_key: ["emp_no", "from_date"], 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", primary_key: ["emp_no", "title", "from_date"], 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
-end

data/spec/postgresql/fk/migrate_change_fk_spec.rb

@@ -1,115 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Ridgepole::Client#diff -> migrate' do
-  context 'when change 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", on_delete: :cascade
-      ERB
-    end
-
-    let(:sorted_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
-
-        add_foreign_key "child", "parent", name: "child_ibfk_1", on_delete: :cascade
-      ERB
-    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
-
-        add_foreign_key "child", "parent", name: "child_ibfk_1"
-      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
-    }
-  end
-
-  context 'when change 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", on_delete: :cascade
-      ERB
-    end
-
-    let(:sorted_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
-
-        add_foreign_key "child", "parent", on_delete: :cascade
-      ERB
-    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
-
-        add_foreign_key "child", "parent"
-      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
-    }
-  end
-end