dnsimple 10.0.0 → 11.1.0

data/spec/dnsimple/client/identity_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Dnsimple::Client, ".identity" do
 
@@ -17,7 +17,7 @@ describe Dnsimple::Client, ".identity" do
       subject.whoami
 
       expect(WebMock).to have_requested(:get, "https://api.dnsimple.test/v2/whoami")
-          .with(headers: { 'Accept' => 'application/json' })
+          .with(headers: { "Accept" => "application/json" })
     end
 
     it "returns the whoami" do

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
@@ -316,4 +316,99 @@ describe Dnsimple::Client, ".zones" do
     end
   end
 
+  describe "#batch_change_zone_records" do
+    let(:account_id) { 1010 }
+    let(:attributes) { { creates: [{ type: "A", content: "3.2.3.4", name: "ab" }, { type: "A", content: "4.2.3.4", name: "ab" }], updates: [{ id: 67622534, content: "3.2.3.40", name: "update1-1757049890" }, { id: 67622537, content: "5.2.3.40", name: "update2-1757049890" }], deletes: [{ id: 67622509 }, { id: 67622527 }] } }
+    let(:zone_id) { "example.com" }
+
+    before do
+      stub_request(:post, %r{/v2/#{account_id}/zones/#{zone_id}/batch$})
+          .to_return(read_http_fixture("batchChangeZoneRecords/success.http"))
+    end
+
+
+    it "builds the correct request" do
+      subject.batch_change_zone_records(account_id, zone_id, attributes)
+
+      expect(WebMock).to have_requested(:post, "https://api.dnsimple.test/v2/#{account_id}/zones/#{zone_id}/batch")
+          .with(body: attributes)
+          .with(headers: { "Accept" => "application/json" })
+    end
+
+    it "returns the result" do
+      response = subject.batch_change_zone_records(account_id, zone_id, attributes)
+      expect(response).to be_a(Dnsimple::Response)
+
+      result = response.data
+      expect(result).to be_a(Dnsimple::Struct::ZoneRecordsBatchChange)
+      expect(result.creates[0].id).to eq(67623409)
+      expect(result.creates[0].type).to eq(attributes.fetch(:creates)[0].fetch(:type))
+      expect(result.creates[0].name).to eq(attributes.fetch(:creates)[0].fetch(:name))
+      expect(result.creates[0].content).to eq(attributes.fetch(:creates)[0].fetch(:content))
+      expect(result.creates[0].regions).to eq(["global"])
+      expect(result.creates[1].id).to eq(67623410)
+      expect(result.updates[0].id).to eq(67622534)
+      expect(result.updates[0].type).to eq("A")
+      expect(result.updates[0].name).to eq(attributes.fetch(:updates)[0].fetch(:name))
+      expect(result.updates[0].content).to eq(attributes.fetch(:updates)[0].fetch(:content))
+      expect(result.updates[1].id).to eq(67622537)
+      expect(result.deletes[0].id).to eq(67622509)
+      expect(result.deletes[1].id).to eq(67622527)
+    end
+
+    context "when there are errors with creation" do
+      it "raises RequestError" do
+        stub_request(:post, %r{/v2/#{account_id}/zones/#{zone_id}/batch$})
+            .to_return(read_http_fixture("batchChangeZoneRecords/error_400_create_validation_failed.http"))
+
+        expect {
+          subject.batch_change_zone_records(account_id, zone_id, attributes)
+        }.to raise_error(Dnsimple::RequestError, "Validation failed") do |exception|
+          expect(exception.attribute_errors["creates"][0]["message"]).to eq("Validation failed")
+          expect(exception.attribute_errors["creates"][0]["index"]).to eq(0)
+          expect(exception.attribute_errors["creates"][0]["errors"]).to eq({ "record_type" => ["unsupported"] })
+        end
+      end
+    end
+
+    context "when there are errors with updates" do
+      it "raises RequestError" do
+        stub_request(:post, %r{/v2/#{account_id}/zones/#{zone_id}/batch$})
+            .to_return(read_http_fixture("batchChangeZoneRecords/error_400_update_validation_failed.http"))
+
+        expect {
+          subject.batch_change_zone_records(account_id, zone_id, attributes)
+        }.to raise_error(Dnsimple::RequestError, "Validation failed") do |exception|
+          expect(exception.attribute_errors["updates"][0]["message"]).to eq("Record not found ID=99999999")
+          expect(exception.attribute_errors["updates"][0]["index"]).to eq(0)
+        end
+      end
+    end
+
+    context "when there are errors with deletes" do
+      it "raises RequestError" do
+        stub_request(:post, %r{/v2/#{account_id}/zones/#{zone_id}/batch$})
+            .to_return(read_http_fixture("batchChangeZoneRecords/error_400_delete_validation_failed.http"))
+
+        expect {
+          subject.batch_change_zone_records(account_id, zone_id, attributes)
+        }.to raise_error(Dnsimple::RequestError, "Validation failed") do |exception|
+          expect(exception.attribute_errors["deletes"][0]["message"]).to eq("Record not found ID=67622509")
+          expect(exception.attribute_errors["deletes"][0]["index"]).to eq(0)
+        end
+      end
+    end
+
+    context "when the zone does not exist" do
+      it "raises NotFoundError" do
+        stub_request(:post, %r{/v2})
+            .to_return(read_http_fixture("notfound-zone.http"))
+
+        expect {
+          subject.batch_change_zone_records(account_id, zone_id, attributes)
+        }.to raise_error(Dnsimple::NotFoundError)
+      end
+    end
+  end
+
 end

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