tests for updatability in before_update method

* removed commented code
* removed attempted attribute update in Component::after_destroy (it was
  breaking some tests when in fact I believe this is correct behaviour
* added specific tests for updatability in before/after destroy methods

spec/models.rb

@@ -48,7 +48,7 @@ class Component < ActiveRecord::Base #:nodoc:
   has_many :sub_components, :dependent => :destroy
   NEW_NAME = 'Something Else!'
 
-  after_destroy :change_name
+  before_destroy :change_name
   def change_name
     self.update_attribute(:name, NEW_NAME)
   end
@@ -147,6 +147,16 @@ def ret_false
   end
 end
 
+class ZombiePirate < ActiveRecord::Base #:nodoc:
+  set_table_name :pirates
+  NEW_NAME = 'Undead rising!'
+
+  after_destroy :change_name
+  def change_name
+    self.update_attribute(:name, NEW_NAME)
+  end
+end
+
 class Uuid < ActiveRecord::Base #:nodoc:
   set_primary_key "uuid"
 

spec/paranoid_spec.rb

@@ -4,24 +4,6 @@
 LUKE = 'Luke Skywalker'
 
 describe Paranoid do
-  before(:each) do
-    # Sticker.delete_all
-    # Place.delete_all
-    # Android.delete_all
-    # Person.delete_all
-    # Component.delete_all
-    #
-    # @luke = Person.create(:name => LUKE)
-    # @r2d2 = Android.create(:name => 'R2D2', :owner_id => @luke.id)
-    # @c3p0 = Android.create(:name => 'C3P0', :owner_id => @luke.id)
-    #
-    # @r2d2.components.create(:name => 'Rotors')
-    #
-    # @r2d2.memories.create(:name => 'A pretty sunset')
-    # @c3p0.sticker = Sticker.create(:name => 'OMG, PONIES!')
-    # @tatooine = Place.create(:name => "Tatooine")
-    # @r2d2.places << @tatooine
-  end
 
   describe 'basic functionality' do
     before(:each) do
@@ -268,6 +250,17 @@
       pirate = RandomPirate.create!(:name => 'Roberts')
       lambda { pirate.destroy rescue nil }.should_not change(RandomPirate, :count)
     end
+
+    it 'should allow updates in before_destroy callbacks' do
+      Component.create!(:name => 'bolt').destroy
+      Component.with_destroyed_only.first.name.should eql(Component::NEW_NAME)
+    end
+
+    it 'should not allow updates in after_destroy callbacks' do
+      soon_to_be_undead = ZombiePirate.create!(:name => 'But I feel fine!')
+      lambda { soon_to_be_undead.destroy }.should raise_error
+    end
+
   end
 
   describe 'association destroy' do