reform 2.2.4

data/test/module_test.rb

@@ -0,0 +1,142 @@
+require "test_helper"
+require 'reform/form/coercion'
+
+class ModuleInclusionTest < MiniTest::Spec
+  module BandPropertyForm
+    include Reform::Form::Module
+
+    property :band do
+      property :title
+
+      validation do
+        key(:title).required
+      end
+
+      def id # gets mixed into Form, too.
+        2
+      end
+    end
+
+    def id # gets mixed into Form, too.
+      1
+    end
+
+    validation do
+      key(:band).required
+    end
+
+    include Dry::Types.module # allows using Types::* in module.
+    property :cool, type: Form::Bool # test coercion.
+  end
+
+  # TODO: test if works, move stuff into inherit_schema!
+  module AirplaysPropertyForm
+    include Reform::Form::Module
+
+    collection :airplays do
+      property :station
+      validation do
+        key(:station).required
+      end
+    end
+    validation do
+      key(:airplays).required
+    end
+  end
+
+
+  # test:
+  # by including BandPropertyForm into multiple classes we assure that options hashes don't get messed up by AM:V.
+  class HitForm < Reform::Form
+    include BandPropertyForm
+  end
+
+  class SongForm < Reform::Form
+    include Coercion
+    property :title
+
+    include BandPropertyForm
+  end
+
+
+  let (:song) { OpenStruct.new(:band => OpenStruct.new(:title => "Time Again")) }
+
+  # nested form from module is present and creates accessor.
+  it { SongForm.new(song).band.title.must_equal "Time Again" }
+
+  # methods from module get included.
+  it { SongForm.new(song).id.must_equal 1 }
+  it { SongForm.new(song).band.id.must_equal 2 }
+
+  # validators get inherited.
+  it do
+    form = SongForm.new(OpenStruct.new)
+    form.validate({})
+    form.errors.messages.must_equal({:band=>["is missing"]})
+  end
+
+  # coercion works
+  it do
+    form = SongForm.new(OpenStruct.new)
+    form.validate({cool: "1"})
+    form.cool.must_equal true
+  end
+
+
+  # include a module into a module into a class :)
+  module AlbumFormModule
+    include Reform::Form::Module
+    include BandPropertyForm
+
+    property :name
+    validation do
+      key(:name).required
+    end
+  end
+
+  class AlbumForm < Reform::Form
+    include AlbumFormModule
+
+    # pp heritage
+    property :band, :inherit => true do
+      property :label
+      validation do
+        key(:label).required
+      end
+    end
+  end
+
+  it do
+    form = AlbumForm.new(OpenStruct.new(:band => OpenStruct.new))
+    form.validate({"band" => {}})
+    form.errors.messages.must_equal({:band=>["must be filled"], :"band.title"=>["is missing"], :"band.label"=>["is missing"], :name=>["is missing"]})
+  end
+
+
+  describe "module with custom accessors" do
+    module SongModule
+      include Reform::Form::Module
+
+      property :id    # no custom accessor for id.
+      property :title # has custom accessor.
+
+      module InstanceMethods
+        def title
+          super.upcase
+        end
+      end
+    end
+
+    class IncludingSongForm < Reform::Form
+      include SongModule
+    end
+
+    let (:song) { OpenStruct.new(id: 1, title: "Instant Mash") }
+
+    it do
+      IncludingSongForm.new(song).id.must_equal 1
+      IncludingSongForm.new(song).title.must_equal "INSTANT MASH"
+    end
+  end
+end
+

data/test/parse_pipeline_test.rb

@@ -0,0 +1,15 @@
+require "test_helper"
+
+class ParsePipelineTest < MiniTest::Spec
+  Album = Struct.new(:name)
+
+  class AlbumForm < Reform::Form
+    property :name, deserializer: { parse_pipeline: ->(input, options) { Representable::Pipeline[->(input, options) { options[:represented].name = input.inspect }] } }
+  end
+
+  it "allows passing :parse_pipeline directly" do
+    form = AlbumForm.new(Album.new)
+    form.validate("name" => "Greatest Hits")
+    form.name.must_equal "{\"name\"=>\"Greatest Hits\"}"
+  end
+end

data/test/populate_test.rb

@@ -0,0 +1,270 @@
+require "test_helper"
+
+class PopulatorTest < MiniTest::Spec
+  Song  = Struct.new(:title, :album, :composer)
+  Album = Struct.new(:name, :songs, :artist)
+  Artist = Struct.new(:name)
+
+  class AlbumForm < Reform::Form
+    property :name, populator: ->(options) { self.name = options[:fragment].reverse }
+    validation do
+      key(:name).required
+    end
+
+    collection :songs,
+      populator: ->(options) {
+        fragment, collection, index = options[:fragment], options[:model], options[:index]
+
+        (item = collection[index]) ? item : collection.insert(index, Song.new) } do
+
+      property :title
+      validation do
+        key(:title).required
+      end
+
+      property :composer, populator: ->(options) { options[:model] || self.composer= Artist.new } do
+        property :name
+        validation do
+          key(:name).required
+        end
+      end
+    end
+
+    # property :artist, populator: lambda { |fragment, options| (item = options.binding.get) ? item : Artist.new } do
+    # NOTE: we have to document that model here is the twin!
+    property :artist, populator: ->(options) { options[:model] || self.artist = Artist.new } do
+      property :name
+    end
+  end
+
+  let (:song)               { Song.new("Broken") }
+  let (:song_with_composer) { Song.new("Resist Stance", nil, composer) }
+  let (:composer)           { Artist.new("Greg Graffin") }
+  let (:artist)             { Artist.new("Bad Religion") }
+  let (:album)              { Album.new("The Dissent Of Man", [song, song_with_composer], artist) }
+
+  let (:form) { AlbumForm.new(album) }
+
+  it "runs populator on scalar" do
+    form.validate(
+      "name" => "override me!"
+    )
+
+    form.name.must_equal "!em edirrevo"
+  end
+
+  # changing existing property :artist.
+  # TODO: check with artist==nil
+  it do
+    old_id = artist.object_id
+
+    form.validate(
+      "artist" => {"name" => "Marcus Miller"}
+    )
+
+    form.artist.model.object_id.must_equal old_id
+  end
+
+  # use populator for default value on scalars?
+
+  # adding to collection via :populator.
+  # valid.
+  it "yyy" do
+    form.validate(
+      "songs"  => [{"title" => "Fallout"}, {"title" => "Roxanne"},
+        {"title" => "Rime Of The Ancient Mariner"}, # new song.
+        {"title" => "Re-Education", "composer" => {"name" => "Rise Against"}}], # new song with new composer.
+    ).must_equal true
+
+    form.errors.messages.inspect.must_equal "{}"
+
+    # form has updated.
+    form.name.must_equal "The Dissent Of Man"
+    form.songs[0].title.must_equal "Fallout"
+    form.songs[1].title.must_equal "Roxanne"
+    form.songs[1].composer.name.must_equal "Greg Graffin"
+
+    form.songs[1].composer.model.must_be_instance_of Artist
+
+    form.songs[1].title.must_equal "Roxanne"
+    form.songs[2].title.must_equal "Rime Of The Ancient Mariner" # new song added.
+    form.songs[3].title.must_equal "Re-Education"
+    form.songs[3].composer.name.must_equal "Rise Against"
+    form.songs.size.must_equal 4
+    form.artist.name.must_equal "Bad Religion"
+
+
+    # model has not changed, yet.
+    album.name.must_equal "The Dissent Of Man"
+    album.songs[0].title.must_equal "Broken"
+    album.songs[1].title.must_equal "Resist Stance"
+    album.songs[1].composer.name.must_equal "Greg Graffin"
+    album.songs.size.must_equal 2
+    album.artist.name.must_equal "Bad Religion"
+  end
+end
+
+class PopulateWithMethodTest < Minitest::Spec
+  Album = Struct.new(:title)
+
+  class AlbumForm < Reform::Form
+    property :title, populator: :title!
+
+    def title!(options)
+      self.title = options[:fragment].reverse
+    end
+  end
+
+  let (:form) { AlbumForm.new(Album.new) }
+
+  it "runs populator method" do
+    form.validate(
+      "title" => "override me!"
+    )
+
+    form.title.must_equal "!em edirrevo"
+  end
+end
+
+class PopulateIfEmptyTest < MiniTest::Spec
+  Song  = Struct.new(:title, :album, :composer)
+  Album = Struct.new(:name, :songs, :artist)
+  Artist = Struct.new(:name)
+
+  let (:song)               { Song.new("Broken") }
+  let (:song_with_composer) { Song.new("Resist Stance", nil, composer) }
+  let (:composer)           { Artist.new("Greg Graffin") }
+  let (:artist)             { Artist.new("Bad Religion") }
+  let (:album)              { Album.new("The Dissent Of Man", [song, song_with_composer], artist) }
+
+
+
+  class AlbumForm < Reform::Form
+    property :name
+
+    collection :songs,
+      populate_if_empty: Song do                                                # class name works.
+
+      property :title
+      validation do
+        key(:title).required
+      end
+
+      property :composer, populate_if_empty: :populate_composer! do # lambda works, too. in form context.
+        property :name
+        validation do
+          key(:name).required
+        end
+      end
+
+    private
+      def populate_composer!(fragment, options)
+        Artist.new
+      end
+    end
+
+    property :artist, populate_if_empty: lambda { |args| create_artist(args[:fragment], args[:user_options]) } do # methods work, too.
+      property :name
+    end
+
+  private
+    class Sting < Artist
+      attr_accessor :args
+    end
+    def create_artist(input, user_options)
+      Sting.new.tap { |artist| artist.args=([input, user_options].to_s) }
+    end
+  end
+
+  let (:form) { AlbumForm.new(album) }
+
+  it do
+    form.songs.size.must_equal 2
+
+    form.validate(
+      "songs"  => [{"title" => "Fallout"}, {"title" => "Roxanne"},
+        {"title" => "Rime Of The Ancient Mariner"}, # new song.
+        {"title" => "Re-Education", "composer" => {"name" => "Rise Against"}}], # new song with new composer.
+    ).must_equal true
+
+    form.errors.messages.inspect.must_equal "{}"
+
+    # form has updated.
+    form.name.must_equal "The Dissent Of Man"
+    form.songs[0].title.must_equal "Fallout"
+    form.songs[1].title.must_equal "Roxanne"
+    form.songs[1].composer.name.must_equal "Greg Graffin"
+    form.songs[1].title.must_equal "Roxanne"
+    form.songs[2].title.must_equal "Rime Of The Ancient Mariner" # new song added.
+    form.songs[3].title.must_equal "Re-Education"
+    form.songs[3].composer.name.must_equal "Rise Against"
+    form.songs.size.must_equal 4
+    form.artist.name.must_equal "Bad Religion"
+
+
+    # model has not changed, yet.
+    album.name.must_equal "The Dissent Of Man"
+    album.songs[0].title.must_equal "Broken"
+    album.songs[1].title.must_equal "Resist Stance"
+    album.songs[1].composer.name.must_equal "Greg Graffin"
+    album.songs.size.must_equal 2
+    album.artist.name.must_equal "Bad Religion"
+  end
+
+  # trigger artist populator. lambda calling form instance method.
+  it "xxxx" do
+    form = AlbumForm.new(album = Album.new)
+    form.validate("artist" => {"name" => "From Autumn To Ashes"})
+
+    form.artist.name.must_equal "From Autumn To Ashes"
+    # test lambda was executed in form context.
+    form.artist.model.must_be_instance_of AlbumForm::Sting
+    # test lambda block arguments.
+    form.artist.model.args.to_s.must_equal "[{\"name\"=>\"From Autumn To Ashes\"}, nil]"
+
+    album.artist.must_equal nil
+  end
+end
+
+
+# delete songs while deserializing.
+class PopulateIfEmptyWithDeletionTest < MiniTest::Spec
+  Song  = Struct.new(:title, :album, :composer)
+  Album = Struct.new(:name, :songs, :artist)
+
+  let (:song)  { Song.new("Broken") }
+  let (:song2) { Song.new("Resist Stance") }
+  let (:album) { Album.new("The Dissent Of Man", [song, song2]) }
+
+
+  class AlbumForm < Reform::Form
+    property :name
+
+    collection :songs,
+      populate_if_empty: Song, skip_if: :delete_song! do
+
+      property :title
+      validation do
+        key(:title).required
+      end
+    end
+
+    def delete_song!(options)
+      songs.delete(songs[0]) and return true if options[:fragment]["title"] == "Broken, delete me!"
+      false
+    end
+  end
+
+  let (:form) { AlbumForm.new(album) }
+
+  it do
+    form.validate(
+      "songs"  => [{"title" => "Broken, delete me!"}, {"title" => "Roxanne"}]
+    ).must_equal true
+
+    form.errors.messages.inspect.must_equal "{}"
+
+    form.songs.size.must_equal 1
+    form.songs[0].title.must_equal "Roxanne"
+  end
+end

data/test/populator_skip_test.rb

@@ -0,0 +1,28 @@
+require "test_helper"
+
+class PopulatorSkipTest < MiniTest::Spec
+  Album = Struct.new(:songs)
+  Song  = Struct.new(:title)
+
+
+  class AlbumForm < Reform::Form
+    collection :songs,
+      populator: ->(options) {
+        return skip! if options[:fragment][:title] == "Good"
+        songs[options[:index]]
+      } do
+        property :title
+    end
+  end
+
+  it do
+    form = AlbumForm.new(Album.new([Song.new, Song.new]))
+    hash = {songs: [{title: "Good"}, {title: "Bad"}]}
+
+    form.validate(hash)
+
+    form.songs.size.must_equal 2
+    form.songs[0].title.must_equal nil
+    form.songs[1].title.must_equal "Bad"
+  end
+end