mailgunner 2.3.0 → 2.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4f2850f4c4d7090a89e92f1f892da73b54dbdce6
-  data.tar.gz: 793b27238fa767fe883dc634442715fa355f958c
+  metadata.gz: e755c6f546bd421940dffab1c41d0b803842b7d4
+  data.tar.gz: ec6bde54051dcdfd4a28dfcc6dfb9ca9b13bd2b0
 SHA512:
-  metadata.gz: 49a2692751d5a67f6fb05aaebf2c73fcd389aa2841311de02dea82a1e156471e3b6802c5e2e7813e08ed5f1bbe83fb3993d451b0f946254ddba7a63274d9cf94
-  data.tar.gz: ddd64af9eb4d81e9c9ddb6734d647681e8f82f863f732967c398a4a6b91a980c1d0f5a98a6dade7fa33d228b669f665209fad453016bac21fbcffa68d4f8a7af
+  metadata.gz: 71f7726130d76a1163d717b5e3437e4a6adecd4fab2a40ceeee62d05bbd7b9c180a930ba95b04744494cfa0288275938d2d8da4619d70d6c38ba719935b32e37
+  data.tar.gz: 3ddcc963e98c68ed4855d0d895c67b5c3511b6a5793c3c0c3f8dda2c2888183a9195615cbc29c677938d59ae9bd7004b91ce7d9b821a2c9138eeda24aca22b17

data/lib/mailgunner.rb

@@ -1,6 +1,7 @@
 require 'net/http'
 require 'json'
 require 'cgi'
+require 'mailgunner/version'
 require 'mailgunner/delivery_method' if defined?(Mail)
 require 'mailgunner/railtie' if defined?(Rails)
 
@@ -96,6 +97,14 @@ module Mailgunner
       delete("/v3/domains/#{escape @domain}/credentials/#{escape login}")
     end
 
+    def get_connection_settings
+      get("/v3/domains/#{escape @domain}/connection")
+    end
+
+    def update_connection_settings(attributes)
+      put("/v3/domains/#{escape @domain}/connection", attributes)
+    end
+
     def get_unsubscribes(params = {})
       get("/v3/#{escape @domain}/unsubscribes", params)
     end
@@ -144,6 +153,10 @@ module Mailgunner
       delete("/v3/#{escape @domain}/bounces/#{escape address}")
     end
 
+    def delete_bounces
+      delete("/v3/#{escape @domain}/bounces")
+    end
+
     def get_stats(params = {})
       Kernel.warn 'Mailgunner::Client#get_stats is deprecated'
 
@@ -158,6 +171,26 @@ module Mailgunner
       get("/v3/#{escape @domain}/events", params)
     end
 
+    def get_tags(params = {})
+      get("/v3/#{escape @domain}/tags", params)
+    end
+
+    def get_tag(id)
+      get("/v3/#{escape @domain}/tags/#{escape id}")
+    end
+
+    def update_tag(id, attributes)
+      put("/v3/#{escape @domain}/tags/#{escape id}", attributes)
+    end
+
+    def get_tag_stats(id, params)
+      get("/v3/#{escape @domain}/tags/#{escape id}/stats", params)
+    end
+
+    def delete_tag(id)
+      delete("/v3/#{escape @domain}/tags/#{escape id}")
+    end
+
     def get_routes(params = {})
       get('/v3/routes', params)
     end
@@ -288,8 +321,11 @@ module Mailgunner
       transmit(Net::HTTP::Delete.new(path))
     end
 
+    USER_AGENT = "Ruby/#{RUBY_VERSION} Mailgunner/#{VERSION}"
+
     def transmit(message)
       message.basic_auth('api', @api_key)
+      message['User-Agent'] = USER_AGENT
 
       yield message if block_given?
 

data/lib/mailgunner/railtie.rb

@@ -1,9 +1,7 @@
 module Mailgunner
   class Railtie < Rails::Railtie
-    initializer 'mailgunner' do
-      ActiveSupport.on_load(:action_mailer) do
-        require 'mailgunner/delivery_method'
-      end
+    config.before_initialize do
+      require 'mailgunner/delivery_method'
     end
   end
 end

data/lib/mailgunner/version.rb

@@ -0,0 +1,3 @@
+module Mailgunner
+  VERSION = '2.4.0'
+end

data/mailgunner.gemspec

@@ -1,6 +1,8 @@
+require_relative './lib/mailgunner/version'
+
 Gem::Specification.new do |s|
   s.name = 'mailgunner'
-  s.version = '2.3.0'
+  s.version = Mailgunner::VERSION
   s.license = 'LGPL-3.0'
   s.platform = Gem::Platform::RUBY
   s.authors = ['Tim Craft']

data/spec/mailgunner_spec.rb

@@ -26,6 +26,8 @@ describe 'Mailgunner::Client' do
     @encoded_address = 'user%40example.com'
 
     @login = 'bob.bar'
+
+    @id = 'idxxx'
   end
 
   describe 'http method' do
@@ -81,37 +83,27 @@ describe 'Mailgunner::Client' do
   end
 
   describe 'get_message method' do
-    it 'fetches the domain message resource with the given key and returns the response object' do
-      key = 'WyIzMmkey'
-
-      stub_request(:get, "#@base_url/domains/#@domain/messages/#{key}").to_return(@json_response)
+    it 'fetches the domain message resource with the given id and returns the response object' do
+      stub_request(:get, "#@base_url/domains/#@domain/messages/#@id").to_return(@json_response)
 
-      @client.get_message(key).must_equal(@json_response_object)
+      @client.get_message(@id).must_equal(@json_response_object)
     end
   end
 
   describe 'get_mime_message method' do
     it 'fetches the domain message resource with the given key and returns the response object' do
-      key = 'WyIzMmkey'
-
-      stub_request(:get, "#@base_url/domains/#@domain/messages/#{key}").
+      stub_request(:get, "#@base_url/domains/#@domain/messages/#@id").
         with(headers: {'Accept' => 'message/rfc2822'}).to_return(@json_response)
 
-      @client.get_mime_message(key).must_equal(@json_response_object)
+      @client.get_mime_message(@id).must_equal(@json_response_object)
     end
   end
 
   describe 'send_message method' do
     it 'posts to the domain messages resource and returns the response object' do
-      stub_request(:post, "#@base_url/#@domain/messages").to_return(@json_response)
+      stub_request(:post, "#@base_url/#@domain/messages").with(body: "to=#@encoded_address").to_return(@json_response)
 
-      @client.send_message({}).must_equal(@json_response_object)
-    end
-
-    it 'encodes the message attributes' do
-      stub_request(:post, "#@base_url/#@domain/messages").with(body: "to=#@encoded_address")
-
-      @client.send_message({to: @address})
+      @client.send_message({to: @address}).must_equal(@json_response_object)
     end
 
     it 'raises an exception if the domain is not provided' do
@@ -158,11 +150,9 @@ describe 'Mailgunner::Client' do
 
   describe 'delete_message method' do
     it 'deletes the domain message resource with the given key and returns the response object' do
-      key = 'WyIzMmkey'
-
-      stub_request(:delete, "#@base_url/domains/#@domain/messages/#{key}").to_return(@json_response)
+      stub_request(:delete, "#@base_url/domains/#@domain/messages/#@id").to_return(@json_response)
 
-      @client.delete_message(key)
+      @client.delete_message(@id).must_equal(@json_response_object)
     end
   end
 
@@ -184,15 +174,9 @@ describe 'Mailgunner::Client' do
 
   describe 'add_domain method' do
     it 'posts to the domains resource and returns the response object' do
-      stub_request(:post, "#@base_url/domains").to_return(@json_response)
-
-      @client.add_domain({}).must_equal(@json_response_object)
-    end
-
-    it 'encodes the domain attributes' do
-      stub_request(:post, "#@base_url/domains").with(body: "name=#@domain")
+      stub_request(:post, "#@base_url/domains").with(body: "name=#@domain").to_return(@json_response)
 
-      @client.add_domain({name: @domain})
+      @client.add_domain({name: @domain}).must_equal(@json_response_object)
     end
   end
 
@@ -222,15 +206,9 @@ describe 'Mailgunner::Client' do
 
   describe 'update_credentials method' do
     it 'updates the domain credentials resource with the given login and returns the response object' do
-      stub_request(:put, "#@base_url/domains/#@domain/credentials/#@login").to_return(@json_response)
+      stub_request(:put, "#@base_url/domains/#@domain/credentials/#@login").with(body: 'password=secret').to_return(@json_response)
 
-      @client.update_credentials(@login, {}).must_equal(@json_response_object)
-    end
-
-    it 'encodes the password attribute' do
-      stub_request(:put, "#@base_url/domains/#@domain/credentials/#@login").with(body: 'password=secret')
-
-      @client.update_credentials(@login, password: 'secret')
+      @client.update_credentials(@login, {password: 'secret'}).must_equal(@json_response_object)
     end
   end
 
@@ -242,6 +220,22 @@ describe 'Mailgunner::Client' do
     end
   end
 
+  describe 'get_connection_settings method' do
+    it 'fetches the domain connection settings resource and returns the response object' do
+      stub_request(:get, "#@base_url/domains/#@domain/connection").to_return(@json_response)
+
+      @client.get_connection_settings.must_equal(@json_response_object)
+    end
+  end
+
+  describe 'update_connection_settings method' do
+    it 'updates the domain connection settings resource and returns the response object' do
+      stub_request(:put, "#@base_url/domains/#@domain/connection").with(body: 'require_tls=true').to_return(@json_response)
+
+      @client.update_connection_settings({require_tls: true}).must_equal(@json_response_object)
+    end
+  end
+
   describe 'get_unsubscribes method' do
     it 'fetches the domain unsubscribes resource and returns the response object' do
       stub_request(:get, "#@base_url/#@domain/unsubscribes").to_return(@json_response)
@@ -274,15 +268,9 @@ describe 'Mailgunner::Client' do
 
   describe 'add_unsubscribe method' do
     it 'posts to the domain unsubscribes resource and returns the response object' do
-      stub_request(:post, "#@base_url/#@domain/unsubscribes").to_return(@json_response)
+      stub_request(:post, "#@base_url/#@domain/unsubscribes").with(body: "address=#@encoded_address").to_return(@json_response)
 
-      @client.add_unsubscribe({}).must_equal(@json_response_object)
-    end
-
-    it 'encodes the unsubscribe attributes' do
-      stub_request(:post, "#@base_url/#@domain/unsubscribes").with(body: "address=#@encoded_address")
-
-      @client.add_unsubscribe({address: @address})
+      @client.add_unsubscribe({address: @address}).must_equal(@json_response_object)
     end
   end
 
@@ -310,15 +298,9 @@ describe 'Mailgunner::Client' do
 
   describe 'add_complaint method' do
     it 'posts to the domain complaints resource and returns the response object' do
-      stub_request(:post, "#@base_url/#@domain/complaints").to_return(@json_response)
-
-      @client.add_complaint({}).must_equal(@json_response_object)
-    end
-
-    it 'encodes the complaint attributes' do
-      stub_request(:post, "#@base_url/#@domain/complaints").with(body: "address=#@encoded_address")
+      stub_request(:post, "#@base_url/#@domain/complaints").with(body: "address=#@encoded_address").to_return(@json_response)
 
-      @client.add_complaint({address: @address})
+      @client.add_complaint({address: @address}).must_equal(@json_response_object)
     end
   end
 
@@ -354,15 +336,9 @@ describe 'Mailgunner::Client' do
 
   describe 'add_bounce method' do
     it 'posts to the domain bounces resource and returns the response object' do
-      stub_request(:post, "#@base_url/#@domain/bounces").to_return(@json_response)
+      stub_request(:post, "#@base_url/#@domain/bounces").with(body: "address=#@encoded_address").to_return(@json_response)
 
-      @client.add_bounce({}).must_equal(@json_response_object)
-    end
-
-    it 'encodes the bounce attributes' do
-      stub_request(:post, "#@base_url/#@domain/bounces").with(body: "address=#@encoded_address")
-
-      @client.add_bounce({address: @address})
+      @client.add_bounce({address: @address}).must_equal(@json_response_object)
     end
   end
 
@@ -374,6 +350,14 @@ describe 'Mailgunner::Client' do
     end
   end
 
+  describe 'delete_bounces method' do
+    it 'deletes the domain bounces resource and returns the response object' do
+      stub_request(:delete, "#@base_url/#@domain/bounces").to_return(@json_response)
+
+      @client.delete_bounces.must_equal(@json_response_object)
+    end
+  end
+
   describe 'get_stats method' do
     before do
       Kernel.stubs(:warn)
@@ -440,6 +424,52 @@ describe 'Mailgunner::Client' do
     end
   end
 
+  describe 'get_tags method' do
+    it 'fetches the domain tags resource and returns the response object' do
+      stub_request(:get, "#@base_url/#@domain/tags").to_return(@json_response)
+
+      @client.get_tags.must_equal(@json_response_object)
+    end
+
+    it 'encodes optional limit parameter' do
+      stub_request(:get, "#@base_url/#@domain/tags?limit=2")
+
+      @client.get_tags(limit: 2)
+    end
+  end
+
+  describe 'get_tag method' do
+    it 'fetches the domain tag resource with the given id and returns the response object' do
+      stub_request(:get, "#@base_url/#@domain/tags/#@id").to_return(@json_response)
+
+      @client.get_tag(@id).must_equal(@json_response_object)
+    end
+  end
+
+  describe 'update_tag method' do
+    it 'updates the domain tag resource with the given id and returns the response object' do
+      stub_request(:put, "#@base_url/#@domain/tags/#@id").with(body: 'description=Tag+description').to_return(@json_response)
+
+      @client.update_tag(@id, {description: 'Tag description'}).must_equal(@json_response_object)
+    end
+  end
+
+  describe 'get_tag_stats method' do
+    it 'fetches the domain tag stats resource with the given id and returns the response object' do
+      stub_request(:get, "#@base_url/#@domain/tags/#@id/stats?event=accepted").to_return(@json_response)
+
+      @client.get_tag_stats(@id, event: 'accepted').must_equal(@json_response_object)
+    end
+  end
+
+  describe 'delete_tag method' do
+    it 'deletes the domain tag resource with the given id and returns the response object' do
+      stub_request(:delete, "#@base_url/#@domain/tags/#@id").to_return(@json_response)
+
+      @client.delete_tag(@id).must_equal(@json_response_object)
+    end
+  end
+
   describe 'get_routes method' do
     it 'fetches the routes resource and returns the response object' do
       stub_request(:get, "#@base_url/routes").to_return(@json_response)
@@ -456,45 +486,33 @@ describe 'Mailgunner::Client' do
 
   describe 'get_route method' do
     it 'fetches the route resource with the given id and returns the response object' do
-      stub_request(:get, "#@base_url/routes/4f3bad2335335426750048c6").to_return(@json_response)
+      stub_request(:get, "#@base_url/routes/#@id").to_return(@json_response)
 
-      @client.get_route('4f3bad2335335426750048c6').must_equal(@json_response_object)
+      @client.get_route(@id).must_equal(@json_response_object)
     end
   end
 
   describe 'add_route method' do
     it 'posts to the routes resource and returns the response object' do
-      stub_request(:post, "#@base_url/routes").to_return(@json_response)
+      stub_request(:post, "#@base_url/routes").with(body: 'description=Example+route&priority=1').to_return(@json_response)
 
-      @client.add_route({}).must_equal(@json_response_object)
-    end
-
-    it 'encodes the route attributes' do
-      stub_request(:post, "#@base_url/routes").with(body: 'description=Example+route&priority=1')
-
-      @client.add_route({description: 'Example route', priority: 1})
+      @client.add_route({description: 'Example route', priority: 1}).must_equal(@json_response_object)
     end
   end
 
   describe 'update_route method' do
     it 'updates the route resource with the given id and returns the response object' do
-      stub_request(:put, "#@base_url/routes/4f3bad2335335426750048c6").to_return(@json_response)
+      stub_request(:put, "#@base_url/routes/#@id").with(body: 'priority=10').to_return(@json_response)
 
-      @client.update_route('4f3bad2335335426750048c6', {}).must_equal(@json_response_object)
-    end
-
-    it 'encodes the route attributes' do
-      stub_request(:put, "#@base_url/routes/4f3bad2335335426750048c6").with(body: 'priority=10')
-
-      @client.update_route('4f3bad2335335426750048c6', {priority: 10})
+      @client.update_route(@id, {priority: 10}).must_equal(@json_response_object)
     end
   end
 
   describe 'delete_route method' do
     it 'deletes the route resource with the given id and returns the response object' do
-      stub_request(:delete, "#@base_url/routes/4f3bad2335335426750048c6").to_return(@json_response)
+      stub_request(:delete, "#@base_url/routes/#@id").to_return(@json_response)
 
-      @client.delete_route('4f3bad2335335426750048c6').must_equal(@json_response_object)
+      @client.delete_route(@id).must_equal(@json_response_object)
     end
   end
 
@@ -514,129 +532,117 @@ describe 'Mailgunner::Client' do
 
   describe 'get_campaign method' do
     it 'fetches the campaign resource with the given id and returns the response object' do
-      stub_request(:get, "#@base_url/#@domain/campaigns/id").to_return(@json_response)
+      stub_request(:get, "#@base_url/#@domain/campaigns/#@id").to_return(@json_response)
 
-      @client.get_campaign('id').must_equal(@json_response_object)
+      @client.get_campaign(@id).must_equal(@json_response_object)
     end
   end
 
   describe 'add_campaign method' do
     it 'posts to the domain campaigns resource and returns the response object' do
-      stub_request(:post, "#@base_url/#@domain/campaigns").to_return(@json_response)
+      stub_request(:post, "#@base_url/#@domain/campaigns").with(body: "id=#@id").to_return(@json_response)
 
-      @client.add_campaign({}).must_equal(@json_response_object)
-    end
-
-    it 'encodes the campaign attributes' do
-      stub_request(:post, "#@base_url/#@domain/campaigns").with(body: 'id=id')
-
-      @client.add_campaign({id: 'id'})
+      @client.add_campaign({id: @id}).must_equal(@json_response_object)
     end
   end
 
   describe 'update_campaign method' do
     it 'updates the campaign resource and returns the response object' do
-      stub_request(:put, "#@base_url/#@domain/campaigns/id").to_return(@json_response)
-
-      @client.update_campaign('id', {}).must_equal(@json_response_object)
-    end
-
-    it 'encodes the campaign attributes' do
-      stub_request(:put, "#@base_url/#@domain/campaigns/id").with(body: 'name=Example+Campaign')
+      stub_request(:put, "#@base_url/#@domain/campaigns/#@id").with(body: 'name=Example+Campaign').to_return(@json_response)
 
-      @client.update_campaign('id', {name: 'Example Campaign'})
+      @client.update_campaign(@id, {name: 'Example Campaign'}).must_equal(@json_response_object)
     end
   end
 
   describe 'delete_campaign method' do
     it 'deletes the domain campaign resource with the given id and returns the response object' do
-      stub_request(:delete, "#@base_url/#@domain/campaigns/id").to_return(@json_response)
+      stub_request(:delete, "#@base_url/#@domain/campaigns/#@id").to_return(@json_response)
 
-      @client.delete_campaign('id').must_equal(@json_response_object)
+      @client.delete_campaign(@id).must_equal(@json_response_object)
     end
   end
 
   describe 'get_campaign_events method' do
     it 'fetches the domain campaign events resource and returns the response object' do
-      stub_request(:get, "#@base_url/#@domain/campaigns/id/events").to_return(@json_response)
+      stub_request(:get, "#@base_url/#@domain/campaigns/#@id/events").to_return(@json_response)
 
-      @client.get_campaign_events('id').must_equal(@json_response_object)
+      @client.get_campaign_events(@id).must_equal(@json_response_object)
     end
 
     it 'encodes the optional parameters' do
-      stub_request(:get, "#@base_url/#@domain/campaigns/id/events?country=US&limit=100")
+      stub_request(:get, "#@base_url/#@domain/campaigns/#@id/events?country=US&limit=100")
 
-      @client.get_campaign_events('id', country: 'US', limit: 100)
+      @client.get_campaign_events(@id, country: 'US', limit: 100)
     end
   end
 
   describe 'get_campaign_stats method' do
     it 'fetches the domain campaign stats resource and returns the response object' do
-      stub_request(:get, "#@base_url/#@domain/campaigns/id/stats").to_return(@json_response)
+      stub_request(:get, "#@base_url/#@domain/campaigns/#@id/stats").to_return(@json_response)
 
-      @client.get_campaign_stats('id').must_equal(@json_response_object)
+      @client.get_campaign_stats(@id).must_equal(@json_response_object)
     end
 
     it 'encodes the optional parameters' do
-      stub_request(:get, "#@base_url/#@domain/campaigns/id/stats?groupby=dailyhour")
+      stub_request(:get, "#@base_url/#@domain/campaigns/#@id/stats?groupby=dailyhour")
 
-      @client.get_campaign_stats('id', groupby: 'dailyhour')
+      @client.get_campaign_stats(@id, groupby: 'dailyhour')
     end
   end
 
   describe 'get_campaign_clicks method' do
     it 'fetches the domain campaign clicks resource and returns the response object' do
-      stub_request(:get, "#@base_url/#@domain/campaigns/id/clicks").to_return(@json_response)
+      stub_request(:get, "#@base_url/#@domain/campaigns/#@id/clicks").to_return(@json_response)
 
-      @client.get_campaign_clicks('id').must_equal(@json_response_object)
+      @client.get_campaign_clicks(@id).must_equal(@json_response_object)
     end
 
     it 'encodes the optional parameters' do
-      stub_request(:get, "#@base_url/#@domain/campaigns/id/clicks?groupby=month&limit=100")
+      stub_request(:get, "#@base_url/#@domain/campaigns/#@id/clicks?groupby=month&limit=100")
 
-      @client.get_campaign_clicks('id', groupby: 'month', limit: 100)
+      @client.get_campaign_clicks(@id, groupby: 'month', limit: 100)
     end
   end
 
   describe 'get_campaign_opens method' do
     it 'fetches the domain campaign opens resource and returns the response object' do
-      stub_request(:get, "#@base_url/#@domain/campaigns/id/opens").to_return(@json_response)
+      stub_request(:get, "#@base_url/#@domain/campaigns/#@id/opens").to_return(@json_response)
 
-      @client.get_campaign_opens('id').must_equal(@json_response_object)
+      @client.get_campaign_opens(@id).must_equal(@json_response_object)
     end
 
     it 'encodes the optional parameters' do
-      stub_request(:get, "#@base_url/#@domain/campaigns/id/opens?groupby=month&limit=100")
+      stub_request(:get, "#@base_url/#@domain/campaigns/#@id/opens?groupby=month&limit=100")
 
-      @client.get_campaign_opens('id', groupby: 'month', limit: 100)
+      @client.get_campaign_opens(@id, groupby: 'month', limit: 100)
     end
   end
 
   describe 'get_campaign_unsubscribes method' do
     it 'fetches the domain campaign unsubscribes resource and returns the response object' do
-      stub_request(:get, "#@base_url/#@domain/campaigns/id/unsubscribes").to_return(@json_response)
+      stub_request(:get, "#@base_url/#@domain/campaigns/#@id/unsubscribes").to_return(@json_response)
 
-      @client.get_campaign_unsubscribes('id').must_equal(@json_response_object)
+      @client.get_campaign_unsubscribes(@id).must_equal(@json_response_object)
     end
 
     it 'encodes the optional parameters' do
-      stub_request(:get, "#@base_url/#@domain/campaigns/id/unsubscribes?groupby=month&limit=100")
+      stub_request(:get, "#@base_url/#@domain/campaigns/#@id/unsubscribes?groupby=month&limit=100")
 
-      @client.get_campaign_unsubscribes('id', groupby: 'month', limit: 100)
+      @client.get_campaign_unsubscribes(@id, groupby: 'month', limit: 100)
     end
   end
 
   describe 'get_campaign_complaints method' do
     it 'fetches the domain campaign complaints resource and returns the response object' do
-      stub_request(:get, "#@base_url/#@domain/campaigns/id/complaints").to_return(@json_response)
+      stub_request(:get, "#@base_url/#@domain/campaigns/#@id/complaints").to_return(@json_response)
 
-      @client.get_campaign_complaints('id').must_equal(@json_response_object)
+      @client.get_campaign_complaints(@id).must_equal(@json_response_object)
     end
 
     it 'encodes the optional parameters' do
-      stub_request(:get, "#@base_url/#@domain/campaigns/id/complaints?groupby=month&limit=100")
+      stub_request(:get, "#@base_url/#@domain/campaigns/#@id/complaints?groupby=month&limit=100")
 
-      @client.get_campaign_complaints('id', groupby: 'month', limit: 100)
+      @client.get_campaign_complaints(@id, groupby: 'month', limit: 100)
     end
   end
 
@@ -664,29 +670,17 @@ describe 'Mailgunner::Client' do
 
   describe 'add_list method' do
     it 'posts to the lists resource and returns the response object' do
-      stub_request(:post, "#@base_url/lists").to_return(@json_response)
+      stub_request(:post, "#@base_url/lists").with(body: 'address=developers%40mailgun.net').to_return(@json_response)
 
-      @client.add_list({}).must_equal(@json_response_object)
-    end
-
-    it 'encodes the list attributes' do
-      stub_request(:post, "#@base_url/lists").with(body: 'address=developers%40mailgun.net')
-
-      @client.add_list({address: 'developers@mailgun.net'})
+      @client.add_list({address: 'developers@mailgun.net'}).must_equal(@json_response_object)
     end
   end
 
   describe 'update_list method' do
     it 'updates the list resource and returns the response object' do
-      stub_request(:put, "#@base_url/lists/developers%40mailgun.net").to_return(@json_response)
-
-      @client.update_list('developers@mailgun.net', {}).must_equal(@json_response_object)
-    end
-
-    it 'encodes the list attributes' do
-      stub_request(:put, "#@base_url/lists/developers%40mailgun.net").with(body: 'name=Example+list')
+      stub_request(:put, "#@base_url/lists/developers%40mailgun.net").with(body: 'name=Example+list').to_return(@json_response)
 
-      @client.update_list('developers@mailgun.net', {name: 'Example list'})
+      @client.update_list('developers@mailgun.net', {name: 'Example list'}).must_equal(@json_response_object)
     end
   end
 
@@ -722,29 +716,17 @@ describe 'Mailgunner::Client' do
 
   describe 'add_list_member method' do
     it 'posts to the list members resource and returns the response object' do
-      stub_request(:post, "#@base_url/lists/developers%40mailgun.net/members").to_return(@json_response)
+      stub_request(:post, "#@base_url/lists/developers%40mailgun.net/members").with(body: "address=#@encoded_address").to_return(@json_response)
 
-      @client.add_list_member('developers@mailgun.net', {}).must_equal(@json_response_object)
-    end
-
-    it 'encodes the list attributes' do
-      stub_request(:post, "#@base_url/lists/developers%40mailgun.net/members").with(body: "address=#@encoded_address")
-
-      @client.add_list_member('developers@mailgun.net', {address: @address})
+      @client.add_list_member('developers@mailgun.net', {address: @address}).must_equal(@json_response_object)
     end
   end
 
   describe 'update_list_member method' do
     it 'updates the list member resource with the given address and returns the response object' do
-      stub_request(:put, "#@base_url/lists/developers%40mailgun.net/members/#@encoded_address").to_return(@json_response)
-
-      @client.update_list_member('developers@mailgun.net', @address, {}).must_equal(@json_response_object)
-    end
+      stub_request(:put, "#@base_url/lists/developers%40mailgun.net/members/#@encoded_address").with(body: 'subscribed=no').to_return(@json_response)
 
-    it 'encodes the list member attributes' do
-      stub_request(:put, "#@base_url/lists/developers%40mailgun.net/members/#@encoded_address").with(body: 'subscribed=no')
-
-      @client.update_list_member('developers@mailgun.net', @address, {subscribed: 'no'})
+      @client.update_list_member('developers@mailgun.net', @address, {subscribed: 'no'}).must_equal(@json_response_object)
     end
   end
 
@@ -755,4 +737,12 @@ describe 'Mailgunner::Client' do
       @client.delete_list_member('developers@mailgun.net', @address).must_equal(@json_response_object)
     end
   end
+
+  it 'sets the user agent header' do
+    headers = {'User-Agent' => /\ARuby\/\d+\.\d+\.\d+ Mailgunner\/\d+\.\d+\.\d+\z/}
+
+    stub_request(:get, "#@base_url/domains/#@domain/messages/#@id").with(headers: headers)
+
+    @client.get_message(@id)
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mailgunner
 version: !ruby/object:Gem::Version
-  version: 2.3.0
+  version: 2.4.0
 platform: ruby
 authors:
 - Tim Craft
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-28 00:00:00.000000000 Z
+date: 2016-03-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -92,6 +92,7 @@ files:
 - lib/mailgunner.rb
 - lib/mailgunner/delivery_method.rb
 - lib/mailgunner/railtie.rb
+- lib/mailgunner/version.rb
 - mailgunner.gemspec
 - spec/mailgunner_delivery_method_spec.rb
 - spec/mailgunner_spec.rb