dnsimple 3.0.0.pre.beta2 → 3.0.0

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
@@ -60,8 +60,8 @@ describe Dnsimple::Client, ".domains" do
     let(:account_id) { 1010 }
 
     it "delegates to client.paginate" do
-      expect(subject).to receive(:paginate).with(:domains, account_id, { foo: "bar" })
-      subject.all_domains(account_id, { foo: "bar" })
+      expect(subject).to receive(:paginate).with(:domains, account_id, foo: "bar")
+      subject.all_domains(account_id, foo: "bar")
     end
   end
 
@@ -69,8 +69,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" } }
@@ -78,9 +78,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
@@ -97,15 +97,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
@@ -128,8 +128,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")
@@ -142,15 +142,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
@@ -163,8 +163,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")
@@ -178,15 +178,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 a response" 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

@@ -11,16 +11,16 @@ describe Dnsimple::Client, ".oauth" do
     let(:code) { "super-code" }
 
     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)
 
-      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, 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, grant_type: "authorization_code" }).
+          with(headers: { 'Accept' => 'application/json' })
     end
 
     it "returns oauth token" do
@@ -37,11 +37,11 @@ describe Dnsimple::Client, ".oauth" do
       let(:redirect_uri) { "super-redirect-uri" }
 
       it "builds the correct request" do
-        subject.exchange_authorization_for_token(code, client_id, client_secret, state:state, redirect_uri: redirect_uri)
+        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("enableAutoRenewal/success.http"))
+      stub_request(:put, %r{/v2/#{account_id}/registrar/domains/#{domain_id}}).
+          to_return(read_http_fixture("enableAutoRenewal/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)
@@ -40,22 +40,22 @@ describe Dnsimple::Client, ".registrar" do
       end
     end
   end
-  
+
   describe "#disable_auto_renewal" do
     let(:account_id) { 1010 }
     let(:domain_id) { "example.com" }
 
     before do
-      stub_request(:delete, %r[/v2/#{account_id}/registrar/domains/#{domain_id}])
-          .to_return(read_http_fixture("disableAutoRenewal/success.http"))
+      stub_request(:delete, %r{/v2/#{account_id}/registrar/domains/#{domain_id}}).
+          to_return(read_http_fixture("disableAutoRenewal/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)

data/spec/dnsimple/client/registrar_delegation_spec.rb

@@ -0,0 +1,55 @@
+require 'spec_helper'
+
+describe Dnsimple::Client, ".registrar" do
+
+  subject { described_class.new(base_url: "https://api.dnsimple.test", access_token: "a1b2c3").registrar }
+
+  describe "#domain_delegation" 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"))
+    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" })
+    end
+
+    it "returns the name servers of the domain" do
+      response = subject.domain_delegation(account_id, "example.com")
+      expect(response).to be_a(Dnsimple::Response)
+
+      expect(response.data).to match_array(%w(ns1.dnsimple.com ns2.dnsimple.com ns3.dnsimple.com ns4.dnsimple.com))
+    end
+  end
+
+  describe "#change_domain_delegation" 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"))
+    end
+
+    let(:attributes) { %w(ns1.dnsimple.com ns2.dnsimple.com ns3.dnsimple.com ns4.dnsimple.com) }
+
+    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" })
+    end
+
+    it "returns the name servers of the domain" do
+      response = subject.change_domain_delegation(account_id, "example.com", attributes)
+      expect(response).to be_a(Dnsimple::Response)
+
+      expect(response.data).to match_array(%w(ns1.dnsimple.com ns2.dnsimple.com ns3.dnsimple.com ns4.dnsimple.com))
+    end
+  end
+end

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
@@ -36,8 +36,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/.+/registration$}).
+          to_return(read_http_fixture("registerDomain/success.http"))
     end
 
     let(:attributes) { { registrant_id: "10" } }
@@ -45,9 +45,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}/registration").
+          with(body: attributes).
+          with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the domain" do
@@ -70,8 +70,8 @@ describe Dnsimple::Client, ".registrar" do
     let(:account_id) { 1010 }
 
     before do
-      stub_request(:post, %r[/v2/#{account_id}/registrar/domains/.+/renew$])
-          .to_return(read_http_fixture("renewDomain/success.http"))
+      stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/renewal$}).
+          to_return(read_http_fixture("renewDomain/success.http"))
     end
 
     let(:attributes) { { period: "3" } }
@@ -79,9 +79,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}/renew")
-          .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}/renewal").
+          with(body: attributes).
+          with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the domain" do
@@ -95,8 +95,8 @@ describe Dnsimple::Client, ".registrar" do
 
     context "when it is too son for the domain to be renewed" do
       it "raises a BadRequestError" do
-        stub_request(:post, %r[/v2/#{account_id}/registrar/domains/.+/renew$])
-            .to_return(read_http_fixture("renewDomain/error-tooearly.http"))
+        stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/renewal$}).
+            to_return(read_http_fixture("renewDomain/error-tooearly.http"))
 
         expect {
           subject.renew_domain(account_id, "example.com", attributes)
@@ -109,8 +109,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/.+/transfer$}).
+          to_return(read_http_fixture("transferDomain/success.http"))
     end
 
     let(:attributes) { { registrant_id: "10", auth_info: "x1y2z3" } }
@@ -118,9 +118,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}/transfer").
+          with(body: attributes).
+          with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the domain" do
@@ -141,8 +141,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/.+/transfer$}).
+            to_return(read_http_fixture("transferDomain/error-indnsimple.http"))
 
         expect {
           subject.transfer_domain(account_id, "example.com", attributes)
@@ -152,8 +152,8 @@ describe Dnsimple::Client, ".registrar" do
 
     context "when :auth_info 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/.+/transfer$}).
+            to_return(read_http_fixture("transferDomain/error-missing-authcode.http"))
 
         expect {
           subject.transfer_domain(account_id, "example.com", registrant_id: 10)
@@ -166,15 +166,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/.+/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}/transfer_out").
+          with(headers: { "Accept" => "application/json" })
     end
 
     it "returns nothing" do