Remove some warnings

rails · Nov 24, 2014 · d91420f · d91420f
1 parent 8fe1aec
commit d91420f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/activerecord/test/cases/finder_test.rb b/activerecord/test/cases/finder_test.rb
@@ -34,10 +34,10 @@ def test_find
   end
 
   def test_find_with_proc_parameter_and_block
-    e = assert_raises(RuntimeError) do
+    error = assert_raises(RuntimeError) do
       Topic.all.find(-> { raise "should happen" }) { |e| e.title == "non-existing-title" }
     end
-    assert_equal "should happen", e.message
+    assert_equal "should happen", error.message
 
     assert_nothing_raised(RuntimeError) do
       Topic.all.find(-> { raise "should not happen" }) { |e| e.title == topics(:first).title }

diff --git a/activerecord/test/cases/relation/merging_test.rb b/activerecord/test/cases/relation/merging_test.rb
@@ -174,7 +174,7 @@ class MergingDifferentRelationsTest < ActiveRecord::TestCase
     rating_1 = comment_1.ratings.create!
 
     comment_2 = dev.comments.create!(body: "I'm John", post: Post.first)
-    rating_2 = comment_2.ratings.create!
+    comment_2.ratings.create!
 
     assert_equal dev.ratings, [rating_1]
   end