dry-validation 1.1.1 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3821e92d2dced61bb892e20a5e5d69b596128fcdbd05596e41fff8716767b08c
-  data.tar.gz: f797d5e46604e777966cb3d9a7a4032265227684164224a8ff109fc8ba51d1c1
+  metadata.gz: aadb0a6e283c3de0438b2ddf7335262e276c206021e95f4aaa7eb70be040f268
+  data.tar.gz: e2c9949f4f69ded243970367602b997c8eba80c5d18b223f7f14d30d914b4886
 SHA512:
-  metadata.gz: 7504498b77c794b9fe37e0384fba50a8fb6c99e3fecd3423e66fe018006288b3ba1a5a7a2232fb53dede256ba0a13630313748a0214b4a0d9865fc22c0b9edae
-  data.tar.gz: 654233d365e7844985f465e16335a234a2d5c2c240b97a485d286a52e6229e3ee80d8719e76d3ec95fe393d75f480617452b7813cb36b4b033021f9b98958461
+  metadata.gz: 7cd158435b3998c6864a594d61315760b974a4b8d8a8728bdff42cc3684209416a23a154e0efe9537ab54839f96819b4b11f434104b2ba5b2be6dd4fce4c39b5
+  data.tar.gz: d796476595c68a33dca5cf26cb11f3aff0c3fc2be73165f5693bb9b62c94c02a917ff8f559cc62055909805d95928dc96189e8a1391657177a21674ab15b58f5

data/CHANGELOG.md

@@ -1,3 +1,19 @@
+# v1.2.0 2019-07-08
+
+### Added
+
+* New extension `:predicates_as_macros` (@waiting-for-dev)
+
+### Fixed
+
+* Guarding rules for nested keys works correctly (issue #560) (@solnic)
+
+### Changed
+
+* `dry-schema` dependency was bumped to `>= 1.3.1` (@solnic)
+
+[Compare v1.1.1...v1.2.0](https://github.com/dry-rb/dry-validation/compare/v1.1.1...v1.2.0)
+
 # v1.1.1 2019-06-24
 
 ### Fixed

data/README.md

@@ -1,5 +1,5 @@
 [gem]: https://rubygems.org/gems/dry-validation
-[travis]: https://travis-ci.org/dry-rb/dry-validation
+[travis]: https://travis-ci.com/dry-rb/dry-validation
 [codeclimate]: https://codeclimate.com/github/dry-rb/dry-validation
 [chat]: https://dry-rb.zulipchat.com
 [inchpages]: http://inch-ci.org/github/dry-rb/dry-validation
@@ -7,7 +7,7 @@
 # dry-validation [![Join the chat at https://dry-rb.zulipchat.com](https://img.shields.io/badge/dry--rb-join%20chat-%23346b7a.svg)][chat]
 
 [![Gem Version](https://badge.fury.io/rb/dry-validation.svg)][gem]
-[![Build Status](https://travis-ci.org/dry-rb/dry-validation.svg?branch=master)][travis]
+[![Build Status](https://travis-ci.com/dry-rb/dry-validation.svg?branch=master)][travis]
 [![Code Climate](https://codeclimate.com/github/dry-rb/dry-validation/badges/gpa.svg)][codeclimate]
 [![Test Coverage](https://codeclimate.com/github/dry-rb/dry-validation/badges/coverage.svg)][codeclimate]
 [![Inline docs](http://inch-ci.org/github/dry-rb/dry-validation.svg?branch=master)][inchpages]

data/lib/dry/validation.rb

@@ -23,6 +23,10 @@ module Dry
       require 'dry/validation/extensions/hints'
     end
 
+    register_extension(:predicates_as_macros) do
+      require 'dry/validation/extensions/predicates_as_macros'
+    end
+
     # Define a contract and build its instance
     #
     # @example

data/lib/dry/validation/contract.rb

@@ -116,11 +116,20 @@ module Dry
       private
 
       # @api private
-      def error?(result, key)
-        path = Schema::Path[key]
+      def error?(result, spec)
+        path = Schema::Path[spec]
 
-        result.error?(path) ||
-          path.map.with_index { |_k, i| result.error?(path.keys[0..i - 2]) }.any?
+        return true if result.error?(path)
+
+        path
+          .to_a[0..-2]
+          .any? { |key|
+            curr_path = Schema::Path[path.keys[0..path.keys.index(key)]]
+
+            return false unless result.error?(curr_path)
+
+            result.errors.any? { |err| Schema::Path[err.path] == curr_path }
+          }
       end
 
       # Get a registered macro

data/lib/dry/validation/contract/class_interface.rb

@@ -7,34 +7,9 @@ require 'dry/schema/key_map'
 
 require 'dry/validation/constants'
 require 'dry/validation/macros'
+require 'dry/validation/schema_ext'
 
 module Dry
-  module Schema
-    # @api private
-    class Key
-      # @api private
-      def to_dot_notation
-        [name.to_s]
-      end
-
-      # @api private
-      class Hash < Key
-        # @api private
-        def to_dot_notation
-          [name].product(members.map(&:to_dot_notation).flatten(1)).map { |e| e.join(DOT) }
-        end
-      end
-    end
-
-    # @api private
-    class KeyMap
-      # @api private
-      def to_dot_notation
-        @to_dot_notation ||= map(&:to_dot_notation).flatten
-      end
-    end
-  end
-
   module Validation
     class Contract
       # Contract's class interface
@@ -179,12 +154,29 @@ module Dry
         private
 
         # @api private
+        # rubocop:disable Metrics/AbcSize
         def ensure_valid_keys(*keys)
           valid_paths = key_map.to_dot_notation.map { |value| Schema::Path[value] }
 
-          invalid_keys = Schema::KeyMap[*keys]
-            .map(&:dump)
-            .reject { |spec| valid_paths.any? { |path| path.include?(Schema::Path[spec]) } }
+          invalid_keys = keys
+            .map { |key|
+              [key, Schema::Path[key]]
+            }
+            .map { |(key, path)|
+              if (last = path.last).is_a?(Array)
+                last.map { |last_key|
+                  path_key = [*path.to_a[0..-2], last_key]
+                  [path_key, Schema::Path[path_key]]
+                }
+              else
+                [[key, path]]
+              end
+            }
+            .flatten(1)
+            .reject { |(_, path)|
+              valid_paths.any? { |valid_path| valid_path.include?(path) }
+            }
+            .map(&:first)
 
           return if invalid_keys.empty?
 
@@ -192,6 +184,7 @@ module Dry
             #{name}.rule specifies keys that are not defined by the schema: #{invalid_keys.inspect}
           STR
         end
+        # rubocop:enable Metrics/AbcSize
 
         # @api private
         def key_map

data/lib/dry/validation/extensions/predicates_as_macros.rb

@@ -0,0 +1,75 @@
+# frozen_string_literal: true
+
+require 'dry/schema/predicate_registry'
+require 'dry/validation/contract'
+
+module Dry
+  module Validation
+    # Predicate registry with additional needed methods.
+    class PredicateRegistry < Schema::PredicateRegistry
+      # List of predicates to be imported by `:predicates_as_macros`
+      # extension.
+      #
+      # @see Dry::Validation::Contract
+      WHITELIST = %i[
+        filled? gt? gteq? included_in? includes? inclusion? is? lt?
+        lteq? max_size? min_size? not_eql? odd? respond_to? size? true?
+        uuid_v4?
+      ].freeze
+
+      # @api private
+      def arg_names(name)
+        arg_list(name).map(&:first)
+      end
+
+      # @api private
+      def call(name, args)
+        self[name].(*args)
+      end
+
+      # @api private
+      def message_opts(name, arg_values)
+        arg_names(name).zip(arg_values).to_h
+      end
+    end
+
+    # Extension to use dry-logic predicates as macros.
+    #
+    # @see Dry::Validation::PredicateRegistry::WHITELIST Available predicates
+    #
+    # @example
+    #   Dry::Validation.load_extensions(:predicates_as_macros)
+    #
+    #   class ApplicationContract < Dry::Validation::Contract
+    #     import_predicates_as_macros
+    #   end
+    #
+    #   class AgeContract < ApplicationContract
+    #     schema do
+    #       required(:age).filled(:integer)
+    #     end
+    #
+    #     rule(:age).validate(gteq?: 18)
+    #   end
+    #
+    #   AgeContract.new.(age: 17).errors.first.text
+    #   # => 'must be greater than or equal to 18'
+    #
+    # @api public
+    class Contract
+      # Make macros available for self and its descendants.
+      def self.import_predicates_as_macros
+        registry = PredicateRegistry.new
+
+        PredicateRegistry::WHITELIST.each do |name|
+          register_macro(name) do |macro:|
+            predicate_args = [*macro.args, value]
+            message_opts = registry.message_opts(name, predicate_args)
+
+            key.failure(name, message_opts) unless registry.(name, predicate_args)
+          end
+        end
+      end
+    end
+  end
+end

data/lib/dry/validation/function.rb

@@ -19,21 +19,24 @@ module Dry
       #   @api private
       option :block
 
+      # @!attribute [r] block_options
+      #   @return [Hash]
+      #   @api private
+      option :block_options, default: -> { block ? map_keywords(block) : EMPTY_HASH }
+
       private
 
       # Extract options for the block kwargs
       #
-      # @return [Hash]
+      # @param [Proc] block Callable
+      # @return Hash
       #
       # @api private
-      def block_options
-        return EMPTY_HASH unless block
-
-        @block_options ||= block
+      def map_keywords(block)
+        block
           .parameters
-          .select { |arg| arg[0].equal?(:keyreq) }
-          .map(&:last)
-          .map { |name| [name, BLOCK_OPTIONS_MAPPINGS[name]] }
+          .select { |arg,| arg.equal?(:keyreq) }
+          .map { |_, name| [name, BLOCK_OPTIONS_MAPPINGS[name]] }
           .to_h
       end
     end

data/lib/dry/validation/rule.rb

@@ -93,6 +93,8 @@ module Dry
           end
         end
 
+        @block_options = map_keywords(block) if block
+
         self
       end
 

data/lib/dry/validation/schema_ext.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+require 'dry/schema/key'
+require 'dry/schema/key_map'
+
+module Dry
+  module Schema
+    # @api private
+    #
+    # TODO: this should be moved to dry-schema at some point
+    class Key
+      # @api private
+      def to_dot_notation
+        [name.to_s]
+      end
+
+      # @api private
+      class Hash < Key
+        # @api private
+        def to_dot_notation
+          [name].product(members.map(&:to_dot_notation).flatten(1)).map { |e| e.join(DOT) }
+        end
+      end
+    end
+
+    # @api private
+    class KeyMap
+      # @api private
+      def to_dot_notation
+        @to_dot_notation ||= map(&:to_dot_notation).flatten
+      end
+    end
+  end
+end

data/lib/dry/validation/version.rb

@@ -2,6 +2,6 @@
 
 module Dry
   module Validation
-    VERSION = '1.1.1'
+    VERSION = '1.2.0'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dry-validation
 version: !ruby/object:Gem::Version
-  version: 1.1.1
+  version: 1.2.0
 platform: ruby
 authors:
 - Piotr Solnica
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-06-24 00:00:00.000000000 Z
+date: 2019-07-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby
@@ -95,7 +95,7 @@ dependencies:
         version: '1.0'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: 1.3.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -105,7 +105,7 @@ dependencies:
         version: '1.0'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: 1.3.1
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -168,6 +168,7 @@ files:
 - lib/dry/validation/evaluator.rb
 - lib/dry/validation/extensions/hints.rb
 - lib/dry/validation/extensions/monads.rb
+- lib/dry/validation/extensions/predicates_as_macros.rb
 - lib/dry/validation/failures.rb
 - lib/dry/validation/function.rb
 - lib/dry/validation/macro.rb
@@ -177,6 +178,7 @@ files:
 - lib/dry/validation/messages/resolver.rb
 - lib/dry/validation/result.rb
 - lib/dry/validation/rule.rb
+- lib/dry/validation/schema_ext.rb
 - lib/dry/validation/values.rb
 - lib/dry/validation/version.rb
 homepage: https://dry-rb.org/gems/dry-validation