mailgun-ruby 1.3.2 → 1.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 58558189e49961ad39adc2090139cfe44158ec4ef56386ce2af0ea1eee0fdaeb
-  data.tar.gz: 82445ca2c1f030e2721c49a0a9226ee39b655ca9864a146d95fb0db303a2e6cf
+  metadata.gz: 898306ea7328a14244502b3d2ff1a17e8bef15cf7e0851bcce8967622a817a7b
+  data.tar.gz: d131c5aa68d89c974e49cbe48a4d225866d035e3b04f39002cd018a3efc806f2
 SHA512:
-  metadata.gz: d7de45e9c8f998fff873aae0c659e0c67a1cd3e59f5fcb9c28c36cbf0c5d52a798f50fc841b8b704837f9e4981dd41f55e9dc7991ada76ccbb1259aeffdcb7f9
-  data.tar.gz: 0dc6b85bcb6517f33c55ee353943e695856d99f2a6b0c7fe956f0f30ecfea3bc436ad962ba676f9780e7bf9959d748ab962f9e6d1c6e9e1a63ae1bb006092f0f
+  metadata.gz: 2fda61d4a1d727a9eecfcde2647414f10db540f3c1409aaa6ef6d35a7825b52c3e8a5f6028b80a6278f435b0fc5b08ba3ddbca73376bab76888b0cba56399b9a
+  data.tar.gz: 27716e8b77cf7e9a3a95f35896f402a62a8f5c31c65caf7fc4ce74f2ceac9f076757a79a5467114443a01a3f9cb983e40631de3921e56729dce4dc39a71170e0

data/.gitignore

@@ -21,3 +21,4 @@ ruby
 .ruby-env.yml
 .rspec-local
 .tool-versions
+.idea

data/lib/mailgun/client.rb

@@ -33,6 +33,7 @@ module Mailgun
       request_options.merge!(request: {timeout: timeout}) if timeout
 
       @http_client =  Faraday.new(request_options) do |conn|
+        conn.request :multipart
         conn.request :authorization, :basic, 'api', api_key
         conn.request :url_encoded
         conn.response :raise_error, include_request: true

data/lib/mailgun/messages/message_builder.rb

@@ -1,5 +1,6 @@
 require 'mime/types'
 require 'time'
+require 'faraday/multipart'
 
 module Mailgun
 
@@ -384,6 +385,16 @@ module Mailgun
       @message[parameter] << (value || '')
     end
 
+    # Adds multipart attachment using Faraday::Multipart::FilePart
+    #
+    # @param [String] parameter The message object parameter name.
+    # @param [String] value The attachment.
+    # @return [void]
+    def add_faraday_attachment(parameter, attachment)
+      content_type = attachment.respond_to?(:content_type) ? attachment.content_type : nil
+      @message[parameter] << Faraday::Multipart::FilePart.new(attachment, content_type)
+    end
+
     # Converts boolean type to string
     #
     # @param [String] value The item to convert
@@ -469,7 +480,7 @@ module Mailgun
         attachment.instance_variable_set :@original_filename, filename
         attachment.instance_eval 'def original_filename; @original_filename; end'
       end
-      set_multi_complex(disposition, attachment)
+      add_faraday_attachment(disposition, attachment)
     end
   end
 

data/lib/mailgun/version.rb

@@ -1,4 +1,4 @@
 # It's the version. Yeay!
 module Mailgun
-  VERSION = '1.3.2'
+  VERSION = '1.3.3'
 end

data/mailgun.gemspec

@@ -28,14 +28,16 @@ Gem::Specification.new do |spec|
   spec.required_ruby_version = '>= 2.2.2'
 
   spec.add_development_dependency 'bundler', '>= 1.16.2'
-  spec.add_development_dependency 'rspec', '~> 3.8.0'
+  spec.add_development_dependency 'rspec', '~> 3.13.0'
+  spec.add_development_dependency 'rspec-its', '~> 2.0.0'
   spec.add_development_dependency 'rake', '~> 12.3.2'
   spec.add_development_dependency 'webmock', '~> 3.7'
-  spec.add_development_dependency 'pry', '~> 0.11.3'
+  spec.add_development_dependency 'pry', '~> 0.15.2'
   spec.add_development_dependency 'vcr', '~> 3.0.3'
   spec.add_development_dependency 'simplecov', '~> 0.16.1'
   spec.add_development_dependency 'rails'
   spec.add_dependency 'mime-types'
   spec.add_dependency 'faraday', "~> 2.1"
+  spec.add_dependency 'faraday-multipart', '~> 1.1.0'
 
 end

data/spec/integration/domains_spec.rb

@@ -4,54 +4,67 @@ require 'mailgun'
 vcr_opts = { :cassette_name => "domains" }
 
 describe 'For the domains endpoint', vcr: vcr_opts do
-  before(:all) do
-    @mg_client = Mailgun::Client.new(APIKEY, APIHOST, APIVERSION, SSL)
-    @mg_obj = Mailgun::Domains.new(@mg_client)
-    @domain = "integration-test.domain.invalid"
-  end
+  let(:api_version) { APIVERSION }
+  let(:mg_client) { Mailgun::Client.new(APIKEY, APIHOST, api_version, SSL) }
+  let(:mg_obj) { Mailgun::Domains.new(mg_client) }
+  let(:domain) { 'integration-test.domain.invalid' }
+
+  describe '#add_domain' do
+    let(:api_version) { 'v4' }
 
-  it 'creates the domain' do
-    result = @mg_obj.add_domain(@domain, { smtp_password: 'super_secret', spam_action: 'tag' })
+    it 'creates a domain' do
+      response = mg_obj.add_domain(domain, { spam_action: 'tag' })
 
-    expect(result['domain']["name"]).to eq(@domain)
-    expect(result['domain']["spam_action"]).to eq("tag")
-    expect(result['domain']["smtp_password"]).to eq("super_secret")
+      expect(response['domain']['name']).to eq(domain)
+      expect(response['domain']['spam_action']).to eq('tag')
+    end
   end
 
-  it 'get the domain.' do
-    result = @mg_obj.get(@domain)
+  describe '#get' do
+    let(:api_version) { 'v4' }
+
+    it 'returns the domain' do
+      response = mg_obj.get(domain)
 
-    expect(result).to include("domain")
-    expect(result["domain"]["name"]).to eq(@domain)
+      expect(response).to include('domain')
+      expect(response['domain']['name']).to eq(domain)
+    end
   end
 
-  it 'gets a list of domains.' do
-    result = @mg_obj.get_domains
+  describe '#get_domains' do
+    let(:api_version) { 'v4' }
+
+    it 'returns a list of domains' do
+      response = mg_obj.get_domains
 
-    expect(result.size).to be > 0
+      expect(response.size).to be > 0
+    end
   end
 
-  it 'deletes a domain.' do
-    result = @mg_obj.delete(@domain)
+  context '#update' do
+    let(:api_version) { 'v4' }
 
-    expect(result).to be_truthy
+    it 'updates the domain' do
+      response = mg_obj.update(domain, { spam_action: 'block', web_scheme: 'https', wildcard: true })
+
+      expect(response['domain']['spam_action']).to eq('block')
+      expect(response['domain']['web_scheme']).to eq('https')
+      expect(response['domain']['wildcard']).to eq(true)
+    end
   end
 
-  it 'updates the domain' do
-    result = @mg_obj.update(@domain, { spam_action: 'block', web_scheme: 'https', wildcard: true })
+  context 'delete a domain' do
+    subject(:response) { mg_obj.delete(domain) }
 
-    expect(result['domain']["spam_action"]).to eq('block')
-    expect(result['domain']["web_scheme"]).to eq('https')
-    expect(result['domain']["wildcard"]).to eq(true)
+    it { is_expected.to be_falsey }
   end
 
   describe '#create_smtp_credentials' do
     it 'creates smtp credentials for domain' do
-      result = @mg_obj.create_smtp_credentials(
-        @domain,
+      result = mg_obj.create_smtp_credentials(
+        domain,
         {
-          login: 'test_login',
-          password: 'test_password'
+          login: 'test_login'
         }
       )
 
@@ -61,11 +74,11 @@ describe 'For the domains endpoint', vcr: vcr_opts do
 
   describe '#update_smtp_credentials' do
     it 'updates smtp credentials for domain' do
-      result = @mg_obj.update_smtp_credentials(
-        @domain,
+      result = mg_obj.update_smtp_credentials(
+        domain,
         'test_login',
         {
-          password: 'test_password2'
+          spec: 'abc'
         }
       )
 
@@ -75,19 +88,8 @@ describe 'For the domains endpoint', vcr: vcr_opts do
 
   describe '#delete_smtp_credentials' do
     it 'deletes smtp credentials for domain' do
-      result = @mg_obj.delete_smtp_credentials(
-        @domain,
-        'test_login'
-      )
-
-      expect(result['message']).to eq('Credentials have been deleted')
-    end
-  end
-
-  describe '#delete_smtp_credentials' do
-    it 'deletes smtp credentials for domain' do
-      result = @mg_obj.delete_smtp_credentials(
-        @domain,
+      result = mg_obj.delete_smtp_credentials(
+        domain,
         'test_login'
       )
 
@@ -97,8 +99,8 @@ describe 'For the domains endpoint', vcr: vcr_opts do
 
   describe '#get_domain_connection_settings' do
     it 'returns delivery connection settings for the defined domain' do
-      result = @mg_obj.get_domain_connection_settings(
-        @domain
+      result = mg_obj.get_domain_connection_settings(
+        domain
       )
 
       expect(result).to include(
@@ -110,8 +112,8 @@ describe 'For the domains endpoint', vcr: vcr_opts do
 
   describe '#update_domain_connection_settings' do
     it 'updates the specified delivery connection settings' do
-      result = @mg_obj.update_domain_connection_settings(
-        @domain,
+      result = mg_obj.update_domain_connection_settings(
+        domain,
         {
           require_tls: true,
           skip_verification: true
@@ -127,19 +129,19 @@ describe 'For the domains endpoint', vcr: vcr_opts do
 
   describe '#get_domain_tracking_settings' do
     it 'returns tracking settings for the defined domain' do
-      result = @mg_obj.get_domain_tracking_settings(
-        @domain
+      result = mg_obj.get_domain_tracking_settings(
+        domain
       )
 
       expect(result).to include('tracking')
-      expect(result['tracking']['click']['active']).to eq(true)
+      expect(result['tracking']['click']['active']).to eq(false)
     end
   end
 
   describe '#update_domain_tracking_open_settings' do
     it 'updates the specified tracking open settings' do
-      result = @mg_obj.update_domain_tracking_open_settings(
-        @domain,
+      result = mg_obj.update_domain_tracking_open_settings(
+        domain,
         {
           active: false
         }
@@ -151,8 +153,8 @@ describe 'For the domains endpoint', vcr: vcr_opts do
 
   describe '#update_domain_tracking_click_settings' do
     it 'updates the specified tracking click settings' do
-      result = @mg_obj.update_domain_tracking_click_settings(
-        @domain,
+      result = mg_obj.update_domain_tracking_click_settings(
+        domain,
         {
           active: false
         }
@@ -164,8 +166,8 @@ describe 'For the domains endpoint', vcr: vcr_opts do
 
   describe '#update_domain_tracking_unsubscribe_settings' do
     it 'updates the specified tracking unsubscribe settings' do
-      result = @mg_obj.update_domain_tracking_unsubscribe_settings(
-        @domain,
+      result = mg_obj.update_domain_tracking_unsubscribe_settings(
+        domain,
         {
           active: false
         }
@@ -177,34 +179,34 @@ describe 'For the domains endpoint', vcr: vcr_opts do
 
   describe '#update_domain_dkim_authority' do
     it 'updates the DKIM authority for a domain' do
-      result = @mg_obj.update_domain_dkim_authority(
-        @domain,
+      result = mg_obj.update_domain_dkim_authority(
+        domain,
         {
-          active: false
+          self: true
         }
       )
 
-      expect(result['message']).to eq('Domain DKIM authority has been changed')
+      expect(result['message']).to eq('Domain DKIM authority has not been changed')
     end
   end
 
   describe '#update_domain_dkim_selector' do
     it 'updates the DKIM selector for a domain' do
-      result = @mg_obj.update_domain_dkim_selector(
-        @domain,
+      result = mg_obj.update_domain_dkim_selector(
+        domain,
         {
           dkim_selector: 'mailo1'
         }
       )
 
-      expect(result['message']).to eq('Domain DKIM authority changed')
+      expect(result['message']).to eq('DKIM selector changed')
     end
   end
 
   describe '#update_domain_web_prefix' do
     it 'updates the the CNAME used for tracking opens and clicks' do
-      result = @mg_obj.update_domain_web_prefix(
-        @domain,
+      result = mg_obj.update_domain_web_prefix(
+        domain,
         {
           web_prefix: 'email'
         }
@@ -215,15 +217,12 @@ describe 'For the domains endpoint', vcr: vcr_opts do
   end
 
   describe 'V4' do
-    before do
-      @mg_client = Mailgun::Client.new(APIKEY, APIHOST, 'v4', SSL)
-      @mg_obj = Mailgun::Domains.new(@mg_client)
-    end
+    let(:api_version) { 'v4' }
 
     describe '#get_domain_keys' do
       it 'lists the domain keys for a specified signing domain' do
-        result = @mg_obj.get_domain_keys(
-          @domain
+        result = mg_obj.get_domain_keys(
+          domain
         )
 
         expect(result).to include('items')
@@ -233,9 +232,10 @@ describe 'For the domains endpoint', vcr: vcr_opts do
 
     describe '#activate_domain_key' do
       it 'activates a domain key' do
-        result = @mg_obj.activate_domain_key(
-          @domain,
-          'smtp'
+
+        result = mg_obj.activate_domain_key(
+          domain,
+          'mailo1'
         )
 
        expect(result['message']).to eq('domain key activated')
@@ -244,11 +244,9 @@ describe 'For the domains endpoint', vcr: vcr_opts do
 
     describe '#deactivate_domain_key' do
       it 'deactivates a domain key' do
-        result = @mg_obj.deactivate_domain_key(
-          {
-            signing_domain: 'x509.zeefarmer.com',
-            selector: 'tetetet'
-          }
+        result = mg_obj.deactivate_domain_key(
+          domain,
+          'tetetet'
         )
 
        expect(result['message']).to eq('domain key deactivated')
@@ -257,15 +255,12 @@ describe 'For the domains endpoint', vcr: vcr_opts do
   end
 
   describe '#delete_domain_key' do
-    before do
-      @mg_client = Mailgun::Client.new(APIKEY, APIHOST, 'v1', SSL)
-      @mg_obj = Mailgun::Domains.new(@mg_client)
-    end
+    let(:api_version) { 'v1' }
 
     it 'deletes a domain key' do
-      result = @mg_obj.delete_domain_key(
+      result = mg_obj.delete_domain_key(
         {
-          signing_domain: @domain,
+          signing_domain: domain,
           selector: 'test'
         }
       )
@@ -276,16 +271,24 @@ describe 'For the domains endpoint', vcr: vcr_opts do
 
   describe '#get_domain_stats' do
     it 'returns total stats for a given domain' do
-      result = @mg_obj.get_domain_stats(
-        @domain,
+      result = mg_obj.get_domain_stats(
+        domain,
         {
-          event: 'clicked',
-          start: 'Sun, 23 Dec 2023 01:23:45 JST',
-          duration: '24h'
+          event: 'clicked'
         }
       )
 
      expect(result).to include('stats')
     end
   end
+
+  def create_mg_object(version)
+    mg_client = Mailgun::Client.new(APIKEY, APIHOST, version, SSL)
+    Mailgun::Domains.new(mg_client)
+  end
+
+  def create_domain
+    mg_obj = create_mg_object('v4')
+    mg_obj.add_domain(domain)
+  end
 end

data/spec/integration/email_validation_spec.rb

@@ -16,6 +16,7 @@ describe 'For the email validation endpoint', order: :defined, vcr: vcr_opts do
   end
 
   it 'returns parsed and unparsable lists' do
+    skip 'is parse method removed?'
     res = @mg_obj.parse(@all_addrs)
 
     expect(res["parsed"]).to eq(@valid)
@@ -58,7 +59,7 @@ describe 'For the email validation endpoint', order: :defined, vcr: vcr_opts do
         "is_role_address" => false,
         "is_valid" => false,
         "mailbox_verification" => "unknown",
-        "reason" => "Validation failed for 'example.org', reason: 'malformed address; missing @ sign'",
+        #"reason" => "Validation failed for 'example.org', reason: 'malformed address; missing @ sign'",
         "parts" => {
             "display_name" => nil,
             "domain" => nil,

data/spec/integration/list_spec.rb

@@ -6,7 +6,7 @@ vcr_opts = { :cassette_name => "mailing_list" }
 describe 'For the Mailing Lists endpoint', vcr: vcr_opts do
   before(:all) do
     @mg_obj = Mailgun::Client.new(APIKEY, APIHOST, APIVERSION, SSL)
-    @domain = TESTDOMAIN
+    @domain = TESTDOMAIN || 'DOMAIN.TEST'
     @ml_address = "integration_test_list@#{@domain}"
   end
 

data/spec/integration/mailer_spec.rb

@@ -9,7 +9,7 @@ ActionMailer::Base.raise_delivery_errors = true
 Rails.logger = Logger.new('/dev/null')
 Rails.logger.level = Logger::DEBUG
 
-class UnitTestMailer < ActionMailer::Base
+class IntegrationUnitTestMailer < ActionMailer::Base
   default from: 'unittest@example.org'
 
   def plain_message(address, from, subject, headers)
@@ -24,14 +24,14 @@ end
 vcr_opts = { :cassette_name => 'message_deliver' }
 
 describe 'Message deliver', vcr: vcr_opts do
-  let(:domain) { TESTDOMAIN }
+  let(:domain) { TESTDOMAIN || 'DOMAIN.TEST' }
   let(:config) do
     {
       api_key: APIKEY,
       domain: domain
     }
   end
-  let(:mail) { UnitTestMailer.plain_message("bob@#{domain}", "bob@#{domain}", 'subject', {}) }
+  let!(:mail) { IntegrationUnitTestMailer.plain_message("bob@#{domain}", "bob@#{domain}", 'subject', {}) }
 
   it 'successfully delivers message' do
     result = Railgun::Mailer.new(config).deliver!(mail)
@@ -53,9 +53,9 @@ describe 'Invalid domain', vcr: vcr_opts do
       domain: domain
     }
   end
-  let(:mail) { UnitTestMailer.plain_message('sally@not-our-doma.in', "bob@#{domain}", 'subject', {}) }
+  let(:mail) { IntegrationUnitTestMailer.plain_message('sally@not-our-doma.in', "bob@#{domain}", 'subject', {}) }
 
   it 'raises expected error' do
-    expect { Railgun::Mailer.new(config).deliver!(mail) }.to raise_error Mailgun::CommunicationError, /Forbidden/
+    expect { Railgun::Mailer.new(config).deliver!(mail) }.to raise_error Mailgun::Unauthorized, /Invalid Domain or API key/
   end
 end

data/spec/integration/mailgun_spec.rb

@@ -15,7 +15,7 @@ vcr_opts = { :cassette_name => "exceptions" }
 describe 'Client exceptions', vcr: vcr_opts do
   before(:all) do
     @mg_obj = Mailgun::Client.new(APIKEY, APIHOST, APIVERSION, SSL)
-    @domain = TESTDOMAIN
+    @domain = TESTDOMAIN || 'DOMAIN.TEST'
   end
 
   it 'display useful error information' do
@@ -27,7 +27,7 @@ describe 'Client exceptions', vcr: vcr_opts do
           :text => 'INTEGRATION TESTING'
       })
     rescue Mailgun::CommunicationError => err
-      expect(err.message).to eq('404 Not Found: Domain not found: not-our-doma.in')
+      expect(err.message).to eq('the server responded with status 404')
     else
       fail
     end
@@ -39,7 +39,7 @@ vcr_opts = { :cassette_name => "exceptions-invalid-api-key" }
 describe 'Client exceptions', vcr: vcr_opts do
   before(:all) do
     @mg_obj = Mailgun::Client.new(APIKEY, APIHOST, APIVERSION, SSL)
-    @domain = TESTDOMAIN
+    @domain = TESTDOMAIN || 'DOMAIN.TEST'
   end
 
   it 'displays error information that API key is invalid' do
@@ -63,7 +63,7 @@ vcr_opts = { :cassette_name => "exceptions-invalid-data" }
 describe 'Client exceptions', vcr: vcr_opts do
   before(:all) do
     @mg_obj = Mailgun::Client.new(APIKEY, APIHOST, APIVERSION, SSL)
-    @domain = TESTDOMAIN
+    @domain = TESTDOMAIN || 'DOMAIN.TEST'
   end
 
   it 'display useful error information' do
@@ -87,7 +87,7 @@ vcr_opts = { :cassette_name => "exceptions-not-allowed" }
 describe 'Client exceptions', vcr: vcr_opts do
   before(:all) do
     @mg_obj = Mailgun::Client.new(APIKEY, APIHOST, APIVERSION, SSL)
-    @domain = TESTDOMAIN
+    @domain = TESTDOMAIN || 'DOMAIN.TEST'
   end
 
   it 'display useful error information' do
@@ -111,10 +111,11 @@ vcr_opts = { :cassette_name => "send_message" }
 describe 'The method send_message()', vcr: vcr_opts do
   before(:all) do
     @mg_obj = Mailgun::Client.new(APIKEY, APIHOST, APIVERSION, SSL)
-    @domain = TESTDOMAIN
+    @domain = TESTDOMAIN || 'DOMAIN.TEST'
   end
 
   it 'sends a standard message in test mode.' do
+    @mg_obj.enable_test_mode!
     result = @mg_obj.send_message(@domain, {:from => "bob@#{@domain}",
                                     :to => "sally@#{@domain}",
                                     :subject => 'Hash Integration Test',
@@ -183,7 +184,8 @@ Sender: me@samples.mailgun.org
 
 Testing some Mailgun awesomness!'
 
-    message_params = {:to => "sally@#{@domain}",
+    message_params = {:from => "bobby@#{@domain}",
+                      :to => "sally@#{@domain}",
                       :message => mime_string}
 
     result = @mg_obj.send_message(@domain, message_params)

data/spec/integration/stats_spec.rb

@@ -6,7 +6,7 @@ vcr_opts = { :cassette_name => "stats" }
 describe 'For the Stats endpoint', vcr: vcr_opts do
   before(:all) do
     @mg_obj = Mailgun::Client.new(APIKEY, APIHOST, APIVERSION, SSL)
-    @domain = TESTDOMAIN
+    @domain = TESTDOMAIN || 'DOMAIN.TEST'
   end
 
   it 'get some stats.' do

data/spec/integration/unsubscribes_spec.rb

@@ -6,8 +6,8 @@ vcr_opts = { :cassette_name => "unsubscribes" }
 describe 'For the Unsubscribes endpoint', order: :defined, vcr: vcr_opts do
   before(:all) do
     @mg_obj = Mailgun::Client.new(APIKEY, APIHOST, APIVERSION, SSL)
-    @domain = TESTDOMAIN
-    @email = "integration-test-email@#{TESTDOMAIN}"
+    @domain = TESTDOMAIN || 'DOMAIN.TEST'
+    @email = "integration-test-email@#{@domain}"
   end
 
   it 'adds an unsubscriber' do

data/spec/integration/webhook_spec.rb

@@ -6,7 +6,7 @@ vcr_opts = { :cassette_name => "webhooks" }
 describe 'For the webhooks endpoint', order: :defined, vcr: vcr_opts do
   before(:all) do
     @mg_obj = Mailgun::Client.new(APIKEY, APIHOST, APIVERSION, SSL)
-    @domain = TESTDOMAIN
+    @domain = 'DOMAIN.TEST'
     @testhook = 'accepted'
     @testhookup = 'accepted'
   end

data/spec/spec_helper.rb

@@ -13,6 +13,7 @@ require 'mailgun'
 
 require 'vcr'
 require 'webmock/rspec'
+require 'rspec/its'
 
 #WebMock.disable_net_connect!(allow_localhost: true)
 require_relative 'unit/connection/test_client'

data/spec/unit/events/events_spec.rb

@@ -20,7 +20,7 @@ describe 'Pagination' do
     json = JSON.parse(result.body)
     expect(json).to include("paging")
     expect(json["paging"]).to include("next")
-    expect(json["paging"]).to include{"previous"}
+    expect(json["paging"]).to include("previous")
   end
 
   it 'should calculate proper next-page url' do

data/spec/unit/messages/message_builder_spec.rb

@@ -265,7 +265,7 @@ describe 'The method add_attachment' do
     @mb_obj.add_attachment io, 'mailgun_icon.png'
 
     expect(@mb_obj.message[:attachment].length).to eq(1)
-    expect(@mb_obj.message[:attachment].first.original_filename).to eq 'mailgun_icon.png'
+    expect(@mb_obj.message[:attachment].first.io.original_filename).to eq 'mailgun_icon.png'
   end
 
   context 'when attachment has unknown type' do