ridgepole 0.9.2 → 0.9.6

data/spec/postgresql/migrate/migrate_rename_table_spec.rb

@@ -1,175 +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", 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
-          t.index ["dept_name"], name: "idx_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", 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
-          t.index ["dept_name"], name: "idx_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", 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 "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.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
-
-    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'/, '')
-    }
-  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.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.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
-end

data/spec/postgresql/migrate/migrate_same_spec.rb

@@ -1,77 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Ridgepole::Client#diff -> migrate' do
-  context 'when database and definition are same' do
-    let(:dsl) do
-      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
-
-    before { restore_tables }
-    subject { client }
-
-    it {
-      delta = subject.diff(dsl)
-      expect(delta.differ?).to be_falsey
-      expect(subject.dump).to match_ruby dsl
-      delta.migrate
-      expect(subject.dump).to match_ruby dsl
-    }
-  end
-end

data/spec/postgresql/ridgepole_test_database.sql

@@ -1,16 +0,0 @@
-CREATE EXTENSION hstore;
-CREATE EXTENSION ltree;
-CREATE EXTENSION citext;
-CREATE EXTENSION IF NOT EXISTS "uuid-ossp";
-
-DO $$
-DECLARE
-  stmts CURSOR FOR
-    SELECT tablename FROM pg_tables
-    WHERE schemaname = 'public';
-BEGIN
-  FOR stmt IN stmts LOOP
-    EXECUTE 'DROP TABLE ' || quote_ident(stmt.tablename) || ';';
-  END LOOP;
-END;
-$$

data/spec/postgresql/ridgepole_test_tables.sql

@@ -1,75 +0,0 @@
-DROP TABLE IF EXISTS clubs;
-CREATE TABLE clubs (
-  id serial PRIMARY KEY,
-  name varchar(255) NOT NULL DEFAULT ''
-);
-CREATE UNIQUE INDEX idx_name ON clubs (name);
-
-DROP TABLE IF EXISTS departments;
-CREATE TABLE departments (
-  dept_no char(4) PRIMARY KEY,
-  dept_name varchar(40) NOT NULL
-);
-CREATE UNIQUE INDEX idx_dept_name ON departments (dept_name);
-
-DROP TABLE IF EXISTS dept_emp ;
-CREATE TABLE dept_emp (
-  emp_no int NOT NULL,
-  dept_no char(4) NOT NULL,
-  from_date date NOT NULL,
-  to_date date NOT NULL,
-  PRIMARY KEY (emp_no,dept_no)
-);
-CREATE INDEX idx_dept_emp_emp_no ON dept_emp (emp_no);
-CREATE INDEX idx_dept_emp_dept_no ON dept_emp (dept_no);
-
-DROP TABLE IF EXISTS dept_manager;
-CREATE TABLE dept_manager (
-  dept_no char(4) NOT NULL,
-  emp_no int NOT NULL,
-  from_date date NOT NULL,
-  to_date date NOT NULL,
-  PRIMARY KEY (emp_no,dept_no)
-);
-CREATE INDEX idx_dept_manager_emp_no ON dept_manager (emp_no);
-CREATE INDEX idx_dept_manager_dept_no ON dept_manager (dept_no);
-
-DROP TABLE IF EXISTS employee_clubs;
-CREATE TABLE employee_clubs (
-  id serial PRIMARY KEY,
-  emp_no int NOT NULL,
-  club_id int NOT NULL
-);
-CREATE INDEX idx_employee_clubs_emp_no_club_id ON employee_clubs (emp_no,club_id);
-
-DROP TABLE IF EXISTS employees;
-CREATE TABLE employees (
-  emp_no int PRIMARY KEY,
-  birth_date date NOT NULL,
-  first_name varchar(14) NOT NULL,
-  last_name varchar(16) NOT NULL,
-  hire_date date NOT NULL
-);
-
-DROP TABLE IF EXISTS salaries;
-CREATE TABLE salaries (
-  emp_no int NOT NULL,
-  salary int NOT NULL,
-  from_date date NOT NULL,
-  to_date date NOT NULL,
-  PRIMARY KEY (emp_no,from_date)
-);
-CREATE INDEX idx_salaries_emp_no ON salaries (emp_no);
-
-DROP TABLE IF EXISTS titles;
-CREATE TABLE titles (
-  emp_no int NOT NULL,
-  title varchar(50) NOT NULL,
-  from_date date NOT NULL,
-  to_date date DEFAULT NULL,
-  PRIMARY KEY (emp_no,title,from_date)
-);
-CREATE INDEX idx_titles_emp_no ON titles (emp_no);
-
-DROP TABLE IF EXISTS child;
-DROP TABLE IF EXISTS parent;

data/spec/postgresql/~default_name_fk/migrate_change_fk_spec.rb

@@ -1,59 +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: "fk_rails_e74ce85cbc", 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: "fk_rails_e74ce85cbc", 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: "fk_rails_e74ce85cbc"
-      ERB
-    end
-
-    before { subject.diff(actual_dsl).migrate }
-
-    subject { client(dump_with_default_fk_name: true) }
-
-    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

data/spec/postgresql/~default_name_fk/migrate_create_fk_spec.rb

@@ -1,134 +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: "fk_rails_e74ce85cbc"
-      RUBY
-    end
-
-    before { subject.diff(actual_dsl).migrate }
-    subject { client(dump_with_default_fk_name: true) }
-
-    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, dump_with_default_fk_name: 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=>"fk_rails_e74ce85cbc"})
-      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: "fk_rails_e74ce85cbc"
-      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: "fk_rails_e74ce85cbc"
-      ERB
-    end
-
-    before { client.diff('').migrate }
-    subject { client(dump_with_default_fk_name: true) }
-
-    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: "fk_rails_e74ce85cbc"
-
-        add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
-      ERB
-    end
-
-    subject { client(dump_with_default_fk_name: true) }
-
-    it {
-      expect do
-        subject.diff(dsl)
-      end.to raise_error('Foreign Key `child(fk_rails_e74ce85cbc)` already defined')
-    }
-  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: "fk_rails_e74ce85cbc"
-      ERB
-    end
-
-    subject { client(dump_with_default_fk_name: true) }
-
-    it {
-      expect do
-        subject.diff(dsl)
-      end.to raise_error('Table `child` to create the foreign key is not defined: fk_rails_e74ce85cbc')
-    }
-  end
-end