virtus 0.1.0 → 0.2.0

data/Changelog.md

@@ -1,11 +1,20 @@
-# v0.1.0 to-be-released
+# v0.2.0 2012-02-08
 
-[Compare v0.0.10..master](https://github.com/solnic/virtus/compare/v0.0.10...master)
+* [feature] Support for Value Objects (emmanuel)
+* [feature] New Symbol attribute (solnic)
+* [feature] Time => Integer coercion (solnic)
+
+[Compare v0.1.0..v0.2.0](https://github.com/solnic/virtus/compare/v0.1.0...v0.2.0)
+
+# v0.1.0 2012-02-05
 
 * [feature] New EmbeddedValue attribute (solnic)
+* [feature] Array and Set attributes support member coercions (emmanuel)
 * [feature] Support for scientific notation handling in string => integer coercion (dkubb)
 * [feature] Handling of string => numeric coercion with a leading + sign (dkubb)
+* [changed] Update Boolean coercion to handle "on", "off", "y", "n", "yes", "no" (dkubb)
 
+[Compare v0.0.10..v0.1.0](https://github.com/solnic/virtus/compare/v0.0.10...v0.1.0)
 
 # v0.0.10 2011-11-21
 

data/README.md

@@ -160,6 +160,39 @@ user.phone_numbers # => [#<PhoneNumber:0x007fdb2d3bef88 @number="212-555-1212">,
 user.addresses # => #<Set: {#<Address:0x007fdb2d3be448 @address="1234 Any St.", @locality="Anytown", @region="DC", @postal_code="21234">}>
 ```
 
+**Value Objects**
+
+``` ruby
+class GeoLocation
+  include Virtus::ValueObject
+
+  attribute :latitude,  Float
+  attribute :longitude, Float
+end
+
+class Venue
+  include Virtus
+
+  attribute :name,     String
+  attribute :location, GeoLocation
+end
+
+venue = Venue.new(
+  :name     => 'Pub',
+  :location => { :latitude => 37.160317, :longitude => -98.437500 })
+
+venue.location.latitude  # => 37.160317
+venue.location.longitude # => -98.4375
+
+# Supports object's equality
+
+venue_other = Venue.new(
+  :name     => 'Other Pub',
+  :location => { :latitude => 37.160317, :longitude => -98.437500 })
+
+venue.location === venue_other.location # => true
+```
+
 **Adding Coercions**
 
 Virtus comes with a builtin coercion library.

data/TODO

@@ -1,8 +1,15 @@
 * Add missing specs:
-  * Add spec file spec/unit/virtus/coercion/time_coercions/to_date_spec.rb for Virtus::Coercion::TimeCoercions#to_date
+  * Add spec file spec/unit/virtus/attribute/collection/member_coercion/coerce_and_append_member_spec.rb for Virtus::Attribute::Collection::MemberCoercion#coerce_and_append_member
+  * Add spec file spec/unit/virtus/attribute/collection/coerce_and_append_member_spec.rb for Virtus::Attribute::Collection#coerce_and_append_member
+  * Add spec file spec/unit/virtus/attribute/collection/member_type_spec.rb for Virtus::Attribute::Collection#member_type
+  * Add spec file spec/unit/virtus/attribute/collection/new_collection_spec.rb for Virtus::Attribute::Collection#new_collection
+  * Add spec file spec/unit/virtus/coercion/string/class_methods/to_symbol_spec.rb for Virtus::Coercion::String.to_symbol
+  * Add spec file spec/unit/virtus/coercion/time/class_methods/to_integer_spec.rb for Virtus::Coercion::Time.to_integer
   * Add spec file spec/unit/virtus/coercion/time_coercions/to_time_spec.rb for Virtus::Coercion::TimeCoercions#to_time
-  * Add spec file spec/unit/virtus/coercion/time_coercions/to_string_spec.rb for Virtus::Coercion::TimeCoercions#to_string
   * Add spec file spec/unit/virtus/coercion/time_coercions/to_datetime_spec.rb for Virtus::Coercion::TimeCoercions#to_datetime
+  * Add spec file spec/unit/virtus/coercion/time_coercions/to_date_spec.rb for Virtus::Coercion::TimeCoercions#to_date
+  * Add spec file spec/unit/virtus/coercion/time_coercions/to_string_spec.rb for Virtus::Coercion::TimeCoercions#to_string
+  * Add spec file spec/unit/virtus/coercion/array/class_methods/to_set_spec.rb for Virtus::Coercion::Array.to_set
   * Add spec file spec/unit/virtus/coercion/decimal/class_methods/to_decimal_spec.rb for Virtus::Coercion::Decimal.to_decimal
   * Add spec file spec/unit/virtus/coercion/float/class_methods/to_float_spec.rb for Virtus::Coercion::Float.to_float
   * Add spec file spec/unit/virtus/coercion/integer/class_methods/to_integer_spec.rb for Virtus::Coercion::Integer.to_integer
@@ -12,9 +19,13 @@
   * Add spec file spec/unit/virtus/coercion/numeric/class_methods/to_decimal_spec.rb for Virtus::Coercion::Numeric.to_decimal
   * Add spec file spec/unit/virtus/coercion/class_methods/element_reference_spec.rb for Virtus::Coercion.[]
   * Add spec file spec/unit/virtus/coercion/class_methods/primitive_spec.rb for Virtus::Coercion.primitive
+  * Add spec file spec/unit/virtus/value_object/class_methods/attribute_spec.rb for Virtus::ValueObject::ClassMethods#attribute
+  * Add spec file spec/unit/virtus/value_object/instance_methods/with_spec.rb for Virtus::ValueObject::InstanceMethods#with
+  * Add spec file spec/unit/virtus/value_object/equalizer/compile_spec.rb for Virtus::ValueObject::Equalizer#compile
+  * Add spec file spec/unit/virtus/value_object/equalizer/append_spec.rb for Virtus::ValueObject::Equalizer#<<
+  * Add spec file spec/unit/virtus/value_object/equalizer/host_name_spec.rb for Virtus::ValueObject::Equalizer#host_name
+  * Add spec file spec/unit/virtus/value_object/equalizer/keys_spec.rb for Virtus::ValueObject::Equalizer#keys
   * Add spec file spec/unit/virtus/attributes_accessor/define_writer_method_spec.rb for Virtus::AttributesAccessor#define_writer_method
-  * Add spec file spec/unit/virtus/attributes_accessor/inspect_spec.rb for Virtus::AttributesAccessor#inspect
   * Add spec file spec/unit/virtus/attributes_accessor/define_reader_method_spec.rb for Virtus::AttributesAccessor#define_reader_method
 
 * Make #to_time #to_date and #to_datetime work on Ruby 1.8.7 instead of typecasting to string and parsing the value
-* Add support for defining attributes on Modules

data/config/flay.yml

@@ -1,3 +1,3 @@
 ---
 threshold: 19
-total_score: 322
+total_score: 354

data/config/roodi.yml

@@ -2,16 +2,16 @@
 AbcMetricMethodCheck:            { score: 12.1 }
 AssignmentInConditionalCheck:    { }
 CaseMissingElseCheck:            { }
-ClassLineCountCheck:             { line_count: 319 }
+ClassLineCountCheck:             { line_count: 325 }
 ClassNameCheck:                  { pattern: !ruby/regexp /\A(?:[A-Z]+|[A-Z][a-z](?:[A-Z]?[a-z])+)\z/ }
 ClassVariableCheck:              { }
 CyclomaticComplexityBlockCheck:  { complexity: 4 }
 CyclomaticComplexityMethodCheck: { complexity: 4 }
 EmptyRescueBodyCheck:            { }
 ForLoopCheck:                    { }
-MethodLineCountCheck:            { line_count: 9 }
+MethodLineCountCheck:            { line_count: 10 }
 MethodNameCheck:                 { pattern: !ruby/regexp /\A(?:[a-z\d](?:_?[a-z\d])+[?!=]?|\[\]=?|==|<=>|<<|[+*&|-])\z/ }
-ModuleLineCountCheck:            { line_count: 325 }
+ModuleLineCountCheck:            { line_count: 331 }
 ModuleNameCheck:                 { pattern: !ruby/regexp /\A(?:[A-Z]+|[A-Z][a-z](?:[A-Z]?[a-z])+)\z/ }
 # TODO: decrease parameter_count to 2 or less
 ParameterNumberCheck:            { parameter_count: 3 }

data/lib/virtus.rb

@@ -35,6 +35,8 @@ require 'virtus/attributes_accessor'
 require 'virtus/class_methods'
 require 'virtus/instance_methods'
 
+require 'virtus/value_object'
+
 require 'virtus/attribute_set'
 
 require 'virtus/coercion'
@@ -69,6 +71,7 @@ require 'virtus/attribute/decimal'
 require 'virtus/attribute/float'
 require 'virtus/attribute/hash'
 require 'virtus/attribute/integer'
+require 'virtus/attribute/symbol'
 require 'virtus/attribute/string'
 require 'virtus/attribute/time'
 require 'virtus/attribute/embedded_value'

data/lib/virtus/attribute/array.rb

@@ -16,21 +16,7 @@ module Virtus
       primitive       ::Array
       coercion_method :to_array
 
-      # Coerce a member of a source collection and append it to the target collection
-      #
-      # @param [Array, Set] collection
-      #   target collection to which the coerced member should be appended
-      #
-      # @param [Object] entry
-      #   the member that should be coerced and appended
-      #
-      # @return [Array, Set]
-      #   collection with the coerced member appended to it
-      #
-      # @api private
-      def coerce_and_append_member(collection, entry)
-        collection << @member_type_instance.coerce(entry)
-      end
+      include Collection::MemberCoercion
 
     end # class Array
   end # class Attribute

data/lib/virtus/attribute/collection.rb

@@ -91,6 +91,26 @@ module Virtus
           "#{self.class}#coerce_and_append_member has not been implemented"
       end
 
+      # Default coercion method for collection members used by Array and Set
+      module MemberCoercion
+
+        # Coerce a member of a source collection and append it to the target collection
+        #
+        # @param [Array, Set] collection
+        #   target collection to which the coerced member should be appended
+        #
+        # @param [Object] entry
+        #   the member that should be coerced and appended
+        #
+        # @return [Array, Set]
+        #   collection with the coerced member appended to it
+        #
+        # @api private
+        def coerce_and_append_member(collection, entry)
+          collection << @member_type_instance.coerce(entry)
+        end
+      end # module MemberCoercion
+
     end # class Array
   end # class Attribute
 end # module Virtus

data/lib/virtus/attribute/set.rb

@@ -16,9 +16,7 @@ module Virtus
       primitive       ::Set
       coercion_method :to_set
 
-      def coerce_and_append_member(collection, entry)
-        collection << @member_type_instance.coerce(entry)
-      end
+      include Collection::MemberCoercion
 
     end # class Set
   end # class Attribute

data/lib/virtus/attribute/symbol.rb

@@ -0,0 +1,21 @@
+module Virtus
+  class Attribute
+
+    # Symbol
+    #
+    # @example
+    #   class Product
+    #     include Virtus
+    #
+    #     attribute :code, Symbol
+    #   end
+    #
+    #   product = Product.new(:code => :red)
+    #
+    class Symbol < Object
+      primitive       ::Symbol
+      coercion_method :to_symbol
+
+    end # class Symbol
+  end # class Attribute
+end # module Virtus

data/lib/virtus/coercion/string.rb

@@ -29,6 +29,20 @@ module Virtus
         ::Object.const_get(value)
       end
 
+      # Coerce give value to a symbol
+      #
+      # @example
+      #   Virtus::Coercion::String.to_symbol('string') # => :string
+      #
+      # @param [String] value
+      #
+      # @return [Symbol]
+      #
+      # @api public
+      def self.to_symbol(value)
+        value.to_sym
+      end
+
       # Coerce given value to Time
       #
       # @example

data/lib/virtus/coercion/time.rb

@@ -21,6 +21,20 @@ module Virtus
         value
       end
 
+      # Creates a Fixnum instance from a Time object
+      #
+      # @example
+      #   Virtus::Coercion::Time.to_integer(time)  # => Fixnum object
+      #
+      # @param [Time] value
+      #
+      # @return [Fixnum]
+      #
+      # @api public
+      def self.to_integer(value)
+        value.to_i
+      end
+
     end # class Time
   end # class Coercion
 end # module Virtus

data/lib/virtus/instance_methods.rb

@@ -35,7 +35,7 @@ module Virtus
     #
     # @api public
     def [](name)
-      __send__(name)
+      get_attribute(name)
     end
 
     # Sets a value of the attribute with the given name
@@ -62,7 +62,7 @@ module Virtus
     #
     # @api public
     def []=(name, value)
-      __send__("#{name}=", value)
+      set_attribute(name, value)
     end
 
     # Returns a hash of all publicly accessible attributes
@@ -165,7 +165,41 @@ module Virtus
     #
     # @api private
     def set_attributes(attribute_values)
-      attribute_values.each { |name, value| self[name] = value }
+      attribute_values.each { |pair| set_attribute(*pair) }
+    end
+
+    # Get values of all attributes defined for this class, ignoring privacy
+    #
+    # @return [Hash]
+    #
+    # @api private
+    def get_attributes
+      self.class.attributes.each_with_object({}) do |attribute, attributes|
+        attribute_name = attribute.name
+        attributes[attribute_name] = get_attribute(attribute_name)
+      end
+    end
+
+    # Returns a value of the attribute with the given name
+    #
+    # @see Virtus::InstanceMethods#[]
+    #
+    # @return [Object]
+    #
+    # @api private
+    def get_attribute(name)
+      __send__(name)
+    end
+
+    # Sets a value of the attribute with the given name
+    #
+    # @see Virtus::InstanceMethods#[]=
+    #
+    # @return [Object]
+    #
+    # @api private
+    def set_attribute(name, value)
+      __send__("#{name}=", value)
     end
 
   end # module InstanceMethods

data/lib/virtus/value_object.rb

@@ -0,0 +1,104 @@
+# -*- encoding: utf-8 -*-
+
+require 'virtus'
+require 'virtus/value_object/equalizer'
+
+module Virtus
+  # Include this Module for Value Object semantics
+  #
+  # The idea is that instances should be immutable and compared based on state
+  #   (rather than identity, as is typically the case)
+  #
+  # @example
+  #   class GeoLocation
+  #     include Virtus::ValueObject
+  #     attribute :latitude,  Float
+  #     attribute :longitude, Float
+  #   end
+  #
+  #   location = GeoLocation.new(:latitude => 10, :longitude => 100)
+  #   same_location = GeoLocation.new(:latitude => 10, :longitude => 100)
+  #   location == same_location       #=> true
+  #   hash = { location => :foo }
+  #   hash[same_location]             #=> :foo
+  module ValueObject
+    # Callback to configure including Class as a Value Object
+    #
+    # Including Class will include Virtus and have additional
+    #   value object semantics defined in this module
+    #
+    # @return [Undefined]
+    #
+    # TODO: stacking modules is getting painful
+    #   time for Facets' module_inheritance, ActiveSupport::Concern or the like
+    #
+    # @api private
+    def self.included(base)
+      base.instance_eval do
+        include ::Virtus
+        include InstanceMethods
+        extend ClassMethods
+      end
+    end
+
+    module InstanceMethods
+      def initialize(attributes = {})
+        set_attributes(attributes)
+      end
+
+      def with(attribute_updates)
+        self.class.new(get_attributes.merge(attribute_updates))
+      end
+    end
+
+    module ClassMethods
+      # Define an attribute on the receiver
+      #
+      # The Attribute will have private writer methods (eg., immutable instances)
+      #   and be used in equality/equivalence comparisons
+      #
+      # @example
+      #   class GeoLocation
+      #     include Virtus::ValueObject
+      #
+      #     attribute :latitude,  Float
+      #     attribute :longitude, Float
+      #   end
+      #
+      # @see Virtus::ClassMethods.attribute
+      #
+      # @return [self]
+      #
+      # @api public
+      def attribute(name, type, options = {})
+        equalizer << name
+        options[:writer] = :private
+
+        super
+      end
+
+      # Define and include a module that provides Value Object semantics
+      #
+      # Included module will have #inspect, #eql?, #== and #hash
+      # methods whose definition is based on the _keys_ argument
+      #
+      # @example
+      #   virtus_class.equalizer
+      #
+      # @return [Equalizer]
+      #   An Equalizer module which defines #inspect, #eql?, #== and #hash
+      #   for instances of this class
+      #
+      # @api public
+      def equalizer
+        return @equalizer if instance_variable_defined?('@equalizer')
+
+        @equalizer = begin
+          equalizer = Equalizer.new(name || inspect)
+          include equalizer
+          equalizer
+        end
+      end
+    end # module ClassMethods
+  end # module ValueObject
+end # module Virtus

data/lib/virtus/value_object/equalizer.rb

@@ -0,0 +1,129 @@
+module Virtus
+  module ValueObject
+    # A type of Module for dynamically defining and hosting equality methods
+    class Equalizer < Module
+      # Name of hosting Class or Module that will be used for #inspect
+      #
+      # @return [String]
+      #
+      # @api private
+      attr_reader :host_name
+
+      # List of methods that will be used to define equality methods
+      #
+      # @return [Array(Symbol)]
+      #
+      # @api private
+      attr_reader :keys
+
+      # Initialize an Equalizer with the given keys
+      #
+      # Will use the keys with which it is initialized to define #eql?, #==,
+      #   and #hash
+      #
+      # @api private
+      def initialize(host_name, keys = [])
+        @host_name = host_name
+        @keys      = keys
+      end
+
+      # Append a key and compile the equality methods
+      #
+      # @return [Equalizer] self
+      #
+      # @api private
+      def <<(key)
+        @keys << key
+        compile
+
+        self
+      end
+
+      # Compile the equalizer methods based on #keys
+      #
+      # @return [self]
+      #
+      # @api private
+      def compile
+        define_inspect_method
+        define_eql_method
+        define_equivalent_method
+        define_hash_method
+
+        self
+      end
+
+    private
+
+      # Define an inspect method that reports the values of the instance's keys
+      #
+      # @return [self]
+      #
+      # @api private
+      def define_inspect_method
+        module_eval <<-RUBY, __FILE__, __LINE__ + 1
+          def inspect
+            "#<#{host_name} #{keys.map { |key| "#{key}=\#{#{key}.inspect}" }.join(' ')}>"
+          end
+        RUBY
+      end
+
+      # Define an #eql? method based on the instance's values identified by #keys
+      #
+      # @return [self]
+      #
+      # @api private
+      def define_eql_method
+        module_eval <<-RUBY, __FILE__, __LINE__ + 1
+          def eql?(other)
+            return true if equal?(other)
+            instance_of?(other.class) &&
+            #{keys.map { |key| "#{key}.eql?(other.#{key})" }.join(' && ')}
+          end
+        RUBY
+      end
+
+      # Define an #== method based on the instance's values identified by #keys
+      #
+      # @return [self]
+      #
+      # @api private
+      def define_equivalent_method
+        respond_to, equivalent = compile_strings_for_equivalent_method
+        module_eval <<-RUBY, __FILE__, __LINE__ + 1
+          def ==(other)
+            return true if equal?(other)
+            return false unless kind_of?(other.class) || other.kind_of?(self.class)
+            #{respond_to.join(' && ')} && #{equivalent.join(' && ')}
+          end
+        RUBY
+      end
+
+      # Define a #hash method based on the instance's values identified by #keys
+      #
+      # @return [self]
+      #
+      # @api private
+      def define_hash_method
+        module_eval <<-RUBY, __FILE__, __LINE__ + 1
+          def hash
+            self.class.hash ^ #{keys.map { |key| "#{key}.hash" }.join(' ^ ')}
+          end
+        RUBY
+      end
+
+      # @api private
+      def compile_strings_for_equivalent_method
+        respond_to = []
+        equivalent = []
+
+        keys.each do |key|
+          respond_to << "other.respond_to?(#{key.inspect})"
+          equivalent << "#{key} == other.#{key}"
+        end
+
+        [ respond_to, equivalent ]
+      end
+    end # class Equalizer
+  end # module ValueObject
+end # module Virtus

data/lib/virtus/version.rb

@@ -1,3 +1,3 @@
 module Virtus
-  VERSION = '0.1.0'
+  VERSION = '0.2.0'
 end

data/spec/integration/collection_member_coercion_spec.rb

@@ -72,4 +72,4 @@ describe User do
     its(:region)      { should eql('DC')           }
     its(:postal_code) { should eql('21234')        }
   end
-end
+end

data/spec/integration/virtus/value_object_spec.rb

@@ -0,0 +1,95 @@
+require 'spec_helper'
+
+describe Virtus::ValueObject do
+  let(:class_under_test) do
+    Class.new do
+      def self.name
+        'GeoLocation'
+      end
+
+      include Virtus::ValueObject
+
+      attribute :latitude,  Float
+      attribute :longitude, Float
+    end
+  end
+  let(:attribute_values) { { :latitude => 10.0, :longitude => 20.0 } }
+  let(:instance_with_equal_state) { class_under_test.new(attribute_values) }
+  let(:instance_with_different_state) do
+    class_under_test.new(:latitude => attribute_values[:latitude])
+  end
+  subject { class_under_test.new(attribute_values) }
+
+  describe 'initialization' do
+    it 'sets the attribute values provided to Class.new' do
+      class_under_test.new(:latitude => 10000.001).latitude.should == 10000.001
+      subject.latitude.should eql(attribute_values[:latitude])
+    end
+  end
+
+  describe 'writer visibility' do
+    it 'attributes are configured for private writers' do
+      class_under_test.attributes[:latitude].writer_visibility.should  == :private
+      class_under_test.attributes[:longitude].writer_visibility.should == :private
+    end
+
+    it 'writer methods are set to private' do
+      private_methods = class_under_test.private_instance_methods
+      private_methods.map! { |m| m.to_s }
+      private_methods.should include('latitude=', 'longitude=')
+    end
+
+    it 'attempts to call attribute writer methods raises NameError' do
+      expect { subject.latitude  = 5.0 }.to raise_exception(NameError)
+      expect { subject.longitude = 5.0 }.to raise_exception(NameError)
+    end
+  end
+
+  describe 'equality' do
+    describe '#==' do
+      it 'returns true for different objects with the same state' do
+        subject.should == instance_with_equal_state
+      end
+
+      it 'returns false for different objects with different state' do
+        subject.should_not == instance_with_different_state
+      end
+    end
+
+    describe '#eql?' do
+      it 'returns true for different objects with the same state' do
+        subject.should eql(instance_with_equal_state)
+      end
+
+      it 'returns false for different objects with different state' do
+        subject.should_not eql(instance_with_different_state)
+      end
+    end
+
+    describe '#equal?' do
+      it 'returns false for different objects with the same state' do
+        subject.should_not equal(instance_with_equal_state)
+      end
+
+      it 'returns false for different objects with different state' do
+        subject.should_not equal(instance_with_different_state)
+      end
+    end
+
+    describe '#hash' do
+      it 'returns the same value for different objects with the same state' do
+        subject.hash.should eql(instance_with_equal_state.hash)
+      end
+
+      it 'returns different values for different objects with different state' do
+        subject.hash.should_not eql(instance_with_different_state.hash)
+      end
+    end
+  end
+
+  describe '#inspect' do
+    it 'includes the class name and attribute values' do
+      subject.inspect.should == '#<GeoLocation latitude=10.0 longitude=20.0>'
+    end
+  end
+end

data/spec/unit/virtus/attribute/class_methods/determine_type_spec.rb

@@ -18,7 +18,7 @@ describe Virtus::Attribute, '.determine_type' do
 
       before do
         if [Virtus::Attribute::EmbeddedValue, Virtus::Attribute::Collection].include? attribute_class
-          pending 
+          pending
         end
       end
 

data/spec/unit/virtus/attribute/collection/class_methods/merge_options_spec.rb

@@ -26,7 +26,7 @@ describe Virtus::Attribute::Collection, '.merge_options' do
     context 'when size is > 1' do
       let(:size) { 2 }
 
-      specify { expect { subject }.to raise_error(NotImplementedError) }
+      specify { expect { subject }.to raise_error(NotImplementedError, "build SumType from list of types (#{type.inspect})") }
     end
   end
 
@@ -37,4 +37,4 @@ describe Virtus::Attribute::Collection, '.merge_options' do
 
     it { should eql(options) }
   end
-end
+end

data/spec/unit/virtus/attribute/collection/coerce_spec.rb

@@ -11,10 +11,10 @@ describe Virtus::Attribute::Collection, '#coerce' do
   before do
     value.should_receive(:respond_to?).with(:inject).and_return(respond_to_inject)
   end
-  
+
   context 'when coerced value responds to #inject' do
     let(:respond_to_inject) { true }
-  
+
     specify { expect { subject }.to raise_error(NotImplementedError) }
   end
 
@@ -23,4 +23,4 @@ describe Virtus::Attribute::Collection, '#coerce' do
 
     specify { should eql(value) }
   end
-end
+end

data/spec/unit/virtus/attribute/integer/coerce_spec.rb

@@ -95,6 +95,12 @@ describe Virtus::Attribute::Integer, '#coerce' do
     it { should eql(-24) }
   end
 
+  context 'with a time object' do
+    let(:value) { Time.now }
+
+    it { should eql(value.to_i) }
+  end
+
   [ Object.new, true, false, '00.0', '0.', '-.0', 'string' ].each do |non_num_value|
     context 'does not coerce non-numeric value #{non_num_value.inspect}' do
       let(:value) { non_num_value }

data/spec/unit/virtus/attribute/object/class_methods/descendants_spec.rb

@@ -1,17 +1,17 @@
 require 'spec_helper'
 
 describe Virtus::Attribute::Object, '.descendants' do
-  subject { described_class.descendants }
+  subject { described_class.descendants.map { |c| c.to_s }.sort }
 
   let(:known_descendants) do
-    [ Virtus::Attribute::EmbeddedValue,
-      Virtus::Attribute::Time,       Virtus::Attribute::String,
-      Virtus::Attribute::Integer,    Virtus::Attribute::Hash,
-      Virtus::Attribute::Float,      Virtus::Attribute::Decimal,
-      Virtus::Attribute::Numeric,    Virtus::Attribute::DateTime,
-      Virtus::Attribute::Date,       Virtus::Attribute::Boolean,
-      Virtus::Attribute::Set,        Virtus::Attribute::Array,
-      Virtus::Attribute::Collection, Virtus::Attribute::Class ]
+    [ Virtus::Attribute::EmbeddedValue, Virtus::Attribute::Symbol,
+      Virtus::Attribute::Time,          Virtus::Attribute::String,
+      Virtus::Attribute::Integer,       Virtus::Attribute::Hash,
+      Virtus::Attribute::Float,         Virtus::Attribute::Decimal,
+      Virtus::Attribute::Numeric,       Virtus::Attribute::DateTime,
+      Virtus::Attribute::Date,          Virtus::Attribute::Boolean,
+      Virtus::Attribute::Set,           Virtus::Attribute::Array,
+      Virtus::Attribute::Collection,    Virtus::Attribute::Class ].map { |a| a.to_s }.sort
   end
 
   it 'should return all known attribute classes' do

data/spec/unit/virtus/attribute/symbol/coerce_spec.rb

@@ -0,0 +1,13 @@
+require 'spec_helper'
+
+describe Virtus::Attribute::Symbol, '#coerce' do
+  subject { attribute.coerce(value) }
+
+  let(:attribute) { described_class.new(:code) }
+
+  context 'with a string' do
+    let(:value) { 'foo' }
+
+    it { should be(:foo) }
+  end
+end

data/spec/unit/virtus/attributes_accessor/inspect_spec.rb

@@ -6,4 +6,4 @@ describe Virtus::AttributesAccessor, '#inspect' do
   let(:object) { described_class.new('Test') }
 
   it { should eql('Test::AttributesAccessor') }
-end
+end

data/spec/unit/virtus/value_object/class_methods/equalizer_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+describe Virtus::ValueObject, '.equalizer' do
+  subject { described_class.equalizer }
+
+  let(:described_class) do
+    Class.new do
+      include Virtus::ValueObject
+
+      attribute :first_name, String
+    end
+  end
+
+  specify { subject.should be_instance_of(Virtus::ValueObject::Equalizer) }
+  specify { described_class.included_modules.should include(subject)      }
+
+  context 'when equalizer is already initialized' do
+    before { subject; described_class.equalizer }
+
+    let(:equalizer) { subject }
+
+    specify { subject.should be(equalizer) }
+  end
+end

data/spec/unit/virtus/value_object/initialize_spec.rb

@@ -0,0 +1,19 @@
+require 'spec_helper'
+
+describe Virtus::ValueObject, '#initialize' do
+  subject { described_class.new(attributes) }
+
+  let(:described_class) do
+    Class.new do
+      include Virtus::ValueObject
+
+      attribute :currency, String
+      attribute :amount,   Integer
+    end
+  end
+
+  let(:attributes) { Hash[:currency => 'USD', :amount => 1] }
+
+  its(:currency) { should eql('USD') }
+  its(:amount)   { should eql(1)     }
+end

data/spec/unit/virtus/value_object/with_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+describe Virtus::ValueObject, '#with' do
+  subject { object.with(attributes) }
+
+  let(:described_class) do
+    Class.new do
+      include Virtus::ValueObject
+
+      attribute :first_name, String
+      attribute :last_name,  String
+    end
+  end
+
+  let(:object)     { described_class.new }
+  let(:attributes) { Hash[:first_name => 'John', :last_name => 'Doe'] }
+
+  let(:described_class) do
+    Class.new do
+      include Virtus::ValueObject
+
+      attribute :first_name, String
+      attribute :last_name,  String
+    end
+  end
+
+  it { should be_instance_of(described_class) }
+
+  its(:first_name) { should eql('John') }
+  its(:last_name)  { should eql('Doe')  }
+end

data/virtus.gemspec

@@ -5,7 +5,6 @@ require File.expand_path('../lib/virtus/version', __FILE__)
 Gem::Specification.new do |gem|
   gem.name        = "virtus"
   gem.version     = Virtus::VERSION
-  gem.date        = "2011-11-21"
   gem.authors     = [ "Piotr Solnica" ]
   gem.email       = [ "piotr.solnica@gmail.com" ]
   gem.description = "Attributes on Steroids for Plain Old Ruby Objects"

metadata

@@ -1,59 +1,82 @@
---- !ruby/object:Gem::Specification
+--- !ruby/object:Gem::Specification 
 name: virtus
-version: !ruby/object:Gem::Version
-  version: 0.1.0
+version: !ruby/object:Gem::Version 
+  hash: 23
   prerelease: 
+  segments: 
+  - 0
+  - 2
+  - 0
+  version: 0.2.0
 platform: ruby
-authors:
+authors: 
 - Piotr Solnica
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-11-21 00:00:00.000000000Z
-dependencies:
-- !ruby/object:Gem::Dependency
+
+date: 2012-02-08 00:00:00 Z
+dependencies: 
+- !ruby/object:Gem::Dependency 
   name: rake
-  requirement: &70295232216940 !ruby/object:Gem::Requirement
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
+      - !ruby/object:Gem::Version 
+        hash: 63
+        segments: 
+        - 0
+        - 9
+        - 2
         version: 0.9.2
   type: :development
-  prerelease: false
-  version_requirements: *70295232216940
-- !ruby/object:Gem::Dependency
+  version_requirements: *id001
+- !ruby/object:Gem::Dependency 
   name: backports
-  requirement: &70295232216140 !ruby/object:Gem::Requirement
+  prerelease: false
+  requirement: &id002 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 2
+        - 3
+        - 0
         version: 2.3.0
   type: :development
-  prerelease: false
-  version_requirements: *70295232216140
-- !ruby/object:Gem::Dependency
+  version_requirements: *id002
+- !ruby/object:Gem::Dependency 
   name: rspec
-  requirement: &70295232215500 !ruby/object:Gem::Requirement
+  prerelease: false
+  requirement: &id003 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
+      - !ruby/object:Gem::Version 
+        hash: 31
+        segments: 
+        - 1
+        - 3
+        - 2
         version: 1.3.2
   type: :development
-  prerelease: false
-  version_requirements: *70295232215500
+  version_requirements: *id003
 description: Attributes on Steroids for Plain Old Ruby Objects
-email:
+email: 
 - piotr.solnica@gmail.com
 executables: []
+
 extensions: []
-extra_rdoc_files:
+
+extra_rdoc_files: 
 - LICENSE
 - README.md
 - TODO
-files:
+files: 
 - .gitignore
 - .rvmrc
 - .travis.yml
@@ -87,6 +110,7 @@ files:
 - lib/virtus/attribute/object.rb
 - lib/virtus/attribute/set.rb
 - lib/virtus/attribute/string.rb
+- lib/virtus/attribute/symbol.rb
 - lib/virtus/attribute/time.rb
 - lib/virtus/attribute_set.rb
 - lib/virtus/attributes_accessor.rb
@@ -111,6 +135,8 @@ files:
 - lib/virtus/support/descendants_tracker.rb
 - lib/virtus/support/options.rb
 - lib/virtus/support/type_lookup.rb
+- lib/virtus/value_object.rb
+- lib/virtus/value_object/equalizer.rb
 - lib/virtus/version.rb
 - spec/integration/collection_member_coercion_spec.rb
 - spec/integration/custom_attributes_spec.rb
@@ -119,6 +145,7 @@ files:
 - spec/integration/embedded_value_spec.rb
 - spec/integration/overriding_virtus_spec.rb
 - spec/integration/virtus/instance_level_attributes_spec.rb
+- spec/integration/virtus/value_object_spec.rb
 - spec/rcov.opts
 - spec/shared/constants_helpers.rb
 - spec/shared/idempotent_method_behaviour.rb
@@ -175,6 +202,7 @@ files:
 - spec/unit/virtus/attribute/set/coerce_spec.rb
 - spec/unit/virtus/attribute/set_spec.rb
 - spec/unit/virtus/attribute/string/coerce_spec.rb
+- spec/unit/virtus/attribute/symbol/coerce_spec.rb
 - spec/unit/virtus/attribute/time/coerce_spec.rb
 - spec/unit/virtus/attribute/value_coerced_spec.rb
 - spec/unit/virtus/attribute/writer_visibility_spec.rb
@@ -238,6 +266,9 @@ files:
 - spec/unit/virtus/options/options_spec.rb
 - spec/unit/virtus/type_lookup/determine_type_spec.rb
 - spec/unit/virtus/type_lookup/primitive_spec.rb
+- spec/unit/virtus/value_object/class_methods/equalizer_spec.rb
+- spec/unit/virtus/value_object/initialize_spec.rb
+- spec/unit/virtus/value_object/with_spec.rb
 - tasks/metrics/ci.rake
 - tasks/metrics/flay.rake
 - tasks/metrics/flog.rake
@@ -251,27 +282,36 @@ files:
 - virtus.gemspec
 homepage: https://github.com/solnic/virtus
 licenses: []
+
 post_install_message: 
 rdoc_options: []
-require_paths:
+
+require_paths: 
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement
+required_ruby_version: !ruby/object:Gem::Requirement 
   none: false
-  requirements:
-  - - ! '>='
-    - !ruby/object:Gem::Version
-      version: '0'
-required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      hash: 3
+      segments: 
+      - 0
+      version: "0"
+required_rubygems_version: !ruby/object:Gem::Requirement 
   none: false
-  requirements:
-  - - ! '>='
-    - !ruby/object:Gem::Version
-      version: '0'
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      hash: 3
+      segments: 
+      - 0
+      version: "0"
 requirements: []
+
 rubyforge_project: 
-rubygems_version: 1.8.11
+rubygems_version: 1.8.15
 signing_key: 
 specification_version: 3
 summary: Attributes on Steroids for Plain Old Ruby Objects
 test_files: []
-has_rdoc: 
+