ridgepole 0.7.0.beta10 → 0.7.0.beta11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 43d1094d18230aec0edeb8cfc3ba63783b074e54
-  data.tar.gz: 3dcc6e2ead2ecd1eae6f5abd09865f272ee0ac6c
+  metadata.gz: ecc5673186f25067b337cfc75180d44b82bba7ce
+  data.tar.gz: 1601ea4b2ab780f97048bdc82b1896cffa3395fe
 SHA512:
-  metadata.gz: '08568bf10b61386b12462595b19c6a9ee527423f7a72224978a202dd6f5a6b0c953d69669c5f3821318b72834b669f2165fa84b081657434af4ccb63224a7958'
-  data.tar.gz: fdd40f8896581ddfb80e1f5921329c8a3bdc6eb202c60cf345e1df72eb2c842b1ae3c071d111ec3e8dc003c5d16ce723e15c99e48b18acae028818f72f390310
+  metadata.gz: ba82871050164aca91fbcc7346a1d892bbc02de322f850ff70c6dd5ab41366b6a6bb1ffa609c7ad74f30be35ba839ccb0720d95bf2551721070d9a57e1514b37
+  data.tar.gz: a7e2278fd069aa686a2a9a652234d3ae069e23a4b46b822272b1310bcfadc4560f99e8e35984fb0e5aebd32bd35555f1d7fe916deeb913ce4d6d6da5ca3d45e8

data/README.md

@@ -9,7 +9,7 @@ It defines DB schema using [Rails DSL](http://guides.rubyonrails.org/migrations.
 [![Build Status](https://travis-ci.org/winebarrel/ridgepole.svg?branch=0.6)](https://travis-ci.org/winebarrel/ridgepole)
 [![Coverage Status](https://coveralls.io/repos/github/winebarrel/ridgepole/badge.svg?branch=0.6)](https://coveralls.io/github/winebarrel/ridgepole?branch=0.6)
 
-[![Edge Version](https://img.shields.io/badge/edge_version-0.7.0.beta10-brightgreen.svg)](https://rubygems.org/gems/ridgepole/versions/0.7.0.beta10)
+[![Edge Version](https://img.shields.io/badge/edge_version-0.7.0.beta11-brightgreen.svg)](https://rubygems.org/gems/ridgepole/versions/0.7.0.beta11)
 [![Build Status](https://travis-ci.org/winebarrel/ridgepole.svg?branch=0.7)](https://travis-ci.org/winebarrel/ridgepole)
 [![Coverage Status](https://coveralls.io/repos/github/winebarrel/ridgepole/badge.svg?branch=0.7)](https://coveralls.io/github/winebarrel/ridgepole?branch=0.7)
 
@@ -77,6 +77,7 @@ It defines DB schema using [Rails DSL](http://guides.rubyonrails.org/migrations.
   * Fix change fk order
   * Add `--check-relation-type` option
   * Add `--skip-column-comment-change` option
+  * Add `--default-bigint-limit` option
 
 ## Installation
 
@@ -107,6 +108,7 @@ Usage: ridgepole [options]
         --bulk-change
         --default-bool-limit LIMIT
         --default-int-limit LIMIT
+        --default-bigint-limit LIMIT
         --default-float-limit LIMIT
         --default-string-limit LIMIT
         --default-text-limit LIMIT

data/bin/ridgepole

@@ -21,6 +21,7 @@ EOS
 COLUMN_TYPES = {
   :boolean => :bool,
   :integer => :int,
+  :bigint  => :bigint,
   :float   => :float,
   :string  => :string,
   :text    => :text,

data/lib/ridgepole/default_limit.rb

@@ -3,6 +3,7 @@ class Ridgepole::DefaultsLimit
     :mysql2 => {
       :boolean => 1,
       :integer => 4,
+      :bigint  => 8,
       :float   => 24,
       :string  => 255,
       :text    => 65535,

data/lib/ridgepole/version.rb

@@ -1,3 +1,3 @@
 module Ridgepole
-  VERSION = '0.7.0.beta10'
+  VERSION = '0.7.0.beta11'
 end

data/spec/mysql/cli/ridgepole_spec.rb

@@ -25,6 +25,7 @@ describe 'ridgepole' do
             --bulk-change
             --default-bool-limit LIMIT
             --default-int-limit LIMIT
+            --default-bigint-limit LIMIT
             --default-float-limit LIMIT
             --default-string-limit LIMIT
             --default-text-limit LIMIT

data/spec/mysql/migrate/migrate_add_column2_spec.rb

@@ -53,7 +53,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
     let(:expected_dsl) {
       <<-EOS
         create_table "dept_emp", id: false, force: :cascade do |t|
-          t.integer "emp_no", limit: 4, null: false
+          t.integer "emp_no", limit: 3, null: false
           t.integer "emp_no2", null: false
           t.string  "dept_no", limit: 4, null: false
           t.date    "from_date", null: false
@@ -63,17 +63,17 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
 
     before { subject.diff(actual_dsl).migrate }
-    subject { client(:default_int_limit => 11) }
+    subject { client(:default_integer_limit => 3) }
 
     it {
       delta = subject.diff(expected_dsl)
       expect(delta.differ?).to be_truthy
-      expect(subject.dump).to match_fuzzy actual_dsl
+      expect(subject.dump).to match_fuzzy actual_dsl.sub(/"emp_no"/, '"emp_no", limit: 3')
       migrated, sql = delta.migrate(:noop => true)
       expect(migrated).to be_truthy
-      expect(subject.dump).to match_fuzzy actual_dsl
+      expect(subject.dump).to match_fuzzy actual_dsl.sub(/"emp_no"/, '"emp_no", limit: 3')
 
-      expect(sql).to match_fuzzy erbh("ALTER TABLE `dept_emp` ADD `emp_no2` int NOT NULL AFTER `emp_no`")
+      expect(sql).to match_fuzzy erbh("ALTER TABLE `dept_emp` ADD `emp_no2` mediumint NOT NULL AFTER `emp_no`")
     }
   end
 
@@ -92,7 +92,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
     let(:expected_dsl) {
       <<-EOS
         create_table "dept_emp", id: false, force: :cascade do |t|
-          t.integer "emp_no", limit: 4, null: false
+          t.integer "emp_no", limit: 3, null: false
           t.integer "emp_no2", limit: 4, null: false
           t.string  "dept_no", limit: 4, null: false
           t.date    "from_date", null: false
@@ -102,7 +102,46 @@ describe 'Ridgepole::Client#diff -> migrate' do
     }
 
     before { subject.diff(actual_dsl).migrate }
-    subject { client(:default_int_limit => 11) }
+    subject { client(:default_integer_limit => 3) }
+
+    it {
+      delta = subject.diff(expected_dsl)
+      expect(delta.differ?).to be_truthy
+      expect(subject.dump).to match_fuzzy actual_dsl.sub(/"emp_no"/, '"emp_no", limit: 3')
+      migrated, sql = delta.migrate(:noop => true)
+      expect(migrated).to be_truthy
+      expect(subject.dump).to match_fuzzy actual_dsl.sub(/"emp_no"/, '"emp_no", limit: 3')
+
+      expect(sql).to match_fuzzy erbh("ALTER TABLE `dept_emp` ADD `emp_no2` int NOT NULL AFTER `emp_no`")
+    }
+  end
+
+  context 'when add column (bigint/noop)' do
+    let(:actual_dsl) {
+      erbh(<<-EOS)
+        create_table "dept_emp", id: false, force: :cascade do |t|
+          t.bigint "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
+      EOS
+    }
+
+    let(:expected_dsl) {
+      <<-EOS
+        create_table "dept_emp", id: false, force: :cascade do |t|
+          t.bigint "emp_no", limit: 9, 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
+      EOS
+    }
+
+    before { subject.diff(actual_dsl).migrate }
+    subject { client(:default_bigint_limit => 9) }
 
     it {
       delta = subject.diff(expected_dsl)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ridgepole
 version: !ruby/object:Gem::Version
-  version: 0.7.0.beta10
+  version: 0.7.0.beta11
 platform: ruby
 authors:
 - Genki Sugawara
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-06-28 00:00:00.000000000 Z
+date: 2017-06-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord