plain_old_model 0.1.0 → 0.1.2

data/lib/plain_old_model/attribute_assignment.rb

@@ -4,17 +4,19 @@ module PlainOldModel
   module AttributeAssignment
     module ClassMethods
       def has_one(attr_name, options={})
-        if options[:class_name]
-          associations[attr_name] = options[:class_name].to_s.capitalize unless options[:class_name].nil?
-        else
-          associations[attr_name] = attr_name.to_s.capitalize
-        end
-       attr_accessor attr_name
+        associations << HasOneAssociation.new(attr_name, options)
+        attr_accessor attr_name
+      end
+
+      def has_many(attr_name, options={})
+        associations << HasManyAssociation.new(attr_name, options)
+        attr_accessor attr_name
       end
 
       def associations
-        @associations ||= {}
+        @associations ||= []
       end
+
     end
 
     def self.included(klass)
@@ -23,21 +25,65 @@ module PlainOldModel
 
     def assign_attributes(new_attributes, options = {})
       return unless new_attributes
-      associations.each do |k, v|
-        if new_attributes.include?(k)
-          new_klass = k.to_s.camelcase.constantize.new(new_attributes[k])
+      associations.each do |association|
+        attr_name = association.attr_name
+        if new_attributes.include?(attr_name)
+          value = association.create_value_from_attributes(new_attributes[attr_name])
 
-          respond_to?("#{k}=") ? (send("#{k}=", new_klass)) : (instance_variable_set("@#{k}".to_sym, new_klass))
-          new_attributes = new_attributes.delete_if { |key, value| key == k }
+          set_attribute(attr_name, value)
+          new_attributes  = new_attributes.delete_if { |key, value| key.to_s == attr_name.to_s }
         end
       end
-      assignment(new_attributes, options)
+      assign_simple_attributes(new_attributes, options)
     end
 
     def associations
       self.class.associations
     end
 
+    class Association
+      attr_reader :attr_name
+
+      def initialize(attr_name, options)
+        @attr_name = attr_name
+        @options = options
+      end
+
+      def create_value_from_attributes(attributes)
+        if @options[:factory_method]
+          klass.send(@options[:factory_method], attributes)
+        else
+          klass.new(attributes)
+        end
+      end
+
+      def klass
+        if @options[:class_name]
+          @options[:class_name].to_s.camelcase.constantize
+        else
+          klass_from_attr_name
+        end
+      end
+
+      def klass_from_attr_name
+        @attr_name.to_s.camelcase.constantize
+      end
+    end
+
+    class HasOneAssociation < Association
+
+    end
+
+    class HasManyAssociation < Association
+      def create_value_from_attributes(items)
+        items.map{|item| super(item)}
+      end
+
+      def klass_from_attr_name
+        @attr_name.to_s.singularize.camelcase.constantize
+      end
+    end
+
     private
     def sanitize_attributes(attributes)
       sanitized_attributes = {}
@@ -49,17 +95,19 @@ module PlainOldModel
       sanitized_attributes
     end
 
-    def assignment(attributes, options)
+    def assign_simple_attributes(attributes, options)
       attributes = sanitize_attributes(attributes).stringify_keys
 
       attributes.each do |k, v|
-        if respond_to?("#{k}=")
-          send("#{k}=", v)
-        elsif respond_to?("#{k}")
-          instance_variable_set("@#{k}".to_sym, v)
-        else
-          raise(Exception)
-        end
+        set_attribute(k, v)
+      end
+    end
+
+    def set_attribute(attr_name, value)
+      if respond_to?("#{attr_name}=")
+        send("#{attr_name}=", value)
+      else
+        instance_variable_set("@#{attr_name}".to_sym, value)
       end
     end
   end

data/lib/plain_old_model/base.rb

@@ -12,7 +12,7 @@ module PlainOldModel
     include ActiveModel::Conversion
     include PlainOldModel::AttributeAssignment
 
-    def initialize(attributes = nil, options = {})
+    def initialize(attributes = {}, options = {})
       assign_attributes(attributes, options) if attributes
     end
 

data/lib/plain_old_model/version.rb

@@ -1,3 +1,3 @@
 module PlainOldModel
-  VERSION = "0.1.0"
+  VERSION = "0.1.2"
 end

data/spec/lib/base_spec.rb

@@ -37,45 +37,76 @@ describe PlainOldModel::Base do
   end
 
   describe "associations" do
-    it "should return empty hash for unassociated class" do
-      @person = Person.new()
-      @person.associations.should == {}
+    it "should return empty array for unassociated class" do
+      @person = Continent.new
+      @person.associations.should == []
     end
     it "should provide all the associations when the class has associations" do
-      @address = Address.new()
-      @address.associations.should == {:country => "Country"}
-    end
-    it "should create a new instance and assign_attributes to the associated class" do
-      @address = Address.new({:fname => "first value", :lname => "second value", :country => {:code => "In", :name => "India"}, :read_test => 'This should not be assigned',:write_test => "this shd be available"})
-      @address.country.class.should == Country
-    end
-    it "should create the nested class instance and assign_attributes to the associated nested class" do
-      @address = Address.new({:fname => "first value", :lname => "second value", :country => {:code => "In", :name => "India", :continent => {:name => "asia"}}, :read_test => 'This should not be assigned',:write_test => "this shd be available"})
-      @address.country.continent.class.should == Continent
+      @address = Address.new
+      @address.associations.length.should == 1
+      @address.associations.first.attr_name.should == :country
     end
-    it "should create a new instance and assign_attributes to the associated class" do
-      @address = Address.new({:fname => "first value", :lname => "second value", :country => {:code => "In", :name => "India", :continent => {:name => "asia", :desc => {:this => "is a test", :actual_desc => "is another test"}}}, :read_test => 'This should not be assigned',:write_test => "this shd be available"})
-      @address.country.continent.class.should == Continent
-      @address.country.continent.name.should == "asia"
-      @continent = @address.country.continent
-      @continent.name.should == "asia"
+    describe "has_one" do
+      it "should create a new instance and assign_attributes to the associated class" do
+        @address = Address.new({:fname => "first value", :lname => "second value", :country => {:code => "In", :name => "India"}, :read_test => 'This should not be assigned',:write_test => "this shd be available"})
+        @address.country.class.should == Country
+      end
+      it "should create the nested class instance and assign_attributes to the associated nested class" do
+        @address = Address.new({:fname => "first value", :lname => "second value", :country => {:code => "In", :name => "India", :continent => {:name => "asia"}}, :read_test => 'This should not be assigned',:write_test => "this shd be available"})
+        @address.country.continent.class.should == Continent
+      end
+      it "should create a new instance and assign_attributes to the associated class" do
+        @address = Address.new({:fname => "first value", :lname => "second value", :country => {:code => "In", :name => "India", :continent => {:name => "asia", :desc => {:this => "is a test", :actual_desc => "is another test"}}}, :read_test => 'This should not be assigned',:write_test => "this shd be available"})
+        @address.country.continent.class.should == Continent
+        @address.country.continent.name.should == "asia"
+        @continent = @address.country.continent
+        @continent.name.should == "asia"
+      end
+      it "should assign values to the read only attributes" do
+        @address = Address.new({:fname => "first value", :lname => "second value", :country => {:code => "In", :name => "India"}, :read_test => 'This should be assigned',:write_test => "this shd be available"})
+        @address.read_test.should == "This should be assigned"
+      end
     end
-    it "should assign values to the read only attributes" do
-      @address = Address.new({:fname => "first value", :lname => "second value", :country => {:code => "In", :name => "India"}, :read_test => 'This should be assigned',:write_test => "this shd be available"})
-      @address.read_test.should == "This should be assigned"
+    describe "has_many" do
+      it "should create a new instance and assign attributes for each value in array" do
+        @person = Person.new({ addresses: [{ fname: "first name 1", lname: "last name 1"}, { fname: "first name 2", lname: "last name 2"}]})
+        @person.addresses.length.should == 2
+        @person.addresses.first.class.should == Address
+        @person.addresses.first.fname.should == "first name 1"
+        @person.addresses.first.lname.should == "last name 1"
+        @person.addresses[1].class.should == Address
+        @person.addresses[1].fname.should == "first name 2"
+        @person.addresses[1].lname.should == "last name 2"
+      end
+
+      it "should create each class via factory_method if one is specified" do
+        @person = Person.new({ phones: [{ number: '423-5841'}, {number: '383-9139'}]})
+        @person.phones.length.should == 2
+        @person.phones[0].number.should == '423-5841'
+        @person.phones[0].extension.should == 'set_via_factory'
+        @person.phones[1].extension.should == 'set_via_factory'
+      end
     end
   end
-
-
 end
 
 
 class Person < PlainOldModel::Base
   attr_accessor :fname, :lname, :address
-  #has_one :address
+  has_many :addresses
+  has_many :phones, factory_method: :create
   validates_presence_of :fname
 end
 
+class Phone < PlainOldModel::Base
+  attr_accessor :number, :extension
+
+  def self.create(attributes)
+    attributes[:extension] = 'set_via_factory'
+    new(attributes)
+  end
+end
+
 class Address < PlainOldModel::Base
   attr_accessor :fname, :lname
   attr_reader :read_test

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: plain_old_model
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.2
   prerelease: 
 platform: ruby
 authors:
@@ -10,11 +10,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-02-11 00:00:00.000000000 Z
+date: 2013-02-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &70282593825980 !ruby/object:Gem::Requirement
+  requirement: &70107737128780 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -22,10 +22,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70282593825980
+  version_requirements: *70107737128780
 - !ruby/object:Gem::Dependency
   name: activemodel
-  requirement: &70282593825520 !ruby/object:Gem::Requirement
+  requirement: &70107737128360 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -33,10 +33,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70282593825520
+  version_requirements: *70107737128360
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70282593825100 !ruby/object:Gem::Requirement
+  requirement: &70107737127940 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -44,10 +44,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70282593825100
+  version_requirements: *70107737127940
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70282593824660 !ruby/object:Gem::Requirement
+  requirement: &70107737127440 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -55,10 +55,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70282593824660
+  version_requirements: *70107737127440
 - !ruby/object:Gem::Dependency
   name: rspec-core
-  requirement: &70282593824180 !ruby/object:Gem::Requirement
+  requirement: &70107737126940 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -66,7 +66,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70282593824180
+  version_requirements: *70107737126940
 description: ! "This gem is created to cater the projects which do not require a backend/database,\n
   \ but still need some of the niceties offered by the ActiveRecord"
 email: