dnsimple 9.0.1 → 11.0.0

data/spec/dnsimple/client/oauth_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Client, ".oauth" do
 
@@ -23,7 +23,7 @@ describe Dnsimple::Client, ".oauth" do
 
       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' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns oauth token" do
@@ -43,7 +43,7 @@ describe Dnsimple::Client, ".oauth" do
 
         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' })
+            .with(headers: { "Accept" => "application/json" })
       end
     end
 

data/spec/dnsimple/client/registrar_auto_renewal_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Client, ".registrar" do
 
@@ -20,7 +20,7 @@ describe Dnsimple::Client, ".registrar" 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' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns nothing" do
@@ -57,7 +57,7 @@ describe Dnsimple::Client, ".registrar" 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' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns nothing" do

data/spec/dnsimple/client/registrar_delegation_spec.rb

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

data/spec/dnsimple/client/registrar_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Client, ".registrar" do
   subject { described_class.new(base_url: "https://api.dnsimple.test", access_token: "a1b2c3").registrar }

data/spec/dnsimple/client/registrar_transfer_lock_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Client, ".registrar" do
   subject { described_class.new(base_url: "https://api.dnsimple.test", access_token: "a1b2c3").registrar }
@@ -18,7 +18,7 @@ describe Dnsimple::Client, ".registrar" do
       subject.get_domain_transfer_lock(account_id, domain_id)
 
       expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_id}/transfer_lock")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the transfer lock state" do
@@ -55,7 +55,7 @@ describe Dnsimple::Client, ".registrar" do
       subject.enable_domain_transfer_lock(account_id, domain_id)
 
       expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_id}/transfer_lock")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the transfer lock state" do
@@ -92,7 +92,7 @@ describe Dnsimple::Client, ".registrar" do
       subject.disable_domain_transfer_lock(account_id, domain_id)
 
       expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/registrar/domains/#{domain_id}/transfer_lock")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the transfer lock state" do

data/spec/dnsimple/client/registrar_whois_privacy_spec.rb

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

data/spec/dnsimple/client/services_domains_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Client, ".services" do
 
@@ -9,7 +9,7 @@ describe Dnsimple::Client, ".services" do
 
   describe "#applied_services" do
     let(:account_id) { 1010 }
-    let(:domain_id)  { 'example.com' }
+    let(:domain_id)  { "example.com" }
 
     before do
       stub_request(:get, %r{/v2/#{account_id}/domains/#{domain_id}/services})
@@ -74,7 +74,7 @@ describe Dnsimple::Client, ".services" do
 
       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' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns empty response" do
@@ -100,7 +100,7 @@ describe Dnsimple::Client, ".services" 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' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns empty response" do

data/spec/dnsimple/client/services_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Client, ".services" do
 

data/spec/dnsimple/client/templates_domains_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Client, ".templates" do
 
@@ -10,7 +10,7 @@ describe Dnsimple::Client, ".templates" do
   describe "#apply_template" do
     let(:account_id)  { 1010 }
     let(:template_id) { 5410 }
-    let(:domain_id)   { 'example.com' }
+    let(:domain_id)   { "example.com" }
 
     before do
       stub_request(:post, %r{/v2/#{account_id}/domains/#{domain_id}/templates/#{template_id}$})

data/spec/dnsimple/client/templates_records_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Client, ".templates" do
 
@@ -92,7 +92,7 @@ describe Dnsimple::Client, ".templates" do
 
       expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/templates/#{template_id}/records")
           .with(body: attributes)
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the record" do
@@ -157,7 +157,7 @@ describe Dnsimple::Client, ".templates" do
       subject.record(account_id, template_id, record_id = 301)
 
       expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/templates/#{template_id}/records/#{record_id}")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the record" do
@@ -191,7 +191,7 @@ describe Dnsimple::Client, ".templates" do
       subject.delete_record(account_id, template_id, record_id = 301)
 
       expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/templates/#{template_id}/records/#{record_id}")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns nothing" do

data/spec/dnsimple/client/templates_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Client, ".templates" do
 
@@ -194,7 +194,7 @@ describe Dnsimple::Client, ".templates" do
   describe "#apply_template" do
     let(:account_id)  { 1010 }
     let(:template_id) { 5410 }
-    let(:domain_id)   { 'example.com' }
+    let(:domain_id)   { "example.com" }
 
     before do
       stub_request(:post, %r{/v2/#{account_id}/domains/#{domain_id}/templates/#{template_id}$})

data/spec/dnsimple/client/tlds_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Client, ".tlds" do
   subject { described_class.new(base_url: "https://api.dnsimple.test", access_token: "a1b2c3").tlds }
@@ -15,7 +15,7 @@ describe Dnsimple::Client, ".tlds" do
       subject.list_tlds
 
       expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/tlds")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "supports pagination" do
@@ -89,7 +89,7 @@ describe Dnsimple::Client, ".tlds" do
       subject.tld(tld = "com")
 
       expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/tlds/#{tld}")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the tld" do
@@ -98,7 +98,7 @@ describe Dnsimple::Client, ".tlds" do
 
       result = response.data
       expect(result).to be_a(Dnsimple::Struct::Tld)
-      expect(result.tld).to eq('com')
+      expect(result.tld).to eq("com")
       expect(result.tld_type).to eq(1)
       expect(result.whois_privacy).to be(true)
       expect(result.auto_renew_only).to be(false)
@@ -121,7 +121,7 @@ describe Dnsimple::Client, ".tlds" do
       subject.tld_extended_attributes(tld = "uk")
 
       expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/tlds/#{tld}/extended_attributes")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the extended attributes" do

data/spec/dnsimple/client/vanity_name_servers_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Client, ".vanity_name_servers" do
 

data/spec/dnsimple/client/webhooks_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Client, ".webhooks" do
 
@@ -18,7 +18,7 @@ describe Dnsimple::Client, ".webhooks" do
       subject.webhooks(account_id)
 
       expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/webhooks")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "supports extra request options" do
@@ -62,7 +62,7 @@ describe Dnsimple::Client, ".webhooks" do
 
       expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/webhooks")
           .with(body: attributes)
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the webhook" do
@@ -87,7 +87,7 @@ describe Dnsimple::Client, ".webhooks" do
       subject.webhook(account_id, webhook_id = "1")
 
       expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/webhooks/#{webhook_id}")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the webhook" do
@@ -124,7 +124,7 @@ describe Dnsimple::Client, ".webhooks" do
       subject.delete_webhook(account_id, webhook_id = "1")
 
       expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/webhooks/#{webhook_id}")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns nothing" do

data/spec/dnsimple/client/zones_distributions_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Client, ".zones" do
 
@@ -18,7 +18,7 @@ describe Dnsimple::Client, ".zones" do
       subject.zone_distribution(account_id, zone = "example.com")
 
       expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone}/distribution")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the zone distribution check with true when the zone is fully distributed" do
@@ -78,7 +78,7 @@ describe Dnsimple::Client, ".zones" do
       subject.zone_record_distribution(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}/distribution")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the zone record distribution check with true when the zone is fully distributed" do

data/spec/dnsimple/client/zones_records_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Client, ".zones" do
 
@@ -20,7 +20,7 @@ describe Dnsimple::Client, ".zones" do
       subject.list_zone_records(account_id, zone_id)
 
       expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone_id}/records")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "supports pagination" do
@@ -126,7 +126,7 @@ describe Dnsimple::Client, ".zones" do
 
       expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone_id}/records")
           .with(body: attributes)
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the record" do
@@ -168,7 +168,7 @@ describe Dnsimple::Client, ".zones" do
       subject.zone_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' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the record" do
@@ -216,7 +216,7 @@ describe Dnsimple::Client, ".zones" do
 
   describe "#update_zone_record" do
     let(:account_id) { 1010 }
-    let(:attributes) { { content: "mxb.example.com", priority: "20", regions: ['global'] } }
+    let(:attributes) { { content: "mxb.example.com", priority: "20", regions: ["global"] } }
     let(:zone_id) { "example.com" }
     let(:record_id) { 5 }
 
@@ -231,7 +231,7 @@ describe Dnsimple::Client, ".zones" do
 
       expect(WebMock).to have_requested(:patch, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone_id}/records/#{record_id}")
           .with(body: attributes)
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the record" do
@@ -282,7 +282,7 @@ describe Dnsimple::Client, ".zones" do
       subject.delete_zone_record(account_id, zone_id, record_id = 2)
 
       expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone_id}/records/#{record_id}")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns nothing" do

data/spec/dnsimple/client/zones_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Client, ".zones" do
   subject { described_class.new(base_url: "https://api.dnsimple.test", access_token: "a1b2c3").zones }
@@ -18,7 +18,7 @@ describe Dnsimple::Client, ".zones" do
       subject.zones(account_id)
 
       expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/zones")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "supports pagination" do
@@ -107,7 +107,7 @@ describe Dnsimple::Client, ".zones" do
       subject.zone(account_id, zone = "example.com")
 
       expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone}")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the zone" do
@@ -151,7 +151,7 @@ describe Dnsimple::Client, ".zones" 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' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the zone id" do
@@ -187,7 +187,7 @@ describe Dnsimple::Client, ".zones" do
       subject.activate_dns(account_id, zone = "example.com")
 
       expect(WebMock).to have_requested(:put, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone}/activation")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the zone" do
@@ -229,7 +229,7 @@ describe Dnsimple::Client, ".zones" do
       subject.deactivate_dns(account_id, zone = "example.com")
 
       expect(WebMock).to have_requested(:delete, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone}/activation")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the zone" do

data/spec/dnsimple/client_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Client do
 
@@ -135,11 +135,11 @@ describe Dnsimple::Client do
     it "performs a request" do
       stub_request(:get, %r{/foo})
 
-      subject.request(:get, 'foo', {})
+      subject.request(:get, "foo", {})
 
       expect(WebMock).to have_requested(:get, "https://api.dnsimple.com/foo")
           .with(basic_auth: %w[user pass],
-                headers: { 'Accept' => 'application/json', 'User-Agent' => Dnsimple::Default::USER_AGENT })
+                headers: { "Accept" => "application/json", "User-Agent" => Dnsimple::Default::USER_AGENT })
     end
 
     it "delegates to HTTParty" do
@@ -150,11 +150,11 @@ describe Dnsimple::Client do
               "#{subject.base_url}foo", {
                 format: :json,
                 basic_auth: { username: "user", password: "pass" },
-                headers: { 'Accept' => 'application/json', 'User-Agent' => Dnsimple::Default::USER_AGENT },
+                headers: { "Accept" => "application/json", "User-Agent" => Dnsimple::Default::USER_AGENT },
               }
             )
 
-      subject.request(:get, 'foo')
+      subject.request(:get, "foo")
     end
 
     it "properly extracts processes options and encodes data" do
@@ -165,12 +165,12 @@ describe Dnsimple::Client do
                 body: JSON.dump(something: "else"),
                 query: { foo: "bar" },
                 basic_auth: { username: "user", password: "pass" },
-                headers: { 'Accept' => 'application/json', 'Content-Type' => 'application/json', 'User-Agent' => Dnsimple::Default::USER_AGENT, "Custom" => "Header" },
+                headers: { "Accept" => "application/json", "Content-Type" => "application/json", "User-Agent" => Dnsimple::Default::USER_AGENT, "Custom" => "Header" },
               }
             )
 
       expect {
-        subject.request(:put, 'foo', { something: "else" }, { query: { foo: "bar" }, headers: { "Custom" => "Header" } })
+        subject.request(:put, "foo", { something: "else" }, { query: { foo: "bar" }, headers: { "Custom" => "Header" } })
       }.not_to raise_error
     end
 
@@ -181,12 +181,12 @@ describe Dnsimple::Client do
                 format: :json,
                 body: { something: "else" },
                 basic_auth: { username: "user", password: "pass" },
-                headers: { 'Accept' => 'application/json', 'Content-Type' => 'application/x-www-form-urlencoded', 'User-Agent' => Dnsimple::Default::USER_AGENT },
+                headers: { "Accept" => "application/json", "Content-Type" => "application/x-www-form-urlencoded", "User-Agent" => Dnsimple::Default::USER_AGENT },
               }
             )
 
       expect {
-        subject.request(:post, 'foo', { something: "else" }, { headers: { "Content-Type" => "application/x-www-form-urlencoded" } })
+        subject.request(:post, "foo", { something: "else" }, { headers: { "Content-Type" => "application/x-www-form-urlencoded" } })
       }.not_to raise_error
     end
 
@@ -197,7 +197,7 @@ describe Dnsimple::Client do
                 format: :json,
                 http_proxyaddr: "example-proxy.com",
                 http_proxyport: "4321",
-                headers: { 'Accept' => 'application/json', 'User-Agent' => Dnsimple::Default::USER_AGENT },
+                headers: { "Accept" => "application/json", "User-Agent" => Dnsimple::Default::USER_AGENT },
               }
             )
 

data/spec/dnsimple/extra_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Extra do
 

data/spec/dnsimple/options/base_spec.rb

@@ -1,19 +1,19 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 RSpec.describe Dnsimple::Options::Base do
-  describe '#initialize' do
-    it 'accepts a hash' do
+  describe "#initialize" do
+    it "accepts a hash" do
       hash = { a: 1 }
       expect(described_class.new(hash).to_h).to eq(hash)
     end
 
-    it 'accepts nil' do
+    it "accepts nil" do
       expect(described_class.new(nil).to_h).to eq({})
     end
 
-    it 'duplicates given hash' do
+    it "duplicates given hash" do
       hash = { a: [1] }
       base = described_class.new(hash)
       base.to_h[:a] << 2