Fixed that pessimistic locking you reference the quoted table name (Josh Susser) [#67 state:resolved]

2 files changed, 9 insertions, 1 deletions

diff --git a/activerecord/test/cases/locking_test.rb b/activerecord/test/cases/locking_test.rb
index e80f902d0d..7db6c570b5 100644
--- a/activerecord/test/cases/locking_test.rb
+++ b/activerecord/test/cases/locking_test.rb
@@ -2,6 +2,7 @@ require "cases/helper"
 require 'models/person'
 require 'models/reader'
 require 'models/legacy_thing'
+require 'models/reference'
 
 class LockWithoutDefault < ActiveRecord::Base; end
 
@@ -15,7 +16,7 @@ class ReadonlyFirstNamePerson < Person
 end
 
 class OptimisticLockingTest < ActiveRecord::TestCase
-  fixtures :people, :legacy_things
+  fixtures :people, :legacy_things, :references
 
   # need to disable transactional fixtures, because otherwise the sqlite3
   # adapter (at least) chokes when we try and change the schema in the middle
@@ -138,6 +139,12 @@ class OptimisticLockingTest < ActiveRecord::TestCase
       end
     end
   end
+  
+  def test_quote_table_name
+    ref = references(:michael_magician)
+    ref.favourite = !ref.favourite
+    assert ref.save
+  end
 
   private
 
diff --git a/activerecord/test/schema/schema.rb b/activerecord/test/schema/schema.rb
index a9b2073b37..e22b78749c 100644
--- a/activerecord/test/schema/schema.rb
+++ b/activerecord/test/schema/schema.rb
@@ -206,6 +206,7 @@ ActiveRecord::Schema.define do
     t.integer :person_id
     t.integer :job_id
     t.boolean :favourite
+    t.integer :lock_version, :default => 0
   end
 
   create_table :minimalistics, :force => true do |t|