representable 1.2.8 → 1.2.9

data/.travis.yml

@@ -2,7 +2,7 @@ notifications:
   irc: "irc.freenode.org#cells"
 matrix:
   include:
-    - rvm: 1.8.7
+    - rvm: 1.9.3
       gemfile: gemfiles/Gemfile.mongoid-2.4
     - rvm: 1.9.3
       gemfile: Gemfile

data/CHANGES.textile

@@ -1,3 +1,9 @@
+h2. 1.2.9
+
+* When @:class@ returns @nil@ we no longer try to create a new instance but use the processed fragment itself.
+* @:instance@ allows overriding the @ObjectBinding#create_object@ workflow by returning an instance from the lambda. This is particularly helpful when you need to inject additional data into the property object created in #deserialize.
+* @:extend@ and @:class@ now also accept procs which allows having polymorphic properties and collections where representer and class can be chosen at runtime.
+
 h2. 1.2.8
 
 * Reverting all the bullshit from 1.2.7 making it even better. @Binding@s now wrap their @Definition@ instance adopting its API. Moved the binding_for_definition mechanics to the respecting @Binding@ subclass.

data/TODO

@@ -0,0 +1,7 @@
+* Pass key/index as first block arg to :class and :extendv
+class: |key, hsh|
+
+* Allow passing options to Binding#serialize.
+	serialize(.., options{:exclude => ..})
+
+* Allow :writer and :reader / :getter/:setter

data/gemfiles/Gemfile.mongoid-2.4

@@ -3,5 +3,4 @@ source "http://rubygems.org"
 # Specify your gem's dependencies in roar-rails.gemspec
 gemspec :path => '../'
 
-gem 'mongoid', '~> 2.4.0'
-gem 'psych'
+gem 'mongoid', '~> 2.4.0'

data/lib/representable.rb

@@ -119,7 +119,7 @@ private
   end
   
   def representable_bindings_for(format)
-    representable_attrs.map {|attr| format.build_for(attr) }
+    representable_attrs.map {|attr| format.build_for(attr, self) }
   end
   
   # Returns the wrapper for the representation. Mostly used in XML.

data/lib/representable/binding.rb

@@ -9,6 +9,11 @@ module Representable
     def definition  # TODO: remove in 1.4.
       raise "Binding#definition is no longer supported as all Definition methods are now delegated automatically."
     end
+
+    def initialize(definition, represented)
+      super(definition)
+      @represented = represented
+    end
     
     # Main entry point for rendering/parsing a property object.
     def serialize(value)
@@ -24,7 +29,7 @@ module Representable
     # at runtime.
     module Extend
       # Extends the object with its representer before serialization.
-      def serialize(object)
+      def serialize(*)
         extend_for(super)
       end
       
@@ -33,12 +38,22 @@ module Representable
       end
       
       def extend_for(object)
-        if mod = representer_module
+        if mod = representer_module_for(object) # :extend.
           object.extend(*mod)
         end
-        
+
         object
       end
+    
+    private
+      def representer_module_for(object, *args)
+        call_proc_for(representer_module, object)   # TODO: how to pass additional data to the computing block?`
+      end
+
+      def call_proc_for(proc, *args)
+        return proc unless proc.is_a?(Proc)
+        @represented.instance_exec(*args, &proc)
+      end
     end
     
     module Object
@@ -47,18 +62,32 @@ module Representable
       def serialize(object)
         return object if object.nil?
         
-        super(object).send(serialize_method, :wrap => false)
+        super.send(serialize_method, :wrap => false)  # TODO: pass :binding => self
       end
       
-      def deserialize(data) 
+      def deserialize(data)
         # DISCUSS: does it make sense to skip deserialization of nil-values here?
-        super(create_object).send(deserialize_method, data)
+        super(create_object(data)).send(deserialize_method, data)
       end
       
-      def create_object
-        sought_type.new
+      def create_object(fragment)
+        instance_for(fragment) or class_for(fragment)
+      end
+
+    private
+      def class_for(fragment, *args)
+        item_class = class_from(fragment) or return fragment # DISCUSS: is it legal to return the very fragment here?
+        item_class.new
+      end
+
+      def class_from(fragment, *args)
+        call_proc_for(sought_type, fragment)
+      end
+
+      def instance_for(fragment, *args)
+        return unless options[:instance] 
+        call_proc_for(options[:instance], fragment)
       end
     end
-    
   end
 end

data/lib/representable/bindings/hash_bindings.rb

@@ -16,10 +16,10 @@ module Representable
     
     
     class PropertyBinding < Representable::Binding
-      def self.build_for(definition)
-        return CollectionBinding.new(definition)  if definition.array?
-        return HashBinding.new(definition)        if definition.hash?
-        new(definition)
+      def self.build_for(definition, represented)
+        return CollectionBinding.new(definition, represented)  if definition.array?
+        return HashBinding.new(definition, represented)        if definition.hash?
+        new(definition, represented)
       end
 
       def initialize(*args) # FIXME. make generic.
@@ -50,6 +50,7 @@ module Representable
     
     class CollectionBinding < PropertyBinding
       def serialize_for(value)
+        # value.enum_for(:each_with_index).collect { |obj, i| serialize(obj, i) } # DISCUSS: provide ary index/hash key for representer_module_for?
         value.collect { |obj| serialize(obj) }
       end
       

data/lib/representable/bindings/xml_bindings.rb

@@ -24,12 +24,12 @@ module Representable
     
     
     class PropertyBinding < Binding
-      def self.build_for(definition)
-        return CollectionBinding.new(definition)      if definition.array?
-        return HashBinding.new(definition)            if definition.hash? and not definition.options[:use_attributes] # FIXME: hate this.
-        return AttributeHashBinding.new(definition)   if definition.hash? and definition.options[:use_attributes]
-        return AttributeBinding.new(definition)       if definition.attribute
-        new(definition)
+      def self.build_for(definition, represented)
+        return CollectionBinding.new(definition, represented)      if definition.array?
+        return HashBinding.new(definition, represented)            if definition.hash? and not definition.options[:use_attributes] # FIXME: hate this.
+        return AttributeHashBinding.new(definition, represented)   if definition.hash? and definition.options[:use_attributes]
+        return AttributeBinding.new(definition, represented)       if definition.attribute
+        new(definition, represented)
       end
 
       def initialize(*args)

data/lib/representable/bindings/yaml_bindings.rb

@@ -19,12 +19,12 @@ module Representable
     end
 
     class PropertyBinding < Representable::Hash::PropertyBinding
-      def self.build_for(definition)
-        return CollectionBinding.new(definition)      if definition.array?
+      def self.build_for(definition, represented)
+        return CollectionBinding.new(definition, represented)      if definition.array?
         #return HashBinding.new(definition)            if definition.hash? and not definition.options[:use_attributes] # FIXME: hate this.
         #return AttributeHashBinding.new(definition)   if definition.hash? and definition.options[:use_attributes]
         #return AttributeBinding.new(definition)       if definition.attribute
-        new(definition)
+        new(definition, represented)
       end
 
       def initialize(*args) # FIXME. make generic.

data/lib/representable/definition.rb

@@ -20,7 +20,7 @@ module Representable
     end
     
     def typed?
-      sought_type.is_a?(Class) or representer_module  # also true if only :extend is set, for people who want solely rendering.
+      sought_type.is_a?(Class) or representer_module or options[:instance]  # also true if only :extend is set, for people who want solely rendering.
     end
     
     def array?

data/lib/representable/json/hash.rb

@@ -1,9 +1,10 @@
+require 'representable/json'
 require 'representable/hash_methods'
 
 module Representable::JSON
   module Hash
     include Representable::JSON
-    include HashMethods
+    include Representable::HashMethods
 
     def self.included(base)
       base.class_eval do

data/lib/representable/readable_writeable.rb

@@ -0,0 +1,29 @@
+module Representable
+  module Feature
+    module ReadableWriteable
+      def deserialize_property(binding, doc, options)
+        return unless binding.writeable?
+        super
+      end
+
+      def serialize_property(binding, doc, options)
+        return unless binding.readable?
+        super
+      end
+    end
+  end
+
+  # TODO: i hate monkey-patching Definition here since it globally adds this options. However, for now this should be ok :-)
+  class Definition
+    # TODO: make this generic like `option :writeable, :default => true`
+    def writeable?
+      return options[:writeable] if options.has_key?(:writeable)
+      true
+    end
+
+    def readable?
+      return options[:readable] if options.has_key?(:readable)
+      true
+    end
+  end
+end

data/lib/representable/version.rb

@@ -1,3 +1,3 @@
 module Representable
-  VERSION = "1.2.8"
+  VERSION = "1.2.9"
 end

data/lib/representable/xml/hash.rb

@@ -1,3 +1,4 @@
+require 'representable/xml'
 require 'representable/hash_methods'
 
 module Representable::XML

data/test/coercion_test.rb

@@ -35,6 +35,8 @@ class VirtusCoercionTest < MiniTest::Spec
       end
       
       it "coerces into the provided type" do
+        skip "Virtus is still not correctly treating coercion on class level?"
+        
         song = ImmigrantSong.new.from_json("{\"composed_at\":\"November 18th, 1983\",\"track\":\"18\"}")
         assert_equal DateTime.parse("Fri, 18 Nov 1983 00:00:00 +0000"), song.composed_at
         assert_equal 18, song.track

data/test/definition_test.rb

@@ -25,6 +25,10 @@ class DefinitionTest < MiniTest::Spec
       it "is true when :extend is present, only" do
         assert Representable::Definition.new(:songs, :extend => Hash).typed?
       end
+
+      it "is true when :instance is present, only" do
+        assert Representable::Definition.new(:songs, :instance => Object.new).typed?
+      end
     end
     
     it "responds to #getter and returns string" do

data/test/example.rb

@@ -59,6 +59,11 @@ puts song.extend(SongRepresenter).to_json
 ######### nesting types
 
 class Album < OpenStruct
+  def name
+    puts @table.inspect
+    #@attributes
+    @table[:name]
+  end
 end
 
 module AlbumRepresenter
@@ -88,6 +93,20 @@ SongRepresenter.module_eval do
 end
 
 
+######### using helpers (customizing the rendering/parsing) 
+module AlbumRepresenter
+  def name
+    super.upper
+  end
+end
+album = Album.new(:name => "The Police", :songs => [song, Song.new(:title => "Synchronicity")])
+puts album.extend(AlbumRepresenter).to_json
+
+SongRepresenter.module_eval do
+  @representable_attrs = nil
+end
+
+
 ######### inheritance
 module SongRepresenter
   include Representable::JSON
@@ -156,4 +175,5 @@ puts song.extend(SongRepresenter).to_yaml
 
 ######### custom methods in representer (using helpers)
 ######### r/w, conditions
-#########
+#########
+######### polymorphic :extend and :class, instance context!, :instance

data/test/hash_bindings_test.rb

@@ -15,7 +15,7 @@ class HashBindingTest < MiniTest::Spec
   describe "PropertyBinding" do
     describe "#read" do
       before do
-        @property = Representable::Hash::PropertyBinding.new(Representable::Definition.new(:song))
+        @property = Representable::Hash::PropertyBinding.new(Representable::Definition.new(:song), nil)
       end
       
       it "returns fragment if present" do
@@ -32,7 +32,7 @@ class HashBindingTest < MiniTest::Spec
     
     describe "with plain text" do
       before do
-        @property = Representable::Hash::PropertyBinding.new(Representable::Definition.new(:song))
+        @property = Representable::Hash::PropertyBinding.new(Representable::Definition.new(:song), nil)
       end
       
       it "extracts with #read" do
@@ -48,7 +48,7 @@ class HashBindingTest < MiniTest::Spec
     
     describe "with an object" do
       before do
-        @property = Representable::Hash::PropertyBinding.new(Representable::Definition.new(:song, :class => SongWithRepresenter))
+        @property = Representable::Hash::PropertyBinding.new(Representable::Definition.new(:song, :class => SongWithRepresenter), nil)
         @doc      = {}
       end
       
@@ -64,7 +64,7 @@ class HashBindingTest < MiniTest::Spec
     
     describe "with an object and :extend" do
       before do
-        @property = Representable::Hash::PropertyBinding.new(Representable::Definition.new(:song, :class => Song, :extend => SongRepresenter))
+        @property = Representable::Hash::PropertyBinding.new(Representable::Definition.new(:song, :class => Song, :extend => SongRepresenter), nil)
         @doc      = {}
       end
       
@@ -83,7 +83,7 @@ class HashBindingTest < MiniTest::Spec
   describe "CollectionBinding" do
     describe "with plain text items" do
       before do
-        @property = Representable::Hash::CollectionBinding.new(Representable::Definition.new(:songs, :collection => true))
+        @property = Representable::Hash::CollectionBinding.new(Representable::Definition.new(:songs, :collection => true), nil)
       end
       
       it "extracts with #read" do
@@ -104,7 +104,7 @@ class HashBindingTest < MiniTest::Spec
   describe "HashBinding" do
     describe "with plain text items" do
       before do
-        @property = Representable::Hash::HashBinding.new(Representable::Definition.new(:songs, :hash => true))
+        @property = Representable::Hash::HashBinding.new(Representable::Definition.new(:songs, :hash => true), nil)
       end
       
       it "extracts with #read" do
@@ -120,7 +120,7 @@ class HashBindingTest < MiniTest::Spec
     
     describe "with objects" do
       before do
-        @property = Representable::Hash::HashBinding.new(Representable::Definition.new(:songs, :hash => true, :class => Song, :extend => SongRepresenter))
+        @property = Representable::Hash::HashBinding.new(Representable::Definition.new(:songs, :hash => true, :class => Song, :extend => SongRepresenter), nil)
       end
       
       it "doesn't change the represented hash in #write" do

data/test/json_test.rb

@@ -128,19 +128,19 @@ module JsonTest
 
       describe "#build_for" do
         it "returns ObjectBinding" do
-          assert_kind_of Representable::Hash::ObjectBinding, Representable::Hash::PropertyBinding.build_for(Def.new(:band, :class => Hash))
+          assert_kind_of Representable::Hash::ObjectBinding, Representable::Hash::PropertyBinding.build_for(Def.new(:band, :class => Hash), nil)
         end
 
         it "returns TextBinding" do
-          assert_kind_of Representable::Hash::PropertyBinding, Representable::Hash::PropertyBinding.build_for(Def.new(:band))
+          assert_kind_of Representable::Hash::PropertyBinding, Representable::Hash::PropertyBinding.build_for(Def.new(:band), nil)
         end
 
         it "returns HashBinding" do
-          assert_kind_of Representable::Hash::HashBinding, Representable::Hash::PropertyBinding.build_for(Def.new(:band, :hash => true))
+          assert_kind_of Representable::Hash::HashBinding, Representable::Hash::PropertyBinding.build_for(Def.new(:band, :hash => true), nil)
         end
 
         it "returns CollectionBinding" do
-          assert_kind_of Representable::Hash::CollectionBinding, Representable::Hash::PropertyBinding.build_for(Def.new(:band, :collection => true))
+          assert_kind_of Representable::Hash::CollectionBinding, Representable::Hash::PropertyBinding.build_for(Def.new(:band, :collection => true), nil)
         end
       end
 

data/test/representable_test.rb

@@ -1,6 +1,15 @@
 require 'test_helper'
 
 class RepresentableTest < MiniTest::Spec
+  def self.representer!(name=:representer, &block)
+    let(name) do
+      Module.new do
+        include Representable::Hash
+        instance_exec(&block)
+      end
+    end
+  end
+
   class Band
     include Representable
     property :name
@@ -403,6 +412,122 @@ class RepresentableTest < MiniTest::Spec
       assert_equal "oh yes", band.fame
     end
   end
+
+  describe ":extend and :class" do
+    module UpcaseRepresenter
+      def to_hash(*); upcase; end
+      def from_hash(hsh); self.class.new hsh.upcase; end   # DISCUSS: from_hash must return self.
+    end
+    module DowncaseRepresenter 
+      def to_hash(*); downcase; end
+      def from_hash(hsh); hsh.downcase; end
+    end
+    class UpcaseString < String; end
+    
+
+    describe "lambda blocks" do
+      representer! do
+        property :name, :extend => lambda { |name| compute_representer(name) }
+      end
+
+      it "executes lambda in represented instance context" do
+        Song.new("Carnage").instance_eval do
+          def compute_representer(name)
+            UpcaseRepresenter
+          end
+          self
+        end.extend(representer).to_hash.must_equal({"name" => "CARNAGE"})
+      end
+    end
+
+    describe ":instance" do
+      obj = String.new("Fate")
+      mod = Module.new { def from_hash(*); self; end }
+      representer! do
+        property :name, :extend => mod, :instance => lambda { |name| obj }
+      end
+
+      it "uses object from :instance but still extends it" do
+        song = Song.new.extend(representer).from_hash("name" => "Eric's Had A Bad Day")
+        song.name.must_equal obj
+        song.name.must_be_kind_of mod
+      end
+    end
+
+    describe "property with :extend" do
+      representer! do
+        property :name, :extend => lambda { |name| name.is_a?(UpcaseString) ? UpcaseRepresenter : DowncaseRepresenter }, :class => String
+      end
+
+      it "uses lambda when rendering" do
+        assert_equal({"name" => "you make me thick"}, Song.new("You Make Me Thick").extend(representer).to_hash )
+        assert_equal({"name" => "STEPSTRANGER"}, Song.new(UpcaseString.new "Stepstranger").extend(representer).to_hash )
+      end
+
+      it "uses lambda when parsing" do
+        Song.new.extend(representer).from_hash({"name" => "You Make Me Thick"}).name.must_equal "you make me thick"
+        Song.new.extend(representer).from_hash({"name" => "Stepstranger"}).name.must_equal "stepstranger" # DISCUSS: we compare "".is_a?(UpcaseString)
+      end
+
+      describe "with :class lambda" do
+        representer! do
+          property :name, :extend => lambda { |name| name.is_a?(UpcaseString) ? UpcaseRepresenter : DowncaseRepresenter },
+                          :class  => lambda { |fragment| fragment == "Still Failing?" ? String : UpcaseString }
+        end
+
+        it "creates instance from :class lambda when parsing" do
+          song = Song.new.extend(representer).from_hash({"name" => "Quitters Never Win"})
+          song.name.must_be_kind_of UpcaseString
+          song.name.must_equal "QUITTERS NEVER WIN"
+
+          song = Song.new.extend(representer).from_hash({"name" => "Still Failing?"})
+          song.name.must_be_kind_of String
+          song.name.must_equal "still failing?"
+        end
+
+        describe "when :class lambda returns nil" do
+          representer! do
+            property :name, :extend => lambda { |name| Module.new { def from_hash(data); data; end } },
+                            :class  => nil
+          end
+
+          it "skips creating new instance" do
+            song = Song.new.extend(representer).from_hash({"name" => string = "Satellite"})
+            song.name.object_id.must_equal string.object_id
+          end
+        end
+      end
+    end
+
+
+    describe "collection with :extend" do
+      representer! do
+        collection :songs, :extend => lambda { |name| name.is_a?(UpcaseString) ? UpcaseRepresenter : DowncaseRepresenter }, :class => String
+      end
+
+      it "uses lambda for each item when rendering" do
+        Album.new([UpcaseString.new("Dean Martin"), "Charlie Still Smirks"]).extend(representer).to_hash.must_equal("songs"=>["DEAN MARTIN", "charlie still smirks"])
+      end
+
+      it "uses lambda for each item when parsing" do
+        album = Album.new.extend(representer).from_hash("songs"=>["DEAN MARTIN", "charlie still smirks"])
+        album.songs.must_equal ["dean martin", "charlie still smirks"] # DISCUSS: we compare "".is_a?(UpcaseString)
+      end
+
+      describe "with :class lambda" do
+        representer! do
+          collection :songs,  :extend => lambda { |name| name.is_a?(UpcaseString) ? UpcaseRepresenter : DowncaseRepresenter },
+                              :class  => lambda { |fragment| fragment == "Still Failing?" ? String : UpcaseString }
+        end
+
+        it "creates instance from :class lambda for each item when parsing" do
+          album = Album.new.extend(representer).from_hash("songs"=>["Still Failing?", "charlie still smirks"])
+          album.songs.must_equal ["still failing?", "CHARLIE STILL SMIRKS"]
+        end
+      end
+    end
+    
+  end
   
   describe "Config" do
     before do

data/test/xml_bindings_test.rb

@@ -26,7 +26,7 @@ class XMLBindingTest < MiniTest::Spec
   describe "PropertyBinding" do
     describe "with plain text" do
       before do
-        @property = Representable::XML::PropertyBinding.new(Representable::Definition.new(:song))
+        @property = Representable::XML::PropertyBinding.new(Representable::Definition.new(:song), nil)
       end
       
       it "extracts with #read" do
@@ -41,7 +41,7 @@ class XMLBindingTest < MiniTest::Spec
     
     describe "with an object" do
       before do
-        @property = Representable::XML::PropertyBinding.new(Representable::Definition.new(:song, :class => SongWithRepresenter))
+        @property = Representable::XML::PropertyBinding.new(Representable::Definition.new(:song, :class => SongWithRepresenter), nil)
       end
       
       it "extracts with #read" do
@@ -56,7 +56,7 @@ class XMLBindingTest < MiniTest::Spec
     
     describe "with an object and :extend" do
       before do
-        @property = Representable::XML::PropertyBinding.new(Representable::Definition.new(:song, :class => Song, :extend => SongRepresenter))
+        @property = Representable::XML::PropertyBinding.new(Representable::Definition.new(:song, :class => Song, :extend => SongRepresenter), nil)
       end
       
       it "extracts with #read" do
@@ -74,7 +74,7 @@ class XMLBindingTest < MiniTest::Spec
   describe "CollectionBinding" do
     describe "with plain text items" do
       before do
-        @property = Representable::XML::CollectionBinding.new(Representable::Definition.new(:song, :collection => true))
+        @property = Representable::XML::CollectionBinding.new(Representable::Definition.new(:song, :collection => true), nil)
       end
       
       it "extracts with #read" do
@@ -90,7 +90,7 @@ class XMLBindingTest < MiniTest::Spec
     
     describe "with objects" do
       before do
-        @property = Representable::XML::PropertyBinding.new(Representable::Definition.new(:song, :collection => true, :class => SongWithRepresenter))
+        @property = Representable::XML::PropertyBinding.new(Representable::Definition.new(:song, :collection => true, :class => SongWithRepresenter), nil)
       end
       
       it "extracts with #read" do
@@ -111,7 +111,7 @@ class XMLBindingTest < MiniTest::Spec
   describe "HashBinding" do
     describe "with plain text items" do
       before do
-        @property = Representable::XML::HashBinding.new(Representable::Definition.new(:songs, :hash => true))
+        @property = Representable::XML::HashBinding.new(Representable::Definition.new(:songs, :hash => true), nil)
       end
       
       it "extracts with #read" do
@@ -127,7 +127,7 @@ class XMLBindingTest < MiniTest::Spec
     
     describe "with objects" do
       before do
-        @property = Representable::XML::HashBinding.new(Representable::Definition.new(:songs, :hash => true, :class => Song, :extend => SongRepresenter))
+        @property = Representable::XML::HashBinding.new(Representable::Definition.new(:songs, :hash => true, :class => Song, :extend => SongRepresenter), nil)
       end
     end
   end
@@ -136,7 +136,7 @@ class XMLBindingTest < MiniTest::Spec
   describe "AttributeBinding" do
     describe "with plain text items" do
       before do
-        @property = Representable::XML::AttributeBinding.new(Representable::Definition.new(:name, :attribute => true))
+        @property = Representable::XML::AttributeBinding.new(Representable::Definition.new(:name, :attribute => true), nil)
       end
       
       it "extracts with #read" do

data/test/xml_test.rb

@@ -126,20 +126,20 @@ class XmlTest < MiniTest::Spec
     
     describe "XML::Binding#build_for" do
       it "returns AttributeBinding" do
-        assert_kind_of XML::AttributeBinding, XML::PropertyBinding.build_for(Def.new(:band, :from => "band", :attribute => true))
+        assert_kind_of XML::AttributeBinding, XML::PropertyBinding.build_for(Def.new(:band, :from => "band", :attribute => true), nil)
       end
       
       it "returns PropertyBinding" do
-        assert_kind_of XML::PropertyBinding, XML::PropertyBinding.build_for(Def.new(:band, :class => Hash))
-        assert_kind_of XML::PropertyBinding, XML::PropertyBinding.build_for(Def.new(:band, :from => :content))
+        assert_kind_of XML::PropertyBinding, XML::PropertyBinding.build_for(Def.new(:band, :class => Hash), nil)
+        assert_kind_of XML::PropertyBinding, XML::PropertyBinding.build_for(Def.new(:band, :from => :content), nil)
       end
       
       it "returns CollectionBinding" do
-        assert_kind_of XML::CollectionBinding, XML::PropertyBinding.build_for(Def.new(:band, :collection => :true))
+        assert_kind_of XML::CollectionBinding, XML::PropertyBinding.build_for(Def.new(:band, :collection => :true), nil)
       end
       
       it "returns HashBinding" do
-        assert_kind_of XML::HashBinding, XML::PropertyBinding.build_for(Def.new(:band, :hash => :true))
+        assert_kind_of XML::HashBinding, XML::PropertyBinding.build_for(Def.new(:band, :hash => :true), nil)
       end
     end
     

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: representable
 version: !ruby/object:Gem::Version
-  version: 1.2.8
+  version: 1.2.9
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-01 00:00:00.000000000 Z
+date: 2013-01-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -155,6 +155,7 @@ files:
 - LICENSE
 - README.rdoc
 - Rakefile
+- TODO
 - gemfiles/Gemfile.mongoid-2.4
 - lib/representable.rb
 - lib/representable/binding.rb
@@ -170,6 +171,7 @@ files:
 - lib/representable/json.rb
 - lib/representable/json/collection.rb
 - lib/representable/json/hash.rb
+- lib/representable/readable_writeable.rb
 - lib/representable/version.rb
 - lib/representable/xml.rb
 - lib/representable/xml/collection.rb