dnsimple 6.0.0 → 7.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a4f667eba7dc3bc3e7718657059de2cc752f966d748b0698326feefad1f93111
-  data.tar.gz: 82eabdcd7a27df808db076d4d65bfb196343c3aa607e42d61ba1ff8f7b328349
+  metadata.gz: 2cf8d057d98c78bf3b8584e15b561624dfb35f1264f80d0c5861e970053b18f3
+  data.tar.gz: acc7e6f33680ba4195243b474dca108e566c953ed2474e36fd9631c9f6b1f2fc
 SHA512:
-  metadata.gz: 8ee87e28d8fd2fef38a9282905146831321e9a443c08216e2399fc577ba957da835df45d8a31052a4c8a12c7db2373ad8b96ed0b7a86072de20b6c2772b3e56c
-  data.tar.gz: dae840d8c7d5f3c86af8f959ba07b5310f6e852e351b84c42166621e181de41604fb1d43be6ffe1c651002840f37dcc3b16eebadf6c9beed90d1c46e9f693aa5
+  metadata.gz: 504d15c395fcbbb08a4c351912f9cb47e0dac082ae9da5ef22161bc5bc6a5fc6e1f46466ab7c482385c996a558a10dad4d12fc2cd1eac130c417f6c21e70cfd0
+  data.tar.gz: 3ebc372cd78a2bc5d4cf3bdcab9168e572856fe0614da843949c5a58a1508399281b25a621e399280b1e4464a72ea5099d9e88e227aec7a564213297d81a6720

data/.github/dependabot.yml

@@ -0,0 +1,12 @@
+version: 2
+updates:
+  - package-ecosystem: bundler
+    directory: /
+    schedule:
+      interval: daily
+      time: '12:00'
+    open-pull-requests-limit: 10
+    labels:
+      - task
+      - dependencies
+      - backlog-dependencies

data/.rubocop.yml

@@ -8,21 +8,12 @@ require:
   - rubocop-rspec
 
 AllCops:
+  TargetRubyVersion: 2.6
   Exclude:
     - '*.gemspec'
     - 'Rakefile'
+    - 'vendor/**/*'
 
-# [codesmell]
-# It's irrelevant here, but it could be a code smell.
-# Hence keep it disabled, but don't include it in the DNSimple suite.
-Lint/Loop:
-  Enabled: false
-
-# It doesn't seem to work as expected.
-Style/IfUnlessModifier:
-  Enabled: false
-
-# This rule suggest to use safe navigation (&.) operator which was introduced
-# with Ruby 2.3. Because we are supporting Ruby 2.0+ we can't use this operator.
-Style/SafeNavigation:
+# This rule conflicts with our specs codebase using Ruby 2.x and 3.x
+Lint/NonDeterministicRequireOrder:
   Enabled: false

data/.rubocop_todo.yml

@@ -1,49 +1,36 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2021-04-05 15:23:33 UTC using RuboCop version 1.12.1.
+# on 2021-06-01 13:07:05 UTC using RuboCop version 1.16.0.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
 # versions of RuboCop, may require this file to be generated again.
 
-# Offense count: 6
-# Configuration parameters: Prefixes.
-# Prefixes: when, with, without
-RSpec/ContextWording:
-  Exclude:
-    - 'spec/dnsimple/client/domains_collaborators_spec.rb'
-    - 'spec/dnsimple/options/list_options_spec.rb'
-
-# Offense count: 70
-# Configuration parameters: Max.
+# Offense count: 71
+# Configuration parameters: CountAsOne.
 RSpec/ExampleLength:
-  Enabled: false
+  Max: 17
 
 # Offense count: 1
 RSpec/LeakyConstantDeclaration:
   Exclude:
     - 'spec/dnsimple/client/client_service_spec.rb'
 
-# Offense count: 21
+# Offense count: 11
 # Configuration parameters: .
 # SupportedStyles: have_received, receive
 RSpec/MessageSpies:
   EnforcedStyle: receive
 
-# Offense count: 117
+# Offense count: 112
 RSpec/MultipleExpectations:
-  Max: 16
+  Max: 15
 
-# Offense count: 374
+# Offense count: 375
 # Configuration parameters: IgnoreSharedExamples.
 RSpec/NamedSubject:
   Enabled: false
 
-# Offense count: 10
-RSpec/StubbedMock:
-  Exclude:
-    - 'spec/dnsimple/client_spec.rb'
-
 # Offense count: 16
 RSpec/SubjectStub:
   Exclude:
@@ -60,13 +47,7 @@ RSpec/SubjectStub:
     - 'spec/dnsimple/client/zones_spec.rb'
     - 'spec/dnsimple/client_spec.rb'
 
-# Offense count: 5
-# Configuration parameters: IgnoreNameless, IgnoreSymbolicNames.
-RSpec/VerifiedDoubles:
-  Exclude:
-    - 'spec/dnsimple/client_spec.rb'
-
-# Offense count: 39
+# Offense count: 37
 # Configuration parameters: AllowedConstants.
 Style/Documentation:
   Enabled: false

data/.travis.yml

@@ -1,16 +1,20 @@
+dist: focal
+
 language: ruby
 
 rvm:
-  - 2.4
-  - 2.5
   - 2.6
   - 2.7
   - 3.0
   - ruby-head
+  - truffleruby-head
+
+cache: bundler
 
-matrix:
+jobs:
   allow_failures:
     - rvm: ruby-head
+    - rvm: truffleruby-head
 
 env:
   - COVERALL=1

data/CHANGELOG.md

@@ -4,6 +4,11 @@ This project uses [Semantic Versioning 2.0.0](http://semver.org/).
 
 ## main
 
+## 0.7.0
+
+- CHANGED: Minimum Ruby version is now 2.6
+- CHANGED: Deprecates `registrar.getDomainPremiumPrice` in favour of `registrar.getDomainPrices`
+
 ## 6.0.0
 
 - NEW: Added `registrar.get_domain_prices` to retrieve whether a domain is premium and the prices to register, transfer, and renew. (dnsimple/dnsimple-ruby#230)

data/Gemfile

@@ -5,7 +5,7 @@ source 'https://rubygems.org'
 gemspec
 
 gem 'coveralls', require: false
-gem 'rubocop', '1.12.1', require: false
+gem 'rubocop', '1.16.0', require: false
 gem 'rubocop-performance', require: false
 gem 'rubocop-rake', require: false
 gem 'rubocop-rspec', require: false

data/dnsimple.gemspec

@@ -11,7 +11,7 @@ Gem::Specification.new do |s|
   s.summary     = 'The DNSimple API client for Ruby'
   s.description = 'The DNSimple API client for Ruby.'
 
-  s.required_ruby_version = ">= 2.4"
+  s.required_ruby_version = ">= 2.6"
 
   s.require_paths    = ['lib']
   s.files            = `git ls-files`.split("\n")

data/lib/dnsimple/client/clients.rb

@@ -98,14 +98,15 @@ module Dnsimple
         options = args.pop
         response = nil
 
-        begin
+        loop do
           current_page += 1
           query = Extra.deep_merge(options, query: { page: current_page, per_page: 100 })
 
           response = send(method, *(args + [query]))
           total_pages ||= response.total_pages
           collection.concat(response.data)
-        end while current_page < total_pages
+          break unless current_page < total_pages
+        end
 
         CollectionResponse.new(response.http_response, collection)
       end

data/lib/dnsimple/client/registrar.rb

@@ -42,7 +42,10 @@ module Dnsimple
       # @return [Struct::DomainPremiumPrice]
       #
       # @raise  [RequestError] When the request fails.
+      #
+      # @deprecated Use {#get_domain_prices} instead of this one as it will soon be removed from the API.
       def domain_premium_price(account_id, domain_name, options = {})
+        Dnsimple.deprecate("Use {#get_domain_prices} instead of this one as it will soon be removed from the API.")
         endpoint = Client.versioned("/%s/registrar/domains/%s/premium_price" % [account_id, domain_name])
         options[:query] = { action: options.delete(:action) } if options.key?(:action)
         response = client.get(endpoint, options)

data/lib/dnsimple/error.rb

@@ -18,7 +18,7 @@ module Dnsimple
 
     def message_from(http_response)
       content_type = http_response.headers["Content-Type"]
-      if content_type && content_type.start_with?("application/json")
+      if content_type&.start_with?("application/json")
         http_response.parsed_response["message"]
       else
         net_http_response = http_response.response

data/lib/dnsimple/extra.rb

@@ -55,7 +55,7 @@ module Dnsimple
     # @raise  [ArgumentError]
     def self.validate_mandatory_attributes(attributes, required)
       required.each do |name|
-        attributes && attributes.key?(name) or raise(ArgumentError, ":#{name} is required")
+        attributes&.key?(name) or raise(ArgumentError, ":#{name} is required")
       end
     end
 

data/lib/dnsimple/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Dnsimple
-  VERSION = "6.0.0"
+  VERSION = "7.0.0"
 end

data/spec/dnsimple/client/domains_collaborators_spec.rb

@@ -63,7 +63,7 @@ describe Dnsimple::Client, ".domains" do
     let(:account_id) { 1010 }
     let(:domain_id)  { "example.com" }
 
-    context "invite user already registered on DNSimple" do
+    context "when inviting a user already registered on DNSimple" do
       before do
         stub_request(:post, %r{/v2/#{account_id}/domains/#{domain_id}/collaborators$})
             .to_return(read_http_fixture("addCollaborator/success.http"))
@@ -93,7 +93,7 @@ describe Dnsimple::Client, ".domains" do
       end
     end
 
-    context "invite not registered on DNSimple" do
+    context "when inviting a user not registered on DNSimple" do
       before do
         stub_request(:post, %r{/v2/#{account_id}/domains/#{domain_id}/collaborators$})
             .to_return(read_http_fixture("addCollaborator/invite-success.http"))

data/spec/dnsimple/client_spec.rb

@@ -49,35 +49,35 @@ describe Dnsimple::Client do
 
   describe "#get" do
     it "delegates to #request" do
-      expect(subject).to receive(:execute).with(:get, "path", nil, foo: "bar").and_return(:returned)
+      allow(subject).to receive(:execute).with(:get, "path", nil, foo: "bar").and_return(:returned)
       expect(subject.get("path", foo: "bar")).to eq(:returned)
     end
   end
 
   describe "#post" do
     it "delegates to #request" do
-      expect(subject).to receive(:execute).with(:post, "path", { foo: "bar" }, {}).and_return(:returned)
+      allow(subject).to receive(:execute).with(:post, "path", { foo: "bar" }, {}).and_return(:returned)
       expect(subject.post("path", foo: "bar")).to eq(:returned)
     end
   end
 
   describe "#put" do
     it "delegates to #request" do
-      expect(subject).to receive(:execute).with(:put, "path", { foo: "bar" }, {}).and_return(:returned)
+      allow(subject).to receive(:execute).with(:put, "path", { foo: "bar" }, {}).and_return(:returned)
       expect(subject.put("path", foo: "bar")).to eq(:returned)
     end
   end
 
   describe "#patch" do
     it "delegates to #request" do
-      expect(subject).to receive(:execute).with(:patch, "path", { foo: "bar" }, {}).and_return(:returned)
+      allow(subject).to receive(:execute).with(:patch, "path", { foo: "bar" }, {}).and_return(:returned)
       expect(subject.patch("path", foo: "bar")).to eq(:returned)
     end
   end
 
   describe "#delete" do
     it "delegates to #request" do
-      expect(subject).to receive(:execute).with(:delete, "path", { foo: "bar" }, {}).and_return(:returned)
+      allow(subject).to receive(:execute).with(:delete, "path", { foo: "bar" }, {}).and_return(:returned)
       expect(subject.delete("path", foo: "bar")).to eq(:returned)
     end
   end
@@ -126,20 +126,20 @@ describe Dnsimple::Client do
     it "delegates to HTTParty" do
       stub_request(:get, %r{/foo})
 
-      expect(HTTParty).to receive(:get)
+      allow(HTTParty).to receive(:get)
           .with(
               "#{subject.base_url}foo",
               format: :json,
               basic_auth: { username: "user", password: "pass" },
               headers: { 'Accept' => 'application/json', 'User-Agent' => Dnsimple::Default::USER_AGENT }
             )
-          .and_return(double('response', code: 200))
+          .and_return(instance_double('response', code: 200))
 
       subject.request(:get, 'foo')
     end
 
     it "properly extracts processes options and encodes data" do
-      expect(HTTParty).to receive(:put)
+      allow(HTTParty).to receive(:put)
           .with(
               "#{subject.base_url}foo",
               format: :json,
@@ -148,13 +148,13 @@ describe Dnsimple::Client do
               basic_auth: { username: "user", password: "pass" },
               headers: { 'Accept' => 'application/json', 'Content-Type' => 'application/json', 'User-Agent' => Dnsimple::Default::USER_AGENT, "Custom" => "Header" }
             )
-          .and_return(double('response', code: 200))
+          .and_return(instance_double('response', code: 200))
 
       subject.request(:put, 'foo', { something: "else" }, { query: { foo: "bar" }, headers: { "Custom" => "Header" } })
     end
 
     it "handles non application/json content types" do
-      expect(HTTParty).to receive(:post)
+      allow(HTTParty).to receive(:post)
           .with(
               "#{subject.base_url}foo",
               format: :json,
@@ -162,13 +162,13 @@ describe Dnsimple::Client do
               basic_auth: { username: "user", password: "pass" },
               headers: { 'Accept' => 'application/json', 'Content-Type' => 'application/x-www-form-urlencoded', 'User-Agent' => Dnsimple::Default::USER_AGENT }
             )
-          .and_return(double('response', code: 200))
+          .and_return(instance_double('response', code: 200))
 
       subject.request(:post, 'foo', { something: "else" }, { headers: { "Content-Type" => "application/x-www-form-urlencoded" } })
     end
 
     it "includes options for proxy support" do
-      expect(HTTParty).to receive(:get)
+      allow(HTTParty).to receive(:get)
           .with(
               "#{subject.base_url}test",
               format: :json,
@@ -176,20 +176,20 @@ describe Dnsimple::Client do
               http_proxyport: "4321",
               headers: { 'Accept' => 'application/json', 'User-Agent' => Dnsimple::Default::USER_AGENT }
             )
-          .and_return(double('response', code: 200))
+          .and_return(instance_double('response', code: 200))
 
       subject = described_class.new(proxy: "example-proxy.com:4321")
       subject.request(:get, "test", nil, {})
     end
 
     it "supports custom user agent" do
-      expect(HTTParty).to receive(:get)
+      allow(HTTParty).to receive(:get)
           .with(
               "#{subject.base_url}test",
               format: :json,
               headers: hash_including("User-Agent" => "customAgent #{Dnsimple::Default::USER_AGENT}")
             )
-          .and_return(double("response", code: 200))
+          .and_return(instance_double("response", code: 200))
 
       subject = described_class.new(user_agent: "customAgent")
       subject.request(:get, "test", nil)

data/spec/dnsimple/options/list_options_spec.rb

@@ -14,14 +14,14 @@ RSpec.describe Dnsimple::Options::ListOptions do
       expect(options.to_h).to eq({})
     end
 
-    context 'query' do
+    describe 'query' do
       it 'adds "query" key if given options are filled' do
         options = described_class.new(a: 1)
         expect(options.to_h).to have_key(:query)
       end
     end
 
-    context 'pagination' do
+    describe 'pagination' do
       it 'adds "page" to "query"' do
         raw      = { page: '23' }
         expected = { query: raw }
@@ -47,7 +47,7 @@ RSpec.describe Dnsimple::Options::ListOptions do
       end
     end
 
-    context 'sorting' do
+    describe 'sorting' do
       it 'adds sorting policy to "query"' do
         raw      = { sort: 'name:desc' }
         expected = { query: raw }
@@ -73,7 +73,7 @@ RSpec.describe Dnsimple::Options::ListOptions do
       end
     end
 
-    context 'filtering' do
+    describe 'filtering' do
       it 'adds filtering policy to "query"' do
         raw      = { filter: { name_like: 'example' } }
         expected = { query: raw.fetch(:filter) }

data/spec/spec_helper.rb

@@ -14,4 +14,4 @@ unless defined?(SPEC_ROOT)
   SPEC_ROOT = File.expand_path(__dir__)
 end
 
-Dir[File.join(SPEC_ROOT, "support/**/*.rb")].sort.each { |f| require f }
+Dir[File.join(SPEC_ROOT, "support/**/*.rb")].each { |f| require f }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dnsimple
 version: !ruby/object:Gem::Version
-  version: 6.0.0
+  version: 7.0.0
 platform: ruby
 authors:
 - DNSimple
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-04-20 00:00:00.000000000 Z
+date: 2021-06-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -88,7 +88,7 @@ extensions: []
 extra_rdoc_files:
 - LICENSE.txt
 files:
-- ".dependabot/config.yml"
+- ".github/dependabot.yml"
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"
@@ -356,7 +356,7 @@ files:
 homepage: https://github.com/dnsimple/dnsimple-ruby
 licenses: []
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -364,15 +364,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.4'
+      version: '2.6'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.3
-signing_key:
+rubygems_version: 3.1.4
+signing_key: 
 specification_version: 4
 summary: The DNSimple API client for Ruby
 test_files:

data/.dependabot/config.yml

@@ -1,9 +0,0 @@
-version: 1
-update_configs:
-  - package_manager: "ruby:bundler"
-    directory: "/"
-    update_schedule: "daily"
-    default_labels:
-      - "task"
-      - "dependencies"
-      - "backlog-dependencies"