dnsimple 4.2.0 → 4.3.0

data/spec/dnsimple/client/domains_email_forwards_spec.rb

@@ -10,15 +10,15 @@ describe Dnsimple::Client, ".domains" do
     let(:domain_id) { "example.com" }
 
     before do
-      stub_request(:get, %r{/v2/#{account_id}/domains/#{domain_id}/email_forwards}).
-          to_return(read_http_fixture("listEmailForwards/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/domains/#{domain_id}/email_forwards})
+          .to_return(read_http_fixture("listEmailForwards/success.http"))
     end
 
     it "builds the correct request" do
       subject.email_forwards(account_id, domain_id)
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/email_forwards").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/email_forwards")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "supports pagination" do
@@ -64,8 +64,8 @@ describe Dnsimple::Client, ".domains" do
 
     context "when the domain does not exist" do
       it "raises NotFoundError" do
-        stub_request(:get, %r{/v2}).
-            to_return(read_http_fixture("notfound-domain.http"))
+        stub_request(:get, %r{/v2})
+            .to_return(read_http_fixture("notfound-domain.http"))
 
         expect {
           subject.email_forwards(account_id, domain_id)
@@ -76,8 +76,8 @@ describe Dnsimple::Client, ".domains" do
 
   describe "#all_email_forwards" do
     before do
-      stub_request(:get, %r{/v2/#{account_id}/domains/#{domain_id}/email_forwards}).
-          to_return(read_http_fixture("listEmailForwards/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/domains/#{domain_id}/email_forwards})
+          .to_return(read_http_fixture("listEmailForwards/success.http"))
     end
 
     let(:account_id) { 1010 }
@@ -100,8 +100,8 @@ describe Dnsimple::Client, ".domains" do
     let(:domain_id) { "example.com" }
 
     before do
-      stub_request(:post, %r{/v2/#{account_id}/domains/#{domain_id}/email_forwards$}).
-          to_return(read_http_fixture("createEmailForward/created.http"))
+      stub_request(:post, %r{/v2/#{account_id}/domains/#{domain_id}/email_forwards$})
+          .to_return(read_http_fixture("createEmailForward/created.http"))
     end
 
     let(:attributes) { { from: "jim", to: "jim@another.com" } }
@@ -109,9 +109,9 @@ describe Dnsimple::Client, ".domains" do
     it "builds the correct request" do
       subject.create_email_forward(account_id, domain_id, attributes)
 
-      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/email_forwards").
-          with(body: attributes).
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/email_forwards")
+          .with(body: attributes)
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns the email forward" do
@@ -130,15 +130,15 @@ describe Dnsimple::Client, ".domains" do
     let(:email_forward_id) { 17706 }
 
     before do
-      stub_request(:get, %r{/v2/#{account_id}/domains/#{domain_id}/email_forwards.+$}).
-          to_return(read_http_fixture("getEmailForward/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/domains/#{domain_id}/email_forwards.+$})
+          .to_return(read_http_fixture("getEmailForward/success.http"))
     end
 
     it "builds the correct request" do
       subject.email_forward(account_id, domain_id, email_forward_id)
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/email_forwards/#{email_forward_id}").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/email_forwards/#{email_forward_id}")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns the email forward" do
@@ -157,8 +157,8 @@ describe Dnsimple::Client, ".domains" do
 
     context "when the email forward does not exist" do
       it "raises NotFoundError" do
-        stub_request(:get, %r{/v2}).
-            to_return(read_http_fixture("notfound-emailforward.http"))
+        stub_request(:get, %r{/v2})
+            .to_return(read_http_fixture("notfound-emailforward.http"))
 
         expect {
           subject.email_forward(account_id, domain_id, email_forward_id)
@@ -173,15 +173,15 @@ describe Dnsimple::Client, ".domains" do
     let(:email_forward_id) { 1 }
 
     before do
-      stub_request(:delete, %r{/v2/#{account_id}/domains/#{domain_id}/email_forwards/#{email_forward_id}$}).
-          to_return(read_http_fixture("deleteEmailForward/success.http"))
+      stub_request(:delete, %r{/v2/#{account_id}/domains/#{domain_id}/email_forwards/#{email_forward_id}$})
+          .to_return(read_http_fixture("deleteEmailForward/success.http"))
     end
 
     it "builds the correct request" do
       subject.delete_email_forward(account_id, domain_id, email_forward_id)
 
-      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/email_forwards/#{email_forward_id}").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/email_forwards/#{email_forward_id}")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns nothing" do
@@ -194,8 +194,8 @@ describe Dnsimple::Client, ".domains" do
 
     context "when the email forward does not exist" do
       it "raises NotFoundError" do
-        stub_request(:delete, %r{/v2}).
-            to_return(read_http_fixture("notfound-emailforward.http"))
+        stub_request(:delete, %r{/v2})
+            .to_return(read_http_fixture("notfound-emailforward.http"))
 
         expect {
           subject.delete_email_forward(account_id, domain_id, email_forward_id)

data/spec/dnsimple/client/domains_pushes_spec.rb

@@ -9,8 +9,8 @@ describe Dnsimple::Client, ".domains" do
     let(:domain_id) { "example.com" }
 
     before do
-      stub_request(:post, %r{/v2/#{account_id}/domains/#{domain_id}/pushes$}).
-          to_return(read_http_fixture("initiatePush/success.http"))
+      stub_request(:post, %r{/v2/#{account_id}/domains/#{domain_id}/pushes$})
+          .to_return(read_http_fixture("initiatePush/success.http"))
     end
 
     let(:attributes) { { new_account_email: "admin@target-account.test" } }
@@ -18,9 +18,9 @@ describe Dnsimple::Client, ".domains" do
     it "builds the correct request" do
       subject.initiate_push(account_id, domain_id, attributes)
 
-      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/pushes").
-          with(body: attributes).
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/pushes")
+          .with(body: attributes)
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns the domain push" do
@@ -37,15 +37,15 @@ describe Dnsimple::Client, ".domains" do
     let(:account_id) { 2020 }
 
     before do
-      stub_request(:get, %r{/v2/#{account_id}/pushes}).
-          to_return(read_http_fixture("listPushes/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/pushes})
+          .to_return(read_http_fixture("listPushes/success.http"))
     end
 
     it "builds the correct request" do
       subject.pushes(account_id)
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/pushes").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/pushes")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "supports pagination" do
@@ -89,8 +89,8 @@ describe Dnsimple::Client, ".domains" do
     let(:push_id) { 1 }
 
     before do
-      stub_request(:post, %r{/v2/#{account_id}/pushes/#{push_id}$}).
-          to_return(read_http_fixture("acceptPush/success.http"))
+      stub_request(:post, %r{/v2/#{account_id}/pushes/#{push_id}$})
+          .to_return(read_http_fixture("acceptPush/success.http"))
     end
 
     let(:attributes) { { contact_id: 2 } }
@@ -98,9 +98,9 @@ describe Dnsimple::Client, ".domains" do
     it "builds the correct request" do
       subject.accept_push(account_id, push_id, attributes)
 
-      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/pushes/#{push_id}").
-          with(body: attributes).
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/pushes/#{push_id}")
+          .with(body: attributes)
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns nothing" do
@@ -113,8 +113,8 @@ describe Dnsimple::Client, ".domains" do
 
     context "when the domain push does not exist" do
       it "raises NotFoundError" do
-        stub_request(:post, %r{/v2}).
-            to_return(read_http_fixture("notfound-domainpush.http"))
+        stub_request(:post, %r{/v2})
+            .to_return(read_http_fixture("notfound-domainpush.http"))
 
         expect {
           subject.accept_push(account_id, push_id, attributes)
@@ -128,15 +128,15 @@ describe Dnsimple::Client, ".domains" do
     let(:push_id) { 1 }
 
     before do
-      stub_request(:delete, %r{/v2/#{account_id}/pushes/#{push_id}$}).
-          to_return(read_http_fixture("rejectPush/success.http"))
+      stub_request(:delete, %r{/v2/#{account_id}/pushes/#{push_id}$})
+          .to_return(read_http_fixture("rejectPush/success.http"))
     end
 
     it "builds the correct request" do
       subject.reject_push(account_id, push_id)
 
-      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/pushes/#{push_id}").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/pushes/#{push_id}")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns nothing" do
@@ -149,8 +149,8 @@ describe Dnsimple::Client, ".domains" do
 
     context "when the domain push does not exist" do
       it "raises NotFoundError" do
-        stub_request(:delete, %r{/v2}).
-            to_return(read_http_fixture("notfound-domainpush.http"))
+        stub_request(:delete, %r{/v2})
+            .to_return(read_http_fixture("notfound-domainpush.http"))
 
         expect {
           subject.reject_push(account_id, push_id)

data/spec/dnsimple/client/domains_spec.rb

@@ -9,15 +9,15 @@ describe Dnsimple::Client, ".domains" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:get, %r{/v2/#{account_id}/domains}).
-          to_return(read_http_fixture("listDomains/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/domains})
+          .to_return(read_http_fixture("listDomains/success.http"))
     end
 
     it "builds the correct request" do
       subject.domains(account_id)
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/domains").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/domains")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "supports pagination" do
@@ -70,8 +70,8 @@ describe Dnsimple::Client, ".domains" do
 
   describe "#all_domains" do
     before do
-      stub_request(:get, %r{/v2/#{account_id}/domains}).
-          to_return(read_http_fixture("listDomains/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/domains})
+          .to_return(read_http_fixture("listDomains/success.http"))
     end
 
     let(:account_id) { 1010 }
@@ -98,8 +98,8 @@ describe Dnsimple::Client, ".domains" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:post, %r{/v2/#{account_id}/domains$}).
-          to_return(read_http_fixture("createDomain/created.http"))
+      stub_request(:post, %r{/v2/#{account_id}/domains$})
+          .to_return(read_http_fixture("createDomain/created.http"))
     end
 
     let(:attributes) { { name: "example.com" } }
@@ -107,9 +107,9 @@ describe Dnsimple::Client, ".domains" do
     it "builds the correct request" do
       subject.create_domain(account_id, attributes)
 
-      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/domains").
-          with(body: attributes).
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/domains")
+          .with(body: attributes)
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns the domain" do
@@ -126,15 +126,15 @@ describe Dnsimple::Client, ".domains" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:get, %r{/v2/#{account_id}/domains/.+$}).
-          to_return(read_http_fixture("getDomain/success.http"))
+      stub_request(:get, %r{/v2/#{account_id}/domains/.+$})
+          .to_return(read_http_fixture("getDomain/success.http"))
     end
 
     it "builds the correct request" do
       subject.domain(account_id, domain = "example.com")
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain}").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain}")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns the domain" do
@@ -157,8 +157,8 @@ describe Dnsimple::Client, ".domains" do
 
     context "when the domain does not exist" do
       it "raises NotFoundError" do
-        stub_request(:get, %r{/v2}).
-            to_return(read_http_fixture("notfound-domain.http"))
+        stub_request(:get, %r{/v2})
+            .to_return(read_http_fixture("notfound-domain.http"))
 
         expect {
           subject.domain(account_id, "example.com")
@@ -171,15 +171,15 @@ describe Dnsimple::Client, ".domains" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:delete, %r{/v2/#{account_id}/domains/.+$}).
-          to_return(read_http_fixture("deleteDomain/success.http"))
+      stub_request(:delete, %r{/v2/#{account_id}/domains/.+$})
+          .to_return(read_http_fixture("deleteDomain/success.http"))
     end
 
     it "builds the correct request" do
       subject.delete_domain(account_id, domain = "example.com")
 
-      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain}").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain}")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns nothing" do
@@ -192,8 +192,8 @@ describe Dnsimple::Client, ".domains" do
 
     context "when the domain does not exist" do
       it "raises NotFoundError" do
-        stub_request(:delete, %r{/v2}).
-            to_return(read_http_fixture("notfound-domain.http"))
+        stub_request(:delete, %r{/v2})
+            .to_return(read_http_fixture("notfound-domain.http"))
 
         expect {
           subject.delete_domain(account_id, "example.com")
@@ -207,15 +207,15 @@ describe Dnsimple::Client, ".domains" do
     let(:domain_id) { "example.com" }
 
     before do
-      stub_request(:post, %r{/v2/#{account_id}/domains/#{domain_id}/token}).
-          to_return(read_http_fixture("resetDomainToken/success.http"))
+      stub_request(:post, %r{/v2/#{account_id}/domains/#{domain_id}/token})
+          .to_return(read_http_fixture("resetDomainToken/success.http"))
     end
 
     it "builds the correct request" do
       subject.reset_domain_token(account_id, domain_id)
 
-      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/token").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/token")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns the domain" do

data/spec/dnsimple/client/identity_spec.rb

@@ -7,15 +7,15 @@ describe Dnsimple::Client, ".identity" do
 
   describe "#whoami" do
     before do
-      stub_request(:get, %r{/v2/whoami$}).
-          to_return(read_http_fixture("whoami/success.http"))
+      stub_request(:get, %r{/v2/whoami$})
+          .to_return(read_http_fixture("whoami/success.http"))
     end
 
     it "builds the correct request" do
       subject.whoami
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/whoami").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/whoami")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns the whoami" do
@@ -28,8 +28,8 @@ describe Dnsimple::Client, ".identity" do
 
     context "when authenticated as account" do
       before do
-        stub_request(:get, %r{/v2/whoami$}).
-            to_return(read_http_fixture("whoami/success-account.http"))
+        stub_request(:get, %r{/v2/whoami$})
+            .to_return(read_http_fixture("whoami/success-account.http"))
       end
 
       it "sets the account" do
@@ -41,8 +41,8 @@ describe Dnsimple::Client, ".identity" do
 
     context "when authenticated as user" do
       before do
-        stub_request(:get, %r{/v2/whoami$}).
-            to_return(read_http_fixture("whoami/success-user.http"))
+        stub_request(:get, %r{/v2/whoami$})
+            .to_return(read_http_fixture("whoami/success-user.http"))
       end
 
       it "sets the user" do

data/spec/dnsimple/client/oauth_spec.rb

@@ -12,16 +12,16 @@ describe Dnsimple::Client, ".oauth" do
     let(:state) { "super-state" }
 
     before do
-      stub_request(:post, %r{/v2/oauth/access_token$}).
-          to_return(read_http_fixture("oauthAccessToken/success.http"))
+      stub_request(:post, %r{/v2/oauth/access_token$})
+          .to_return(read_http_fixture("oauthAccessToken/success.http"))
     end
 
     it "builds the correct request" do
       subject.exchange_authorization_for_token(code, client_id, client_secret, state: state)
 
-      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/oauth/access_token").
-          with(body: { client_id: client_id, client_secret: client_secret, code: code, state: state, grant_type: "authorization_code" }).
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/oauth/access_token")
+          .with(body: { client_id: client_id, client_secret: client_secret, code: code, state: state, grant_type: "authorization_code" })
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns oauth token" do
@@ -39,9 +39,9 @@ describe Dnsimple::Client, ".oauth" do
       it "builds the correct request" do
         subject.exchange_authorization_for_token(code, client_id, client_secret, state: state, redirect_uri: redirect_uri)
 
-        expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/oauth/access_token").
-            with(body: { client_id: client_id, client_secret: client_secret, code: code, state: state, redirect_uri: redirect_uri, grant_type: "authorization_code" }).
-            with(headers: { 'Accept' => 'application/json' })
+        expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/oauth/access_token")
+            .with(body: { client_id: client_id, client_secret: client_secret, code: code, state: state, redirect_uri: redirect_uri, grant_type: "authorization_code" })
+            .with(headers: { 'Accept' => 'application/json' })
       end
     end
   end

data/spec/dnsimple/client/registrar_auto_renewal_spec.rb

@@ -9,16 +9,16 @@ describe Dnsimple::Client, ".registrar" do
     let(:domain_id) { "example.com" }
 
     before do
-      stub_request(:put, %r{/v2/#{account_id}/registrar/domains/#{domain_id}}).
-          to_return(read_http_fixture("enableDomainAutoRenewal/success.http"))
+      stub_request(:put, %r{/v2/#{account_id}/registrar/domains/#{domain_id}})
+          .to_return(read_http_fixture("enableDomainAutoRenewal/success.http"))
     end
 
 
     it "builds the correct request" do
       subject.enable_auto_renewal(account_id, domain_id)
 
-      expect(WebMock).to have_requested(:put, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_id}/auto_renewal").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:put, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_id}/auto_renewal")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns nothing" do
@@ -31,8 +31,8 @@ describe Dnsimple::Client, ".registrar" do
 
     context "when the domain does not exist" do
       it "raises NotFoundError" do
-        stub_request(:put, %r{/v2}).
-            to_return(read_http_fixture("notfound-domain.http"))
+        stub_request(:put, %r{/v2})
+            .to_return(read_http_fixture("notfound-domain.http"))
 
         expect {
           subject.enable_auto_renewal(account_id, domain_id)
@@ -46,16 +46,16 @@ describe Dnsimple::Client, ".registrar" do
     let(:domain_id) { "example.com" }
 
     before do
-      stub_request(:delete, %r{/v2/#{account_id}/registrar/domains/#{domain_id}}).
-          to_return(read_http_fixture("disableDomainAutoRenewal/success.http"))
+      stub_request(:delete, %r{/v2/#{account_id}/registrar/domains/#{domain_id}})
+          .to_return(read_http_fixture("disableDomainAutoRenewal/success.http"))
     end
 
 
     it "builds the correct request" do
       subject.disable_auto_renewal(account_id, domain_id)
 
-      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_id}/auto_renewal").
-          with(headers: { 'Accept' => 'application/json' })
+      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_id}/auto_renewal")
+          .with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns nothing" do
@@ -68,8 +68,8 @@ describe Dnsimple::Client, ".registrar" do
 
     context "when the domain does not exist" do
       it "raises NotFoundError" do
-        stub_request(:delete, %r{/v2}).
-            to_return(read_http_fixture("notfound-domain.http"))
+        stub_request(:delete, %r{/v2})
+            .to_return(read_http_fixture("notfound-domain.http"))
 
         expect {
           subject.disable_auto_renewal(account_id, domain_id)