Don't mutate that one variable

lib/griddler/email.rb

@@ -74,12 +74,10 @@ def clean_html(html)
 
     def clean_invalid_utf8_bytes(text)
       if text && !text.valid_encoding?
-        text = text
-          .force_encoding('ISO-8859-1')
-          .encode('UTF-8')
+        text.force_encoding('ISO-8859-1').encode('UTF-8')
+      else
+        text
       end
-
-      text
     end
   end
 end

spec/griddler/email_spec.rb

@@ -3,7 +3,6 @@
 require 'spec_helper'
 
 describe Griddler::Email, 'body formatting' do
-
   it 'uses the html field and sanitizes it when text param missing' do
     body = <<-EOF
       <p>Hello.</p><span>Reply ABOVE THIS LINE</span><p>original message</p>
@@ -624,6 +623,10 @@ def header_from_email(header)
 end
 
 describe Griddler::Email, 'with custom configuration' do
+  before do
+    Griddler.configure
+  end
+
   let(:params) do
     {
       to: ['Some Identifier <some-identifier@example.com>'],
@@ -639,10 +642,6 @@ def header_from_email(header)
     }
   end
 
-  before do
-    Griddler.configure
-  end
-
   describe 'accepts and works with a string reply delimiter' do
     it 'does not split on Reply ABOVE THIS LINE' do
       allow(Griddler.configuration).to receive_messages(reply_delimiter: 'Stuff and things')