diff options
| author | Curtis Hawthorne <curtis@curtis.hawthorne.name> | 2009-03-09 14:56:09 +0000 |
|---|---|---|
| committer | Pratik Naik <pratiknaik@gmail.com> | 2009-03-09 14:56:09 +0000 |
| commit | 0d922885fb54c19f04680482f024452859218910 (patch) | |
| tree | e8ec3cbf4434c9f6a8b26ff27e890a5c1cb7097a /activerecord/test/cases | |
| parent | 1e6c50e21bdb8c99116a7dc6921ef3eb4ed9531a (diff) | |
| download | rails-0d922885fb54c19f04680482f024452859218910.tar.gz rails-0d922885fb54c19f04680482f024452859218910.tar.bz2 rails-0d922885fb54c19f04680482f024452859218910.zip | |
Ensure Model#destroy respects optimistic locking [#1966 state:resolved]
Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
Diffstat (limited to 'activerecord/test/cases')
| -rw-r--r-- | activerecord/test/cases/locking_test.rb | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/activerecord/test/cases/locking_test.rb b/activerecord/test/cases/locking_test.rb index 8d2059d63f..e177235591 100644 --- a/activerecord/test/cases/locking_test.rb +++ b/activerecord/test/cases/locking_test.rb @@ -38,6 +38,24 @@ class OptimisticLockingTest < ActiveRecord::TestCase assert_raise(ActiveRecord::StaleObjectError) { p2.save! } end + def test_lock_destroy + p1 = Person.find(1) + p2 = Person.find(1) + assert_equal 0, p1.lock_version + assert_equal 0, p2.lock_version + + p1.first_name = 'stu' + p1.save! + assert_equal 1, p1.lock_version + assert_equal 0, p2.lock_version + + assert_raises(ActiveRecord::StaleObjectError) { p2.destroy } + + assert p1.destroy + assert_equal true, p1.frozen? + assert_raises(ActiveRecord::RecordNotFound) { Person.find(1) } + end + def test_lock_repeating p1 = Person.find(1) p2 = Person.find(1) |