dnsimple 4.2.0 → 4.3.0

data/spec/dnsimple/client/zones_records_spec.rb

@@ -10,15 +10,15 @@ describe Dnsimple::Client, ".zones" do
     let(:zone_id) { "example.com" }
 
     before do
-      stub_request(:get, %r{/v2/#{account_id}/zones/#{zone_id}/records}).
-          to_return(read_http_fixture("listZoneRecords/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/zones/#{zone_id}/records})
+          .to_return(read_http_fixture("listZoneRecords/success.http"))
     end
 
     it "builds the correct request" do
       subject.records(account_id, zone_id)
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone_id}/records").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone_id}/records")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "supports pagination" do
@@ -71,8 +71,8 @@ describe Dnsimple::Client, ".zones" do
 
     context "when the zone does not exist" do
       it "raises NotFoundError" do
-        stub_request(:get, %r{/v2}).
-            to_return(read_http_fixture("notfound-zone.http"))
+        stub_request(:get, %r{/v2})
+            .to_return(read_http_fixture("notfound-zone.http"))
 
         expect {
           subject.records(account_id, zone_id)
@@ -83,8 +83,8 @@ describe Dnsimple::Client, ".zones" do
 
   describe "#all_records" do
     before do
-      stub_request(:get, %r{/v2/#{account_id}/zones/#{zone_id}/records}).
-          to_return(read_http_fixture("listZoneRecords/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/zones/#{zone_id}/records})
+          .to_return(read_http_fixture("listZoneRecords/success.http"))
     end
 
     let(:account_id) { 1010 }
@@ -113,8 +113,8 @@ describe Dnsimple::Client, ".zones" do
     let(:zone_id) { "example.com" }
 
     before do
-      stub_request(:post, %r{/v2/#{account_id}/zones/#{zone_id}/records$}).
-          to_return(read_http_fixture("createZoneRecord/created.http"))
+      stub_request(:post, %r{/v2/#{account_id}/zones/#{zone_id}/records$})
+          .to_return(read_http_fixture("createZoneRecord/created.http"))
     end
 
     let(:attributes) { { type: "A", name: "www", content: "127.0.0.1", regions: %w(global) } }
@@ -122,9 +122,9 @@ describe Dnsimple::Client, ".zones" do
     it "builds the correct request" do
       subject.create_record(account_id, zone_id, attributes)
 
-      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone_id}/records").
-          with(body: attributes).
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone_id}/records")
+          .with(body: attributes)
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns the record" do
@@ -142,8 +142,8 @@ describe Dnsimple::Client, ".zones" do
 
     context "when the zone does not exist" do
       it "raises NotFoundError" do
-        stub_request(:post, %r{/v2}).
-            to_return(read_http_fixture("notfound-zone.http"))
+        stub_request(:post, %r{/v2})
+            .to_return(read_http_fixture("notfound-zone.http"))
 
         expect {
           subject.create_record(account_id, zone_id, attributes)
@@ -158,15 +158,15 @@ describe Dnsimple::Client, ".zones" do
     let(:record_id) { 5 }
 
     before do
-      stub_request(:get, %r{/v2/#{account_id}/zones/#{zone_id}/records/.+$}).
-          to_return(read_http_fixture("getZoneRecord/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/zones/#{zone_id}/records/.+$})
+          .to_return(read_http_fixture("getZoneRecord/success.http"))
     end
 
     it "builds the correct request" do
       subject.record(account_id, zone_id, record_id)
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone_id}/records/#{record_id}").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone_id}/records/#{record_id}")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns the record" do
@@ -191,8 +191,8 @@ describe Dnsimple::Client, ".zones" do
 
     context "when the zone does not exist" do
       it "raises NotFoundError" do
-        stub_request(:get, %r{/v2}).
-            to_return(read_http_fixture("notfound-zone.http"))
+        stub_request(:get, %r{/v2})
+            .to_return(read_http_fixture("notfound-zone.http"))
 
         expect {
           subject.record(account_id, zone_id, "0")
@@ -202,8 +202,8 @@ describe Dnsimple::Client, ".zones" do
 
     context "when the record does not exist" do
       it "raises NotFoundError" do
-        stub_request(:get, %r{/v2}).
-            to_return(read_http_fixture("notfound-record.http"))
+        stub_request(:get, %r{/v2})
+            .to_return(read_http_fixture("notfound-record.http"))
 
         expect {
           subject.record(account_id, zone_id, "0")
@@ -218,8 +218,8 @@ describe Dnsimple::Client, ".zones" do
     let(:record_id) { 5 }
 
     before do
-      stub_request(:patch, %r{/v2/#{account_id}/zones/#{zone_id}/records/.+$}).
-          to_return(read_http_fixture("updateZoneRecord/success.http"))
+      stub_request(:patch, %r{/v2/#{account_id}/zones/#{zone_id}/records/.+$})
+          .to_return(read_http_fixture("updateZoneRecord/success.http"))
     end
 
     let(:attributes) { { content: "mxb.example.com", priority: "20", regions: ['global'] } }
@@ -227,9 +227,9 @@ describe Dnsimple::Client, ".zones" do
     it "builds the correct request" do
       subject.update_record(account_id, zone_id, record_id, attributes)
 
-      expect(WebMock).to have_requested(:patch, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone_id}/records/#{record_id}").
-          with(body: attributes).
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:patch, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone_id}/records/#{record_id}")
+          .with(body: attributes)
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns the record" do
@@ -246,8 +246,8 @@ describe Dnsimple::Client, ".zones" do
 
     context "when the zone does not exist" do
       it "raises NotFoundError" do
-        stub_request(:patch, %r{/v2}).
-            to_return(read_http_fixture("notfound-zone.http"))
+        stub_request(:patch, %r{/v2})
+            .to_return(read_http_fixture("notfound-zone.http"))
 
         expect {
           subject.update_record(account_id, zone_id, "0", {})
@@ -257,8 +257,8 @@ describe Dnsimple::Client, ".zones" do
 
     context "when the record does not exist" do
       it "raises NotFoundError" do
-        stub_request(:patch, %r{/v2}).
-            to_return(read_http_fixture("notfound-record.http"))
+        stub_request(:patch, %r{/v2})
+            .to_return(read_http_fixture("notfound-record.http"))
 
         expect {
           subject.update_record(account_id, zone_id, "0", {})
@@ -272,15 +272,15 @@ describe Dnsimple::Client, ".zones" do
     let(:zone_id) { "example.com" }
 
     before do
-      stub_request(:delete, %r{/v2/#{account_id}/zones/#{zone_id}/records/.+$}).
-          to_return(read_http_fixture("deleteZoneRecord/success.http"))
+      stub_request(:delete, %r{/v2/#{account_id}/zones/#{zone_id}/records/.+$})
+          .to_return(read_http_fixture("deleteZoneRecord/success.http"))
     end
 
     it "builds the correct request" do
       subject.delete_record(account_id, zone_id, record_id = 2)
 
-      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone_id}/records/#{record_id}").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone_id}/records/#{record_id}")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns nothing" do
@@ -293,8 +293,8 @@ describe Dnsimple::Client, ".zones" do
 
     context "when the zone does not exist" do
       it "raises NotFoundError" do
-        stub_request(:delete, %r{/v2}).
-            to_return(read_http_fixture("notfound-zone.http"))
+        stub_request(:delete, %r{/v2})
+            .to_return(read_http_fixture("notfound-zone.http"))
 
         expect {
           subject.delete_record(account_id, zone_id, "0")
@@ -304,8 +304,8 @@ describe Dnsimple::Client, ".zones" do
 
     context "when the record does not exist" do
       it "raises NotFoundError" do
-        stub_request(:delete, %r{/v2}).
-            to_return(read_http_fixture("notfound-record.http"))
+        stub_request(:delete, %r{/v2})
+            .to_return(read_http_fixture("notfound-record.http"))
 
         expect {
           subject.delete_record(account_id, zone_id, "0")

data/spec/dnsimple/client/zones_spec.rb

@@ -9,15 +9,15 @@ describe Dnsimple::Client, ".zones" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:get, %r{/v2/#{account_id}/zones}).
-          to_return(read_http_fixture("listZones/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/zones})
+          .to_return(read_http_fixture("listZones/success.http"))
     end
 
     it "builds the correct request" do
       subject.zones(account_id)
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/zones").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/zones")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "supports pagination" do
@@ -70,8 +70,8 @@ describe Dnsimple::Client, ".zones" do
 
   describe "#all_zones" do
     before do
-      stub_request(:get, %r{/v2/#{account_id}/zones}).
-          to_return(read_http_fixture("listZones/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/zones})
+          .to_return(read_http_fixture("listZones/success.http"))
     end
 
     let(:account_id) { 1010 }
@@ -98,15 +98,15 @@ describe Dnsimple::Client, ".zones" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:get, %r{/v2/#{account_id}/zones/.+$}).
-          to_return(read_http_fixture("getZone/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/zones/.+$})
+          .to_return(read_http_fixture("getZone/success.http"))
     end
 
     it "builds the correct request" do
       subject.zone(account_id, zone = "example.com")
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone}").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone}")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns the zone" do
@@ -125,8 +125,8 @@ describe Dnsimple::Client, ".zones" do
 
     context "when the zone does not exist" do
       it "raises NotFoundError" do
-        stub_request(:get, %r{/v2}).
-            to_return(read_http_fixture("notfound-zone.http"))
+        stub_request(:get, %r{/v2})
+            .to_return(read_http_fixture("notfound-zone.http"))
 
         expect {
           subject.zone(account_id, "example.com")
@@ -139,15 +139,15 @@ describe Dnsimple::Client, ".zones" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:get, %r{/v2/#{account_id}/zones/.+$}).
-          to_return(read_http_fixture("getZoneFile/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/zones/.+$})
+          .to_return(read_http_fixture("getZoneFile/success.http"))
     end
 
     it "builds the correct request" do
       subject.zone_file(account_id, zone = "example.com")
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone}/file").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone}/file")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns the zone id" do
@@ -161,8 +161,8 @@ describe Dnsimple::Client, ".zones" do
 
     context "when the zone file does not exist" do
       it "raises NotFoundError" do
-        stub_request(:get, %r{/v2}).
-            to_return(read_http_fixture("notfound-zone.http"))
+        stub_request(:get, %r{/v2})
+            .to_return(read_http_fixture("notfound-zone.http"))
 
         expect {
           subject.zone_file(account_id, "example.com")

data/spec/dnsimple/client_spec.rb

@@ -40,8 +40,8 @@ describe Dnsimple::Client do
       subject = described_class.new(access_token: "access-token")
       subject.execute(:get, "test", {})
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.com/test").
-          with { |req| req.headers["Authorization"] == "Bearer access-token" }
+      expect(WebMock).to(have_requested(:get, "https://api.dnsimple.com/test")
+          .with { |req| req.headers["Authorization"] == "Bearer access-token" })
     end
   end
 
@@ -116,78 +116,78 @@ describe Dnsimple::Client do
 
       subject.request(:get, 'foo', {})
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.com/foo").
-          with(basic_auth: %w(user pass),
-               headers: { 'Accept' => 'application/json', 'User-Agent' => Dnsimple::Default::USER_AGENT })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.com/foo")
+          .with(basic_auth: %w(user pass),
+                headers: { 'Accept' => 'application/json', 'User-Agent' => Dnsimple::Default::USER_AGENT })
     end
 
     it "delegates to HTTParty" do
       stub_request(:get, %r{/foo})
 
-      expect(HTTParty).to receive(:get).
-          with(
+      expect(HTTParty).to receive(:get)
+          .with(
               "#{subject.base_url}foo",
               format: :json,
               basic_auth: { username: "user", password: "pass" },
               headers: { 'Accept' => 'application/json', 'User-Agent' => Dnsimple::Default::USER_AGENT }
-          ).
-          and_return(double('response', code: 200))
+          )
+          .and_return(double('response', code: 200))
 
       subject.request(:get, 'foo')
     end
 
     it "properly extracts processes options and encodes data" do
-      expect(HTTParty).to receive(:put).
-          with(
+      expect(HTTParty).to receive(:put)
+          .with(
               "#{subject.base_url}foo",
               format: :json,
               body: JSON.dump(something: "else"),
               query: { foo: "bar" },
               basic_auth: { username: "user", password: "pass" },
               headers: { 'Accept' => 'application/json', 'Content-Type' => 'application/json', 'User-Agent' => Dnsimple::Default::USER_AGENT, "Custom" => "Header" }
-          ).
-          and_return(double('response', code: 200))
+          )
+          .and_return(double('response', code: 200))
 
       subject.request(:put, 'foo', { something: "else" }, { query: { foo: "bar" }, headers: { "Custom" => "Header" } })
     end
 
     it "handles non application/json content types" do
-      expect(HTTParty).to receive(:post).
-          with(
+      expect(HTTParty).to receive(:post)
+          .with(
               "#{subject.base_url}foo",
               format: :json,
               body: { something: "else" },
               basic_auth: { username: "user", password: "pass" },
               headers: { 'Accept' => 'application/json', 'Content-Type' => 'application/x-www-form-urlencoded', 'User-Agent' => Dnsimple::Default::USER_AGENT }
-          ).
-          and_return(double('response', code: 200))
+          )
+          .and_return(double('response', code: 200))
 
       subject.request(:post, 'foo', { something: "else" }, { headers: { "Content-Type" => "application/x-www-form-urlencoded" } })
     end
 
     it "includes options for proxy support" do
-      expect(HTTParty).to receive(:get).
-          with(
+      expect(HTTParty).to receive(:get)
+          .with(
               "#{subject.base_url}test",
               format: :json,
               http_proxyaddr: "example-proxy.com",
               http_proxyport: "4321",
               headers: { 'Accept' => 'application/json', 'User-Agent' => Dnsimple::Default::USER_AGENT }
-          ).
-          and_return(double('response', code: 200))
+          )
+          .and_return(double('response', code: 200))
 
       subject = described_class.new(proxy: "example-proxy.com:4321")
       subject.request(:get, "test", nil, {})
     end
 
     it "supports custom user agent" do
-      expect(HTTParty).to receive(:get).
-          with(
+      expect(HTTParty).to receive(:get)
+          .with(
               "#{subject.base_url}test",
               format: :json,
               headers: hash_including("User-Agent" => "#{Dnsimple::Default::USER_AGENT} customAgent")
-          ).
-          and_return(double("response", code: 200))
+          )
+          .and_return(double("response", code: 200))
 
       subject = described_class.new(user_agent: "customAgent")
       subject.request(:get, "test", nil)

metadata

@@ -1,18 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: dnsimple
 version: !ruby/object:Gem::Version
-  version: 4.2.0
+  version: 4.3.0
 platform: ruby
 authors:
 - Anthony Eden
 - Simone Carletti
 - Javier Acero
-- Joseph Caudle
 - Luca Guidi
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-03-07 00:00:00.000000000 Z
+date: 2017-06-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -89,7 +88,6 @@ email:
 - anthony.eden@dnsimple.com
 - simone.carletti@dnsimple.com
 - javier.acero@dnsimple.com
-- joseph.caudle@dnsimple.com
 - luca.guidi@dnsimple.com
 executables: []
 extensions: []
@@ -346,7 +344,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.8
+rubygems_version: 2.6.10
 signing_key: 
 specification_version: 4
 summary: The DNSimple API client for Ruby