dry-validation 1.4.0 → 1.5.2

data/LICENSE

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2015-2019 dry-rb team
+Copyright (c) 2015-2020 dry-rb team
 
 Permission is hereby granted, free of charge, to any person obtaining a copy of
 this software and associated documentation files (the "Software"), to deal in

data/README.md

@@ -1,29 +1,28 @@
 [gem]: https://rubygems.org/gems/dry-validation
-[ci]: https://github.com/dry-rb/dry-validation/actions?query=workflow%3Aci
-[codeclimate]: https://codeclimate.com/github/dry-rb/dry-validation
+[actions]: https://github.com/dry-rb/dry-validation/actions
+[codacy]: https://www.codacy.com/gh/dry-rb/dry-validation
 [chat]: https://dry-rb.zulipchat.com
 [inchpages]: http://inch-ci.org/github/dry-rb/dry-validation
 
 # 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://github.com/dry-rb/dry-monads/workflows/ci/badge.svg)][ci]
-[![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]
+[![CI Status](https://github.com/dry-rb/dry-validation/workflows/ci/badge.svg)][actions]
+[![Codacy Badge](https://api.codacy.com/project/badge/Grade/f30e3ff5ec304c55a73868cdbf055c67)][codacy]
+[![Codacy Badge](https://api.codacy.com/project/badge/Coverage/f30e3ff5ec304c55a73868cdbf055c67)][codacy]
 [![Inline docs](http://inch-ci.org/github/dry-rb/dry-validation.svg?branch=master)][inchpages]
 
 ## Links
 
-- [User documentation](https://dry-rb.org/gems/dry-validation)
-- [API documentation](http://rubydoc.info/gems/dry-validation)
-- [Guidelines for contributing](CONTRIBUTING.md)
+* [User documentation](http://dry-rb.org/gems/dry-validation)
+* [API documentation](http://rubydoc.info/gems/dry-validation)
 
 ## Supported Ruby versions
 
 This library officially supports the following Ruby versions:
 
-- MRI >= `2.4`
-- jruby >= `9.2`
+* MRI >= `2.4`
+* jruby >= `9.2`
 
 ## License
 

data/dry-validation.gemspec

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+# this file is managed by dry-rb/devtools project
+
+lib = File.expand_path('lib', __dir__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+require 'dry/validation/version'
+
+Gem::Specification.new do |spec|
+  spec.name          = 'dry-validation'
+  spec.authors       = ["Piotr Solnica"]
+  spec.email         = ["piotr.solnica@gmail.com"]
+  spec.license       = 'MIT'
+  spec.version       = Dry::Validation::VERSION.dup
+
+  spec.summary       = "Validation library"
+  spec.description   = spec.summary
+  spec.homepage      = 'https://dry-rb.org/gems/dry-validation'
+  spec.files         = Dir["CHANGELOG.md", "LICENSE", "README.md", "dry-validation.gemspec", "lib/**/*", "config/*.yml"]
+  spec.bindir        = 'bin'
+  spec.executables   = []
+  spec.require_paths = ['lib']
+
+  spec.metadata['allowed_push_host'] = 'https://rubygems.org'
+  spec.metadata['changelog_uri']     = 'https://github.com/dry-rb/dry-validation/blob/master/CHANGELOG.md'
+  spec.metadata['source_code_uri']   = 'https://github.com/dry-rb/dry-validation'
+  spec.metadata['bug_tracker_uri']   = 'https://github.com/dry-rb/dry-validation/issues'
+
+  spec.required_ruby_version = ">= 2.4.0"
+
+  # to update dependencies edit project.yml
+  spec.add_runtime_dependency "concurrent-ruby", "~> 1.0"
+  spec.add_runtime_dependency "dry-container", "~> 0.7", ">= 0.7.1"
+  spec.add_runtime_dependency "dry-core", "~> 0.4"
+  spec.add_runtime_dependency "dry-equalizer", "~> 0.2"
+  spec.add_runtime_dependency "dry-initializer", "~> 3.0"
+  spec.add_runtime_dependency "dry-schema", "~> 1.5"
+
+  spec.add_development_dependency "bundler"
+  spec.add_development_dependency "rake"
+  spec.add_development_dependency "rspec"
+end

data/lib/dry-validation.rb

@@ -1,3 +1,3 @@
 # frozen_string_literal: true
 
-require 'dry/validation'
+require "dry/validation"

data/lib/dry/validation.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require 'dry/validation/constants'
-require 'dry/validation/contract'
-require 'dry/validation/macros'
+require "dry/validation/constants"
+require "dry/validation/contract"
+require "dry/validation/macros"
 
 # Main namespace
 #
@@ -16,15 +16,15 @@ module Dry
     extend Macros::Registrar
 
     register_extension(:monads) do
-      require 'dry/validation/extensions/monads'
+      require "dry/validation/extensions/monads"
     end
 
     register_extension(:hints) do
-      require 'dry/validation/extensions/hints'
+      require "dry/validation/extensions/hints"
     end
 
     register_extension(:predicates_as_macros) do
-      require 'dry/validation/extensions/predicates_as_macros'
+      require "dry/validation/extensions/predicates_as_macros"
     end
 
     # Define a contract and build its instance
@@ -46,11 +46,9 @@ module Dry
     #
     # @api public
     #
-    # rubocop:disable Naming/MethodName
     def self.Contract(options = EMPTY_HASH, &block)
       Contract.build(options, &block)
     end
-    # rubocop:enable Naming/MethodName
 
     # This is needed by Macros::Registrar
     #

data/lib/dry/validation/config.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'dry/schema/config'
-require 'dry/validation/macros'
+require "dry/schema/config"
+require "dry/validation/macros"
 
 module Dry
   module Validation

data/lib/dry/validation/constants.rb

@@ -1,22 +1,22 @@
 # frozen_string_literal: true
 
-require 'pathname'
-require 'dry/core/constants'
+require "pathname"
+require "dry/core/constants"
 
 module Dry
   module Validation
     include Dry::Core::Constants
 
-    DOT = '.'
+    DOT = "."
 
     # Root path is used for base errors in hash representation of error messages
     ROOT_PATH = [nil].freeze
 
     # Path to the default errors locale file
-    DEFAULT_ERRORS_NAMESPACE = 'dry_validation'
+    DEFAULT_ERRORS_NAMESPACE = "dry_validation"
 
     # Path to the default errors locale file
-    DEFAULT_ERRORS_PATH = Pathname(__FILE__).join('../../../../config/errors.yml').realpath.freeze
+    DEFAULT_ERRORS_PATH = Pathname(__FILE__).join("../../../../config/errors.yml").realpath.freeze
 
     # Mapping for block kwarg options used by block_options
     #

data/lib/dry/validation/contract.rb

@@ -1,18 +1,18 @@
 # frozen_string_literal: true
 
-require 'concurrent/map'
-
-require 'dry/equalizer'
-require 'dry/initializer'
-require 'dry/schema/path'
-
-require 'dry/validation/config'
-require 'dry/validation/constants'
-require 'dry/validation/rule'
-require 'dry/validation/evaluator'
-require 'dry/validation/messages/resolver'
-require 'dry/validation/result'
-require 'dry/validation/contract/class_interface'
+require "concurrent/map"
+
+require "dry/equalizer"
+require "dry/initializer"
+require "dry/schema/path"
+
+require "dry/validation/config"
+require "dry/validation/constants"
+require "dry/validation/rule"
+require "dry/validation/evaluator"
+require "dry/validation/messages/resolver"
+require "dry/validation/result"
+require "dry/validation/contract/class_interface"
 
 module Dry
   module Validation

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

@@ -1,13 +1,13 @@
 # frozen_string_literal: true
 
-require 'dry/schema'
-require 'dry/schema/messages'
-require 'dry/schema/path'
-require 'dry/schema/key_map'
+require "dry/schema"
+require "dry/schema/messages"
+require "dry/schema/path"
+require "dry/schema/key_map"
 
-require 'dry/validation/constants'
-require 'dry/validation/macros'
-require 'dry/validation/schema_ext'
+require "dry/validation/constants"
+require "dry/validation/macros"
+require "dry/validation/schema_ext"
 
 module Dry
   module Validation
@@ -23,7 +23,7 @@ module Dry
         # @api private
         def inherited(klass)
           super
-          klass.instance_variable_set('@config', config.dup)
+          klass.instance_variable_set("@config", config.dup)
         end
 
         # Configuration
@@ -154,28 +154,15 @@ 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 = keys
-            .map { |key|
-              [key, Schema::Path[key]]
-            }
-            .flat_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
-            }
-            .reject { |(_, path)|
-              valid_paths.any? { |valid_path| valid_path.include?(path) }
-            }
-            .map(&:first)
+          valid_paths = key_map.to_dot_notation
+          key_paths = key_paths(keys)
+
+          invalid_keys = key_paths.map { |(key, path)|
+            unless valid_paths.any? { |vp| vp.include?(path) || vp.include?("#{path}[]") }
+              key
+            end
+          }.compact.uniq
 
           return if invalid_keys.empty?
 
@@ -183,7 +170,28 @@ 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_paths(keys)
+          keys.map { |key|
+            case key
+            when Hash
+              path = Schema::Path[key]
+              if path.multi_value?
+                *head, tail = Array(path)
+                [key].product(
+                  tail.map { |el| [*head, *el] }.map { |parts| parts.join(DOT) }
+                )
+              else
+                [[key, path.to_a.join(DOT)]]
+              end
+            when Array
+              [[key, Schema::Path[key].to_a.join(DOT)]]
+            else
+              [[key, key.to_s]]
+            end
+          }.flatten(1)
+        end
 
         # @api private
         def key_map
@@ -192,7 +200,7 @@ module Dry
 
         # @api private
         def core_schema_opts
-          { parent: superclass&.__schema__, config: config }
+          {parent: superclass&.__schema__, config: config}
         end
 
         # @api private
@@ -200,7 +208,7 @@ module Dry
           return __schema__ if external_schemas.empty? && block.nil?
 
           unless __schema__.nil?
-            raise ::Dry::Validation::DuplicateSchemaError, 'Schema has already been defined'
+            raise ::Dry::Validation::DuplicateSchemaError, "Schema has already been defined"
           end
 
           schema_opts = core_schema_opts

data/lib/dry/validation/evaluator.rb

@@ -1,9 +1,10 @@
 # frozen_string_literal: true
 
-require 'dry/initializer'
+require "dry/initializer"
+require "dry/core/deprecations"
 
-require 'dry/validation/constants'
-require 'dry/validation/failures'
+require "dry/validation/constants"
+require "dry/validation/failures"
 
 module Dry
   module Validation
@@ -16,6 +17,9 @@ module Dry
     # @api public
     class Evaluator
       extend Dry::Initializer
+      extend Dry::Core::Deprecations[:'dry-validation']
+
+      deprecate :error?, :schema_error?
 
       # @!attribute [r] _contract
       #   @return [Contract]
@@ -142,7 +146,7 @@ module Dry
       #
       # @return [Object]
       #
-      # @public
+      # @api public
       def value
         values[key_name]
       end
@@ -163,17 +167,26 @@ module Dry
         values.key?(key_name)
       end
 
-      # Check if there are any errors under the provided path
+      # Check if there are any errors on the schema under the provided path
       #
-      # @param [Symbol, String, Array] A Path-compatible spec
+      # @param path [Symbol, String, Array] A Path-compatible spec
       #
       # @return [Boolean]
       #
       # @api public
-      def error?(path)
+      def schema_error?(path)
         result.error?(path)
       end
 
+      # Check if there are any errors on the current rule
+      #
+      # @return [Boolean]
+      #
+      # @api public
+      def rule_error?
+        !key(path).empty?
+      end
+
       # @api private
       def respond_to_missing?(meth, include_private = false)
         super || _contract.respond_to?(meth, true)
@@ -192,6 +205,7 @@ module Dry
           super
         end
       end
+      ruby2_keywords(:method_missing) if respond_to?(:ruby2_keywords, true)
     end
   end
 end

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

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require 'dry/monads/result'
-
 module Dry
   module Validation
     # Hints extension
@@ -46,7 +44,7 @@ module Dry
         #
         # @api public
         def messages(new_options = EMPTY_HASH)
-          errors.with(hints.to_a, options.merge(**new_options))
+          errors.with(hints(new_options).to_a, options.merge(**new_options))
         end
 
         # Return hint messages

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

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'dry/monads/result'
+require "dry/monads/result"
 
 module Dry
   module Validation

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

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'dry/schema/predicate_registry'
-require 'dry/validation/contract'
+require "dry/schema/predicate_registry"
+require "dry/validation/contract"
 
 module Dry
   module Validation
@@ -12,7 +12,7 @@ module Dry
       #
       # @see Dry::Validation::Contract
       WHITELIST = %i[
-        filled? gt? gteq? included_in? includes? inclusion? is? lt?
+        filled? format? gt? gteq? included_in? includes? inclusion? is? lt?
         lteq? max_size? min_size? not_eql? odd? respond_to? size? true?
         uuid_v4?
       ].freeze

data/lib/dry/validation/failures.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'dry/schema/path'
-require 'dry/validation/constants'
+require "dry/schema/path"
+require "dry/validation/constants"
 
 module Dry
   module Validation
@@ -57,9 +57,14 @@ module Dry
       #
       # @api public
       def failure(message, tokens = EMPTY_HASH)
-        opts << { message: message, tokens: tokens, path: path }
+        opts << {message: message, tokens: tokens, path: path}
         self
       end
+
+      # @api private
+      def empty?
+        opts.empty?
+      end
     end
   end
 end