From 8298bef72eac980e5a5abab12a9539c0ae1b8ceb Mon Sep 17 00:00:00 2001 From: Jacob Lewallen Date: Wed, 7 Jul 2010 10:55:40 +0900 Subject: [PATCH] Set destroyed=true in opt locking's destroy [#5058 state:resolved] MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: José Valim --- activerecord/lib/active_record/locking/optimistic.rb | 1 + activerecord/test/cases/locking_test.rb | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/activerecord/lib/active_record/locking/optimistic.rb b/activerecord/lib/active_record/locking/optimistic.rb index 373f75e63b..147aa86c60 100644 --- a/activerecord/lib/active_record/locking/optimistic.rb +++ b/activerecord/lib/active_record/locking/optimistic.rb @@ -128,6 +128,7 @@ def destroy_with_lock #:nodoc: end end + @destroyed = true freeze end diff --git a/activerecord/test/cases/locking_test.rb b/activerecord/test/cases/locking_test.rb index cdd6591fca..a2d9d92429 100644 --- 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 -- GitLab