closes #195

Author: catmando

ruby/hyper-model/lib/reactive_record/active_record/reactive_record/base.rb

@@ -428,6 +428,7 @@ def load_from_db(*args)
 
     def destroy_associations
       @destroyed = false
+      @being_destroyed = true
       model.reflect_on_all_associations.each do |association|
         if association.collection?
           @attributes[association.attribute].replace([]) if @attributes[association.attribute]

ruby/hyper-model/lib/reactive_record/active_record/reactive_record/setters.rb

@@ -115,9 +115,8 @@ def set_change_status_and_notify_only(attr, changed)
     end
 
     def change_status_and_notify_helper(attr, changed)
+      return if @being_destroyed
       empty_before = changed_attributes.empty?
-      # TODO: confirm this works:
-      # || data_loading? added so that model.new can be wrapped in a ReactiveRecord.load_data
       if !changed || data_loading?
         changed_attributes.delete(attr)
       elsif !changed_attributes.include?(attr)

ruby/hyper-model/spec/batch2/relationships_spec.rb

@@ -120,7 +120,7 @@ def self.add_child
     evaluate_ruby("TestComponent2.add_child")
     page.should have_content("parent has 3 children")
   end
-  
+
   it "preserves the order of children" do
     isomorphic do
       ChildModel.class_eval do
@@ -211,7 +211,7 @@ class TestComponent3 < HyperComponent
       ChildModel.create(child_attribute: :foo, test_model: TestModel.find(1))
       page.should have_content('child id = 2')
       ChildModel.find(1).destroy
-      sleep 0.1 # necessary for poltergeist to work with pusher faker
+      sleep 0.1 # necessary for chrome driver to work with pusher faker
       page.should_not have_content('child id = 1', wait: 2)
     end
 
@@ -228,7 +228,7 @@ class TestComponent3 < HyperComponent
       evaluate_ruby do
         ChildModel.find(1).destroy
       end
-      sleep 0.1 # necessary for poltergeist to work with pusher faker
+      sleep 0.1 # necessary for chrome driver to work with pusher faker
       page.should_not have_content('child id = 1', wait: 2)
     end
 
@@ -242,10 +242,14 @@ class TestComponent3 < HyperComponent
       FactoryBot.create(:child_model, test_model: m)
 
       mount "TestComponent3" do
+        class InnerComponent < HyperComponent
+          param :child
+          render { LI { "child id = #{@Child.id} #{@Child.test_model.test_attribute}"} }
+        end
         class TestComponent3 < HyperComponent
           render(OL) do
             TestModel.all[0].child_models.each do |child|
-              LI { "child id = #{child.id} "}
+              InnerComponent(child: child)
             end
           end
         end
@@ -257,8 +261,9 @@ class TestComponent3 < HyperComponent
       ChildModel.create(child_attribute: :foo, test_model: TestModel.find(1))
       page.should have_content('child id = 2')
       ChildModel.find(1).destroy
-      sleep 0.1 # necessary for poltergeist to work with pusher faker
+      sleep 0.1 # necessary for chrome driver to work with pusher faker
       page.should_not have_content('child id = 1', wait: 2)
+      page.should have_content('child id = 2')
     end
 
     it "will update when sent from the client" do
@@ -275,7 +280,7 @@ class TestComponent3 < HyperComponent
 
         ChildModel.find(1).destroy
       end
-      sleep 0.1 # necessary for poltergeist to work with pusher faker
+      sleep 0.1 # necessary for chrome driver to work with pusher faker
       page.should_not have_content('child id = 1', wait: 2)
     end