Merge pull request #1318 from sebbASF/part_spec_master

Don't shadow local variable
mikel · Jan 18, 2023 · 9c46620 · 9c46620
2 parents f81a410 + 942cb29
commit 9c46620
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/spec/mail/part_spec.rb b/spec/mail/part_spec.rb
@@ -206,15 +206,15 @@
     end
 
     it "retains specified encoding even though it isn't lowest cost" do
-      part = Mail::Part.new.tap do |part|
+      mail = Mail::Part.new.tap do |part|
         part.header[:content_disposition] = 'attachment; filename="unnamed"'
         part.content_type  = 'text/plain'
         # part.body          = 'a' * 998
         part.body          = 'a' * 999
         part.body.encoding = 'base64'
       end
 
-      part_body_encoding = part.to_s.scan(/Content-Transfer-Encoding: (.+)\r$/).last.first
+      part_body_encoding = mail.to_s.scan(/Content-Transfer-Encoding: (.+)\r$/).last.first
       expect(part_body_encoding).to eq('base64')
     end
   end