ridgepole 0.2.6 → 0.2.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: dc9a12435829f7fc6154b970fa7999a33ad0b062
-  data.tar.gz: 174873096bf770276973d159ac74a91237eb63e7
+  metadata.gz: b195dc41a13b87e24223916ac26701adc7cc4cb2
+  data.tar.gz: 4d3ffa051d81f4f5b3366afe3106b4f3b592bb3c
 SHA512:
-  metadata.gz: e7968a0fb170b70c2227c4e434a6e4b1b33fb2a5f47983281a9f818e2a621c3030729e227f93f73f209783db106044586cb315d21ef3761ee5e35455b7b327fc
-  data.tar.gz: b4c880bd3e74c435802691258f43f9517d56eb76c436f856a5067ace9975bc9561c8071be21c53e1d79ce91872a7fa57c1247ac36463e57e7f14e03cc24a2403
+  metadata.gz: a77bd763ef07424ae3db9870732029ed4a167d336f8517e02e7a7ef99c2a6ab0e306c80fc698837bd27c3d5602f3be5fd57a961e96518713bbc71de88b5d1383
+  data.tar.gz: ded7b3fd166e9b64d03e860d2b712ebade9d7bef5fa8eafebc873aafcca047c1ea03f5a5119032e758fa203b81998eda277ad3328c133471c1f248e843e6a82c

data/lib/ridgepole/dsl_parser.rb

@@ -97,6 +97,18 @@ class Ridgepole::DSLParser
   end
 
   def parse(dsl, opts = {})
-    Context.eval(dsl, opts)
+    parsed = Context.eval(dsl, opts)
+    check_orphan_index(parsed)
+    parsed
+  end
+
+  private
+
+  def check_orphan_index(parsed)
+    parsed.each do |table_name, attrs|
+      if attrs.length == 1 and attrs[:indices]
+        raise "Table `#{table_name}` to create the index is not defined: #{attrs[:indices].keys.join(',')}"
+      end
+    end
   end
 end

data/lib/ridgepole/version.rb

@@ -1,3 +1,3 @@
 module Ridgepole
-  VERSION = '0.2.6'
+  VERSION = '0.2.7'
 end

data/spec/migrate/check_orphan_index_spec.rb

@@ -0,0 +1,81 @@
+describe 'Ridgepole::Client#diff -> migrate' do
+  context 'when database is empty' do
+    let(:actual_dsl) { '' }
+    let(:expected_dsl) {
+      <<-RUBY
+        create_table "clubs", force: true do |t|
+          t.string "name", default: "", null: false
+        end
+
+        add_index "clubs", ["name"], name: "idx_name", unique: true, using: :btree
+
+        create_table "departments", primary_key: "dept_no", force: true do |t|
+          t.string "dept_name", limit: 40, null: false
+        end
+
+        add_index "departments", ["dept_name"], name: "dept_name", unique: true, using: :btree
+
+        create_table "dept_emp", id: false, force: true 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: "dept_no", using: :btree
+        add_index "dept_emp", ["emp_no"], name: "emp_no", using: :btree
+
+        create_table "dept_manager", id: false, force: true 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 "Xdept_manager", ["dept_no"], name: "dept_no", using: :btree
+        add_index "Xdept_manager", ["emp_no"], name: "emp_no", using: :btree
+
+        create_table "employee_clubs", force: true do |t|
+          t.integer "emp_no",  unsigned: true, null: false
+          t.integer "club_id", unsigned: true, null: false
+        end
+
+        add_index "employee_clubs", ["emp_no", "club_id"], name: "idx_emp_no_club_id", using: :btree
+
+        create_table "employees", primary_key: "emp_no", force: true 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.string "gender",     limit: 1,  null: false
+          t.date   "hire_date",             null: false
+        end
+
+        create_table "salaries", id: false, force: true 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: "emp_no", using: :btree
+
+        create_table "titles", id: false, force: true 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: "emp_no", using: :btree
+      RUBY
+    }
+
+    subject { client }
+
+    it {
+      expect {
+        subject.diff(expected_dsl)
+      }.to raise_error('Table `Xdept_manager` to create the index is not defined: dept_no,emp_no')
+    }
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ridgepole
 version: !ruby/object:Gem::Version
-  version: 0.2.6
+  version: 0.2.7
 platform: ruby
 authors:
 - Genki Sugawara
@@ -127,6 +127,7 @@ files:
 - spec/dump/dump_class_method_spec.rb
 - spec/dump/dump_some_tables_spec.rb
 - spec/dump/dump_spec.rb
+- spec/migrate/check_orphan_index_spec.rb
 - spec/migrate/migrate_add_column_spec.rb
 - spec/migrate/migrate_change_column_spec.rb
 - spec/migrate/migrate_change_table_option_spec.rb
@@ -182,6 +183,7 @@ test_files:
 - spec/dump/dump_class_method_spec.rb
 - spec/dump/dump_some_tables_spec.rb
 - spec/dump/dump_spec.rb
+- spec/migrate/check_orphan_index_spec.rb
 - spec/migrate/migrate_add_column_spec.rb
 - spec/migrate/migrate_change_column_spec.rb
 - spec/migrate/migrate_change_table_option_spec.rb