representable 3.0.4 → 3.1.1

data/lib/representable/xml/binding.rb

@@ -1,10 +1,8 @@
 require 'representable/binding'
-require 'representable/hash/binding.rb'
 
 module Representable
   module XML
-    module_function
-    def Node(document, name, attributes={})
+    module_function def Node(document, name, attributes={})
       node = Nokogiri::XML::Node.new(name.to_s, document) # Java::OrgW3cDom::DOMException: NAMESPACE_ERR: An attempt is made to create or change an object in a way which is incorrect with regard to namespaces.
 
       attributes.each { |k, v| node[k] = v } # TODO: benchmark.
@@ -18,6 +16,7 @@ module Representable
         return AttributeHash.new(definition)   if definition.hash? and definition[:use_attributes]
         return Attribute.new(definition)       if definition[:attribute]
         return Content.new(definition)         if definition[:content]
+
         new(definition)
       end
 

data/lib/representable/yaml.rb

@@ -1,9 +1,9 @@
 require 'psych'
-require 'representable/hash'
-require 'representable/yaml/binding'
+require 'representable'
 
 module Representable
   module YAML
+    autoload :Binding, 'representable/yaml/binding'
     include Hash
 
     def self.included(base)
@@ -40,7 +40,7 @@ module Representable
       stream.to_yaml
     end
 
-     alias_method :render, :to_yaml
+    alias_method :render, :to_yaml
     alias_method :parse, :from_yaml
   end
 end

data/lib/representable/yaml/binding.rb

@@ -5,6 +5,7 @@ module Representable
     class Binding < Representable::Hash::Binding
       def self.build_for(definition)
         return Collection.new(definition) if definition.array?
+
         new(definition)
       end
 

data/representable.gemspec

@@ -19,16 +19,16 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
   spec.license       = "MIT"
 
-  spec.required_ruby_version = '>= 1.9.3'
+  spec.required_ruby_version = '>= 2.4.0'
 
   spec.add_dependency "uber",               "< 0.2.0"
   spec.add_dependency "declarative",        "< 0.1.0"
-  spec.add_dependency "declarative-option", "< 0.2.0"
+  spec.add_dependency "trailblazer-option", ">= 0.1.1", "< 0.2.0"
 
   spec.add_development_dependency "rake"
   spec.add_development_dependency "test_xml", ">= 0.1.6"
   spec.add_development_dependency "minitest"
   spec.add_development_dependency "virtus"
-  spec.add_development_dependency "multi_json"
+  spec.add_development_dependency "dry-types"
   spec.add_development_dependency "ruby-prof" if RUBY_ENGINE == "ruby" # mri
 end

data/test/as_test.rb

@@ -17,7 +17,7 @@ class AsTest < MiniTest::Spec
       end
 
       it { render(song).must_equal_document output }
-      it { parse(song, input).name.must_equal "Wie Es Geht" }
+      it { _(parse(song, input).name).must_equal "Wie Es Geht" }
     end
 
 
@@ -27,7 +27,7 @@ class AsTest < MiniTest::Spec
       end
 
       it { render(song).must_equal_document({"Song" => "Revolution"}) }
-      it { parse(song, {"Song" => "Wie Es Geht"}).name.must_equal "Wie Es Geht" }
+      it { _(parse(song, {"Song" => "Wie Es Geht"}).name).must_equal "Wie Es Geht" }
     end
 
 
@@ -37,7 +37,7 @@ class AsTest < MiniTest::Spec
       end
 
       it { render(song, user_options:{volume: 1}).must_equal_document({"{:volume=>1}" => "Revolution"}) }
-      it { parse(song, {"{:volume=>1}" => "Wie Es Geht"}, user_options: {volume: 1}).name.must_equal "Wie Es Geht" }
+      it { _(parse(song, {"{:volume=>1}" => "Wie Es Geht"}, user_options: {volume: 1}).name).must_equal "Wie Es Geht" }
     end
   end
 end
@@ -60,6 +60,6 @@ class AsXmlTest < MiniTest::Spec
 
   it do
     skip
-    representer.new(Album.new(Band.new("Offspring"))).to_xml.must_equal ""
+    _(representer.new(Album.new(Band.new("Offspring"))).to_xml).must_equal ""
   end
 end

data/test/binding_test.rb

@@ -8,16 +8,16 @@ class BindingTest < MiniTest::Spec
     let(:binding) { Binding.new(render_nil_definition) }
 
     # don't skip when present.
-    it { binding.skipable_empty_value?("Disconnect, Disconnect").must_equal false }
+    it { _(binding.skipable_empty_value?("Disconnect, Disconnect")).must_equal false }
 
     # don't skip when it's nil and render_nil: true
-    it { binding.skipable_empty_value?(nil).must_equal false }
+    it { _(binding.skipable_empty_value?(nil)).must_equal false }
 
     # skip when nil and :render_nil undefined.
-    it { Binding.new(Representable::Definition.new(:song)).skipable_empty_value?(nil).must_equal true }
+    it { _(Binding.new(Representable::Definition.new(:song)).skipable_empty_value?(nil)).must_equal true }
 
     # don't skip when nil and :render_nil undefined.
-    it { Binding.new(Representable::Definition.new(:song)).skipable_empty_value?("Fatal Flu").must_equal false }
+    it { _(Binding.new(Representable::Definition.new(:song)).skipable_empty_value?("Fatal Flu")).must_equal false }
   end
 
 
@@ -26,21 +26,21 @@ class BindingTest < MiniTest::Spec
     let(:binding) { Binding.new(definition) }
 
     # return value when value present.
-    it { binding.default_for("Black And Blue").must_equal "Black And Blue" }
+    it { _(binding.default_for("Black And Blue")).must_equal "Black And Blue" }
 
     # return false when value false.
-    it { binding.default_for(false).must_equal false }
+    it { _(binding.default_for(false)).must_equal false }
 
     # return default when value nil.
-    it { binding.default_for(nil).must_equal "Insider" }
+    it { _(binding.default_for(nil)).must_equal "Insider" }
 
     # return nil when value nil and render_nil: true.
-    it { Binding.new(render_nil_definition).default_for(nil).must_be_nil }
+    it { _(Binding.new(render_nil_definition).default_for(nil)).must_be_nil }
 
     # return nil when value nil and render_nil: true, even when :default is set" do
-    it { Binding.new(Representable::Definition.new(:song, :render_nil => true, :default => "The Quest")).default_for(nil).must_be_nil }
+    it { _(Binding.new(Representable::Definition.new(:song, :render_nil => true, :default => "The Quest")).default_for(nil)).must_be_nil }
 
     # return nil if no :default
-    it { Binding.new(Representable::Definition.new(:song)).default_for(nil).must_be_nil }
+    it { _(Binding.new(Representable::Definition.new(:song)).default_for(nil)).must_be_nil }
   end
 end

data/test/cached_test.rb

@@ -64,7 +64,7 @@ class CachedTest < MiniTest::Spec
       # album2 = Model::Album.new("Louder And Even More Dangerous", [song2, song])
 
       # makes sure options are passed correctly.
-      representer.to_hash(user_options: {volume: 9}).must_equal({"name"=>"Live And Dangerous",
+      _(representer.to_hash(user_options: {volume: 9})).must_equal({"name"=>"Live And Dangerous",
         "songs"=>[{"title"=>"Jailbreak:{:volume=>9}"}, {"title"=>"Southbound:{:volume=>9}"}, {"title"=>"Emerald:{:volume=>9}"}]}) # called in Deserializer/Serializer
 
       # representer becomes reusable as it is stateless.
@@ -81,22 +81,22 @@ class CachedTest < MiniTest::Spec
       data = Profiler.profile { representer.to_hash }
 
       # 3 songs get decorated.
-      data.must_match(/3\s*Representable::Function::Decorate#call/m)
+      _(data).must_match(/3\s*Representable::Function::Decorate#call/m)
       # These weird Regexp bellow are a quick workaround to accomodate
       # the different profiler result formats.
       #   - "3   <Class::Representable::Decorator>#prepare" -> At MRI Ruby
       #   - "3  Representable::Decorator.prepare"           -> At JRuby
 
       # 3 nested decorator is instantiated for 3 Songs, though.
-      data.must_match(/3\s*(<Class::)?Representable::Decorator\>?[\#.]prepare/m)
+      _(data).must_match(/3\s*(<Class::)?Representable::Decorator\>?[\#.]prepare/m)
       # no Binding is instantiated at runtime.
-      data.wont_match "Representable::Binding#initialize"
+      _(data).wont_match "Representable::Binding#initialize"
       # 2 mappers for Album, Song
       # data.must_match "2   Representable::Mapper::Methods#initialize"
       # title, songs, 3x title, composer
-      data.must_match(/8\s*Representable::Binding[#\.]render_pipeline/m)
-      data.wont_match "render_functions"
-      data.wont_match "Representable::Binding::Factories#render_functions"
+      _(data).must_match(/8\s*Representable::Binding[#\.]render_pipeline/m)
+      _(data).wont_match "render_functions"
+      _(data).wont_match "Representable::Binding::Factories#render_functions"
     end
   end
 
@@ -118,14 +118,14 @@ class CachedTest < MiniTest::Spec
 
       AlbumRepresenter.new(album).from_hash(album_hash)
 
-      album.songs.size.must_equal 3
-      album.name.must_equal "Louder And Even More Dangerous"
-      album.songs[0].title.must_equal "Southbound"
-      album.songs[0].composer.name.must_equal "Lynott"
-      album.songs[1].title.must_equal "Jailbreak"
-      album.songs[1].composer.name.must_equal "Phil Lynott"
-      album.songs[2].title.must_equal "Emerald"
-      album.songs[2].composer.must_be_nil
+      _(album.songs.size).must_equal 3
+      _(album.name).must_equal "Louder And Even More Dangerous"
+      _(album.songs[0].title).must_equal "Southbound"
+      _(album.songs[0].composer.name).must_equal "Lynott"
+      _(album.songs[1].title).must_equal "Jailbreak"
+      _(album.songs[1].composer.name).must_equal "Phil Lynott"
+      _(album.songs[2].title).must_equal "Emerald"
+      _(album.songs[2].composer).must_be_nil
 
       # TODO: test options.
     end
@@ -139,12 +139,12 @@ class CachedTest < MiniTest::Spec
       # only 2 nested decorators are instantiated, Song, and Artist.
       # Didn't like the regexp?
       # MRI and JRuby has different output formats. See note above.
-      data.must_match(/5\s*(<Class::)?Representable::Decorator>?[#\.]prepare/)
+      _(data).must_match(/5\s*(<Class::)?Representable::Decorator>?[#\.]prepare/)
       # a total of 5 properties in the object graph.
-      data.wont_match "Representable::Binding#initialize"
+      _(data).wont_match "Representable::Binding#initialize"
 
-      data.wont_match "parse_functions" # no pipeline creation.
-      data.must_match(/10\s*Representable::Binding[#\.]parse_pipeline/)
+      _(data).wont_match "parse_functions" # no pipeline creation.
+      _(data).must_match(/10\s*Representable::Binding[#\.]parse_pipeline/)
       # three mappers for Album, Song, composer
       # data.must_match "3   Representable::Mapper::Methods#initialize"
       # # 6 deserializers as the songs collection uses 2.

data/test/class_test.rb

@@ -20,8 +20,8 @@ class ClassTest < BaseTest
 
     it "creates fresh instance and doesn't extend" do
       song = representer.prepare(OpenStruct.new).from_hash({"song" => {"__name__" => "Captured"}}).song
-      song.must_be_instance_of RepresentingSong
-      song.name.must_equal "Captured"
+      _(song).must_be_instance_of RepresentingSong
+      _(song.name).must_equal "Captured"
     end
   end
 
@@ -33,8 +33,8 @@ class ClassTest < BaseTest
 
     it "creates fresh instance and doesn't extend" do
       song = representer.prepare(OpenStruct.new).from_hash({"song" => {"__name__" => "Captured"}}).song
-      song.must_be_instance_of RepresentingSong
-      song.name.must_equal "Captured"
+      _(song).must_be_instance_of RepresentingSong
+      _(song.name).must_equal "Captured"
     end
   end
 
@@ -69,7 +69,7 @@ class ClassTest < BaseTest
       property :song, :class => lambda { |options| _klass.args=([options[:fragment],options[:user_options]]); _klass }
     end
 
-    it { representer.prepare(OpenStruct.new).from_hash({"song" => {"name" => "Captured"}}, user_options: {volume: true}).song.class.args.
+    it { _(representer.prepare(OpenStruct.new).from_hash({"song" => {"name" => "Captured"}}, user_options: {volume: true}).song.class.args).
       must_equal([{"name"=>"Captured"}, {:volume=>true}]) }
   end
 
@@ -90,7 +90,7 @@ class ClassTest < BaseTest
       collection :songs, :class => lambda { |options| _klass.args=([options[:fragment],options[:index],options[:user_options]]); _klass }
     end
 
-    it { representer.prepare(OpenStruct.new).from_hash({"songs" => [{"name" => "Captured"}]}, user_options: {volume: true}).songs.first.class.args.
+    it { _(representer.prepare(OpenStruct.new).from_hash({"songs" => [{"name" => "Captured"}]}, user_options: {volume: true}).songs.first.class.args).
       must_equal([{"name"=>"Captured"}, 0, {:volume=>true}]) }
   end
 
@@ -109,7 +109,7 @@ class ClassTest < BaseTest
     end
 
     it "allows returning arbitrary objects in #from_hash" do
-      representer.prepare(OpenStruct.new).from_hash({"song" => 1}).song.must_equal [1,2,3,4]
+      _(representer.prepare(OpenStruct.new).from_hash({"song" => 1}).song).must_equal [1,2,3,4]
     end
   end
 end

data/test/coercion_test.rb

@@ -1,52 +1,63 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 require 'representable/coercion'
 
-class VirtusCoercionTest < MiniTest::Spec
+class CoercionTest < MiniTest::Spec
   representer! do
     include Representable::Coercion
 
     property :title # no coercion.
-    property :length, :type => Float
+    property :length, type: Representable::Coercion::Types::Params::Float
 
-    property :band, :class => OpenStruct do
-      property :founded, :type => Integer
+    property :band, class: OpenStruct do
+      property :founded, type: Representable::Coercion::Types::Params::Integer
     end
 
-    collection :songs, :class => OpenStruct do
-      property :ok, :type => Virtus::Attribute::Boolean
+    collection :songs, class: OpenStruct do
+      property :ok, type: Representable::Coercion::Types::Params::Bool
     end
   end
 
-  let(:album) { OpenStruct.new(:title => "Dire Straits", :length => 41.34,
-    :band  => OpenStruct.new(:founded => "1977"),
-    :songs => [OpenStruct.new(:ok => 1), OpenStruct.new(:ok => 0)]) }
+  let(:album) do
+    OpenStruct.new(title: 'Dire Straits', length: 41.34,
+                   band: OpenStruct.new(founded: '1977'),
+                   songs: [OpenStruct.new(ok: 1), OpenStruct.new(ok: 0)])
+  end
 
-  it { album.extend(representer).to_hash.must_equal({"title"=>"Dire Straits", "length"=>41.34, "band"=>{"founded"=>1977}, "songs"=>[{"ok"=>true}, {"ok"=>false}]}) }
+  it do
+    _(album.extend(representer).to_hash).must_equal({ 'title' => 'Dire Straits',
+                                                      'length' => 41.34,
+                                                      'band' => { 'founded' => 1977 },
+                                                      'songs' => [{ 'ok' => true }, { 'ok' => false }] })
+  end
 
   it {
     album = OpenStruct.new
     album.extend(representer)
-    album.from_hash({"title"=>"Dire Straits", "length"=>"41.34", "band"=>{"founded"=>"1977"}, "songs"=>[{"ok"=>1}, {"ok"=>0}]})
+    album.from_hash({ 'title' => 'Dire Straits',
+                      'length' => '41.34',
+                      'band' => { 'founded' => '1977' },
+                      'songs' => [{ 'ok' => 1 }, { 'ok' => 0 }] })
 
     # it
-    album.length.must_equal 41.34
-    album.band.founded.must_equal 1977
-    album.songs[0].ok.must_equal true
+    _(album.length).must_equal 41.34
+    _(album.band.founded).must_equal 1977
+    _(album.songs[0].ok).must_equal true
   }
 
-
-  describe "with user :parse_filter and :render_filter" do
+  describe 'with user :parse_filter and :render_filter' do
     representer! do
       include Representable::Coercion
 
-      property :length, :type => Float,
-      :parse_filter  => lambda { |input, options| "#{input}.1" }, # happens BEFORE coercer.
-      :render_filter => lambda { |fragment,*| "#{fragment}.1" }
+      property :length, type: Representable::Coercion::Types::Params::Float,
+                        parse_filter: ->(input, _options) { "#{input}.1" }, # happens BEFORE coercer.
+                        render_filter: ->(fragment, *) { "#{fragment}.1" }
     end
 
     # user's :parse_filter(s) are run before coercion.
-    it { OpenStruct.new.extend(representer).from_hash("length"=>"1").length.must_equal 1.1 }
+    it { _(OpenStruct.new.extend(representer).from_hash('length' => '1').length).must_equal 1.1 }
     # user's :render_filter(s) are run before coercion.
-    it { OpenStruct.new(:length=>1).extend(representer).to_hash.must_equal({"length" => 1.1}) }
+    it { _(OpenStruct.new(length: 1).extend(representer).to_hash).must_equal({ 'length' => 1.1 }) }
   end
-end
+end

data/test/config/inherit_test.rb

@@ -9,8 +9,8 @@ class ConfigInheritTest < MiniTest::Spec
 
     child_def.merge!(:alias => property)
 
-    child_def[:alias].wont_equal parent_def[:alias]
-    child_def.object_id.wont_equal parent_def.object_id
+    _(child_def[:alias]).wont_equal parent_def[:alias]
+    _(child_def.object_id).wont_equal parent_def.object_id
   end
   # class Object
 
@@ -30,7 +30,7 @@ class ConfigInheritTest < MiniTest::Spec
     end
   end
 
-  it { Decorator.definitions.keys.must_equal ["title", "artist"] }
+  it { _(Decorator.definitions.keys).must_equal ["title", "artist"] }
 
   # in inheriting Decorator
 
@@ -38,10 +38,10 @@ class ConfigInheritTest < MiniTest::Spec
     property :location
   end
 
-  it { InheritingDecorator.definitions.keys.must_equal ["title", "artist", "location"] }
+  it { _(InheritingDecorator.definitions.keys).must_equal ["title", "artist", "location"] }
   it { assert_cloned(InheritingDecorator, Decorator, "title") }
   it do
-    InheritingDecorator.representable_attrs.get(:artist).representer_module.object_id.wont_equal Decorator.representable_attrs.get(:artist).representer_module.object_id
+    _(InheritingDecorator.representable_attrs.get(:artist).representer_module.object_id).wont_equal Decorator.representable_attrs.get(:artist).representer_module.object_id
   end
 
   # in inheriting and including Decorator
@@ -51,7 +51,7 @@ class ConfigInheritTest < MiniTest::Spec
     property :location
   end
 
-  it { InheritingAndIncludingDecorator.definitions.keys.must_equal ["title", "artist", "genre", "location"] }
+  it { _(InheritingAndIncludingDecorator.definitions.keys).must_equal ["title", "artist", "genre", "location"] }
   it { assert_cloned(InheritingAndIncludingDecorator, GenreModule, :genre) }
 
 
@@ -61,7 +61,7 @@ class ConfigInheritTest < MiniTest::Spec
     property :title
   end
 
-  it { Module.definitions.keys.must_equal ["title"] }
+  it { _(Module.definitions.keys).must_equal ["title"] }
 
 
   # in module including module
@@ -72,7 +72,7 @@ class ConfigInheritTest < MiniTest::Spec
     property :location
   end
 
-  it { SubModule.definitions.keys.must_equal ["title", "location"] }
+  it { _(SubModule.definitions.keys).must_equal ["title", "location"] }
   it { assert_cloned(SubModule, Module, :title) }
 
   # including preserves order
@@ -84,7 +84,7 @@ class ConfigInheritTest < MiniTest::Spec
     property :location
   end
 
-  it { IncludingModule.definitions.keys.must_equal ["genre", "title", "location"] }
+  it { _(IncludingModule.definitions.keys).must_equal ["genre", "title", "location"] }
 
 
   # included in class -------------------------------------------
@@ -93,7 +93,7 @@ class ConfigInheritTest < MiniTest::Spec
     include IncludingModule
   end
 
-  it { Class.definitions.keys.must_equal ["genre", "title", "location"] }
+  it { _(Class.definitions.keys).must_equal ["genre", "title", "location"] }
   it { assert_cloned(Class, IncludingModule, :title) }
   it { assert_cloned(Class, IncludingModule, :location) }
   it { assert_cloned(Class, IncludingModule, :genre) }
@@ -105,7 +105,7 @@ class ConfigInheritTest < MiniTest::Spec
     include IncludingModule
   end
 
-  it { DefiningClass.definitions.keys.must_equal ["street_cred", "genre", "title", "location"] }
+  it { _(DefiningClass.definitions.keys).must_equal ["street_cred", "genre", "title", "location"] }
 
   # in class
   class RepresenterClass
@@ -113,7 +113,7 @@ class ConfigInheritTest < MiniTest::Spec
     property :title
   end
 
-  it { RepresenterClass.definitions.keys.must_equal ["title"] }
+  it { _(RepresenterClass.definitions.keys).must_equal ["title"] }
 
 
   # in inheriting class
@@ -122,7 +122,7 @@ class ConfigInheritTest < MiniTest::Spec
     property :location
   end
 
-  it { InheritingClass.definitions.keys.must_equal ["title", "location"] }
+  it { _(InheritingClass.definitions.keys).must_equal ["title", "location"] }
   it { assert_cloned(InheritingClass, RepresenterClass, :title) }
 
   # in inheriting class and including
@@ -131,5 +131,5 @@ class ConfigInheritTest < MiniTest::Spec
     include GenreModule
   end
 
-  it { InheritingAndIncludingClass.definitions.keys.must_equal ["title", "location", "genre"] }
+  it { _(InheritingAndIncludingClass.definitions.keys).must_equal ["title", "location", "genre"] }
 end