From fcf535ac915d5286673858696332450cff4d4558 Mon Sep 17 00:00:00 2001
From: Benjamin Quorning <benjamin@quorning.net>
Date: Tue, 30 Jan 2024 18:36:49 +0100
Subject: [PATCH] Use getter methods instead of instance variables

---
 .../cop/rspec/rails/minitest_assertions.rb    | 22 ++++++++++---------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/lib/rubocop/cop/rspec/rails/minitest_assertions.rb b/lib/rubocop/cop/rspec/rails/minitest_assertions.rb
index 9bdaf6315..8f25d47c5 100644
--- a/lib/rubocop/cop/rspec/rails/minitest_assertions.rb
+++ b/lib/rubocop/cop/rspec/rails/minitest_assertions.rb
@@ -30,22 +30,24 @@ class MinitestAssertions < Base
           class BasicAssertion
             extend NodePattern::Macros
 
+            attr_reader :expected, :actual, :failure_message
+
             def self.minitest_assertion
               raise NotImplementedError
             end
 
-            def initialize(expected, actual, fail_message)
+            def initialize(expected, actual, failure_message)
               @expected = expected&.source
               @actual = actual.source
-              @fail_message = fail_message&.source
+              @failure_message = failure_message&.source
             end
 
             def replaced(node)
               runner = negated?(node) ? 'not_to' : 'to'
-              if @fail_message.nil?
-                "expect(#{@actual}).#{runner} #{assertion}"
+              if failure_message.nil?
+                "expect(#{actual}).#{runner} #{assertion}"
               else
-                "expect(#{@actual}).#{runner}(#{assertion}, #{@fail_message})"
+                "expect(#{actual}).#{runner}(#{assertion}, #{failure_message})"
               end
             end
 
@@ -76,7 +78,7 @@ def self.match(expected, actual, failure_message)
             end
 
             def assertion
-              "eq(#{@expected})"
+              "eq(#{expected})"
             end
           end
 
@@ -98,7 +100,7 @@ def self.match(expected, actual, failure_message)
             end
 
             def assertion
-              "be_an_instance_of(#{@expected})"
+              "be_an_instance_of(#{expected})"
             end
           end
 
@@ -120,7 +122,7 @@ def self.match(collection, expected, failure_message)
             end
 
             def assertion
-              "include(#{@expected})"
+              "include(#{expected})"
             end
           end
 
@@ -144,7 +146,7 @@ def self.match(subject, predicate, failure_message)
             end
 
             def assertion
-              "be_#{@expected.delete_prefix(':').delete_suffix('?')}"
+              "be_#{expected.delete_prefix(':').delete_suffix('?')}"
             end
           end
 
@@ -165,7 +167,7 @@ def self.match(matcher, actual, failure_message)
             end
 
             def assertion
-              "match(#{@expected})"
+              "match(#{expected})"
             end
           end