ridgepole 0.8.5 → 0.8.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8b49db08934182bd164f7910335dd04f3843e2e88c8593ab1d89f9bb4d0005d8
-  data.tar.gz: ff49a0ada6cac848407838596d23d93d75714079c0e95528015c4fcb9b2e4f56
+  metadata.gz: 6f850327d50c5d732987209c8c8eb5c9db3090e1b638f2ee9dbc8ced0959b128
+  data.tar.gz: 242e925ebc5c1a94d0c68e605e894a55efab238f1184b67d1a6a56f4aee51078
 SHA512:
-  metadata.gz: dc24b664af9f8dad9f6c6d21b6eeaa083ad915c42b63b58ce0b7b50386e11c3462b2f30f9819c80f03e5c2a682dadd5a980a355993e8c4e2c8f4a0dc06703242
-  data.tar.gz: c40af5068b9ccbc226d9375b8b7501584a161bdc81554d6612e2c1135346f9dcdbe4a1371657e23bc6d9f70df795e67c8addc6a8b5db538ce658e3588ad3867c
+  metadata.gz: 0451a8c71155ff9fd991045e04973cdf063c405dc1013c63883db20ce11bad93ce4f7cdd0533d5ae12f4a9c35bbf03e18e04a6487ad217baf614a090837095d4
+  data.tar.gz: f4bb253e35a0f7904a0b17effd29a5e388449d4a295282f9f1652f1dd11f6d8a9c1df8dbd176efcf884cda5e15e15bf80db61b69bfd7f702ee43de2e0b70a186

data/.github/workflows/test.yml

@@ -0,0 +1,62 @@
+name: test
+
+on:
+  push:
+  pull_request:
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby:
+          - 2.4
+          - 2.5
+          - 2.6
+          - 2.7
+        env:
+          - MYSQL56=1
+          - MYSQL57=1
+          - POSTGRESQL=1
+        gemfile:
+          - gemfiles/activerecord_5.0.gemfile
+          - gemfiles/activerecord_5.1.gemfile
+          - gemfiles/activerecord_5.2.gemfile
+          - gemfiles/activerecord_6.0.gemfile
+        exclude:
+          - ruby: 2.4
+            gemfile: gemfiles/activerecord_6.0.gemfile
+
+    steps:
+      - uses: actions/checkout@v2
+      - uses: actions/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+      - uses: actions/cache@v2
+        with:
+          path: gemfiles/vendor/bundle
+          key: ${{ runner.os }}-gems-${{ matrix.ruby }}-${{ matrix.gemfile }}-${{ hashFiles('ridgepole.gemspec', '**/Gemfile', '${{ matrix.gemfile }}') }}
+          restore-keys: |
+            ${{ runner.os }}-gems-${{ matrix.ruby }}-${{ matrix.gemfile }}-
+
+      - name: Setup dependencies
+        run: |
+          for i in {1..60}; do docker-compose up -d && break; sleep 1; done
+
+          bundle config path vendor/bundle
+          bundle install --jobs 4 --retry 3
+
+          # Wait until database servers start
+          function mysql_ping { mysqladmin -u root -h 127.0.0.1 -P 13316 -ppassword ping; }
+          function mysql57_ping { mysqladmin -u root -h 127.0.0.1 -P 13317 -ppassword ping; }
+          function pg_ping { PGPASSWORD=password pg_isready -U postgres -h 127.0.0.1 -p 15442; }
+          for i in {1..60}; do mysql_ping && break; sleep 1; done
+          for i in {1..60}; do mysql57_ping && break; sleep 1; done
+          for i in {1..60}; do pg_ping && break; sleep 1; done
+        env:
+          BUNDLE_GEMFILE: ${{ matrix.gemfile }}
+
+      - run: ${{ matrix.env }} bundle exec rake
+        env:
+          BUNDLE_GEMFILE: ${{ matrix.gemfile }}

data/.rubocop.yml

@@ -6,7 +6,7 @@ AllCops:
 Bundler/OrderedGems:
   Include:
     - 'Appraisals'
-Layout/IndentHeredoc:
+Layout/HeredocIndentation:
   Enabled: false
 Metrics/AbcSize:
   Enabled: false
@@ -18,7 +18,7 @@ Metrics/ClassLength:
   Enabled: false
 Metrics/CyclomaticComplexity:
   Enabled: false
-Metrics/LineLength:
+Layout/LineLength:
   Enabled: false
 Metrics/MethodLength:
   Enabled: false
@@ -30,8 +30,6 @@ Style/Documentation:
   Enabled: false
 Style/GuardClause:
   Enabled: false
-Style/MethodMissingSuper:
-  Enabled: false
 Style/MixinUsage:
   Exclude:
     - 'spec/**/*'
@@ -41,3 +39,39 @@ Layout/ClosingHeredocIndentation:
   Enabled: false
 Style/NumericPredicate:
   Enabled: false
+Layout/EmptyLinesAroundAttributeAccessor:
+  Enabled: true
+Layout/SpaceAroundMethodCallOperator:
+  Enabled: true
+Lint/DeprecatedOpenSSLConstant:
+  Enabled: true
+Lint/MissingSuper:
+  Enabled: false
+Lint/MixedRegexpCaptureTypes:
+  Enabled: true
+Lint/RaiseException:
+  Enabled: true
+Lint/StructNewOverride:
+  Enabled: true
+Style/AccessorGrouping:
+  Enabled: true
+Style/BisectedAttrAccessor:
+  Enabled: true
+Style/ExponentialNotation:
+  Enabled: true
+Style/HashEachMethods:
+  Enabled: true
+Style/HashTransformKeys:
+  Enabled: true
+Style/HashTransformValues:
+  Enabled: true
+Style/RedundantAssignment:
+  Enabled: true
+Style/RedundantFetchBlock:
+  Enabled: true
+Style/RedundantRegexpCharacterClass:
+  Enabled: true
+Style/RedundantRegexpEscape:
+  Enabled: true
+Style/SlicingWithRange:
+  Enabled: true

data/README.md

@@ -6,7 +6,7 @@ It defines DB schema using [Rails DSL](http://guides.rubyonrails.org/migrations.
 (like Chef/Puppet)
 
 [![Gem Version](https://badge.fury.io/rb/ridgepole.svg)](http://badge.fury.io/rb/ridgepole)
-[![Build Status](https://travis-ci.org/winebarrel/ridgepole.svg?branch=0.8)](https://travis-ci.org/winebarrel/ridgepole)
+[![Build Status](https://github.com/winebarrel/ridgepole/workflows/test/badge.svg?branch=0.8)](https://github.com/winebarrel/ridgepole/actions)
 [![Coverage Status](https://coveralls.io/repos/github/winebarrel/ridgepole/badge.svg?branch=0.8)](https://coveralls.io/github/winebarrel/ridgepole?branch=0.8)
 
 <details><summary>ChangeLog</summary>
@@ -111,6 +111,17 @@ It defines DB schema using [Rails DSL](http://guides.rubyonrails.org/migrations.
   * Display a warning if an InnoDB table doesn't have any indexes on a column where it has a foreign key ([pull#290](https://github.com/winebarrel/ridgepole/pull/290))
 * `>= 0.8.5`
   * Improve warning message on table options ([pull#291](https://github.com/winebarrel/ridgepole/pull/291))
+* `>= 0.8.6`
+  * Support multiple databases feature ([pull#297](https://github.com/winebarrel/ridgepole/pull/297))
+* `>= 0.8.7`
+  * Support `require_relative` ([pull#298](https://github.com/winebarrel/ridgepole/pull/298))
+* `>= 0.8.8`
+  * Fix keyword arguments warnings in Ruby 2.7 ([pull#303](https://github.com/winebarrel/ridgepole/pull/303))
+* `>= 0.8.9`
+  * Fix unexpected differences on text types and blob types on Rails 6 ([pull#306](https://github.com/winebarrel/ridgepole/pull/306))
+  * Fix unexpected warning when a foreign key is added on the primary key ([pull#307](https://github.com/winebarrel/ridgepole/pull/307))
+* `>= 0.8.10`
+  * Raise an error if an InnoDB column has a foreign key but no index ([pull#310](https://github.com/winebarrel/ridgepole/pull/310))
 </details>
 
 ## Installation
@@ -152,6 +163,7 @@ sudo /opt/ridgepole/embedded/bin/gem install mysql2
 Usage: ridgepole [options]
     -c, --config CONF_OR_FILE
     -E, --env ENVIRONMENT
+    -s, --spec-name SPEC_NAME
     -a, --apply
     -m, --merge
     -f, --file SCHEMAFILE
@@ -188,6 +200,7 @@ Usage: ridgepole [options]
         --ignore-table-comment
         --skip-column-comment-change
         --create-table-with-index
+        --allow-pk-change
         --mysql-dump-auto-increment
     -r, --require LIBS
         --log-file LOG_FILE
@@ -304,7 +317,6 @@ end
 ```
 
 ## Collation/Charset
-You can use the column collation by passing `--enable-mysql-awesome` ([activerecord-mysql-awesome](https://github.com/kamipo/activerecord-mysql-awesome) is required)
 
 ```ruby
 create_table "articles", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|

data/bin/ridgepole

@@ -38,6 +38,7 @@ split = false
 diff_files = nil
 diff_with_apply = false
 exit_code = 0
+spec_name = ''
 
 options = {
   dry_run: false,
@@ -76,6 +77,7 @@ ARGV.options do |opt|
   begin
     opt.on('-c', '--config CONF_OR_FILE') { |v| config = v }
     opt.on('-E', '--env ENVIRONMENT') { |v| env = v }
+    opt.on('-s', '--spec-name SPEC_NAME') { |v| spec_name = v }
     opt.on('-a', '--apply') { set_mode[:apply] }
     opt.on('-m', '--merge') do
       set_mode[:apply]
@@ -168,7 +170,7 @@ begin
   logger = Ridgepole::Logger.instance
   logger.debug = options[:debug]
 
-  client = Ridgepole::Client.new(Ridgepole::Config.load(config, env), options) if config
+  client = Ridgepole::Client.new(Ridgepole::Config.load(config, env, spec_name), options) if config
 
   ActiveRecord::Base.logger = logger
   ActiveSupport::LogSubscriber.colorize_logging = options[:color]
@@ -247,7 +249,7 @@ begin
         file_ext = File.extname(diff_file)
 
         if %w[.yml .yaml].include?(file_ext)
-          Ridgepole::Config.load(diff_file, env)
+          Ridgepole::Config.load(diff_file, env, spec_name)
         else
           File.open(diff_file)
         end

data/lib/ridgepole/cli/config.rb

@@ -6,7 +6,7 @@ require 'yaml'
 module Ridgepole
   class Config
     class << self
-      def load(config, env = 'development')
+      def load(config, env = 'development', spec_name = '')
         config = ENV.fetch(Regexp.last_match(1)) if config =~ /\Aenv:(.+)\z/
 
         parsed_config = if File.exist?(config)
@@ -26,8 +26,10 @@ module Ridgepole
 
         parsed_config = parse_database_url(config) unless parsed_config.is_a?(Hash)
 
-        if parsed_config.key?(env.to_s)
-          parsed_config.fetch(env.to_s)
+        parsed_config = parsed_config.fetch(env.to_s) if parsed_config.key?(env.to_s)
+
+        if parsed_config.key?(spec_name.to_s)
+          parsed_config.fetch(spec_name.to_s)
         else
           parsed_config
         end

data/lib/ridgepole/delta.rb

@@ -246,7 +246,7 @@ create_table(#{table_name.inspect}, #{inspect_options_include_default_proc(optio
       end
 
       buf.puts(<<-RUBY)
-  end
+end
       RUBY
 
       if !(@options[:create_table_with_index]) && !indices.empty?
@@ -326,7 +326,7 @@ execute "ALTER TABLE #{ActiveRecord::Base.connection.quote_table_name(table_name
     end
 
     def append_change_table(table_name, buf)
-      buf.puts "change_table(#{table_name.inspect}, {:bulk => true}) do |t|" if @options[:bulk_change]
+      buf.puts "change_table(#{table_name.inspect}, bulk: true) do |t|" if @options[:bulk_change]
       yield
       buf.puts 'end' if @options[:bulk_change]
     end
@@ -429,11 +429,11 @@ remove_column(#{table_name.inspect}, #{column_name.inspect})
 
       if force_bulk_change || @options[:bulk_change]
         buf.puts(<<-RUBY)
-  t.index(#{column_name.inspect}, #{options.inspect})
+  t.index(#{column_name.inspect}, **#{options.inspect})
       RUBY
       else
         buf.puts(<<-RUBY)
-add_index(#{table_name.inspect}, #{column_name.inspect}, #{options.inspect})
+add_index(#{table_name.inspect}, #{column_name.inspect}, **#{options.inspect})
       RUBY
       end
     end
@@ -441,15 +441,20 @@ add_index(#{table_name.inspect}, #{column_name.inspect}, #{options.inspect})
     def append_remove_index(table_name, _index_name, attrs, buf)
       column_name = attrs.fetch(:column_name)
       options = attrs[:options] || {}
-      target = options[:name] ? { name: options[:name] } : column_name
+      target =
+        if options[:name]
+          "name: #{options[:name].inspect}"
+        else
+          column_name.inspect
+        end
 
       if @options[:bulk_change]
         buf.puts(<<-RUBY)
-  t.remove_index(#{target.inspect})
+  t.remove_index(#{target})
       RUBY
       else
         buf.puts(<<-RUBY)
-remove_index(#{table_name.inspect}, #{target.inspect})
+remove_index(#{table_name.inspect}, #{target})
       RUBY
       end
     end
@@ -469,7 +474,7 @@ remove_index(#{table_name.inspect}, #{target.inspect})
       attrs_options = attrs[:options] || {}
 
       buf.puts(<<-RUBY)
-add_foreign_key(#{table_name.inspect}, #{to_table.inspect}, #{attrs_options.inspect})
+add_foreign_key(#{table_name.inspect}, #{to_table.inspect}, **#{attrs_options.inspect})
     RUBY
     end
 
@@ -478,13 +483,13 @@ add_foreign_key(#{table_name.inspect}, #{to_table.inspect}, #{attrs_options.insp
       fk_name = attrs_options[:name]
 
       target = if fk_name
-                 { name: fk_name }
+                 "name: #{fk_name.inspect}"
                else
-                 attrs.fetch(:to_table)
+                 attrs.fetch(:to_table).inspect
                end
 
       buf.puts(<<-RUBY)
-remove_foreign_key(#{table_name.inspect}, #{target.inspect})
+remove_foreign_key(#{table_name.inspect}, #{target})
     RUBY
     end
 
@@ -500,17 +505,19 @@ remove_foreign_key(#{table_name.inspect}, #{target.inspect})
     def inspect_options_include_default_proc(options)
       options = options.dup
 
-      if options[:default].is_a?(Proc)
-        proc_default = options.delete(:default)
-        proc_default = ":default=>proc{#{proc_default.call.inspect}}"
-        options_inspect = options.inspect
-        options_inspect.sub!(/\}\z/, '')
-        options_inspect << ', ' if options_inspect !~ /\{\z/
-        options_inspect << proc_default << '}'
-        options_inspect
-      else
-        options.inspect
-      end
+      kwargs =
+        if options[:default].is_a?(Proc)
+          proc_default = options.delete(:default)
+          proc_default = ":default=>proc{#{proc_default.call.inspect}}"
+          options_inspect = options.inspect
+          options_inspect.sub!(/\}\z/, '')
+          options_inspect << ', ' if options_inspect !~ /\{\z/
+          options_inspect << proc_default << '}'
+          options_inspect
+        else
+          options.inspect
+        end
+      "**#{kwargs}"
     end
   end
 end

data/lib/ridgepole/diff.rb

@@ -386,6 +386,18 @@ module Ridgepole
           attrs[:type] = :bigint
           opts.delete(:limit)
         end
+
+        if opts[:size] && (attrs[:type] == :text || attrs[:type] == :blob || attrs[:type] == :binary)
+          case opts.delete(:size)
+          when :tiny
+            attrs[:type] = :blob if attrs[:type] == :binary
+            opts[:limit] = 255
+          when :medium
+            opts[:limit] = 16_777_215
+          when :long
+            opts[:limit] = 4_294_967_295
+          end
+        end
       end
     end
 

data/lib/ridgepole/dsl_parser.rb

@@ -37,11 +37,9 @@ module Ridgepole
       attrs[:foreign_keys].each do |_, foreign_key_attrs|
         fk_index = foreign_key_attrs[:options][:column] || "#{foreign_key_attrs[:to_table].singularize}_id"
         next if attrs[:indices]&.any? { |_k, v| v[:column_name].first == fk_index }
+        next if attrs[:options][:primary_key] == fk_index
 
-        Ridgepole::Logger.instance.warn(<<-MSG)
-[WARNING] Table `#{table_name}` has a foreign key on `#{fk_index}` column, but doesn't have any indexes on the column.
-          Although an index will be added automatically by InnoDB, please add an index explicitly for your future operations.
-        MSG
+        raise "The column `#{fk_index}` of the table `#{table_name}` has a foreign key but no index. Although InnoDB creates an index automatically, please add one explicitly in order for ridgepole to manage it."
       end
     end
   end

data/lib/ridgepole/dsl_parser/context.rb

@@ -3,8 +3,7 @@
 module Ridgepole
   class DSLParser
     class Context
-      attr_reader :__definition
-      attr_reader :__execute
+      attr_reader :__definition, :__execute
 
       def initialize(opts = {})
         @__working_dir = File.expand_path(opts[:path] ? File.dirname(opts[:path]) : Dir.pwd)
@@ -104,6 +103,10 @@ module Ridgepole
         end
       end
 
+      def require_relative(relative_path)
+        require(File.expand_path(relative_path, File.dirname(caller[0])))
+      end
+
       def execute(sql, _name = nil, &cond)
         @__execute << {
           sql: sql,

data/lib/ridgepole/external_sql_executer.rb

@@ -41,7 +41,7 @@ module Ridgepole
               end
             end
           end
-        rescue EOFError # rubocop:disable Lint/HandleExceptions
+        rescue EOFError
           # nothing to do
         end
 

data/lib/ridgepole/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Ridgepole
-  VERSION = '0.8.5'
+  VERSION = '0.8.10'
 end

data/spec/dsl_parser/context_spec.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+describe Ridgepole::DSLParser::Context do
+  describe '#require_relative' do
+    subject { context.require_relative(relative_path) }
+
+    let!(:context) do
+      Ridgepole::DSLParser::Context.new
+    end
+    let!(:relative_path) do
+      '../fixtures/for_require_relative_spec.rb'
+    end
+
+    it { is_expected.to be_truthy }
+  end
+end