ridgepole 0.6.0.beta2 → 0.6.0.beta3

data/spec/postgresql/fk/migrate_change_fk_spec.rb

@@ -0,0 +1,62 @@
+if postgresql?
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when change fk' do
+    let(:actual_dsl) {
+      <<-RUBY
+create_table "parent", force: :cascade do |t|
+end
+
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent", name: "child_ibfk_1", on_delete: :cascade
+      RUBY
+    }
+
+    let(:sorted_actual_dsl) {
+      <<-RUBY
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent", force: :cascade do |t|
+end
+
+add_foreign_key "child", "parent", name: "child_ibfk_1", on_delete: :cascade
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent", force: :cascade do |t|
+end
+
+add_foreign_key "child", "parent", name: "child_ibfk_1"
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+
+    subject { client(enable_foreigner: true) }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump.delete_empty_lines).to eq sorted_actual_dsl.strip_heredoc.strip.delete_empty_lines
+      delta.migrate
+      expect(subject.dump.delete_empty_lines).to eq expected_dsl.strip_heredoc.strip.delete_empty_lines
+    }
+  end
+end
+end

data/spec/postgresql/fk/migrate_create_fk_spec.rb

@@ -0,0 +1,173 @@
+if postgresql?
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when create fk' do
+    let(:actual_dsl) {
+      <<-RUBY
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent", force: :cascade do |t|
+end
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      actual_dsl + (<<-RUBY)
+
+add_foreign_key "child", "parent", name: "child_ibfk_1"
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump.delete_empty_lines).to eq actual_dsl.strip_heredoc.strip.delete_empty_lines
+      delta.migrate
+      expect(subject.dump.delete_empty_lines).to eq expected_dsl.strip_heredoc.strip.delete_empty_lines
+    }
+
+    it {
+      delta = Ridgepole::Client.diff(actual_dsl, expected_dsl, reverse: true, default_int_limit: 4)
+      expect(delta.differ?).to be_truthy
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        remove_foreign_key("child", {:name=>"child_ibfk_1"})
+      RUBY
+    }
+
+    it {
+      delta = client(bulk_change: true).diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump.delete_empty_lines).to eq actual_dsl.strip_heredoc.strip.delete_empty_lines
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        add_foreign_key("child", "parent", {:name=>"child_ibfk_1"})
+      RUBY
+      delta.migrate
+      expect(subject.dump.delete_empty_lines).to eq expected_dsl.strip_heredoc.strip.delete_empty_lines
+    }
+  end
+
+  context 'when create fk when create table' do
+    let(:dsl) {
+      <<-RUBY
+# 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
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent", name: "child_ibfk_1"
+      RUBY
+    }
+
+    let(:sorted_dsl) {
+      <<-RUBY
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent", force: :cascade do |t|
+end
+
+add_foreign_key "child", "parent", name: "child_ibfk_1"
+      RUBY
+    }
+
+    before { client.diff('').migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff(dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump.strip).to eq ''
+      delta.migrate
+      expect(subject.dump.delete_empty_lines).to eq sorted_dsl.strip_heredoc.strip.delete_empty_lines
+    }
+  end
+
+  context 'already defined' do
+    let(:dsl) {
+      <<-RUBY
+# 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
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent", name: "child_ibfk_1"
+
+add_foreign_key "child", "parent", name: "child_ibfk_1"
+      RUBY
+    }
+
+    subject { client }
+
+    it {
+      expect {
+        subject.diff(dsl)
+      }.to raise_error('Foreign Key `child(child_ibfk_1)` already defined')
+    }
+  end
+
+  context 'no name' do
+    let(:dsl) {
+      <<-RUBY
+# 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
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent"
+      RUBY
+    }
+
+    subject { client }
+
+    it {
+      expect {
+        subject.diff(dsl)
+      }.to raise_error('Foreign key name in `child` is undefined')
+    }
+  end
+
+  context 'orphan fk' do
+    let(:dsl) {
+      <<-RUBY
+# Define parent before child
+create_table "parent", force: :cascade do |t|
+end
+
+add_foreign_key "child", "parent", name: "child_ibfk_1"
+      RUBY
+    }
+
+    subject { client }
+
+    it {
+      expect {
+        subject.diff(dsl)
+      }.to raise_error('Table `child` to create the foreign key is not defined: child_ibfk_1')
+    }
+  end
+end
+end

data/spec/postgresql/fk/migrate_drop_fk_spec.rb

@@ -0,0 +1,114 @@
+if postgresql?
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when drop fk' do
+    let(:actual_dsl) {
+      <<-RUBY
+create_table "parent", force: :cascade do |t|
+end
+
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent", name: "child_ibfk_1"
+      RUBY
+    }
+
+    let(:sorted_actual_dsl) {
+      expected_dsl + (<<-RUBY)
+
+add_foreign_key "child", "parent", name: "child_ibfk_1"
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent", force: :cascade do |t|
+end
+      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 sorted_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, default_int_limit: 4)
+      expect(delta.differ?).to be_truthy
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        add_foreign_key("child", "parent", {:name=>"child_ibfk_1"})
+      RUBY
+    }
+
+    it {
+      delta = client(bulk_change: true).diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq sorted_actual_dsl.strip_heredoc.strip
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        remove_foreign_key("child", {:name=>"child_ibfk_1"})
+      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
+
+  context 'when drop fk when drop table' do
+    let(:dsl) {
+      <<-RUBY
+create_table "parent", force: :cascade do |t|
+end
+
+
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id", unsigned: true
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+add_foreign_key "child", "parent", name: "child_ibfk_1"
+      RUBY
+    }
+
+    let(:sorted_dsl) {
+      <<-RUBY
+create_table "child", force: :cascade do |t|
+  t.integer "parent_id"
+end
+
+add_index "child", ["parent_id"], name: "par_id", using: :btree
+
+create_table "parent", force: :cascade do |t|
+end
+
+add_foreign_key "child", "parent", name: "child_ibfk_1"
+      RUBY
+    }
+
+    before { subject.diff(dsl).migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff('')
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to eq sorted_dsl.strip_heredoc.strip
+      delta.migrate
+      expect(subject.dump.strip).to eq ''
+    }
+  end
+end
+end

data/spec/postgresql/migrate/migrate_add_column_spec.rb

@@ -0,0 +1,220 @@
+if postgresql?
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when add 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
+
+        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
+
+        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
+          t.string  "any_col", limit: 255, 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
+          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
+        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 = Ridgepole::Client.diff(actual_dsl, expected_dsl, reverse: true)
+      expect(delta.differ?).to be_truthy
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        remove_column("employee_clubs", "any_col")
+
+        remove_column("employees", "age")
+        remove_column("employees", "updated_at")
+      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("employee_clubs", {:bulk => true}) do |t|
+          t.column("any_col", :string, {:limit=>255, :null=>false, :after=>"club_id"})
+        end
+
+        change_table("employees", {:bulk => true}) do |t|
+          t.column("age", :integer, {:null=>false, :after=>"hire_date"})
+          t.column("updated_at", :date, {:after=>"age"})
+        end
+      RUBY
+      delta.migrate
+      expect(subject.dump).to eq expected_dsl.strip_heredoc.strip
+    }
+  end
+
+  context 'when add column (int/noop)' do
+    let(:actual_dsl) {
+      <<-RUBY
+        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
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+        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
+      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
+      migrated, sql = delta.migrate(:noop => true)
+      expect(migrated).to be_truthy
+      expect(subject.dump).to eq actual_dsl.strip_heredoc.strip
+
+      sql = sql.each_line.map {|i| i.strip }.join("\n")
+      expect(sql).to eq("ALTER TABLE \"dept_emp\" ADD \"emp_no2\" integer NOT NULL")
+    }
+  end
+end
+end