nandi 0.9.0 → 0.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ba61fdd052ec58212031846c92ad947cb1f7cb8c033271b101f5973a2b7074f6
-  data.tar.gz: b0c30216fb5544927d1d3b47ef0ad87d935a9ea8859c37a8779662a65f88d9f5
+  metadata.gz: ff51dca733bf2e4a5bff9ff71ac930c6d849fa734106a31b1244777fb25db9c0
+  data.tar.gz: ceeaebbaee681e1a4e6e857ec32289d59d821b9c627f899b371b607c214b077a
 SHA512:
-  metadata.gz: f4c0ecfcb5e1a84c4a8b2e0721f4e48a8abff3f6bde78f04184544e9004987a4f155eebb3342b780fc793bc2d5104a8a98d7d25c6fcc9f8da43cdd70e0b5f9fd
-  data.tar.gz: aedffb405097c5d9e988e001eab0c929ed0fab64b1b9b04a534051fa59aa16634c3c9071bf5daf31b7c416bfa94679b020685bd8c201c78d0552ebbcf60e0a75
+  metadata.gz: aa53ecf85370d256a09197547d0d99ceb4a3f731620f25341ea05b44364749ba452a3f375554f180879426b1b757f0b551cca1e86ede08f755070adcb62a879f
+  data.tar.gz: 80f8945b42a0931dced7e02e83ff0c43e4173009d9d5e6f74c876f18f5fe037280488cd524fce8683301001a86eacdcc9df6674ea649ef1ae71f2b9a1fa6c2f5

data/README.md

@@ -202,10 +202,10 @@ class AddReferenceOnFoosToBars < ActiveRecord::Migration[5.2]
   set_statement_timeout(1_500)
 
   def up
-    add_reference(:foos, :bar)
+    add_column(:foos, :bar_id, :bigint)
   end
   def down
-    remove_reference(:foos, :bar)
+    remove_column(:foos, :bar_id)
   end
 end
 
@@ -268,10 +268,9 @@ Add a foreign key constraint. The generated SQL will include the NOT VALID param
 ### `#add_index(table, fields, **kwargs)`
 Adds a new index to the database.
 
-Nandi will:
-
+Nandi will
 - add the `CONCURRENTLY` option, which means the change takes a less restrictive lock at the cost of not running in a DDL transaction
-- use the `BTREE` index type which is the safest to create.
+- default to the `BTREE` index type, as it is commonly a good fit.
 
 Because index creation is particularly failure-prone, and because we cannot run in a transaction and therefore risk partially applied migrations that (in a Rails environment) require manual intervention, Nandi Validates that, if there is a add_index statement in the migration, it must be the only statement.
 
@@ -286,6 +285,12 @@ create_table :widgets do |t|
 end
 ```
 
+### `#add_reference(table, ref_name, **extra_args)`
+Adds a new reference column. Nandi will validate that the foreign key flag is not set to true; use `add_foreign_key` and `validate_foreign_key` instead! Nandi will also set the `index: false` flag, as index creation is unsafe unless done concurrently in a separate migration.
+
+### `#remove_reference(table, ref_name, **extra_args)`
+Removes a reference column.
+
 ### `#remove_column(table, name, **extra_args)`
 Remove an existing column.
 

data/lib/nandi/formatting.rb

@@ -64,7 +64,7 @@ module Nandi
         key = if k.is_a?(Symbol)
                 symbol_key(k)
               else
-                format_value(k) + " =>"
+                "#{format_value(k)} =>"
               end
         "#{key} #{format_value(v)}"
       end

data/lib/nandi/instructions.rb

@@ -5,6 +5,8 @@ require "nandi/instructions/remove_index"
 require "nandi/instructions/create_table"
 require "nandi/instructions/drop_table"
 require "nandi/instructions/add_column"
+require "nandi/instructions/add_reference"
+require "nandi/instructions/remove_reference"
 require "nandi/instructions/remove_column"
 require "nandi/instructions/add_foreign_key"
 require "nandi/instructions/drop_constraint"

data/lib/nandi/instructions/add_index.rb

@@ -21,11 +21,10 @@ module Nandi
           name: name,
 
           # Overrides and extra options
-          **@extra_args,
+          **extra_args_with_default_index_type,
 
           # Mandatory values
           algorithm: :concurrently,
-          using: :btree,
         }
       end
 
@@ -38,13 +37,19 @@ module Nandi
       private
 
       def name
-        :"idx_#{table.to_s}_on_#{field_names}"
+        :"idx_#{table}_on_#{field_names}"
       end
 
       def field_names
         field_names = fields.respond_to?(:map) ? fields.map(&:to_s).join("_") : fields
         field_names.to_s.scan(/\w+/).join("_")
       end
+
+      def extra_args_with_default_index_type
+        {
+          using: :btree,
+        }.merge(@extra_args)
+      end
     end
   end
 end

data/lib/nandi/instructions/add_reference.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+module Nandi
+  module Instructions
+    class AddReference
+      DEFAULT_EXTRA_ARGS = { index: false }.freeze
+      attr_reader :table, :ref_name, :extra_args
+
+      def initialize(table:, ref_name:, **kwargs)
+        @table = table
+        @ref_name = ref_name
+        @extra_args = DEFAULT_EXTRA_ARGS.merge(kwargs)
+      end
+
+      def procedure
+        :add_reference
+      end
+
+      def lock
+        Nandi::Migration::LockWeights::ACCESS_EXCLUSIVE
+      end
+    end
+  end
+end

data/lib/nandi/instructions/remove_reference.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module Nandi
+  module Instructions
+    class RemoveReference
+      attr_reader :table, :ref_name, :extra_args
+
+      def initialize(table:, ref_name:, **kwargs)
+        @table = table
+        @ref_name = ref_name
+        @extra_args = kwargs
+      end
+
+      def procedure
+        :remove_reference
+      end
+
+      def lock
+        Nandi::Migration::LockWeights::ACCESS_EXCLUSIVE
+      end
+    end
+  end
+end

data/lib/nandi/migration.rb

@@ -43,6 +43,7 @@ module Nandi
 
     class << self
       attr_reader :lock_timeout, :statement_timeout
+
       # For sake both of correspondence with Postgres syntax and familiarity
       # with activerecord-safe_migrations's identically named macros, we
       # disable this cop.
@@ -111,7 +112,7 @@ module Nandi
     # Nandi will:
     # * add the `CONCURRENTLY` option, which means the change takes a less
     #   restrictive lock at the cost of not running in a DDL transaction
-    # * use the `BTREE` index type which is the safest to create.
+    # * default to the `BTREE` index type, as it is commonly a good fit.
     #
     # Because index creation is particularly failure-prone, and because
     # we cannot run in a transaction and therefore risk partially applied
@@ -188,6 +189,31 @@ module Nandi
       )
     end
 
+    # Adds a new reference column. Nandi will validate that the foreign key flag
+    # is not set to true; use `add_foreign_key` and `validate_foreign_key` instead!
+    # @param table [Symbol, String] The name of the table to add the column to
+    # @param ref_name [Symbol, String] The referenced column name
+    # @param kwargs [Hash] Arbitrary options to be passed to the backend.
+    def add_reference(table, ref_name, **kwargs)
+      current_instructions << Instructions::AddReference.new(
+        table: table,
+        ref_name: ref_name,
+        **kwargs,
+      )
+    end
+
+    # Removes a reference column.
+    # @param table [Symbol, String] The name of the table to remove the reference from
+    # @param ref_name [Symbol, String] The referenced column name
+    # @param kwargs [Hash] Arbitrary options to be passed to the backend.
+    def remove_reference(table, ref_name, **kwargs)
+      current_instructions << Instructions::RemoveReference.new(
+        table: table,
+        ref_name: ref_name,
+        **kwargs,
+      )
+    end
+
     # Remove an existing column.
     # @param table [Symbol, String] The name of the table to remove the column
     #   from.

data/lib/nandi/renderers/active_record/instructions.rb

@@ -82,6 +82,18 @@ module Nandi
           formatted_property :extra_args
         end
 
+        class AddReferenceCell < Base
+          formatted_property :table
+          formatted_property :ref_name
+          formatted_property :extra_args
+        end
+
+        class RemoveReferenceCell < Base
+          formatted_property :table
+          formatted_property :ref_name
+          formatted_property :extra_args
+        end
+
         class RemoveColumnCell < Base
           formatted_property :table
           formatted_property :name

data/lib/nandi/validation.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "nandi/validation/add_column_validator"
+require "nandi/validation/add_reference_validator"
 require "nandi/validation/remove_index_validator"
 require "nandi/validation/each_validator"
 require "nandi/validation/result"

data/lib/nandi/validation/add_reference_validator.rb

@@ -0,0 +1,50 @@
+# frozen_string_literal: true
+
+require "nandi/validation/failure_helpers"
+
+module Nandi
+  module Validation
+    class AddReferenceValidator
+      include Nandi::Validation::FailureHelpers
+
+      def self.call(instruction)
+        new(instruction).call
+      end
+
+      def initialize(instruction)
+        @instruction = instruction
+      end
+
+      def call
+        foreign_key = instruction.extra_args.fetch(:foreign_key, false)
+        index = instruction.extra_args.fetch(:index, false)
+
+        collect_errors(
+          assert(
+            !foreign_key,
+            foreign_key_message,
+          ),
+          assert(
+            !index,
+            index_message,
+          ),
+        )
+      end
+
+      private
+
+      def index_message
+        "Indexing a reference column on creation can make the table unavailable." \
+          "Use the `add_index` method in a separate migration to index the column."
+      end
+
+      def foreign_key_message
+        "Adding a foreign key constraint must be done in two separate migrations. " \
+          "Use the `add_foreign_key` and `validate_foreign_key` methods, or the " \
+          "nandi:foreign_key generator, to do this."
+      end
+
+      attr_reader :instruction
+    end
+  end
+end

data/lib/nandi/validation/each_validator.rb

@@ -21,6 +21,8 @@ module Nandi
           RemoveIndexValidator.call(instruction)
         when :add_column
           AddColumnValidator.call(instruction)
+        when :add_reference
+          AddReferenceValidator.call(instruction)
         else
           success
         end

data/lib/nandi/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Nandi
-  VERSION = "0.9.0"
+  VERSION = "0.10.0"
 end

data/lib/templates/nandi/renderers/active_record/instructions/add_reference/show.rb.erb

@@ -0,0 +1,5 @@
+add_reference(
+  <%= table %>,
+  <%= ref_name %>,
+  <%= extra_args %>
+)

data/lib/templates/nandi/renderers/active_record/instructions/remove_reference/show.rb.erb

@@ -0,0 +1,5 @@
+remove_reference(
+  <%= table %>,
+  <%= ref_name %>,
+  <%= extra_args %>
+)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: nandi
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.10.0
 platform: ruby
 authors:
 - GoCardless Engineering
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-03-16 00:00:00.000000000 Z
+date: 2020-11-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -100,28 +100,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.3.14
+        version: 2.15.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.3.14
+        version: 2.15.0
 - !ruby/object:Gem::Dependency
   name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.7.0
+        version: 3.9.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.7.0
+        version: 3.9.0
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
@@ -140,22 +140,22 @@ dependencies:
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '12.3'
     - - ">="
       - !ruby/object:Gem::Version
         version: 12.3.3
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '12.3'
     - - ">="
       - !ruby/object:Gem::Version
         version: 12.3.3
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -190,14 +190,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.61'
+        version: '1.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.61'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: yard
   requirement: !ruby/object:Gem::Requirement
@@ -252,6 +252,7 @@ files:
 - lib/nandi/instructions/add_column.rb
 - lib/nandi/instructions/add_foreign_key.rb
 - lib/nandi/instructions/add_index.rb
+- lib/nandi/instructions/add_reference.rb
 - lib/nandi/instructions/change_column_default.rb
 - lib/nandi/instructions/create_table.rb
 - lib/nandi/instructions/drop_constraint.rb
@@ -260,6 +261,7 @@ files:
 - lib/nandi/instructions/remove_column.rb
 - lib/nandi/instructions/remove_index.rb
 - lib/nandi/instructions/remove_not_null_constraint.rb
+- lib/nandi/instructions/remove_reference.rb
 - lib/nandi/instructions/validate_constraint.rb
 - lib/nandi/lockfile.rb
 - lib/nandi/migration.rb
@@ -273,6 +275,7 @@ files:
 - lib/nandi/timeout_policies/concurrent.rb
 - lib/nandi/validation.rb
 - lib/nandi/validation/add_column_validator.rb
+- lib/nandi/validation/add_reference_validator.rb
 - lib/nandi/validation/each_validator.rb
 - lib/nandi/validation/failure_helpers.rb
 - lib/nandi/validation/remove_index_validator.rb
@@ -285,6 +288,7 @@ files:
 - lib/templates/nandi/renderers/active_record/instructions/add_column/show.rb.erb
 - lib/templates/nandi/renderers/active_record/instructions/add_foreign_key/show.rb.erb
 - lib/templates/nandi/renderers/active_record/instructions/add_index/show.rb.erb
+- lib/templates/nandi/renderers/active_record/instructions/add_reference/show.rb.erb
 - lib/templates/nandi/renderers/active_record/instructions/change_column_default/show.rb.erb
 - lib/templates/nandi/renderers/active_record/instructions/create_table/show.rb.erb
 - lib/templates/nandi/renderers/active_record/instructions/drop_constraint/show.rb.erb
@@ -293,6 +297,7 @@ files:
 - lib/templates/nandi/renderers/active_record/instructions/remove_column/show.rb.erb
 - lib/templates/nandi/renderers/active_record/instructions/remove_index/show.rb.erb
 - lib/templates/nandi/renderers/active_record/instructions/remove_not_null_constraint/show.rb.erb
+- lib/templates/nandi/renderers/active_record/instructions/remove_reference/show.rb.erb
 - lib/templates/nandi/renderers/active_record/instructions/validate_constraint/show.rb.erb
 homepage: https://github.com/gocardless/nandi
 licenses:
@@ -306,14 +311,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.4.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Fear-free migrations for PostgreSQL