Find, test and fix cache_versioning exception message

kaspth · Feb 13, 2025 · e21bb24 · e21bb24
1 parent fa9e214
commit e21bb24
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/active_record/associated_object.rb b/lib/active_record/associated_object.rb
@@ -44,7 +44,7 @@ def cache_key_with_version
     def cache_key
       case
       when !record.cache_versioning?
-        raise "ActiveRecord::AssociatedObject#cache_key only supports #{record_klass}.cache_versioning = true"
+        raise "ActiveRecord::AssociatedObject#cache_key only supports #{record.class}.cache_versioning = true"
       when new_record?
         "#{model_name.cache_key}/new"
       else

diff --git a/test/active_record/associated_object_test.rb b/test/active_record/associated_object_test.rb
@@ -117,7 +117,8 @@ class ActiveRecord::AssociatedObjectTest < ActiveSupport::TestCase
 
   test "cache_key integration without cache_versioning" do
     previous_versioning, Post.cache_versioning = Post.cache_versioning, false
-    assert_raises { @publisher.cache_key }
+    error = assert_raises { @publisher.cache_key }
+    assert_match /cache_key.*?Post.cache_versioning = true/, error.message
   ensure
     Post.cache_versioning = previous_versioning
   end