ridgepole 0.6.0.beta2 → 0.6.0.beta3

data/spec/postgresql/migrate/migrate_drop_column_with_index_spec.rb

@@ -0,0 +1,180 @@
+if postgresql?
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when drop column and index' do
+    let(:actual_dsl) {
+      <<-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
+    }
+
+    let(:expected_dsl) {
+      <<-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.string "dept_no", limit: 4, null: false
+        end
+
+        add_index "dept_emp", ["dept_no"], name: "idx_dept_emp_dept_no", using: :btree
+
+        create_table "dept_manager", id: false, force: :cascade do |t|
+          t.string "dept_no", limit: 4, null: false
+        end
+
+        add_index "dept_manager", ["dept_no"], name: "idx_dept_manager_dept_no", using: :btree
+
+        create_table "employee_clubs", force: :cascade do |t|
+          t.integer "emp_no", null: false
+        end
+
+        add_index "employee_clubs", ["emp_no"], name: "idx_employee_clubs_emp_no", 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
+        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
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      delta.migrate
+      expect(subject.dump).to eq expected_dsl.strip_heredoc.strip
+    }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      delta.migrate(:noop => true)
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+    }
+
+    it {
+      delta = client(:bulk_change => true).diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+change_table("dept_emp", {:bulk => true}) do |t|
+  t.remove("emp_no")
+  t.remove("from_date")
+  t.remove("to_date")
+end
+
+change_table("dept_manager", {:bulk => true}) do |t|
+  t.remove("emp_no")
+  t.remove("from_date")
+  t.remove("to_date")
+end
+
+change_table("employee_clubs", {:bulk => true}) do |t|
+  t.remove("club_id")
+  t.index(["emp_no"], {:name=>"idx_employee_clubs_emp_no", :using=>:btree})
+end
+
+change_table("employees", {:bulk => true}) do |t|
+  t.remove("last_name")
+  t.remove("hire_date")
+end
+      RUBY
+      delta.migrate
+      expect(subject.dump).to eq expected_dsl.strip_heredoc.strip
+    }
+  end
+end
+end

data/spec/postgresql/migrate/migrate_drop_index_spec.rb

@@ -0,0 +1,125 @@
+if postgresql?
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when drop index' do
+    let(:dsl) {
+      <<-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
+    }
+
+    let(:actual_dsl) { dsl }
+
+    let(:expected_dsl) {
+      dsl.delete_add_index('clubs', ['name']).
+          delete_add_index('employee_clubs', ['emp_no', 'club_id']).
+          delete_add_index('titles', ['emp_no'])
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      delta.migrate
+      expect(subject.dump.each_line.select {|i| i !~ /\A\Z/ }.join).to eq expected_dsl.strip_heredoc.strip.each_line.select {|i| i !~ /\A\Z/ }.join
+    }
+
+    it {
+      delta = Ridgepole::Client.diff(actual_dsl, expected_dsl, reverse: true)
+      expect(delta.differ?).to be_truthy
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        add_index("clubs", ["name"], {:name=>"idx_name", :unique=>true, :using=>:btree})
+
+        add_index("employee_clubs", ["emp_no", "club_id"], {:name=>"idx_employee_clubs_emp_no_club_id", :using=>:btree})
+
+        add_index("titles", ["emp_no"], {:name=>"idx_titles_emp_no", :using=>:btree})
+      RUBY
+    }
+
+    it {
+      delta = client(:bulk_change => true).diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        change_table("clubs", {:bulk => true}) do |t|
+          t.remove_index({:name=>"idx_name"})
+        end
+
+        change_table("employee_clubs", {:bulk => true}) do |t|
+          t.remove_index({:name=>"idx_employee_clubs_emp_no_club_id"})
+        end
+
+        change_table("titles", {:bulk => true}) do |t|
+          t.remove_index({:name=>"idx_titles_emp_no"})
+        end
+      RUBY
+      delta.migrate
+      expect(subject.dump.each_line.select {|i| i !~ /\A\Z/ }.join).to eq expected_dsl.strip_heredoc.strip.each_line.select {|i| i !~ /\A\Z/ }.join
+    }
+  end
+end
+end

data/spec/postgresql/migrate/migrate_drop_table_spec.rb

@@ -0,0 +1,116 @@
+if postgresql?
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when drop table' do
+    let(:dsl) {
+      <<-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
+    }
+
+    let(:actual_dsl) { dsl }
+
+    let(:expected_dsl) {
+      dsl.delete_create_table('clubs')
+         .delete_create_table('employee_clubs')
+         .delete_create_table('employees')
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      delta.migrate
+      expect(subject.dump.each_line.select {|i| i !~ /\A\Z/ }.join).to eq expected_dsl.strip_heredoc.strip.each_line.select {|i| i !~ /\A\Z/ }.join
+    }
+
+    it {
+      delta = Ridgepole::Client.diff(actual_dsl, expected_dsl, reverse: true)
+      expect(delta.differ?).to be_truthy
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        create_table("clubs", {#{unsigned_if_enabled2('')}}) do |t|
+          t.string("name", {:limit=>255, :default=>"", :null=>false})
+        end
+        add_index("clubs", ["name"], {:name=>"idx_name", :unique=>true, :using=>:btree})
+
+        create_table("employee_clubs", {}) 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"}) 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
+      RUBY
+    }
+  end
+end
+end

data/spec/postgresql/migrate/migrate_rename_column_spec.rb

@@ -0,0 +1,179 @@
+if postgresql?
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when rename column' do
+    let(:actual_dsl) {
+      <<-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
+    }
+
+    let(:expected_dsl) {
+      <<-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_date2",           null: false, renamed_from: 'from_date'
+          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_date2",            null: false, renamed_from: 'to_date'
+        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
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      delta.migrate
+      expect(subject.dump).to eq expected_dsl.strip_heredoc.strip.gsub(/\s*,\s*renamed_from:.*$/, '')
+    }
+
+    it {
+      delta = Ridgepole::Client.diff(actual_dsl, expected_dsl, reverse: true)
+      expect(delta.differ?).to be_truthy
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+       rename_column("dept_emp", "from_date2", "from_date")
+
+       rename_column("dept_manager", "to_date2", "to_date")
+      RUBY
+    }
+
+    it {
+      delta = client(:bulk_change => true).diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        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
+      RUBY
+      delta.migrate
+      expect(subject.dump).to eq expected_dsl.strip_heredoc.strip.gsub(/\s*,\s*renamed_from:.*$/, '')
+    }
+  end
+end
+end