Set destroyed=true in opt locking's destroy [rails#5058 state:resolved]

Signed-off-by: José Valim <jose.valim@gmail.com>
mislav · Jul 14, 2010 · d10ecfe · d10ecfe
1 parent 16bae77
commit d10ecfe
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/activerecord/lib/active_record/locking/optimistic.rb b/activerecord/lib/active_record/locking/optimistic.rb
@@ -124,6 +124,7 @@ def destroy #:nodoc:
             end
           end
 
+          @destroyed = true
           freeze
         end
 

diff --git a/activerecord/test/cases/locking_test.rb b/activerecord/test/cases/locking_test.rb
@@ -53,7 +53,8 @@ def test_lock_destroy
     assert_raises(ActiveRecord::StaleObjectError) { p2.destroy }
 
     assert p1.destroy
-    assert_equal true, p1.frozen?
+    assert p1.frozen?
+    assert p1.destroyed?
     assert_raises(ActiveRecord::RecordNotFound) { Person.find(1) }
   end