reform 2.3.0.rc2 → 2.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: d820c5f61dcb3ef74ab8ac15bbc662bd8e9490a8
-  data.tar.gz: aa023d388b19459ebe7dd4c19862f8bc85364697
+SHA256:
+  metadata.gz: f22b2268e248120b5738d5387d464d8563d661d6749a6c371ed1172d9c3fb18b
+  data.tar.gz: 534162094e74586a1c73634f1394f8ef8f37bb79b970c7003d37773c1f2109a1
 SHA512:
-  metadata.gz: 48f9fcbc0917b9478d1124900146d6597e32e613a2ab4d61d504d359fb546b42d44ae958fff089df1bc315983c1fb70701b4246fb4a85b79341113f9460e5278
-  data.tar.gz: eac447cc3824c9e09926e64fb33437e8e9c0b3e86a7dfd24ec1cc71407c23550854fd6c64ea25538de01cb1350fb6302bb033e542bebb1f52a65ea1a7d8674e9
+  metadata.gz: d1f26ff2eb879a16cea3a230fb0fccf97525cf72321941b48312d47ed47e5a9759f67e44c9cd68057adf020ec0374706ae4991b9b29141b46ee963ef4e218d31
+  data.tar.gz: 77c4f8dc39bb7616e12b2b8c74f3f45e0b9285bd95dcc0a9a7206e35905c1b4ef956ab87eef49b20b87efc322578e047caa0f84cc30024ff236320af93764308

data/.gitignore

@@ -4,7 +4,6 @@
 .config
 .yardoc
 Gemfile.lock
-gemfiles/*.lock
 InstalledFiles
 _yardoc
 coverage
@@ -18,3 +17,6 @@ test/version_tmp
 tmp
 .rubocop-https*
 .byebug_history
+.idea
+*.iml
+gemfiles/*.gemfile.lock

data/.travis.yml

@@ -3,30 +3,14 @@ cache: bundler
 bundler_args: --without benchmarks tools
 rvm:
   - ruby-head
+  - 2.7
+  - 2.6
   - 2.5
   - 2.4
-env:
-  - "DRY_VALIDATION='~> 1.3.0'"
-  - "DRY_VALIDATION='~> 1.2.0'"
-  - "DRY_VALIDATION='~> 1.1.0'"
-  - "DRY_VALIDATION='~> 1.0.0'"
-  - "DRY_VALIDATION='~> 0.13.0'"
-  - "DRY_VALIDATION='~> 0.12.0'"
+gemfile:
+  - gemfiles/1.5.0.gemfile
+  - gemfiles/0.13.0.gemfile
 matrix:
   fast_finish: true
   allow_failures:
     - rvm: ruby-head
-  include:
-    - { rvm: 2.2, env: "DRY_VALIDATION='~> 0.11.0'" }
-    - { rvm: 2.3, env: "DRY_VALIDATION='~> 0.12.0'" }
-    - { rvm: 2.3, env: "DRY_VALIDATION='~> 0.13.0'" }
-
-
-# maybe move this to slack in a private channel
-# notifications:
-#   webhooks:
-#     urls:
-#       - https://webhooks.gitter.im/e/680e86d98056f2ae2fd7
-#     on_success: change  # options: [always|never|change] default: always
-#     on_failure: always  # options: [always|never|change] default: always
-#     on_start: never     # options: [always|never|change] default: always

data/Appraisals

@@ -0,0 +1,8 @@
+appraise "1.5.0" do
+  gem 'dry-monads', "1.3.5"
+  gem 'dry-validation', '~> 1.5.0'
+end
+
+appraise "0.13.0" do
+  gem 'dry-validation', '~> 0.13.0'
+end

data/CHANGES.md

@@ -3,6 +3,11 @@
 [* Removed `Reform::Contract` ?]
 [* Move Form#deserializer to Form::deserializer]
 
+## 2.3.1
+* With dry-validation 1.5 the form is always injected. Just add option :form to access it in the schema.
+* Removed global monkey patching of Dry::Schema::DSL
+* Tests in ruby 2.7
+
 ## 2.3.0
 
 You can upgrade from 2.2.0 without worries.

data/Gemfile

@@ -2,10 +2,4 @@ source "https://rubygems.org"
 
 gemspec
 
-# gem "disposable", path: "../disposable"
-
-# just trying to add `dry-monads` correct version in base on dry-validation
-dry_v_version = ENV.fetch('DRY_VALIDATION', '~> 0.13.0')
-dry_m_version = "~> #{dry_v_version.gsub("~>", "").to_f}.0"
-gem 'dry-monads', dry_m_version if dry_v_version.gsub("~>", "").to_f >= 1
-gem 'dry-validation', ENV.fetch('DRY_VALIDATION', '~> 0.13.0')
+gem "appraisal", "~> 2.2"

data/LICENSE.txt

@@ -1,4 +1,4 @@
-Copyright (c) 2013 - 2016 Nick Sutterer
+Copyright (c) 2013 - 2020 Nick Sutterer
 
 MIT License
 

data/Rakefile

@@ -1,6 +1,7 @@
 require "bundler/gem_tasks"
 require "rake/testtask"
 require "rubocop/rake_task"
+require "dry/types/version"
 
 task default: %i[test]
 
@@ -10,8 +11,7 @@ TEST_WITH_OLD_AND_NEW_API = %w[
 ].freeze
 
 def dry_v_test_files
-  dry_v_version = ENV.fetch("DRY_VALIDATION", "~> 0.13.0")
-  api = dry_v_version.gsub("~>", "").to_f >= 1.0 ? "new" : "old"
+  api = Gem::Version.new(Dry::Types::VERSION).to_s.split('.').first.to_i >= 1 ? "new" : "old"
   TEST_WITH_OLD_AND_NEW_API.map { |file| "test/#{file}_#{api}_api.rb" }
 end
 

data/gemfiles/0.13.0.gemfile

@@ -0,0 +1,8 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "appraisal", "~> 2.2"
+gem "dry-validation", "~> 0.13.0"
+
+gemspec path: "../"

data/gemfiles/1.5.0.gemfile

@@ -0,0 +1,9 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "appraisal", "~> 2.2"
+gem "dry-monads", "1.3.5"
+gem "dry-validation", "~> 1.5.0"
+
+gemspec path: "../"

data/lib/reform/form/dry/new_api.rb

@@ -1,6 +1,9 @@
+::Dry::Validation.load_extensions(:hints)
+
 module Reform::Form::Dry
   module NewApi
-    class Contract < Dry::Validation::Contract
+
+    class Contract < ::Dry::Validation::Contract
     end
 
     module Validations
@@ -18,39 +21,24 @@ module Reform::Form::Dry
         include InputHash
 
         def initialize(options = {})
-          options ||= {}
-          @validator = options[:schema] || Reform::Form::Dry::NewApi::Contract
-
-          @schema_inject_params = options[:with] || {}
+          @validator = options.fetch(:schema, Contract)
+          @schema_inject_params = options.fetch(:with, {})
         end
 
         def instance_exec(&block)
-          Dry::Validation.load_extensions(:hints)
           @block = block
         end
 
         def call(form)
-          dynamic_options = {}
-          dynamic_options[:form] = form if @schema_inject_params[:form]
-          inject_options = @schema_inject_params.merge(dynamic_options)
-
-          Dry::Schema::DSL.class_eval do
-            inject_options.each do |key, value|
-              define_method(key) { value }
-            end
-          end
-
           # when passing options[:schema] the class instance is already created so we just need to call
           # "call"
-          @validator = @validator.build(&@block) if @validator == Reform::Form::Dry::NewApi::Contract
-
-          # TODO: only pass submitted values to Schema#call?
-          dry_result = @validator.call(input_hash(form))
-          # dry_messages    = dry_result.messages
-
-          return dry_result
+          if @validator.is_a?(Class) && @validator <= ::Dry::Validation::Contract
+            dynamic_options = {form: form}
+            inject_options = @schema_inject_params.merge(dynamic_options)
+            @validator = @validator.build(inject_options, &@block)
+          end
 
-          _reform_errors = Reform::Contract::Errors.new(dry_result) # TODO: dry should be merged here.
+          @validator.call(input_hash(form))
         end
       end
     end

data/lib/reform/version.rb

@@ -1,3 +1,3 @@
 module Reform
-  VERSION = "2.3.0.rc2".freeze
+  VERSION = "2.3.1".freeze
 end

data/reform.gemspec

@@ -24,7 +24,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "bundler"
   spec.add_development_dependency "minitest"
   spec.add_development_dependency "minitest-line"
-  spec.add_development_dependency "byebug"
+  spec.add_development_dependency "pry-byebug"
   spec.add_development_dependency "multi_json"
   spec.add_development_dependency "rake"
   spec.add_development_dependency "rubocop"

data/test/test_helper.rb

@@ -3,7 +3,7 @@ require "minitest/autorun"
 require "representable/debug"
 require "declarative/testing"
 require "pp"
-require "byebug"
+require "pry-byebug"
 
 require "reform/form/dry"
 

data/test/validation/dry_validation_new_api.rb

@@ -73,21 +73,21 @@ class DryValidationErrorsAPITest < Minitest::Spec
     form.to_result.hints.must_equal(title: ["size cannot be less than 2"])
     form.artist.to_result.errors.must_equal(email: ["must be filled"])
     form.artist.to_result.messages.must_equal(email: ["must be filled"])
-    form.artist.to_result.hints.must_equal(email: [])
+    form.artist.to_result.hints.must_equal({})
     form.artist.label.to_result.errors.must_equal(location: ["must be filled"])
     form.artist.label.to_result.messages.must_equal(location: ["must be filled"])
-    form.artist.label.to_result.hints.must_equal(location: [])
+    form.artist.label.to_result.hints.must_equal({})
     form.songs[0].to_result.errors.must_equal({})
     form.songs[0].to_result.messages.must_equal({})
     form.songs[0].to_result.hints.must_equal({})
     form.songs[1].to_result.errors.must_equal(title: ["must be filled"])
     form.songs[1].to_result.messages.must_equal(title: ["must be filled"])
-    form.songs[1].to_result.hints.must_equal(title: [])
+    form.songs[1].to_result.hints.must_equal({})
     form.songs[1].to_result.errors(locale: :de).must_equal(title: ["muss abgefüllt sein"])
     # seems like dry-v when calling Dry::Schema::Result#messages locale option is ignored
     # started a topic in their forum https://discourse.dry-rb.org/t/dry-result-messages-ignore-locale-option/910
     # form.songs[1].to_result.messages(locale: :de).must_equal(title: ["muss abgefüllt sein"])
-    form.songs[1].to_result.hints(locale: :de).must_equal(title: [])
+    form.songs[1].to_result.hints(locale: :de).must_equal({})
   end
 
   it "only nested property is invalid." do
@@ -195,8 +195,14 @@ class DryValidationDefaultGroupTest < Minitest::Spec
       params { required(:confirm_password).filled }
     end
 
-    validation name: :dynamic_args, with: {form: true} do
-      params { required(:color).maybe(included_in?: form.colors) }
+    validation name: :dynamic_args do
+      option :form
+      params { optional(:color) }
+      rule(:color) do
+        if value
+          key.failure("must be one of: #{form.colors}") unless form.colors.include? value
+        end
+      end
     end
 
     def colors
@@ -208,15 +214,15 @@ class DryValidationDefaultGroupTest < Minitest::Spec
 
   # valid.
   it do
-    form.validate(
+    assert form.validate(
       username: "Helloween",
       email:    "yep",
       starts_at: "01/01/2000 - 11:00",
       active: "true",
       confirm_password: "pA55w0rd"
-    ).must_equal true
-    form.active.must_equal true
-    form.errors.messages.inspect.must_equal "{}"
+    )
+    assert form.active
+    assert_equal "{}", form.errors.messages.inspect
   end
 
   it "invalid" do
@@ -309,8 +315,12 @@ class ValidationGroupsTest < MiniTest::Spec
         property :username
 
         validation name: :default, with: {user: OpenStruct.new(name: "Nick")} do
+          option :user
           params do
-            required(:username).filled(eql?: user.name)
+            required(:username).filled
+          end
+          rule(:username) do
+            key.failure("must be equal to #{user.name}") unless user.name == value
           end
         end
       end
@@ -471,7 +481,7 @@ class ValidationGroupsTest < MiniTest::Spec
       # TODO: use the same form structure as the top one and do the same test against messages, errors and hints.
       form.producers[0].to_result.errors.must_equal(name: ["must be filled"])
       form.producers[0].to_result.messages.must_equal(name: ["must be filled"])
-      form.producers[0].to_result.hints.must_equal(name: [])
+      form.producers[0].to_result.hints.must_equal({})
     end
 
     # FIXME: fix the "must be filled error"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: reform
 version: !ruby/object:Gem::Version
-  version: 2.3.0.rc2
+  version: 2.3.1
 platform: ruby
 authors:
 - Nick Sutterer
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-11-05 00:00:00.000000000 Z
+date: 2020-05-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: disposable
@@ -108,7 +108,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: byebug
+  name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -175,6 +175,7 @@ files:
 - ".rubocop.yml"
 - ".rubocop_todo.yml"
 - ".travis.yml"
+- Appraisals
 - CHANGES.md
 - CONTRIBUTING.md
 - Gemfile
@@ -183,6 +184,8 @@ files:
 - README.md
 - Rakefile
 - TODO.md
+- gemfiles/0.13.0.gemfile
+- gemfiles/1.5.0.gemfile
 - lib/reform.rb
 - lib/reform/contract.rb
 - lib/reform/contract/custom_error.rb
@@ -271,12 +274,11 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.14.3
+rubygems_version: 3.0.6
 signing_key: 
 specification_version: 4
 summary: Form object decoupled from models with validation, population and presentation.