dnsimple 4.2.0 → 4.3.0

data/spec/dnsimple/client/registrar_delegation_spec.rb

@@ -8,15 +8,15 @@ describe Dnsimple::Client, ".registrar" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:get, %r{/v2/#{account_id}/registrar/domains/.+/delegation$}).
-          to_return(read_http_fixture("getDomainDelegation/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/registrar/domains/.+/delegation$})
+          .to_return(read_http_fixture("getDomainDelegation/success.http"))
     end
 
     it "builds the correct request" do
       subject.domain_delegation(account_id, domain_name = "example.com")
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/delegation").
-          with(headers: { "Accept" => "application/json" })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/delegation")
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the name servers of the domain" do
@@ -31,8 +31,8 @@ describe Dnsimple::Client, ".registrar" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:put, %r{/v2/#{account_id}/registrar/domains/.+/delegation$}).
-          to_return(read_http_fixture("changeDomainDelegation/success.http"))
+      stub_request(:put, %r{/v2/#{account_id}/registrar/domains/.+/delegation$})
+          .to_return(read_http_fixture("changeDomainDelegation/success.http"))
     end
 
     let(:attributes) { %w(ns1.dnsimple.com ns2.dnsimple.com ns3.dnsimple.com ns4.dnsimple.com) }
@@ -40,9 +40,9 @@ describe Dnsimple::Client, ".registrar" do
     it "builds the correct request" do
       subject.change_domain_delegation(account_id, domain_name = "example.com", attributes)
 
-      expect(WebMock).to have_requested(:put, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/delegation").
-          with(body: JSON.dump(attributes)).
-          with(headers: { "Accept" => "application/json" })
+      expect(WebMock).to have_requested(:put, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/delegation")
+          .with(body: JSON.dump(attributes))
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the name servers of the domain" do
@@ -57,8 +57,8 @@ describe Dnsimple::Client, ".registrar" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:put, %r{/v2/#{account_id}/registrar/domains/.+/delegation/vanity$}).
-          to_return(read_http_fixture("changeDomainDelegationToVanity/success.http"))
+      stub_request(:put, %r{/v2/#{account_id}/registrar/domains/.+/delegation/vanity$})
+          .to_return(read_http_fixture("changeDomainDelegationToVanity/success.http"))
     end
 
     let(:attributes) { %w(ns1.example.com ns2.example.com) }
@@ -66,9 +66,9 @@ describe Dnsimple::Client, ".registrar" do
     it "builds the correct request" do
       subject.change_domain_delegation_to_vanity(account_id, domain_name = "example.com", attributes)
 
-      expect(WebMock).to have_requested(:put, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/delegation/vanity").
-          with(body: JSON.dump(attributes)).
-          with(headers: { "Accept" => "application/json" })
+      expect(WebMock).to have_requested(:put, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/delegation/vanity")
+          .with(body: JSON.dump(attributes))
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns vanity name servers of the domain" do
@@ -84,15 +84,15 @@ describe Dnsimple::Client, ".registrar" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:delete, %r{/v2/#{account_id}/registrar/domains/.+/delegation/vanity$}).
-          to_return(read_http_fixture("changeDomainDelegationFromVanity/success.http"))
+      stub_request(:delete, %r{/v2/#{account_id}/registrar/domains/.+/delegation/vanity$})
+          .to_return(read_http_fixture("changeDomainDelegationFromVanity/success.http"))
     end
 
     it "builds the correct request" do
       subject.change_domain_delegation_from_vanity(account_id, domain_name = "example.com")
 
-      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/delegation/vanity").
-          with(headers: { "Accept" => "application/json" })
+      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/delegation/vanity")
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns empty response" do

data/spec/dnsimple/client/registrar_spec.rb

@@ -9,15 +9,15 @@ describe Dnsimple::Client, ".registrar" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:get, %r{/v2/#{account_id}/registrar/domains/.+/check$}).
-          to_return(read_http_fixture("checkDomain/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/registrar/domains/.+/check$})
+          .to_return(read_http_fixture("checkDomain/success.http"))
     end
 
     it "builds the correct request" do
       subject.check_domain(account_id, domain_name = "example.com")
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/check").
-          with(headers: { "Accept" => "application/json" })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/check")
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the availability" do
@@ -37,15 +37,15 @@ describe Dnsimple::Client, ".registrar" do
 
     context "when premium price" do
       before do
-        stub_request(:get, %r{/v2/#{account_id}/registrar/domains/.+/premium_price[\?action]*}).
-            to_return(read_http_fixture("getDomainPremiumPrice/success.http"))
+        stub_request(:get, %r{/v2/#{account_id}/registrar/domains/.+/premium_price[\?action]*})
+            .to_return(read_http_fixture("getDomainPremiumPrice/success.http"))
       end
 
       it "builds the correct request" do
         subject.domain_premium_price(account_id, domain_name = "ruby.codes")
 
-        expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/premium_price").
-            with(headers: { "Accept" => "application/json" })
+        expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/premium_price")
+            .with(headers: { "Accept" => "application/json" })
       end
 
       it "returns the premium price" do
@@ -61,15 +61,15 @@ describe Dnsimple::Client, ".registrar" do
       it "builds the correct request when action is passed" do
         subject.domain_premium_price(account_id, domain_name = "ruby.codes", action: "registration")
 
-        expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/premium_price?action=registration").
-            with(headers: { "Accept" => "application/json" })
+        expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/premium_price?action=registration")
+            .with(headers: { "Accept" => "application/json" })
       end
     end
 
     context "when not premium price" do
       before do
-        stub_request(:get, %r{/v2/#{account_id}/registrar/domains/.+/premium_price$}).
-            to_return(read_http_fixture("getDomainPremiumPrice/failure.http"))
+        stub_request(:get, %r{/v2/#{account_id}/registrar/domains/.+/premium_price$})
+            .to_return(read_http_fixture("getDomainPremiumPrice/failure.http"))
       end
 
       it "raises error" do
@@ -84,8 +84,8 @@ describe Dnsimple::Client, ".registrar" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/registration$}).
-          to_return(read_http_fixture("registerDomain/success.http"))
+      stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/registrations$})
+          .to_return(read_http_fixture("registerDomain/success.http"))
     end
 
     let(:attributes) { { registrant_id: "10" } }
@@ -93,9 +93,9 @@ describe Dnsimple::Client, ".registrar" do
     it "builds the correct request" do
       subject.register_domain(account_id, domain_name = "example.com", attributes)
 
-      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/registration").
-          with(body: attributes).
-          with(headers: { "Accept" => "application/json" })
+      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/registrations")
+          .with(body: attributes)
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the domain" do
@@ -119,8 +119,8 @@ describe Dnsimple::Client, ".registrar" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/renewal$}).
-          to_return(read_http_fixture("renewDomain/success.http"))
+      stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/renewals$})
+          .to_return(read_http_fixture("renewDomain/success.http"))
     end
 
     let(:attributes) { { period: "3" } }
@@ -128,9 +128,9 @@ describe Dnsimple::Client, ".registrar" do
     it "builds the correct request" do
       subject.renew_domain(account_id, domain_name = "example.com", attributes)
 
-      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/renewal").
-          with(body: attributes).
-          with(headers: { "Accept" => "application/json" })
+      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/renewals")
+          .with(body: attributes)
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the domain" do
@@ -145,8 +145,8 @@ describe Dnsimple::Client, ".registrar" do
 
     context "when it is too soon for the domain to be renewed" do
       it "raises a BadRequestError" do
-        stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/renewal$}).
-            to_return(read_http_fixture("renewDomain/error-tooearly.http"))
+        stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/renewals$})
+            .to_return(read_http_fixture("renewDomain/error-tooearly.http"))
 
         expect {
           subject.renew_domain(account_id, "example.com", attributes)
@@ -159,8 +159,8 @@ describe Dnsimple::Client, ".registrar" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/transfer$}).
-          to_return(read_http_fixture("transferDomain/success.http"))
+      stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/transfers$})
+          .to_return(read_http_fixture("transferDomain/success.http"))
     end
 
     let(:attributes) { { registrant_id: "10", auth_code: "x1y2z3" } }
@@ -168,9 +168,9 @@ describe Dnsimple::Client, ".registrar" do
     it "builds the correct request" do
       subject.transfer_domain(account_id, domain_name = "example.com", attributes)
 
-      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/transfer").
-          with(body: attributes).
-          with(headers: { "Accept" => "application/json" })
+      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/transfers")
+          .with(body: attributes)
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the domain" do
@@ -191,8 +191,8 @@ describe Dnsimple::Client, ".registrar" do
 
     context "when the domain is already in DNSimple" do
       it "raises a BadRequestError" do
-        stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/transfer$}).
-            to_return(read_http_fixture("transferDomain/error-indnsimple.http"))
+        stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/transfers$})
+            .to_return(read_http_fixture("transferDomain/error-indnsimple.http"))
 
         expect {
           subject.transfer_domain(account_id, "example.com", attributes)
@@ -202,8 +202,8 @@ describe Dnsimple::Client, ".registrar" do
 
     context "when :auth_code wasn't provided and is required by the TLD" do
       it "raises a BadRequestError" do
-        stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/transfer$}).
-            to_return(read_http_fixture("transferDomain/error-missing-authcode.http"))
+        stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/transfers$})
+            .to_return(read_http_fixture("transferDomain/error-missing-authcode.http"))
 
         expect {
           subject.transfer_domain(account_id, "example.com", registrant_id: 10)
@@ -216,15 +216,15 @@ describe Dnsimple::Client, ".registrar" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/transfer_out$}).
-          to_return(read_http_fixture("transferDomainOut/success.http"))
+      stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/authorize_transfer_out$})
+          .to_return(read_http_fixture("transferDomainOut/success.http"))
     end
 
     it "builds the correct request" do
       subject.transfer_domain_out(account_id, domain_name = "example.com")
 
-      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/transfer_out").
-          with(headers: { "Accept" => "application/json" })
+      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/authorize_transfer_out")
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns nothing" do

data/spec/dnsimple/client/registrar_whois_privacy_spec.rb

@@ -9,15 +9,15 @@ describe Dnsimple::Client, ".registrar" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:get, %r{/v2/#{account_id}/registrar/domains/.+/whois_privacy$}).
-          to_return(read_http_fixture("getWhoisPrivacy/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/registrar/domains/.+/whois_privacy$})
+          .to_return(read_http_fixture("getWhoisPrivacy/success.http"))
     end
 
     it "builds the correct request" do
       subject.whois_privacy(account_id, domain_name = "example.com")
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/whois_privacy").
-          with(headers: { "Accept" => "application/json" })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/whois_privacy")
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the whois privacy" do
@@ -37,15 +37,15 @@ describe Dnsimple::Client, ".registrar" do
 
     context "when the whois privacy had already been purchased" do
       before do
-        stub_request(:put, %r{/v2/#{account_id}/registrar/domains/.+/whois_privacy$}).
-            to_return(read_http_fixture("enableWhoisPrivacy/success.http"))
+        stub_request(:put, %r{/v2/#{account_id}/registrar/domains/.+/whois_privacy$})
+            .to_return(read_http_fixture("enableWhoisPrivacy/success.http"))
       end
 
       it "builds the correct request" do
         subject.enable_whois_privacy(account_id, domain_name = "example.com")
 
-        expect(WebMock).to have_requested(:put, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/whois_privacy").
-            with(headers: { "Accept" => "application/json" })
+        expect(WebMock).to have_requested(:put, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/whois_privacy")
+            .with(headers: { "Accept" => "application/json" })
       end
 
       it "returns the whois privacy" do
@@ -63,15 +63,15 @@ describe Dnsimple::Client, ".registrar" do
 
     context "when the whois privacy is newly purchased" do
       before do
-        stub_request(:put, %r{/v2/#{account_id}/registrar/domains/.+/whois_privacy$}).
-            to_return(read_http_fixture("enableWhoisPrivacy/created.http"))
+        stub_request(:put, %r{/v2/#{account_id}/registrar/domains/.+/whois_privacy$})
+            .to_return(read_http_fixture("enableWhoisPrivacy/created.http"))
       end
 
       it "builds the correct request" do
         subject.enable_whois_privacy(account_id, domain_name = "example.com")
 
-        expect(WebMock).to have_requested(:put, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/whois_privacy").
-            with(headers: { "Accept" => "application/json" })
+        expect(WebMock).to have_requested(:put, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/whois_privacy")
+            .with(headers: { "Accept" => "application/json" })
       end
 
       it "returns the whois privacy" do
@@ -93,15 +93,15 @@ describe Dnsimple::Client, ".registrar" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:delete, %r{/v2/#{account_id}/registrar/domains/.+/whois_privacy$}).
-          to_return(read_http_fixture("disableWhoisPrivacy/success.http"))
+      stub_request(:delete, %r{/v2/#{account_id}/registrar/domains/.+/whois_privacy$})
+          .to_return(read_http_fixture("disableWhoisPrivacy/success.http"))
     end
 
     it "builds the correct request" do
       subject.disable_whois_privacy(account_id, domain_name = "example.com")
 
-      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/whois_privacy").
-          with(headers: { "Accept" => "application/json" })
+      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/whois_privacy")
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the whois privacy" do

data/spec/dnsimple/client/services_domains_spec.rb

@@ -10,15 +10,15 @@ describe Dnsimple::Client, ".services" do
     let(:domain_id)  { 'example.com' }
 
     before do
-      stub_request(:get, %r{/v2/#{account_id}/domains/#{domain_id}/services}).
-          to_return(read_http_fixture("appliedServices/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/domains/#{domain_id}/services})
+          .to_return(read_http_fixture("appliedServices/success.http"))
     end
 
     it "builds the correct request" do
       subject.applied_services(account_id, domain_id)
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/services").
-          with(headers: { "Accept" => "application/json" })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/services")
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "supports pagination" do
@@ -63,8 +63,8 @@ describe Dnsimple::Client, ".services" do
     let(:service_id) { "service1" }
 
     before do
-      stub_request(:post, %r{/v2/#{account_id}/domains/#{domain_id}/services/#{service_id}$}).
-          to_return(read_http_fixture("applyService/success.http"))
+      stub_request(:post, %r{/v2/#{account_id}/domains/#{domain_id}/services/#{service_id}$})
+          .to_return(read_http_fixture("applyService/success.http"))
     end
 
     let(:settings) { { app: "foo" } }
@@ -72,9 +72,9 @@ describe Dnsimple::Client, ".services" do
     it "builds the correct request" do
       subject.apply_service(account_id, service_id, domain_id, settings)
 
-      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/services/#{service_id}").
-          with(body: settings).
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/services/#{service_id}")
+          .with(body: settings)
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns empty response" do
@@ -92,15 +92,15 @@ describe Dnsimple::Client, ".services" do
     let(:service_id) { "service1" }
 
     before do
-      stub_request(:delete, %r{/v2/#{account_id}/domains/#{domain_id}/services/#{service_id}$}).
-          to_return(read_http_fixture("unapplyService/success.http"))
+      stub_request(:delete, %r{/v2/#{account_id}/domains/#{domain_id}/services/#{service_id}$})
+          .to_return(read_http_fixture("unapplyService/success.http"))
     end
 
     it "builds the correct request" do
       subject.unapply_service(account_id, service_id, domain_id)
 
-      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/services/#{service_id}").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/services/#{service_id}")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns empty response" do

data/spec/dnsimple/client/services_spec.rb

@@ -7,15 +7,15 @@ describe Dnsimple::Client, ".services" do
 
   describe "#list_services" do
     before do
-      stub_request(:get, %r{/v2/services}).
-          to_return(read_http_fixture("listServices/success.http"))
+      stub_request(:get, %r{/v2/services})
+          .to_return(read_http_fixture("listServices/success.http"))
     end
 
     it "builds the correct request" do
       subject.list_services
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/services").
-          with(headers: { "Accept" => "application/json" })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/services")
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "supports pagination" do
@@ -56,8 +56,8 @@ describe Dnsimple::Client, ".services" do
 
   describe "#all_services" do
     before do
-      stub_request(:get, %r{/v2/services}).
-          to_return(read_http_fixture("listServices/success.http"))
+      stub_request(:get, %r{/v2/services})
+          .to_return(read_http_fixture("listServices/success.http"))
     end
 
     it "delegates to client.paginate" do
@@ -76,15 +76,15 @@ describe Dnsimple::Client, ".services" do
     let(:service_id) { 1 }
 
     before do
-      stub_request(:get, %r{/v2/services/#{service_id}$}).
-          to_return(read_http_fixture("getService/success.http"))
+      stub_request(:get, %r{/v2/services/#{service_id}$})
+          .to_return(read_http_fixture("getService/success.http"))
     end
 
     it "builds the correct request" do
       subject.service(service_id)
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/services/#{service_id}").
-          with(headers: { "Accept" => "application/json" })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/services/#{service_id}")
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the service" do