dnsimple 4.6.0 → 7.0.0

data/spec/dnsimple/client/domains_pushes_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Dnsimple::Client, ".domains" do
@@ -6,6 +8,7 @@ describe Dnsimple::Client, ".domains" do
 
   describe "#initiate_push" do
     let(:account_id) { 1010 }
+    let(:attributes) { { new_account_email: "admin@target-account.test" } }
     let(:domain_id) { "example.com" }
 
     before do
@@ -13,7 +16,6 @@ describe Dnsimple::Client, ".domains" do
           .to_return(read_http_fixture("initiatePush/success.http"))
     end
 
-    let(:attributes) { { new_account_email: "admin@target-account.test" } }
 
     it "builds the correct request" do
       subject.initiate_push(account_id, domain_id, attributes)
@@ -86,6 +88,7 @@ describe Dnsimple::Client, ".domains" do
 
   describe "#accept_push" do
     let(:account_id) { 2020 }
+    let(:attributes) { { contact_id: 2 } }
     let(:push_id) { 1 }
 
     before do
@@ -93,7 +96,6 @@ describe Dnsimple::Client, ".domains" do
           .to_return(read_http_fixture("acceptPush/success.http"))
     end
 
-    let(:attributes) { { contact_id: 2 } }
 
     it "builds the correct request" do
       subject.accept_push(account_id, push_id, attributes)

data/spec/dnsimple/client/domains_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Dnsimple::Client, ".domains" do
@@ -33,9 +35,9 @@ describe Dnsimple::Client, ".domains" do
     end
 
     it "supports sorting" do
-      subject.domains(account_id, sort: "expires_on:asc")
+      subject.domains(account_id, sort: "expiration:asc")
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/domains?sort=expires_on:asc")
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/domains?sort=expiration:asc")
     end
 
     it "supports filtering" do
@@ -82,9 +84,9 @@ describe Dnsimple::Client, ".domains" do
     end
 
     it "supports sorting" do
-      subject.all_domains(account_id, sort: "expires_on:asc")
+      subject.all_domains(account_id, sort: "expiration:asc")
 
-      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/domains?page=1&per_page=100&sort=expires_on:asc")
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/domains?page=1&per_page=100&sort=expiration:asc")
     end
 
     it "supports filtering" do
@@ -96,13 +98,13 @@ describe Dnsimple::Client, ".domains" do
 
   describe "#create_domain" do
     let(:account_id) { 1010 }
+    let(:attributes) { { name: "example.com" } }
 
     before do
       stub_request(:post, %r{/v2/#{account_id}/domains$})
           .to_return(read_http_fixture("createDomain/created.http"))
     end
 
-    let(:attributes) { { name: "example.com" } }
 
     it "builds the correct request" do
       subject.create_domain(account_id, attributes)
@@ -131,28 +133,28 @@ describe Dnsimple::Client, ".domains" do
     end
 
     it "builds the correct request" do
-      subject.domain(account_id, domain = "example.com")
+      subject.domain(account_id, domain = "example-alpha.com")
 
       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
-      response = subject.domain(account_id, "example.com")
+      response = subject.domain(account_id, "example-alpha.com")
       expect(response).to be_a(Dnsimple::Response)
 
       result = response.data
       expect(result).to be_a(Dnsimple::Struct::Domain)
-      expect(result.id).to eq(1)
-      expect(result.account_id).to eq(1010)
-      expect(result.registrant_id).to eq(nil)
+      expect(result.id).to eq(181984)
+      expect(result.account_id).to eq(1385)
+      expect(result.registrant_id).to eq(2715)
       expect(result.name).to eq("example-alpha.com")
-      expect(result.state).to eq("hosted")
+      expect(result.state).to eq("registered")
       expect(result.auto_renew).to be(false)
       expect(result.private_whois).to be(false)
-      expect(result.expires_on).to eq(nil)
-      expect(result.created_at).to eq("2014-12-06T15:56:55Z")
-      expect(result.updated_at).to eq("2015-12-09T00:20:56Z")
+      expect(result.expires_at).to eq("2021-06-05T02:15:00Z")
+      expect(result.created_at).to eq("2020-06-04T19:15:14Z")
+      expect(result.updated_at).to eq("2020-06-04T19:15:21Z")
     end
 
     context "when the domain does not exist" do
@@ -201,31 +203,4 @@ describe Dnsimple::Client, ".domains" do
       end
     end
   end
-
-  describe "#reset_domain_token" do
-    let(:account_id) { 1010 }
-    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"))
-    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' })
-    end
-
-    it "returns the domain" do
-      response = subject.reset_domain_token(account_id, domain_id)
-      expect(response).to be_a(Dnsimple::Response)
-
-      result = response.data
-      expect(result).to be_a(Dnsimple::Struct::Domain)
-      expect(result.id).to be_a(Integer)
-    end
-  end
-
 end

data/spec/dnsimple/client/identity_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Dnsimple::Client, ".identity" do

data/spec/dnsimple/client/oauth_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Dnsimple::Client, ".oauth" do

data/spec/dnsimple/client/registrar_auto_renewal_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Dnsimple::Client, ".registrar" do

data/spec/dnsimple/client/registrar_delegation_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Dnsimple::Client, ".registrar" do
@@ -29,13 +31,13 @@ describe Dnsimple::Client, ".registrar" do
 
   describe "#change_domain_delegation" do
     let(:account_id) { 1010 }
+    let(:attributes) { %w(ns1.dnsimple.com ns2.dnsimple.com ns3.dnsimple.com ns4.dnsimple.com) }
 
     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)
@@ -55,13 +57,13 @@ describe Dnsimple::Client, ".registrar" do
 
   describe "#change_domain_delegation_to_vanity" do
     let(:account_id) { 1010 }
+    let(:attributes) { %w(ns1.example.com ns2.example.com) }
 
     before do
       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) }
 
     it "builds the correct request" do
       subject.change_domain_delegation_to_vanity(account_id, domain_name = "example.com", attributes)

data/spec/dnsimple/client/registrar_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Dnsimple::Client, ".registrar" do
@@ -37,7 +39,7 @@ describe Dnsimple::Client, ".registrar" do
 
     context "when premium price" do
       before do
-        stub_request(:get, %r{/v2/#{account_id}/registrar/domains/.+/premium_price[\?action]*})
+        stub_request(:get, %r{/v2/#{account_id}/registrar/domains/.+/premium_price[?action]*})
             .to_return(read_http_fixture("getDomainPremiumPrice/success.http"))
       end
 
@@ -80,15 +82,58 @@ describe Dnsimple::Client, ".registrar" do
     end
   end
 
+  describe "#get_domain_prices" do
+    let(:account_id) { 1010 }
+
+    before do
+      stub_request(:get, %r{/v2/#{account_id}/registrar/domains/bingo.pizza/prices$})
+          .to_return(read_http_fixture("getDomainPrices/success.http"))
+    end
+
+    it "builds the correct request" do
+      subject.get_domain_prices(account_id, "bingo.pizza")
+
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/bingo.pizza/prices")
+          .with(headers: { "Accept" => "application/json" })
+    end
+
+    it "returns the prices" do
+      response = subject.get_domain_prices(account_id, "bingo.pizza")
+      expect(response).to be_a(Dnsimple::Response)
+
+      result = response.data
+
+      expect(result).to be_a(Dnsimple::Struct::DomainPrice)
+      expect(result.domain).to eq("bingo.pizza")
+      expect(result.premium).to be(true)
+      expect(result.registration_price).to eq(20.0)
+      expect(result.renewal_price).to eq(20.0)
+      expect(result.transfer_price).to eq(20.0)
+    end
+
+    context "when the TLD is not supported" do
+      before do
+        stub_request(:get, %r{/v2/#{account_id}/registrar/domains/bingo.pineapple/prices$})
+            .to_return(read_http_fixture("getDomainPrices/failure.http"))
+      end
+
+      it "raises error" do
+        expect {
+          subject.get_domain_prices(account_id, "bingo.pineapple")
+        }.to raise_error(Dnsimple::RequestError, "TLD .PINEAPPLE is not supported")
+      end
+    end
+  end
+
   describe "#register_domain" do
     let(:account_id) { 1010 }
+    let(:attributes) { { registrant_id: "10" } }
 
     before do
       stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/registrations$})
           .to_return(read_http_fixture("registerDomain/success.http"))
     end
 
-    let(:attributes) { { registrant_id: "10" } }
 
     it "builds the correct request" do
       subject.register_domain(account_id, domain_name = "example.com", attributes)
@@ -117,13 +162,13 @@ describe Dnsimple::Client, ".registrar" do
 
   describe "#renew_domain" do
     let(:account_id) { 1010 }
+    let(:attributes) { { period: "3" } }
 
     before do
       stub_request(:post, %r{/v2/#{account_id}/registrar/domains/.+/renewals$})
           .to_return(read_http_fixture("renewDomain/success.http"))
     end
 
-    let(:attributes) { { period: "3" } }
 
     it "builds the correct request" do
       subject.renew_domain(account_id, domain_name = "example.com", attributes)
@@ -157,13 +202,13 @@ describe Dnsimple::Client, ".registrar" do
 
   describe "#transfer_domain" do
     let(:account_id) { 1010 }
+    let(:attributes) { { registrant_id: "10", auth_code: "x1y2z3" } }
 
     before do
       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" } }
 
     it "builds the correct request" do
       subject.transfer_domain(account_id, domain_name = "example.com", attributes)
@@ -212,6 +257,73 @@ describe Dnsimple::Client, ".registrar" do
     end
   end
 
+  describe "#get_transfer_domain" do
+    let(:account_id) { 1010 }
+
+    before do
+      stub_request(:get, %r{/v2/#{account_id}/registrar/domains/.+/transfers/.+$})
+          .to_return(read_http_fixture("getDomainTransfer/success.http"))
+    end
+
+    it "builds the correct request" do
+      subject.get_domain_transfer(account_id, domain_name = "example.com", transfer_id = 361)
+
+      expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/transfers/#{transfer_id}")
+          .with(headers: { "Accept" => "application/json" })
+    end
+
+    it "returns the domain transfer" do
+      response = subject.get_domain_transfer(account_id, "example.com", 361)
+      expect(response).to be_a(Dnsimple::Response)
+
+      result = response.data
+      expect(result).to be_a(Dnsimple::Struct::DomainTransfer)
+      expect(result.id).to eq(361)
+      expect(result.domain_id).to eq(182245)
+      expect(result.registrant_id).to eq(2715)
+      expect(result.state).to eq("cancelled")
+      expect(result.auto_renew).to be(false)
+      expect(result.whois_privacy).to be(false)
+      expect(result.status_description).to eq("Canceled by customer")
+      expect(result.created_at).to eq("2020-06-05T18:08:00Z")
+      expect(result.updated_at).to eq("2020-06-05T18:10:01Z")
+    end
+  end
+
+  describe "#cancel_domain_transfer" do
+    let(:account_id) { 1010 }
+
+    before do
+      stub_request(:delete, %r{/v2/#{account_id}/registrar/domains/.+/transfers/.+$})
+          .to_return(read_http_fixture("cancelDomainTransfer/success.http"))
+    end
+
+    it "builds the correct request" do
+      subject.cancel_domain_transfer(account_id, domain_name = "example.com", transfer_id = 361)
+
+      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_name}/transfers/#{transfer_id}")
+          .with(headers: { "Accept" => "application/json" })
+    end
+
+    it "returns the domain transfer" do
+      response = subject.cancel_domain_transfer(account_id, "example.com", 361)
+      expect(response).to be_a(Dnsimple::Response)
+
+      result = response.data
+      expect(result).to be_a(Dnsimple::Struct::DomainTransfer)
+      expect(result.id).to eq(361)
+      expect(result.domain_id).to eq(182245)
+      expect(result.registrant_id).to eq(2715)
+      expect(result.state).to eq("transferring")
+      expect(result.auto_renew).to be(false)
+      expect(result.whois_privacy).to be(false)
+      expect(result.status_description).to eq(nil)
+      expect(result.created_at).to eq("2020-06-05T18:08:00Z")
+      expect(result.updated_at).to eq("2020-06-05T18:08:04Z")
+    end
+  end
+
+
   describe "#transfer_domain_out" do
     let(:account_id) { 1010 }
 

data/spec/dnsimple/client/registrar_whois_privacy_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Dnsimple::Client, ".registrar" do

data/spec/dnsimple/client/services_domains_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Dnsimple::Client, ".services" do
@@ -50,15 +52,14 @@ describe Dnsimple::Client, ".services" do
         expect(service.sid).to be_a(String)
         expect(service.description).to be_a(String)
 
-        service.settings.each do |service_setting|
-          expect(service_setting).to be_a(Dnsimple::Struct::Service::Setting)
-        end
+        expect(service.settings).to all(be_a(Dnsimple::Struct::Service::Setting))
       end
     end
   end
 
   describe "#apply_service" do
     let(:account_id) { 1010 }
+    let(:settings) { { app: "foo" } }
     let(:domain_id)  { "example.com" }
     let(:service_id) { "service1" }
 
@@ -67,7 +68,6 @@ describe Dnsimple::Client, ".services" do
           .to_return(read_http_fixture("applyService/success.http"))
     end
 
-    let(:settings) { { app: "foo" } }
 
     it "builds the correct request" do
       subject.apply_service(account_id, service_id, domain_id, settings)

data/spec/dnsimple/client/services_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Dnsimple::Client, ".services" do
@@ -47,9 +49,7 @@ describe Dnsimple::Client, ".services" do
         expect(service.sid).to be_a(String)
         expect(service.description).to be_a(String)
 
-        service.settings.each do |service_setting|
-          expect(service_setting).to be_a(Dnsimple::Struct::Service::Setting)
-        end
+        expect(service.settings).to all(be_a(Dnsimple::Struct::Service::Setting))
       end
     end
   end

data/spec/dnsimple/client/templates_domains_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Dnsimple::Client, ".templates" do

data/spec/dnsimple/client/templates_records_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Dnsimple::Client, ".templates" do
@@ -76,6 +78,7 @@ describe Dnsimple::Client, ".templates" do
 
   describe "#create_record" do
     let(:account_id) { 1010 }
+    let(:attributes) { { type: "MX", name: "", content: "mx.example.com", priority: 10, ttl: 600 } }
     let(:template_id) { "alpha" }
 
     before do
@@ -83,7 +86,6 @@ describe Dnsimple::Client, ".templates" do
           .to_return(read_http_fixture("createTemplateRecord/created.http"))
     end
 
-    let(:attributes) { { type: "MX", name: "", content: "mx.example.com", priority: 10, ttl: 600 } }
 
     it "builds the correct request" do
       subject.create_record(account_id, template_id, attributes)