dnsimple 3.1.0 → 4.0.0

data/spec/dnsimple/client/services_spec.rb

@@ -42,7 +42,7 @@ describe Dnsimple::Client, ".services" do
 
       response.data.each do |service|
         expect(service).to be_a(Dnsimple::Struct::Service)
-        expect(service.id).to be_a(Fixnum)
+        expect(service.id).to be_a(Integer)
         expect(service.name).to be_a(String)
         expect(service.short_name).to be_a(String)
         expect(service.description).to be_a(String)

data/spec/dnsimple/client/templates_domains_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+describe Dnsimple::Client, ".templates" do
+
+  subject { described_class.new(base_url: "https://api.dnsimple.test", access_token: "a1b2c3").templates }
+
+
+  describe "#apply_template" do
+    let(:account_id)  { 1010 }
+    let(:template_id) { 5410 }
+    let(:domain_id)   { 'example.com' }
+
+    before do
+      stub_request(:post, %r{/v2/#{account_id}/domains/#{domain_id}/templates/#{template_id}$}).
+          to_return(read_http_fixture("applyTemplate/success.http"))
+    end
+
+    it "builds the correct request" do
+      subject.apply_template(account_id, template_id, domain_id)
+
+      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/templates/#{template_id}").
+          with(headers: { "Accept" => "application/json" })
+    end
+
+    it "returns nil" do
+      response = subject.apply_template(account_id, template_id, domain_id)
+      expect(response).to be_a(Dnsimple::Response)
+      expect(response.data).to be_nil
+    end
+  end
+
+end

data/spec/dnsimple/client/templates_records_spec.rb

@@ -45,7 +45,7 @@ describe Dnsimple::Client, ".templates" do
 
       response.data.each do |result|
         expect(result).to be_a(Dnsimple::Struct::TemplateRecord)
-        expect(result.id).to be_a(Fixnum)
+        expect(result.id).to be_a(Integer)
         expect(result.type).to be_a(String)
         expect(result.name).to be_a(String)
         expect(result.content).to be_a(String)

data/spec/dnsimple/client/templates_spec.rb

@@ -182,11 +182,35 @@ describe Dnsimple::Client, ".templates" do
           with(headers: { "Accept" => "application/json" })
     end
 
-    it "returns the list of templates" do
+    it "returns nil" do
       response = subject.delete_template(account_id, template_id)
       expect(response).to be_a(Dnsimple::Response)
       expect(response.data).to be_nil
     end
   end
 
+  describe "#apply_template" do
+    let(:account_id)  { 1010 }
+    let(:template_id) { 5410 }
+    let(:domain_id)   { 'example.com' }
+
+    before do
+      stub_request(:post, %r{/v2/#{account_id}/domains/#{domain_id}/templates/#{template_id}$}).
+          to_return(read_http_fixture("applyTemplate/success.http"))
+    end
+
+    it "builds the correct request" do
+      subject.apply_template(account_id, template_id, domain_id)
+
+      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/domains/#{domain_id}/templates/#{template_id}").
+          with(headers: { "Accept" => "application/json" })
+    end
+
+    it "returns nil" do
+      response = subject.apply_template(account_id, template_id, domain_id)
+      expect(response).to be_a(Dnsimple::Response)
+      expect(response.data).to be_nil
+    end
+  end
+
 end

data/spec/dnsimple/client/tlds_spec.rb

@@ -43,7 +43,7 @@ describe Dnsimple::Client, ".tlds" do
 
       response.data.each do |result|
         expect(result).to be_a(Dnsimple::Struct::Tld)
-        expect(result.tld_type).to be_a(Fixnum)
+        expect(result.tld_type).to be_a(Integer)
         expect(result.tld).to be_a(String)
       end
     end
@@ -53,9 +53,9 @@ describe Dnsimple::Client, ".tlds" do
 
       expect(response.respond_to?(:page)).to be_truthy
       expect(response.page).to eq(1)
-      expect(response.per_page).to be_a(Fixnum)
-      expect(response.total_entries).to be_a(Fixnum)
-      expect(response.total_pages).to be_a(Fixnum)
+      expect(response.per_page).to be_a(Integer)
+      expect(response.total_entries).to be_a(Integer)
+      expect(response.total_pages).to be_a(Integer)
     end
   end
 
@@ -101,6 +101,10 @@ describe Dnsimple::Client, ".tlds" do
       expect(result.whois_privacy).to eq(true)
       expect(result.auto_renew_only).to eq(false)
       expect(result.idn).to eq(true)
+      expect(result.minimum_registration).to eq(1)
+      expect(result.registration_enabled).to eq(true)
+      expect(result.renewal_enabled).to eq(true)
+      expect(result.transfer_enabled).to eq(true)
     end
   end
 

data/spec/dnsimple/client/vanity_name_servers_spec.rb

@@ -0,0 +1,54 @@
+require 'spec_helper'
+
+describe Dnsimple::Client, ".vanity_name_servers" do
+
+  subject { described_class.new(base_url: "https://api.dnsimple.test", access_token: "a1b2c3").vanity_name_servers }
+
+  describe "#enable_vanity_name_servers" do
+    let(:account_id) { 1010 }
+
+    before do
+      stub_request(:put, %r{/v2/#{account_id}/vanity/.+$}).
+          to_return(read_http_fixture("enableVanityNameServers/success.http"))
+    end
+
+    it "builds the correct request" do
+      subject.enable_vanity_name_servers(account_id, domain_name = "example.com")
+
+      expect(WebMock).to have_requested(:put, "https://api.dnsimple.test/v2/#{account_id}/vanity/#{domain_name}").
+          with(headers: { "Accept" => "application/json" })
+    end
+
+    it "returns vanity name servers of the domain" do
+      response = subject.enable_vanity_name_servers(account_id, "example.com")
+      expect(response).to be_a(Dnsimple::Response)
+
+      vanity_name_servers = response.data.map { |ns| ns["name"] }
+      expect(vanity_name_servers).to match_array(%w(ns1.example.com ns2.example.com ns3.example.com ns4.example.com))
+    end
+  end
+
+  describe "#disable_vanity_name_servers" do
+    let(:account_id) { 1010 }
+
+    before do
+      stub_request(:delete, %r{/v2/#{account_id}/vanity/.+$}).
+          to_return(read_http_fixture("disableVanityNameServers/success.http"))
+    end
+
+    it "builds the correct request" do
+      subject.disable_vanity_name_servers(account_id, domain_name = "example.com")
+
+      expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/vanity/#{domain_name}").
+          with(headers: { "Accept" => "application/json" })
+    end
+
+    it "returns empty response" do
+      response = subject.disable_vanity_name_servers(account_id, "example.com")
+      expect(response).to be_a(Dnsimple::Response)
+
+      result = response.data
+      expect(result).to be_nil
+    end
+  end
+end

data/spec/dnsimple/client/webhooks_spec.rb

@@ -40,7 +40,7 @@ describe Dnsimple::Client, ".webhooks" do
 
       response.data.each do |result|
         expect(result).to be_a(Dnsimple::Struct::Webhook)
-        expect(result.id).to be_a(Fixnum)
+        expect(result.id).to be_a(Integer)
       end
     end
   end
@@ -69,7 +69,7 @@ describe Dnsimple::Client, ".webhooks" do
 
       result = response.data
       expect(result).to be_a(Dnsimple::Struct::Webhook)
-      expect(result.id).to be_a(Fixnum)
+      expect(result.id).to be_a(Integer)
     end
   end
 

data/spec/dnsimple/client/zones_records_spec.rb

@@ -53,8 +53,9 @@ describe Dnsimple::Client, ".zones" do
       expect(response.data.size).to eq(5)
 
       response.data.each do |result|
-        expect(result).to be_a(Dnsimple::Struct::Record)
-        expect(result.id).to be_a(Fixnum)
+        expect(result).to be_a(Dnsimple::Struct::ZoneRecord)
+        expect(result.id).to be_a(Integer)
+        expect(result.regions).to be_a(Array)
       end
     end
 
@@ -63,9 +64,9 @@ describe Dnsimple::Client, ".zones" do
 
       expect(response.respond_to?(:page)).to be_truthy
       expect(response.page).to eq(1)
-      expect(response.per_page).to be_a(Fixnum)
-      expect(response.total_entries).to be_a(Fixnum)
-      expect(response.total_pages).to be_a(Fixnum)
+      expect(response.per_page).to be_a(Integer)
+      expect(response.total_entries).to be_a(Integer)
+      expect(response.total_pages).to be_a(Integer)
     end
 
     context "when the zone does not exist" do
@@ -116,7 +117,7 @@ describe Dnsimple::Client, ".zones" do
           to_return(read_http_fixture("createZoneRecord/created.http"))
     end
 
-    let(:attributes) { { type: "A", name: "www", content: "127.0.0.1" } }
+    let(:attributes) { { type: "MX", name: "", content: "mxa.example.com", regions: %w(SV1 IAD) } }
 
     it "builds the correct request" do
       subject.create_record(account_id, zone_id, attributes)
@@ -131,8 +132,12 @@ describe Dnsimple::Client, ".zones" do
       expect(response).to be_a(Dnsimple::Response)
 
       result = response.data
-      expect(result).to be_a(Dnsimple::Struct::Record)
-      expect(result.id).to eq(64784)
+      expect(result).to be_a(Dnsimple::Struct::ZoneRecord)
+      expect(result.id).to eq(5)
+      expect(result.type).to eq(attributes.fetch(:type))
+      expect(result.name).to eq(attributes.fetch(:name))
+      expect(result.content).to eq(attributes.fetch(:content))
+      expect(result.regions).to eq(attributes.fetch(:regions))
     end
 
     context "when the zone does not exist" do
@@ -150,6 +155,7 @@ describe Dnsimple::Client, ".zones" do
   describe "#record" do
     let(:account_id) { 1010 }
     let(:zone_id) { "example.com" }
+    let(:record_id) { 5 }
 
     before do
       stub_request(:get, %r{/v2/#{account_id}/zones/#{zone_id}/records/.+$}).
@@ -157,29 +163,30 @@ describe Dnsimple::Client, ".zones" do
     end
 
     it "builds the correct request" do
-      subject.record(account_id, zone_id, record_id = 2)
+      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' })
     end
 
     it "returns the record" do
-      response = subject.record(account_id, zone_id, 2)
+      response = subject.record(account_id, zone_id, record_id)
       expect(response).to be_a(Dnsimple::Response)
 
       result = response.data
-      expect(result).to be_a(Dnsimple::Struct::Record)
-      expect(result.id).to eq(64784)
+      expect(result).to be_a(Dnsimple::Struct::ZoneRecord)
+      expect(result.id).to eq(record_id)
       expect(result.zone_id).to eq("example.com")
       expect(result.parent_id).to eq(nil)
-      expect(result.type).to eq("A")
-      expect(result.name).to eq("www")
-      expect(result.content).to eq("127.0.0.1")
+      expect(result.type).to eq("MX")
+      expect(result.name).to eq("")
+      expect(result.content).to eq("mxa.example.com")
       expect(result.ttl).to eq(600)
-      expect(result.priority).to eq(nil)
+      expect(result.priority).to eq(10)
       expect(result.system_record).to eq(false)
-      expect(result.created_at).to eq("2016-01-07T17:45:13.653Z")
-      expect(result.updated_at).to eq("2016-01-07T17:45:13.653Z")
+      expect(result.regions).to eq(%w(SV1 IAD))
+      expect(result.created_at).to eq("2016-10-05T09:51:35.313Z")
+      expect(result.updated_at).to eq("2016-10-05T09:51:35.313Z")
     end
 
     context "when the zone does not exist" do
@@ -208,16 +215,17 @@ describe Dnsimple::Client, ".zones" do
   describe "#update_record" do
     let(:account_id) { 1010 }
     let(:zone_id) { "example.com" }
+    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"))
     end
 
-    let(:attributes) { { content: "127.0.0.1", priority: "1" } }
+    let(:attributes) { { content: "mxb.example.com", priority: "20", regions: ['global'] } }
 
     it "builds the correct request" do
-      subject.update_record(account_id, zone_id, record_id = 2, attributes)
+      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).
@@ -225,12 +233,15 @@ describe Dnsimple::Client, ".zones" do
     end
 
     it "returns the record" do
-      response = subject.update_record(account_id, zone_id, 2, attributes)
+      response = subject.update_record(account_id, zone_id, record_id, attributes)
       expect(response).to be_a(Dnsimple::Response)
 
       result = response.data
-      expect(result).to be_a(Dnsimple::Struct::Record)
-      expect(result.id).to eq(64784)
+      expect(result).to be_a(Dnsimple::Struct::ZoneRecord)
+      expect(result.id).to eq(record_id)
+      expect(result.content).to eq(attributes.fetch(:content))
+      expect(result.priority).to eq(attributes.fetch(:priority).to_i)
+      expect(result.regions).to eq(attributes.fetch(:regions))
     end
 
     context "when the zone does not exist" do

data/spec/dnsimple/client/zones_spec.rb

@@ -53,7 +53,7 @@ describe Dnsimple::Client, ".zones" do
 
       response.data.each do |result|
         expect(result).to be_a(Dnsimple::Struct::Zone)
-        expect(result.id).to be_a(Fixnum)
+        expect(result.id).to be_a(Integer)
       end
     end
 
@@ -62,9 +62,9 @@ describe Dnsimple::Client, ".zones" do
 
       expect(response.respond_to?(:page)).to be_truthy
       expect(response.page).to eq(1)
-      expect(response.per_page).to be_a(Fixnum)
-      expect(response.total_entries).to be_a(Fixnum)
-      expect(response.total_pages).to be_a(Fixnum)
+      expect(response.per_page).to be_a(Integer)
+      expect(response.total_entries).to be_a(Integer)
+      expect(response.total_pages).to be_a(Integer)
     end
   end
 
@@ -135,4 +135,40 @@ describe Dnsimple::Client, ".zones" do
     end
   end
 
+  describe "#zone_file" do
+    let(:account_id) { 1010 }
+
+    before do
+      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' })
+    end
+
+    it "returns the zone id" do
+      response = subject.zone_file(account_id, "example.com")
+      expect(response).to be_a(Dnsimple::Response)
+
+      result = response.data
+      expect(result).to be_a(Dnsimple::Struct::ZoneFile)
+      expect(result.zone).to eq("$ORIGIN example.com.\n$TTL 1h\nexample.com. 3600 IN SOA ns1.dnsimple.com. admin.dnsimple.com. 1453132552 86400 7200 604800 300\nexample.com. 3600 IN NS ns1.dnsimple.com.\nexample.com. 3600 IN NS ns2.dnsimple.com.\nexample.com. 3600 IN NS ns3.dnsimple.com.\nexample.com. 3600 IN NS ns4.dnsimple.com.\n")
+    end
+
+    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"))
+
+        expect {
+          subject.zone_file(account_id, "example.com")
+        }.to raise_error(Dnsimple::NotFoundError)
+      end
+    end
+  end
+
 end

data/spec/fixtures.http/acceptPush/success.http

@@ -0,0 +1,17 @@
+HTTP/1.1 204 No Content
+Server: nginx
+Date: Thu, 11 Aug 2016 10:23:46 GMT
+Connection: keep-alive
+X-RateLimit-Limit: 2400
+X-RateLimit-Remaining: 2392
+X-RateLimit-Reset: 1470913058
+Cache-Control: no-cache
+X-Request-Id: 5867ab50-6237-4cf8-b220-51f5ef57debd
+X-Runtime: 0.582482
+X-Content-Type-Options: nosniff
+X-Download-Options: noopen
+X-Frame-Options: DENY
+X-Permitted-Cross-Domain-Policies: none
+X-XSS-Protection: 1; mode=block
+Strict-Transport-Security: max-age=31536000
+

data/spec/fixtures.http/addCollaborator/invite-success.http

@@ -0,0 +1,21 @@
+HTTP/1.1 201 Created
+Server: nginx
+Date: Fri, 07 Oct 2016 08:51:12 GMT
+Content-Type: application/json; charset=utf-8
+Transfer-Encoding: chunked
+Connection: keep-alive
+X-RateLimit-Limit: 2400
+X-RateLimit-Remaining: 2398
+X-RateLimit-Reset: 1475833772
+ETag: W/"3e59e2eafbf1d4b626413a5703309b57"
+Cache-Control: max-age=0, private, must-revalidate
+X-Request-Id: 9f27e164-2448-4d23-ad67-4ecedaec063e
+X-Runtime: 0.387144
+X-Content-Type-Options: nosniff
+X-Download-Options: noopen
+X-Frame-Options: DENY
+X-Permitted-Cross-Domain-Policies: none
+X-XSS-Protection: 1; mode=block
+Strict-Transport-Security: max-age=31536000
+
+{"data":{"id":101,"domain_id":1,"domain_name":"example.com","user_id":null,"user_email":"invited-user@example.com","invitation":true,"created_at":"2016-10-07T08:51:12.630Z","updated_at":"2016-10-07T08:51:12.630Z","accepted_at":null}}

data/spec/fixtures.http/addCollaborator/success.http

@@ -0,0 +1,21 @@
+HTTP/1.1 201 Created
+Server: nginx
+Date: Fri, 07 Oct 2016 08:53:41 GMT
+Content-Type: application/json; charset=utf-8
+Transfer-Encoding: chunked
+Connection: keep-alive
+X-RateLimit-Limit: 2400
+X-RateLimit-Remaining: 2397
+X-RateLimit-Reset: 1475833772
+ETag: W/"d35fa3da40b4f6586503174563d71bfc"
+Cache-Control: max-age=0, private, must-revalidate
+X-Request-Id: 42de3144-37c7-4329-88c4-fdc103cb150f
+X-Runtime: 0.156436
+X-Content-Type-Options: nosniff
+X-Download-Options: noopen
+X-Frame-Options: DENY
+X-Permitted-Cross-Domain-Policies: none
+X-XSS-Protection: 1; mode=block
+Strict-Transport-Security: max-age=31536000
+
+{"data":{"id":100,"domain_id":1,"domain_name":"example.com","user_id":999,"user_email":"existing-user@example.com","invitation":false,"created_at":"2016-10-07T08:53:41.643Z","updated_at":"2016-10-07T08:53:41.643Z","accepted_at":"2016-10-07T08:53:41.643Z"}}