dbview_cti 0.1.1 → 0.1.2

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 0.1.2 (27/10/2013)
+
+* Fundamental change to how associations are handled in order to solve problems with build
+* Added support for accepts_nested_attributes_for
+* Validation errors on associated models are now correctly handled
+
 ## 0.1.1 (23/10/2013)
 
 * Fixed handling of the remote part of associations (i.e. the belongs_to part when a cti-class has e.g. a has_many association)

data/README.md

@@ -175,8 +175,29 @@ end
 
 ## Associations
 
-Associations (`has_many`, `has_one`, etc.) work and are inherited as you would expect. The only caveat is that 
-in Rails 4 it might be necessary to explicitly specify the join table when using `has_and_belongs_to_many`.
+Associations (`has_many`, `has_one`, etc.) work and are inherited as you would expect. There are two caveats:
+
+* In the base class, you have to call `cti_base_class` before defining any associations:
+
+```ruby
+class Vehicle < ActiveRecord::Base
+  # call cti_base_class first...
+  cti_base_class
+  
+  # ...before defining any associations
+  has_many :parts
+end
+```
+
+* In Rails 4 it might be necessary to explicitly specify the join table when using `has_and_belongs_to_many`:
+
+```ruby
+class SpaceShip < Vehicle
+  cti_derived_class
+
+  has_and_belongs_to_many :astronauts, :join_table => 'astronauts_space_ships'
+end
+```
 
 ## API
 

data/lib/db_view_cti/model/cti.rb

@@ -31,6 +31,7 @@ module DBViewCTI
       end
       
       def convert_to(type)
+        return nil unless persisted?
         type_string = type.to_s
         type_string = type_string.camelize if type.is_a?(Symbol)
         return self if type_string == self.class.name
@@ -48,26 +49,73 @@ module DBViewCTI
         type_string.constantize.find(id.to_i)
       end
       
-      # change destroy and delete methods to operate on most specialized obect
+      # change destroy and delete methods to operate on most specialized object
       included do
-        alias_method_chain :destroy, :specialize
-        alias_method_chain :delete, :specialize
+        alias_method_chain :destroy, :cti
+        alias_method_chain :delete, :cti
         # destroy! seems te be defined in Rails 4
-        alias_method_chain :destroy!, :specialize if self.method_defined?(:destroy!) 
+        alias_method_chain :destroy!, :cti if self.method_defined?(:destroy!)
+
+        # for associations:
+        alias_method_chain :association, :cti
+        validate :cti_validate_associations
+
+        # save callbacks (necessary for saving associations)
+        after_save :cti_save_associations
       end
       
-      def destroy_with_specialize
-        specialize.destroy_without_specialize
+      def destroy_with_cti
+        specialize.destroy_without_cti
       end
       
-      def destroy_with_specialize!
-        specialize.destroy_without_specialize!
+      def destroy_with_cti!
+        specialize.destroy_without_cti!
       end
       
-      def delete_with_specialize
-        specialize.delete_without_specialize
+      def delete_with_cti
+        specialize.delete_without_cti
+      end
+      
+      def cti_validate_associations
+        return_value = true
+        self.class.cti_association_proxies.each_key do |proxy_name|
+          proxy = instance_variable_get(proxy_name)
+          if proxy && !proxy.valid?
+            errors.messages.merge!(proxy.errors.messages)
+            return_value = false
+          end
+        end
+        return_value
+      end
+
+      def cti_save_associations
+        self.class.cti_association_proxies.each_key do |proxy_name|
+          proxy = instance_variable_get(proxy_name)
+          proxy.save if proxy
+        end
+        true
       end
       
+      def association_with_cti(*args)
+        return association_without_cti(*args) unless args.length == 1
+        association_name = args[0]
+        proxy = cti_association_proxy(association_name)
+        proxy ||= self
+        proxy.association_without_cti(association_name)
+      end
+      
+      def cti_association_proxy(association_name)
+        return nil if self.class.reflect_on_all_associations(:belongs_to).map(&:name).include?(association_name.to_sym)
+        proxy_name = self.class.cti_association_proxy_name(association_name)
+        proxy = instance_variable_get(proxy_name)
+        if !proxy && !self.class.cti_has_association?(association_name)
+          instance_variable_set(proxy_name, 
+                                ModelDelegator.new(self, self.class.cti_association_proxies[proxy_name]))
+          proxy = instance_variable_get(proxy_name)
+        end
+        proxy
+      end
+
       module ClassMethods
         
         def cti_base_class?
@@ -78,7 +126,7 @@ module DBViewCTI
           !!@cti_derived_class
         end
         
-        attr_accessor :cti_descendants, :cti_ascendants
+        attr_accessor :cti_descendants, :cti_ascendants, :cti_association_proxies
         
         # registers a derived class and its descendants in the current class
         # class_name: name of derived class (the one calling cti_register_descendants on this class)
@@ -114,7 +162,7 @@ module DBViewCTI
           result
         end
         
-        # redefine association class methods (except for belongs_to)
+        # redefine association class methods
         [:has_many, :has_and_belongs_to_many, :has_one].each do |name|
           self.class_eval <<-eos, __FILE__, __LINE__+1
             def #{name}(*args, &block)
@@ -125,45 +173,20 @@ module DBViewCTI
           eos
         end
 
-        def cti_redefine_associations
-          # redefine associations defined in ascendant classes so they keep working
+        def cti_create_association_proxies
+          # create hash with proxy and class names. The proxies themselves will be created
+          # by the 'association' instance method when the association is used for the first time.
+          @cti_association_proxies ||= {}
           @cti_ascendants.each do |ascendant|
-            [:has_many, :has_and_belongs_to_many].each do |association_type|
+            [:has_many, :has_and_belongs_to_many, :has_one].each do |association_type|
               ascendant.constantize.cti_associations[association_type].each do |association|
-                cti_redefine_to_many(ascendant, association)
+                proxy_name = cti_association_proxy_name(association)
+                @cti_association_proxies[proxy_name] = ascendant
               end
             end
-            ascendant.constantize.cti_associations[:has_one].each do |association|
-              cti_redefine_has_one(ascendant, association)
-            end
           end
         end
         
-        # redefine has_many and has_and_belongs_to_many association
-        def cti_redefine_to_many(class_name, association)
-          plural = association.to_s
-          singular = association.to_s.singularize
-          [ plural, "#{plural}=", "#{singular}_ids", "#{singular}_ids=" ].each do |name|
-            cti_redefine_single_association(name, class_name)
-          end
-        end
-        
-        # redefine has_many and has_and_belongs_to_many association
-        def cti_redefine_has_one(class_name, association)
-          singular = association
-          [ association, "#{association}=", "build_#{association}", "create_#{association}", "create_#{association}!" ].each do |name|
-            cti_redefine_single_association(name, class_name)
-          end
-        end
-        
-        def cti_redefine_single_association(name, class_name)
-          self.class_eval <<-eos, __FILE__, __LINE__+1
-            def #{name}(*args, &block)
-              self.convert_to('#{class_name}').send('#{name}', *args, &block)
-            end
-          eos
-        end
-        
          # fix the 'remote' (i.e. belongs_to) part of any has_one of has_many association in this class
         def cti_redefine_remote_associations
           cti_initialize_cti_associations
@@ -214,12 +237,26 @@ module DBViewCTI
             end
           eos
         end
+        
+        def cti_association_proxy_name(association)
+          "@cti_#{association}_association_proxy"
+        end
 
         def cti_associations
           cti_initialize_cti_associations
           @cti_associations
         end
         
+        def cti_has_association?(association_name)
+          if !@cti_all_associations
+            @cti_all_associations = @cti_associations.keys.inject([]) do |result, key|
+              result += @cti_associations[key]
+              result 
+            end
+          end
+          @cti_all_associations.include?(association_name.to_sym)
+        end
+        
         include DBViewCTI::SQLGeneration::Model
         
         # this method is only used in testing. It returns the number of rows present in the real database
@@ -236,6 +273,7 @@ module DBViewCTI
             @cti_associations[name] ||= []
             @cti_redefined_remote_associations[name] ||= []
           end
+          @cti_association_proxies ||= {}
         end
         
       end

data/lib/db_view_cti/model/extensions.rb

@@ -18,7 +18,7 @@ module DBViewCTI
           @cti_derived_class = true
           self.table_name = DBViewCTI::Names.view_name(self)
           self.superclass.cti_register_descendants(self.name)
-          cti_redefine_associations
+          cti_create_association_proxies
           cti_redefine_remote_associations
           # call redefine_remote_associations on superclass to deal with associations
           # that were defined after the call to cti_derived_class or cti_base_class

data/lib/db_view_cti/model/model_delegator.rb

@@ -0,0 +1,48 @@
+require 'delegate'
+
+module DBViewCTI
+  module Model
+
+    class ModelDelegator < SimpleDelegator
+      
+      attr_reader :cti_target_class
+
+      def initialize(object, target_class)
+        @cti_object = object
+        @cti_converted_object = object.convert_to(target_class)
+        if !@cti_converted_object
+          @cti_converted_object = object.becomes(target_class.constantize)
+          @cti_is_new = true
+        end
+        @cti_target_class = target_class
+        super( @cti_converted_object )
+      end
+      
+      def cti_is_new?
+        @cti_is_new
+      end
+      
+      def save(*args, &block)
+        return super unless cti_is_new?
+        # special case for new objects, we need som hackish id-juggling
+        old_id = @cti_object.id
+        new_id = @cti_object.convert_to( @cti_target_class ).id
+        # since @cti_converted_object was created using 'becomes', @cti_object.id changes
+        # as well in the following statement. So we saved it in old_id and restore it after the
+        # call to save (i.e. super)
+        @cti_object.reload  # only needed in rails 4
+        self.id = new_id
+        self.created_at = @cti_object.created_at  # only needed in rails 4
+        self.updated_at = @cti_object.updated_at  # only needed in rails 4
+        retval = !!super
+        @cti_is_new = false
+        @cti_object.id = old_id
+        @cti_converted_object = @cti_object.convert_to( @cti_target_class )
+        __setobj__(@cti_converted_object)
+        retval
+      end
+      
+    end
+    
+  end
+end

data/lib/db_view_cti/version.rb

@@ -1,3 +1,3 @@
 module DBViewCTI
-  VERSION = "0.1.1"
+  VERSION = "0.1.2"
 end

data/lib/dbview_cti.rb

@@ -28,6 +28,7 @@ module DBViewCTI
     extend ActiveSupport::Autoload
     autoload :CTI
     autoload :Extensions
+    autoload :ModelDelegator
     autoload :CollectionDelegator
   end
 end

data/spec/dummy-rails-3/app/models/category.rb

@@ -0,0 +1,5 @@
+class Category < ActiveRecord::Base
+  attr_accessible :name unless Rails::VERSION::MAJOR > 3
+  has_many :space_ships
+  validates :name, :presence => true
+end

data/spec/dummy-rails-3/app/models/launch.rb

@@ -2,4 +2,6 @@ class Launch < ActiveRecord::Base
   attr_accessible :spache_ship_id, :date unless Rails::VERSION::MAJOR > 3
 
   belongs_to :space_ship
+  
+  validates :date, :presence => true
 end

data/spec/dummy-rails-3/app/models/space_ship.rb

@@ -2,9 +2,18 @@ class SpaceShip < Vehicle
   attr_accessible :single_use, :reliability unless Rails::VERSION::MAJOR > 3
   cti_derived_class
   
+  belongs_to :category
+
   has_many :launches
+  accepts_nested_attributes_for :launches
+
   has_one :captain
+  accepts_nested_attributes_for :captain
+  
   has_and_belongs_to_many :astronauts, :join_table => 'astronauts_space_ships'
+  accepts_nested_attributes_for :astronauts
+
   has_many :experiment_space_ship_performances
   has_many :experiments, :through => :experiment_space_ship_performances
+  accepts_nested_attributes_for :experiments
 end

data/spec/dummy-rails-3/db/migrate/20130816022112_create_categories.rb

@@ -0,0 +1,9 @@
+class CreateCategories < ActiveRecord::Migration
+  def change
+    create_table :categories do |t|
+      t.string :name
+
+      t.timestamps
+    end
+  end
+end

data/spec/dummy-rails-3/db/migrate/20130816022212_create_space_ships.rb

@@ -2,12 +2,15 @@ class CreateSpaceShips < ActiveRecord::Migration
   def change
     create_table :space_ships do |t|
       t.references :vehicle
+
+      t.references :category
       t.boolean :single_use
 
       t.timestamps
     end
     
     add_foreign_key(:space_ships, :vehicles)
+    add_foreign_key(:space_ships, :categories)
     cti_create_view('SpaceShip')
   end
 end

data/spec/dummy-rails-3/db/schema.rb

@@ -39,6 +39,12 @@ ActiveRecord::Schema.define(:version => 20131022030720) do
     t.datetime "updated_at",       :null => false
   end
 
+  create_table "categories", :force => true do |t|
+    t.string   "name"
+    t.datetime "created_at", :null => false
+    t.datetime "updated_at", :null => false
+  end
+
   create_table "experiment_space_ship_performances", :force => true do |t|
     t.integer  "experiment_id"
     t.integer  "space_ship_id"
@@ -84,6 +90,7 @@ ActiveRecord::Schema.define(:version => 20131022030720) do
 
   create_table "space_ships", :force => true do |t|
     t.integer  "vehicle_id"
+    t.integer  "category_id"
     t.boolean  "single_use"
     t.datetime "created_at",  :null => false
     t.datetime "updated_at",  :null => false
@@ -122,6 +129,7 @@ ActiveRecord::Schema.define(:version => 20131022030720) do
 
   add_foreign_key "rocket_engines", "space_ships", :name => "rocket_engines_space_ship_id_fk"
 
+  add_foreign_key "space_ships", "categories", :name => "space_ships_category_id_fk"
   add_foreign_key "space_ships", "vehicles", :name => "space_ships_vehicle_id_fk"
 
   add_foreign_key "space_shuttles", "space_ships", :name => "space_shuttles_space_ship_id_fk"

data/spec/models/space_shuttle_spec.rb

@@ -44,6 +44,39 @@ describe SpaceShuttle do
       launch3.save!
       @shuttle.launch_ids = [ launch1.id, launch3.id ]
       @shuttle.launch_ids.sort.should eq [ launch1.id, launch3.id ]
+      # test build functionality for adding to existing collection
+      expect {
+        @shuttle.launches.build(:date => Date.yesterday)
+        @shuttle.save!
+      }.to change(Launch, :count).by(1)
+      @shuttle.launches.order(:id).last.date.should eq Date.yesterday
+      # test build functionality for new collection
+      @shuttle.launches.clear
+      @shuttle.launch_ids.should eq []
+      expect {
+        @shuttle.name = 'Shuttle'
+        @shuttle.launches.build(:date => Date.yesterday)
+        @shuttle.save!
+      }.to change(Launch, :count).by(1)
+      @shuttle.name.should eq 'Shuttle'
+      # test build functionality for new collection and new object
+      shuttle = SpaceShuttle.new(:name => 'Endeavour', :reliability => 100)
+      expect {
+        expect {
+          shuttle.name = 'Shuttle'
+          shuttle.launches.build(:date => Date.yesterday)
+          shuttle.launches.build(:date => Date.tomorrow)
+          shuttle.save!
+        }.to change(Launch, :count).by(2)
+      }.to change(SpaceShuttle, :count).by(1)
+      shuttle.reload
+      shuttle.name.should eq 'Shuttle'
+      # test adding onto existing collection
+      expect {
+        shuttle.launches.build(:date => Date.today)
+        shuttle.save!
+      }.to change(Launch, :count).by(1)
+      shuttle.launches.order(:id).last.date.should eq Date.today
     end
   
     it "supports assignment on the 'remote' side of a has_many association" do
@@ -59,6 +92,27 @@ describe SpaceShuttle do
         launch.save!
       }.to change(Launch, :count).by(1)
     end
+
+    it "supports accepts_nested_attributes for has_many associations defined in ascendant classes" do
+      expect {
+        @shuttle.launches_attributes = [
+          {:date => Date.today }, {:date => Date.yesterday }
+        ]
+        @shuttle.save!
+      }.to change(Launch, :count).by(2)
+      @shuttle.launches.order(:date).map(&:date).should eq [ Date.yesterday, Date.today ]
+      # do the same for a new object
+      shuttle = SpaceShuttle.new(:name => 'Endeavour', :reliability => 100)
+      expect {
+        expect {
+          shuttle.launches_attributes = [
+            {:date => Date.today }, {:date => Date.yesterday }
+          ]
+          shuttle.save!
+        }.to change(Launch, :count).by(2)
+      }.to change(SpaceShuttle, :count).by(1)
+      shuttle.launches.order(:date).map(&:date).should eq [ Date.yesterday, Date.today ]
+    end
     
     it "can use has_many :through associations defined in ascendant classes" do
       experiment1 = Experiment.new(:name => 'Zero-gravity')
@@ -77,6 +131,41 @@ describe SpaceShuttle do
       @shuttle.experiment_ids = [ experiment1.id, experiment3.id ]
       @shuttle.experiment_ids.sort.should eq [ experiment1.id, experiment3.id ]
       Experiment.last.space_ships.first.specialize.id.should eq @shuttle.id
+      # test build functionality for adding to existing collection
+      expect {
+        @shuttle.experiments.build(:name => 'Superconductivity')
+        @shuttle.save!
+      }.to change(Experiment, :count).by(1)
+      @shuttle.experiments.order(:id).last.name.should eq 'Superconductivity'
+      # test build functionality for new collection
+      @shuttle.experiments.clear
+      @shuttle.experiment_ids.should eq []
+      expect {
+        @shuttle.name = 'Shuttle'
+        @shuttle.experiments.build(:name => 'Failed experiment')
+        @shuttle.save!
+      }.to change(Experiment, :count).by(1)
+      @shuttle.experiments.first.name.should eq 'Failed experiment'
+      @shuttle.name.should eq 'Shuttle'
+      # test build functionality for new collection and new object
+      shuttle = SpaceShuttle.new(:name => 'Endeavour', :reliability => 100)
+      expect {
+        expect {
+          shuttle.name = 'Shuttle'
+          shuttle.experiments.build(:name => 'Exp1')
+          shuttle.experiments.build(:name => 'Exp2')
+          shuttle.save!
+        }.to change(Experiment, :count).by(2)
+      }.to change(SpaceShuttle, :count).by(1)
+      shuttle.reload
+      shuttle.name.should eq 'Shuttle'
+      shuttle.experiments.order(:name).map(&:name).should eq ['Exp1', 'Exp2']
+      # test adding onto existing collection (new object)
+      expect {
+        shuttle.experiments.build(:name => 'Exp3')
+        shuttle.save!
+      }.to change(Experiment, :count).by(1)
+      shuttle.experiments.order(:name).map(&:name).should eq ['Exp1', 'Exp2', 'Exp3']
     end
   
     it "supports operations on the 'remote' side of a has_many :through association" do
@@ -97,6 +186,27 @@ describe SpaceShuttle do
       }.to change(ExperimentSpaceShipPerformance, :count).by(-1)
     end
     
+    it "supports accepts_nested_attributes for has_many :through associations defined in ascendant classes" do
+      expect {
+        @shuttle.experiments_attributes = [
+          {:name => 'Exp1'}, {:name => 'Exp2'}
+        ]
+        @shuttle.save!
+      }.to change(Experiment, :count).by(2)
+      @shuttle.experiments.order(:name).map(&:name).should eq [ 'Exp1', 'Exp2' ]
+      # do the same for a new object
+      shuttle = SpaceShuttle.new(:name => 'Endeavour', :reliability => 100)
+      expect {
+        expect {
+          shuttle.experiments_attributes = [
+            {:name => 'Exp1'}, {:name => 'Exp2'}
+          ]
+          shuttle.save!
+        }.to change(Experiment, :count).by(2)
+      }.to change(SpaceShuttle, :count).by(1)
+      shuttle.experiments.order(:name).map(&:name).should eq [ 'Exp1', 'Exp2' ]
+    end
+    
     it "can use has_one associations defined in ascendant classes" do
       captain = Captain.new(:name => 'Armstrong')
       expect {
@@ -110,11 +220,24 @@ describe SpaceShuttle do
         @shuttle.create_captain(:name => 'Glenn')
       }.to change(Captain, :count).by(1)
       @shuttle.captain.space_ship_id.should eq @shuttle.convert_to(:space_ship).id
+      Captain.all.map(&:destroy)
+      # test build for existing object
       expect {
         cap = @shuttle.build_captain(:name => 'Aldrinn')
-        cap.save!
+        @shuttle.save!
       }.to change(Captain, :count).by(1)
       @shuttle.captain.space_ship_id.should eq @shuttle.convert_to(:space_ship).id
+      # test build for new object
+      shuttle = SpaceShuttle.new(:name => 'Endeavour', :reliability => 100)
+      expect {
+        expect {
+          shuttle.name = 'Shuttle'
+          cap = shuttle.build_captain(:name => 'Aldrinn')
+          shuttle.save!
+        }.to change(Captain, :count).by(1)
+      }.to change(SpaceShuttle, :count).by(1)
+      shuttle.name.should eq 'Shuttle'
+      shuttle.captain.space_ship_id.should eq shuttle.convert_to(:space_ship).id
     end
   
     it "supports operations on the 'remote' side of a has_one association" do
@@ -131,6 +254,23 @@ describe SpaceShuttle do
       }.to change(Captain, :count).by(1)
     end
     
+    it "supports accepts_nested_attributes for has_one associations defined in ascendant classes" do
+      expect {
+        @shuttle.captain_attributes = {:name => 'Haddock'}
+        @shuttle.save!
+      }.to change(Captain, :count).by(1)
+      @shuttle.captain.name.should eq 'Haddock'
+      # do the same for a new object
+      shuttle = SpaceShuttle.new(:name => 'Endeavour', :reliability => 100)
+      expect {
+        expect {
+          shuttle.captain_attributes = {:name => 'Haddock'}
+          shuttle.save!
+        }.to change(Captain, :count).by(1)
+      }.to change(SpaceShuttle, :count).by(1)
+      shuttle.captain.name.should eq 'Haddock'
+    end
+    
     it "can use has_and_belongs_to_many associations defined in ascendant classes" do
       astronaut1 = Astronaut.new(:name => 'Armstrong')
       astronaut2 = Astronaut.new(:name => 'Glenn')
@@ -148,6 +288,41 @@ describe SpaceShuttle do
       astronaut3.save!
       @shuttle.astronaut_ids = [ astronaut1.id, astronaut3.id ]
       @shuttle.astronaut_ids.sort.should eq [ astronaut1.id, astronaut3.id ]
+      # test build functionality for adding to existing collection
+      expect {
+        @shuttle.astronauts.build(:name => 'astro1')
+        @shuttle.save!
+      }.to change(Astronaut, :count).by(1)
+      @shuttle.astronauts.order(:id).last.name.should eq 'astro1'
+      # test build functionality for new collection
+      @shuttle.astronauts.clear
+      @shuttle.astronaut_ids.should eq []
+      expect {
+        @shuttle.name = 'Shuttle'
+        @shuttle.astronauts.build(:name => 'astro2')
+        @shuttle.save!
+      }.to change(Astronaut, :count).by(1)
+      @shuttle.astronauts.first.name.should eq 'astro2'
+      @shuttle.name.should eq 'Shuttle'
+      # test build functionality for new collection and new object
+      shuttle = SpaceShuttle.new(:name => 'Endeavour', :reliability => 100)
+      expect {
+        expect {
+          shuttle.name = 'Shuttle'
+          shuttle.astronauts.build(:name => 'Astro1')
+          shuttle.astronauts.build(:name => 'Astro2')
+          shuttle.save!
+        }.to change(Astronaut, :count).by(2)
+      }.to change(SpaceShuttle, :count).by(1)
+      shuttle.reload
+      shuttle.name.should eq 'Shuttle'
+      shuttle.astronauts.order(:name).map(&:name).should eq ['Astro1', 'Astro2']
+      # test adding onto existing collection (new object)
+      expect {
+        shuttle.astronauts.build(:name => 'Astro3')
+        shuttle.save!
+      }.to change(Astronaut, :count).by(1)
+      shuttle.astronauts.order(:name).map(&:name).should eq ['Astro1', 'Astro2', 'Astro3']
     end
 
     it "supports operations on the 'remote' side of a has_and_belongs_to_many association" do
@@ -167,7 +342,45 @@ describe SpaceShuttle do
       astronaut.space_ships.destroy(@shuttle)
       ActiveRecord::Base.connection().execute(query)[0]['count'].to_i.should be_zero
     end
+
+    it "supports accepts_nested_attributes for has_and_belongs_to_many associations defined in ascendant classes" do
+      expect {
+        @shuttle.astronauts_attributes = [
+          {:name => 'Astro1'}, {:name => 'Astro2'}
+        ]
+        @shuttle.save!
+      }.to change(Astronaut, :count).by(2)
+      @shuttle.astronauts.order(:name).map(&:name).should eq [ 'Astro1', 'Astro2' ]
+      # do the same for a new object
+      shuttle = SpaceShuttle.new(:name => 'Endeavour', :reliability => 100)
+      expect {
+        expect {
+          shuttle.astronauts_attributes = [
+            {:name => 'Astro1'}, {:name => 'Astro2'}
+          ]
+          shuttle.save!
+        }.to change(Astronaut, :count).by(2)
+      }.to change(SpaceShuttle, :count).by(1)
+      shuttle.astronauts.order(:name).map(&:name).should eq [ 'Astro1', 'Astro2' ]
+    end
+        
+    it "doesn't choke on belongs_to associations" do
+      @shuttle.category # should not rais exception
+    end    
     
+    it "doesn't save in case of validation errors in associations defined in ascendant classes" do
+      expect {
+        @shuttle.launches.build
+        @shuttle.save!
+      }.to raise_exception(ActiveRecord::RecordInvalid)
+      # same with new object
+      shuttle = SpaceShuttle.new(:name => 'Endeavour', :reliability => 100)
+      expect {
+        shuttle.launches.build
+        shuttle.save!
+      }.to raise_exception(ActiveRecord::RecordInvalid)
+    end
+
   end
 
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dbview_cti
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-23 00:00:00.000000000 Z
+date: 2013-10-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -81,6 +81,7 @@ files:
 - lib/db_view_cti/model/collection_delegator.rb
 - lib/db_view_cti/model/cti.rb
 - lib/db_view_cti/model/extensions.rb
+- lib/db_view_cti/model/model_delegator.rb
 - lib/db_view_cti/names.rb
 - lib/db_view_cti/railtie.rb
 - lib/db_view_cti/schema_dumper.rb
@@ -104,6 +105,7 @@ files:
 - spec/dummy-rails-3/app/models/astronaut.rb
 - spec/dummy-rails-3/app/models/captain.rb
 - spec/dummy-rails-3/app/models/car.rb
+- spec/dummy-rails-3/app/models/category.rb
 - spec/dummy-rails-3/app/models/experiment.rb
 - spec/dummy-rails-3/app/models/experiment_space_ship_performance.rb
 - spec/dummy-rails-3/app/models/launch.rb
@@ -135,6 +137,7 @@ files:
 - spec/dummy-rails-3/db/migrate/20130713133605_create_motor_vehicles.rb
 - spec/dummy-rails-3/db/migrate/20130713133705_create_cars.rb
 - spec/dummy-rails-3/db/migrate/20130713153631_create_motor_cycles.rb
+- spec/dummy-rails-3/db/migrate/20130816022112_create_categories.rb
 - spec/dummy-rails-3/db/migrate/20130816022212_create_space_ships.rb
 - spec/dummy-rails-3/db/migrate/20130817014120_create_space_shuttles.rb
 - spec/dummy-rails-3/db/migrate/20130817024220_create_rocket_engines.rb
@@ -211,7 +214,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 3228810689469855643
+      hash: -748447197345162789
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -220,7 +223,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 3228810689469855643
+      hash: -748447197345162789
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.25