reform 2.3.0.rc1 → 2.5.0

data/test/validation/result_test.rb

@@ -8,72 +8,70 @@ class ErrorsResultTest < Minitest::Spec
   # TODO: errors(args) not tested.
 
   describe "Contract::Result#success?" do
-    let (:failed) { MyResult.new(false) }
-    let (:succeeded) { MyResult.new(true) }
+    let(:failed) { MyResult.new(false) }
+    let(:succeeded) { MyResult.new(true) }
 
-    it { Reform::Contract::Result.new([failed, failed]).success?.must_equal false }
-    it { Reform::Contract::Result.new([succeeded, failed]).success?.must_equal false }
-    it { Reform::Contract::Result.new([failed, succeeded]).success?.must_equal false }
-    it { Reform::Contract::Result.new([succeeded, succeeded]).success?.must_equal true }
+    it { assert_equal Reform::Contract::Result.new([failed, failed]).success?, false }
+    it { assert_equal Reform::Contract::Result.new([succeeded, failed]).success?, false }
+    it { assert_equal Reform::Contract::Result.new([failed, succeeded]).success?, false }
+    it { assert Reform::Contract::Result.new([succeeded, succeeded]).success? }
   end
 
   describe "Contract::Result#errors" do
-    let (:results) {
+    let(:results) do
       [
-        MyResult.new(false, { title: ["must be filled"], nested: { something: [] } }),
-        MyResult.new(false, { length: ["no Int"] })
+        MyResult.new(false, {length: ["no Int"]}),
+        MyResult.new(false, {title: ["must be filled"], nested: {something: []}}),
+        MyResult.new(false, {title: ["must be filled"], nested: {something: []}}),
+        MyResult.new(false, {title: ["something more"], nested: {something: []}})
       ]
-    }
+    end
 
-    it { Reform::Contract::Result.new(results).errors.must_equal({:title=>["must be filled"], :length=>["no Int"]}) }
+    it { assert_equal Reform::Contract::Result.new(results).errors, {title: ["must be filled", "something more"], length: ["no Int"]} }
   end
 
   describe "Result::Pointer" do
-    let (:errors) do # dry result #errors format.
+    let(:errors) do # dry result #errors format.
       {
         title: ["ignore"],
-        artist: { age: ["too old"],
+        artist: {age: ["too old"],
           bands: {
-            0 => { name: "too new school" },
-            1 => { name: "too boring" },
+            0 => {name: "too new school"},
+            1 => {name: "too boring"},
           }
         }
       }
     end
 
-    let (:top) { Reform::Contract::Result::Pointer.new(MyResult.new(false, errors), []) }
-    it { top.success?.must_equal false }
-    it { top.errors.must_equal errors }
+    let(:top) { Reform::Contract::Result::Pointer.new(MyResult.new(false, errors), []) }
+    it { assert_equal top.success?, false }
+    it { assert_equal top.errors, errors }
 
-    let (:artist) { Reform::Contract::Result::Pointer.new(MyResult.new(false, errors), [:artist]) }
-    it { artist.success?.must_equal false }
-    it { artist.errors.must_equal({:age=>["too old"], :bands=>{0=>{:name=>"too new school"}, 1=>{:name=>"too boring"}}}) }
+    let(:artist) { Reform::Contract::Result::Pointer.new(MyResult.new(false, errors), [:artist]) }
+    it { assert_equal artist.success?, false }
+    it { assert_equal artist.errors,({age: ["too old"], bands: {0 => {name: "too new school"}, 1 => {name: "too boring"}}}) }
 
-    let (:band) { Reform::Contract::Result::Pointer.new(MyResult.new(false, errors), [:artist, :bands, 1]) }
-    it { band.success?.must_equal false }
-    it { band.errors.must_equal({:name=>"too boring"}) }
+    let(:band) { Reform::Contract::Result::Pointer.new(MyResult.new(false, errors), [:artist, :bands, 1]) }
+    it { assert_equal band.success?, false }
+    it { assert_equal band.errors,({name: "too boring"}) }
 
     describe "advance" do
       let(:advanced) { artist.advance(:bands, 1) }
 
-      it { advanced.success?.must_equal false }
-      it { advanced.errors.must_equal({:name=>"too boring"}) }
+      it { assert_equal advanced.success?, false }
+      it { assert_equal advanced.errors,({name: "too boring"}) }
 
-      it { artist.advance([:absolute, :nonsense]).must_equal nil }
+      it { assert_nil artist.advance(%i[absolute nonsense]) }
     end
   end
 end
 
-
 # validation group:
 
 # form.errors/messages/hint(*args)            ==> {:title: [..]}
 #   @call_result.errors/messages/hint(*args) }
 
-
-
 # # result = Result(original_result => [:band, :label], my_local_result => [] )
 # # result.messages(locale: :en) merges original_result and my_local_result
 
 # form.errors => Result(fetch tree of all nested forms.messages(*args))
-

data/test/validation_library_provided_test.rb

@@ -1,8 +1,8 @@
-require 'reform'
-require 'minitest/autorun'
+require "reform"
+require "minitest/autorun"
 
 class ValidationLibraryProvidedTest < MiniTest::Spec
-  it 'no validation library loaded' do
+  it "no validation library loaded" do
     assert_raises Reform::Validation::NoValidationLibraryError do
       class PersonForm < Reform::Form
         property :name

data/test/virtual_test.rb

@@ -1,16 +1,20 @@
-require 'test_helper'
+require "test_helper"
 
 class VirtualTest < MiniTest::Spec
   class CreditCardForm < TestForm
     property :credit_card_number, virtual: true # no read, no write, it's virtual.
+    collection :transactions, virtual: true, populate_if_empty: OpenStruct do
+      property :id
+    end
   end
 
-  let (:form) { CreditCardForm.new(Object.new) }
+  let(:form) { CreditCardForm.new(Object.new) }
 
   it {
-    form.validate("credit_card_number" => "123")
+    form.validate(credit_card_number: "123", transactions: [id: 1])
 
-    form.credit_card_number.must_equal "123"  # this is still readable in the UI.
+    assert_equal form.credit_card_number, "123" # this is still readable in the UI.
+    assert_equal form.transactions.first.id, 1 # this is still readable in the UI.
 
     form.sync
 
@@ -19,6 +23,42 @@ class VirtualTest < MiniTest::Spec
       hash = nested
     end
 
-    hash.must_equal("credit_card_number"=> "123")
+    assert_equal hash, "credit_card_number" => "123", "transactions" => ["id" => 1]
+  }
+end
+
+class VirtualAndDefaultTest < MiniTest::Spec
+  class CreditCardForm < TestForm
+    property :credit_card_number, virtual: true, default: "123" # no read, no write, it's virtual.
+    collection :transactions, virtual: true, populate_if_empty: OpenStruct, default: [OpenStruct.new(id: 2)] do
+      property :id
+    end
+  end
+
+  def hash(form)
+    hash = {}
+    form.save do |nested|
+      hash = nested
+    end
+    hash
+  end
+
+  let(:form) { CreditCardForm.new(Object.new) }
+
+  it {
+    form = CreditCardForm.new(Object.new)
+    form.validate({})
+
+    assert_equal hash(form), "credit_card_number" => "123", "transactions" => ["id" => 2]
+
+    form = CreditCardForm.new(Object.new)
+    form.validate(credit_card_number: "123", transactions: [id: 1])
+
+    assert_equal form.credit_card_number,  "123" # this is still readable in the UI.
+    assert_equal form.transactions.first.id,  1 # this is still readable in the UI.
+
+    form.sync
+
+    assert_equal hash(form), "credit_card_number" => "123", "transactions" => ["id" => 1]
   }
-end
+end

data/test/writeable_test.rb

@@ -1,4 +1,4 @@
-require 'test_helper'
+require "test_helper"
 
 class WriteableTest < MiniTest::Spec
   Location = Struct.new(:country)
@@ -7,24 +7,24 @@ class WriteableTest < MiniTest::Spec
     property :country, writeable: false
   end
 
-  let (:loc) { Location.new("Australia") }
-  let (:form) { LocationForm.new(loc) }
+  let(:loc) { Location.new("Australia") }
+  let(:form) { LocationForm.new(loc) }
 
   it do
-    form.country.must_equal "Australia"
+    assert_equal form.country, "Australia"
 
     form.validate("country" => "Germany") # this usually won't change when submitting.
-    form.country.must_equal "Germany"
+    assert_equal form.country, "Germany"
 
     form.sync
-    loc.country.must_equal "Australia" # the writer wasn't called.
+    assert_equal loc.country, "Australia" # the writer wasn't called.
 
     hash = {}
     form.save do |nested|
       hash = nested
     end
 
-    hash.must_equal("country"=> "Germany")
+    assert_equal hash, "country" => "Germany"
   end
 end
 
@@ -36,23 +36,23 @@ class WritableTest < MiniTest::Spec
     property :country, writable: false
   end
 
-  let (:loc) { Location.new("Australia") }
-  let (:form) { LocationForm.new(loc) }
+  let(:loc) { Location.new("Australia") }
+  let(:form) { LocationForm.new(loc) }
 
   it do
-    form.country.must_equal "Australia"
+    assert_equal form.country, "Australia"
 
     form.validate("country" => "Germany") # this usually won't change when submitting.
-    form.country.must_equal "Germany"
+    assert_equal form.country, "Germany"
 
     form.sync
-    loc.country.must_equal "Australia" # the writer wasn't called.
+    assert_equal loc.country, "Australia" # the writer wasn't called.
 
     hash = {}
     form.save do |nested|
       hash = nested
     end
 
-    hash.must_equal("country"=> "Germany")
+    assert_equal hash, "country" => "Germany"
   end
 end

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: reform
 version: !ruby/object:Gem::Version
-  version: 2.3.0.rc1
+  version: 2.5.0
 platform: ruby
 authors:
 - Nick Sutterer
 - Fran Worley
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2017-02-26 00:00:00.000000000 Z
+date: 2021-02-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: disposable
@@ -32,39 +32,39 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 0.5.0
 - !ruby/object:Gem::Dependency
-  name: uber
+  name: representable
   requirement: !ruby/object:Gem::Requirement
     requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.4.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: 0.2.0
+        version: 3.1.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.4.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: 0.2.0
+        version: 3.1.0
 - !ruby/object:Gem::Dependency
-  name: representable
+  name: uber
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 2.4.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: 3.1.0
+        version: 0.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 2.4.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: 3.1.0
+        version: 0.2.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -80,7 +80,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -94,7 +94,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: minitest
+  name: minitest-line
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -108,7 +108,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: dry-types
+  name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -136,19 +136,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: dry-validation
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.10.1
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.10.1
+        version: '0'
 description: Form object decoupled from models.
 email:
 - apotonick@gmail.com
@@ -169,6 +169,7 @@ files:
 - TODO.md
 - lib/reform.rb
 - lib/reform/contract.rb
+- lib/reform/contract/custom_error.rb
 - lib/reform/contract/validate.rb
 - lib/reform/errors.rb
 - lib/reform/form.rb
@@ -176,6 +177,7 @@ files:
 - lib/reform/form/coercion.rb
 - lib/reform/form/composition.rb
 - lib/reform/form/dry.rb
+- lib/reform/form/dry/input_hash.rb
 - lib/reform/form/module.rb
 - lib/reform/form/populator.rb
 - lib/reform/form/prepopulate.rb
@@ -190,9 +192,11 @@ files:
 - test/changed_test.rb
 - test/coercion_test.rb
 - test/composition_test.rb
+- test/contract/custom_error_test.rb
 - test/contract_test.rb
 - test/default_test.rb
 - test/deserialize_test.rb
+- test/docs/validation_test.rb
 - test/errors_test.rb
 - test/feature_test.rb
 - test/fixtures/dry_error_messages.yml
@@ -208,7 +212,6 @@ files:
 - test/prepopulator_test.rb
 - test/read_only_test.rb
 - test/readable_test.rb
-- test/readonly_test.rb
 - test/reform_test.rb
 - test/save_test.rb
 - test/setup_test.rb
@@ -225,7 +228,7 @@ homepage: https://github.com/trailblazer/reform
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -236,13 +239,12 @@ 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.3
-signing_key: 
+rubygems_version: 3.0.3
+signing_key:
 specification_version: 4
 summary: Form object decoupled from models with validation, population and presentation.
 test_files:
@@ -251,9 +253,11 @@ test_files:
 - test/changed_test.rb
 - test/coercion_test.rb
 - test/composition_test.rb
+- test/contract/custom_error_test.rb
 - test/contract_test.rb
 - test/default_test.rb
 - test/deserialize_test.rb
+- test/docs/validation_test.rb
 - test/errors_test.rb
 - test/feature_test.rb
 - test/fixtures/dry_error_messages.yml
@@ -269,7 +273,6 @@ test_files:
 - test/prepopulator_test.rb
 - test/read_only_test.rb
 - test/readable_test.rb
-- test/readonly_test.rb
 - test/reform_test.rb
 - test/save_test.rb
 - test/setup_test.rb