ridgepole 0.6.0.beta2 → 0.6.0.beta3

data/spec/{migrate → mysql/migrate}/migrate_change_index2_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when change index (no change)' do
     let(:actual_dsl) {
@@ -87,3 +88,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_change_index3_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when change index without using (no change)' do
     let(:actual_dsl) {
@@ -175,3 +176,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_change_index4_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when change index (same name)' do
     let(:actual_dsl) {
@@ -114,3 +115,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/mysql/migrate/migrate_change_index5_spec.rb

@@ -0,0 +1,78 @@
+unless postgresql?
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when change index (unique: false)' do
+    let(:actual_dsl) {
+      <<-RUBY
+        create_table "salaries"#{unsigned_if_enabled}, force: :cascade do |t|
+          t.integer "emp_no",    limit: 4, null: false
+          t.integer "salary",    limit: 4, null: false
+          t.date    "from_date",           null: false
+          t.date    "to_date",             null: false
+        end
+
+        add_index "salaries", ["emp_no", "id"], name: "emp_no", using: :btree
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+        create_table "salaries"#{unsigned_if_enabled}, force: :cascade do |t|
+          t.integer "emp_no",    limit: 4, null: false
+          t.integer "salary",    limit: 4, null: false
+          t.date    "from_date",           null: false
+          t.date    "to_date",             null: false
+        end
+
+        add_index "salaries", ["emp_no", "id"], unique: false, name: "emp_no", unique: false, using: :btree
+      RUBY
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_falsy
+    }
+  end
+
+  context 'when change index (unique: true)' do
+    let(:actual_dsl) {
+      <<-RUBY
+        create_table "salaries"#{unsigned_if_enabled}, force: :cascade do |t|
+          t.integer "emp_no",    limit: 4, null: false
+          t.integer "salary",    limit: 4, null: false
+          t.date    "from_date",           null: false
+          t.date    "to_date",             null: false
+        end
+
+        add_index "salaries", ["emp_no", "id"], name: "emp_no", using: :btree
+      RUBY
+    }
+
+    let(:expected_dsl) {
+      <<-RUBY
+        create_table "salaries"#{unsigned_if_enabled}, force: :cascade do |t|
+          t.integer "emp_no",    limit: 4, null: false
+          t.integer "salary",    limit: 4, null: false
+          t.date    "from_date",           null: false
+          t.date    "to_date",             null: false
+        end
+
+        add_index "salaries", ["emp_no", "id"], name: "emp_no", unique: true, 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.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
+    }
+  end
+end
+end

data/spec/{migrate → mysql/migrate}/migrate_change_index_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when change index' do
     let(:dsl) {
@@ -261,3 +262,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_change_table_option_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when change column' do
     let(:actual_dsl) {
@@ -44,3 +45,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_create_index_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when create index' do
     let(:dsl) {
@@ -122,3 +123,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_create_table_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when create table' do
     let(:dsl) {
@@ -101,3 +102,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_create_table_with_options_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when create table' do
     let(:expected_dsl) {
@@ -27,3 +28,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_drop_column_and_index_2_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when drop column and index (2)' do
     let(:actual_dsl) {
@@ -150,3 +151,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_drop_column_and_index_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when drop column and index' do
     let(:actual_dsl) {
@@ -177,3 +178,4 @@ end
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_drop_column_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when drop column' do
     let(:actual_dsl) {
@@ -184,3 +185,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_drop_index_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when drop index' do
     let(:dsl) {
@@ -122,3 +123,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_drop_table_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when drop table' do
     let(:dsl) {
@@ -114,3 +115,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_duplicate_index_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when index already defined' do
     let(:dsl) {
@@ -23,3 +24,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_duplicate_table_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when table already defined' do
     let(:dsl) {
@@ -33,3 +34,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_empty_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when database is empty' do
     let(:actual_dsl) { '' }
@@ -103,3 +104,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_execute_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when execute' do
     let(:dsl) {
@@ -292,3 +293,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_merge_mode_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when marge table' do
     let(:actual_dsl) {
@@ -160,3 +161,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_noop_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when no operation' do
     let(:actual_dsl) { '' }
@@ -235,3 +236,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_rename_column_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when rename column' do
     let(:actual_dsl) {
@@ -206,3 +207,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_rename_table_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when rename table' do
     let(:actual_dsl) {
@@ -223,3 +224,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_same_default_null_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when database and definition are same (default null / nothing -> null:true)' do
     let(:actual_dsl) {
@@ -73,3 +74,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_same_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when database and definition are same' do
     let(:dsl) {
@@ -81,3 +82,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_script_error_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when there is an error in the script' do
     let(:actual_dsl) { '' }
@@ -87,3 +88,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_skip_rename_column_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when rename column' do
     let(:actual_dsl) {
@@ -155,3 +156,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_skip_rename_table_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when rename table' do
     let(:actual_dsl) {
@@ -155,3 +156,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_with_ignore_tables_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when with ignore tables option (same)' do
     let(:current_schema) {
@@ -153,3 +154,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_with_pre_post_query_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'migrate with pre/post query' do
     let(:actual_dsl) { '' }
@@ -163,3 +164,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/{migrate → mysql/migrate}/migrate_with_tables_spec.rb

@@ -1,3 +1,4 @@
+unless postgresql?
 describe 'Ridgepole::Client#diff -> migrate' do
   context 'when with tables option (same)' do
     let(:current_schema) {
@@ -153,3 +154,4 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
   end
 end
+end

data/spec/postgresql/diff/diff_spec.rb

@@ -0,0 +1,153 @@
+if postgresql?
+describe 'Ridgepole::Client.diff' do
+  context 'when change 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_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
+    }
+
+    subject { Ridgepole::Client }
+
+    it {
+      delta = subject.diff(actual_dsl, expected_dsl, enable_mysql_unsigned: true)
+      expect(delta.differ?).to be_truthy
+      expect(delta.script).to eq <<-RUBY.strip_heredoc.strip
+        change_column("employee_clubs", "club_id", :integer, {:null=>true, :default=>nil})
+
+        change_column("employees", "last_name", :string, {:limit=>20, :default=>"XXX", :null=>false})
+      RUBY
+    }
+  end
+end
+end

data/spec/postgresql/dump/dump_spec.rb

@@ -0,0 +1,76 @@
+if postgresql?
+describe 'Ridgepole::Client#dump' do
+  context 'when there is a tables' do
+    before { restore_tables }
+    subject { client }
+
+    it {
+      expect(subject.dump).to eq <<-RUBY.strip_heredoc.strip
+        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
+end
+end