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) { <<-EOS 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 EOS } let(:expected_dsl) { <<-EOS 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 EOS } subject { Ridgepole::Client } it { delta = subject.diff(actual_dsl, expected_dsl) expect(delta.differ?).to be_truthy expect(delta.script).to match_fuzzy <<-EOS change_column("employee_clubs", "club_id", :integer, {:null=>true, :default=>nil}) change_column("employees", "last_name", :string, {:limit=>20, :default=>"XXX"}) EOS } end describe 'column position warning' do subject { Ridgepole::Client } context 'when adding a column to the last' do let(:actual_dsl) { <<-EOS } create_table "users", force: :cascade do |t| t.string "name", null: false end EOS let(:expected_dsl) { <<-EOS } 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 EOS # 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) { <<-EOS } create_table "users", force: :cascade do |t| t.datetime "created_at", null: false end EOS let(:expected_dsl) { <<-EOS } 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 EOS # 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