Do not use this gem in production

Add env specific stuff not during initialization

Author: Neeraj Singh

lib/mail_interceptor.rb

@@ -12,7 +12,6 @@ def initialize options = {}
       @forward_emails_to = options.fetch :forward_emails_to
       @env               = options.fetch :env, InterceptorEnv.new
 
-      add_env_info_to_subject_prefix
       sanitize_forward_emails_to
     end
 
@@ -40,7 +39,8 @@ def normalize_recipients recipients
     def add_subject_prefix message
       return if subject_prefix.blank?
 
-      message.subject = "#{subject_prefix} #{message.subject}"
+      modified_subject_prefix = add_env_info_to_subject_prefix
+      message.subject = "#{modified_subject_prefix} #{message.subject}"
     end
 
     def sanitize_forward_emails_to
@@ -52,10 +52,7 @@ def sanitize_forward_emails_to
     end
 
     def add_env_info_to_subject_prefix
-      return if subject_prefix.blank?
-
-      _prefix = env.intercept? ? "#{subject_prefix} #{env.name}" : subject_prefix
-      self.subject_prefix = "[#{_prefix}]"
+      "[#{subject_prefix} #{env.name}]"
     end
 
     def forward_emails_to_empty?

test/mail_interceptor_test.rb

@@ -60,16 +60,6 @@ def test_subject_prefix_in_test
     assert_equal "[wheel TEST] Another Forgot password", @message.subject
   end
 
-  def test_subject_prefix_in_production
-    interceptor = ::MailInterceptor::Interceptor.new env: env('production'),
-                                                     forward_emails_to: 'test@example.com',
-                                                     subject_prefix: 'wheel'
-    @message.subject = 'Forgot password'
-
-    interceptor.delivering_email @message
-    assert_equal "[wheel] Forgot password", @message.subject
-  end
-
   def test_error_if_forward_emails_to_is_empty
     message = "forward_emails_to should not be empty"