dnsimple 12.1.0 → 12.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 371112dc13ce900d770aa434d297b19f75892d3a90a3b2f4292ddffb88574c3f
-  data.tar.gz: c312bc89ea319f11c94bf3ed157c663264d206480033076c9b122a5d31dee2a2
+  metadata.gz: 7f407171557923f494c7021338188386bc8960ce0f4b5d8d054a599370e1c9bb
+  data.tar.gz: 9770d96251d504107ce14afcedf3cade478902ac1533f1836103fca8cf4fe584
 SHA512:
-  metadata.gz: 40334df4d23d15a5a70fde859aacfaffe0407f71fb8cde5524a03b4be98b4df710d71b8eaa7fb3c8484c38fb7845b3b54c95326c84328d44524d771700a39f39
-  data.tar.gz: 190c114666115887b7724778e72ad511a204a084f06d30494fed5cc47f8a4ca76a4465e1e8d3b44773b71b55105bdfcfca00c38aaf50a20beb714926e2fd2b5f
+  metadata.gz: 519a34dc4c836058cb9edae0c185abb3c270f1ebac1b5b22202a1b198e148736aff3cc4e002082901656621e71e26a78e0fe5012df6d9421f7129e876e30c6d5
+  data.tar.gz: d5fad291abd99d373d66158a954780f2b278ccc3d95d01f7552f9689ae8bd5d7f9a91816b127818fc38c1c53f44680c9cce9542cd073f5f4ab8c0c2443543453

data/.github/workflows/ci.yml

@@ -15,7 +15,6 @@ concurrency:
 
 jobs:
   markdown-lint:
-    name: Lint markdown
     runs-on: ubuntu-latest
     steps:
       - name: Checkout Code
@@ -27,10 +26,10 @@ jobs:
           config_file: ".markdownlint.yaml"
 
   test:
+    name: Ruby ${{ matrix.ruby-version }}
+    runs-on: ubuntu-latest
     needs:
       - markdown-lint
-    runs-on: ubuntu-latest
-    name: Ruby ${{ matrix.ruby-version }}
     strategy:
       matrix:
         ruby-version:

data/.github/workflows/release.yml

@@ -1,54 +1,27 @@
 name: Release
 
 on:
-  workflow_run:
-    workflows:
-      - "CI"
-    types:
-      - completed
+  push:
+    tags:
+      - v*.*.*
+
 jobs:
-  validate-tag:
-    name: Check tag
+  wait-for-ci:
+    name: Wait for CI
     runs-on: ubuntu-latest
-    outputs:
-      valid_tag: ${{ steps.validation.outputs.valid_tag }}
-    if: ${{ github.event.workflow_run.conclusion == 'success' && github.event.workflow_run.event == 'push' && startsWith(github.event.workflow_run.head_branch, 'v') }}
     steps:
-      - name: Check out the repository including tags
-        uses: actions/checkout@v6
+      - name: Wait for CI to pass
+        uses: lewagon/wait-on-check-action@v1.5.0
         with:
-          ref: ${{ github.event.workflow_run.head_sha }}
-          fetch-depth: 0
-          fetch-tags: true
-      - name: Validate tag
-        id: validation
-        run: |
-          # Validation is necessary in the unlikely case that a branch matching the tag naming pattern is pushed
-          # and the CI workflow in that branch is modified to run upon a push to that branch
-          REF='${{ github.event.workflow_run.head_branch }}' # This can be a branch or tag name
-          if [[ "$REF" != v*.*.* ]]; then
-            echo "valid_tag=false" >> "$GITHUB_OUTPUT"; exit 0
-          fi
-          # Validate that the tag exists
-          if ! git rev-parse -q --verify "refs/tags/$REF" >/dev/null; then
-            echo "There is no tag matching $REF - $REF is a branch"
-            echo "valid_tag=false" >> "$GITHUB_OUTPUT"; exit 0
-          fi
-          # Validate that the tag is for the same commit that was pushed
-          TAG_SHA="$(git rev-parse "$REF^{commit}")"
-          COMMIT_SHA="${{ github.event.workflow_run.head_sha }}"
-          if [ "$TAG_SHA" != "$COMMIT_SHA" ]; then
-            echo "Tag SHA $TAG_SHA does not match pushed commit SHA $COMMIT_SHA"
-            echo "valid_tag=false" >> "$GITHUB_OUTPUT"; exit 0
-          fi
-          echo "Tag $REF exists and is valid. Tag $TAG_SHA matches the pushed commit $COMMIT_SHA."
-          echo "valid_tag=true"  >> "$GITHUB_OUTPUT"
+          ref: ${{ github.sha }}
+          running-workflow-name: Wait for CI
+          repo-token: ${{ secrets.GITHUB_TOKEN }}
+          wait-interval: 10
+
   publish:
     name: Release to RubyGems
-    needs: validate-tag
+    needs: wait-for-ci
     runs-on: ubuntu-latest
-    if: ${{ needs.validate-tag.outputs.valid_tag == 'true' && github.event.workflow_run.conclusion == 'success' }}
-
     steps:
       - uses: actions/checkout@v6
 

data/.github/workflows/sync-test-fixtures.yml

@@ -11,45 +11,24 @@ jobs:
       - name: Checkout repository
         uses: actions/checkout@v6
       - name: Sync fixtures
+        id: sync
         run: |
           TMP_DIR=$(mktemp -d)
           curl -fsSL "https://codeload.github.com/dnsimple/dnsimple-developer/tar.gz/refs/heads/main" \
             | tar -xz -C "$TMP_DIR"
-          rsync -a --delete "$TMP_DIR/dnsimple-developer-main/fixtures/v2/api/" spec/fixtures.http/
-      - name: Create a PR to sync fixtures if there are changes
-        run: |
-          cat <<'MD' > pr_body.md
-          This PR replaces the fixtures in this repository with those from https://github.com/dnsimple/dnsimple-developer/
-
-          ## Checklist
-          - [ ] Close and re-open this PR to trigger the CI workflow (it does not run automatically)
-          - [ ] Ensure the CI workflow passes before merging
-          MD
-
-          # Configure Git identity for the workflow
-          git config --global user.name "github-actions"
-          git config --global user.email "github-actions@github.com"
-
-          # Create and switch to a new branch
-          BRANCH="chore/sync-fixtures-${GITHUB_RUN_ID}"
-          NOW="$(date +'%Y-%m-%d %H:%M:%S')"
-          git checkout -b "$BRANCH"
-
-          # Stage changes
-          git add spec/fixtures.http
-
-          # Commit if there are staged changes
-          if ! git diff --cached --quiet; then
-            git commit -m "chore: sync test fixtures as of $NOW"
-            git push -u origin "$BRANCH"
-
-            gh pr create \
-              --title "chore: Sync fixtures as of $NOW" \
-              --body-file pr_body.md \
-              --base main \
-              --head "$BRANCH"
-          else
-            echo "No fixture detected; nothing to do"
-          fi
-        env:
-          GITHUB_TOKEN: ${{ github.token }}
+          rsync -a --delete "$TMP_DIR/dnsimple-developer-main/fixtures/v2/api/" test/fixtures.http/
+          echo "now=$(date +'%Y-%m-%d %H:%M:%S')" >> "$GITHUB_OUTPUT"
+      - name: Create PR with synced fixtures
+        uses: peter-evans/create-pull-request@v7
+        with:
+          sign-commits: true
+          add-paths: test/fixtures.http
+          commit-message: "chore: sync test fixtures as of ${{ steps.sync.outputs.now }}"
+          branch: "chore/sync-fixtures-${{ github.run_id }}"
+          title: "chore: Sync fixtures as of ${{ steps.sync.outputs.now }}"
+          body: |
+            This PR replaces the fixtures in this repository with those from https://github.com/dnsimple/dnsimple-developer/
+
+            ## Checklist
+            - [ ] Close and re-open this PR to trigger the CI workflow (it does not run automatically)
+            - [ ] Ensure the CI workflow passes before merging

data/.rubocop.yml

@@ -6,7 +6,7 @@ inherit_from:
 plugins:
   - rubocop-performance
   - rubocop-rake
-  - rubocop-rspec
+  - rubocop-minitest
 
 AllCops:
   TargetRubyVersion: 3.2

data/.rubocop_todo.yml

@@ -1,49 +1,11 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2025-12-27 17:14:14 UTC using RuboCop version 1.82.1.
+# on 2026-01-14 23:11:38 UTC using RuboCop version 1.82.1.
 # 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: 90
-# Configuration parameters: CountAsOne.
-RSpec/ExampleLength:
-  Max: 17
-
-# Offense count: 1
-RSpec/LeakyConstantDeclaration:
-  Exclude:
-    - 'spec/dnsimple/client/client_service_spec.rb'
-
-# Offense count: 12
-# Configuration parameters: .
-# SupportedStyles: have_received, receive
-RSpec/MessageSpies:
-  EnforcedStyle: receive
-
-# Offense count: 134
-RSpec/MultipleExpectations:
+# Offense count: 79
+Minitest/MultipleAssertions:
   Max: 15
-
-# Offense count: 422
-# Configuration parameters: EnforcedStyle, IgnoreSharedExamples.
-# SupportedStyles: always, named_only
-RSpec/NamedSubject:
-  Enabled: false
-
-# Offense count: 16
-RSpec/SubjectStub:
-  Exclude:
-    - 'spec/dnsimple/client/certificates_spec.rb'
-    - 'spec/dnsimple/client/contacts_spec.rb'
-    - 'spec/dnsimple/client/domains_delegation_signer_records_spec.rb'
-    - 'spec/dnsimple/client/domains_email_forwards_spec.rb'
-    - 'spec/dnsimple/client/domains_spec.rb'
-    - 'spec/dnsimple/client/services_spec.rb'
-    - 'spec/dnsimple/client/templates_records_spec.rb'
-    - 'spec/dnsimple/client/templates_spec.rb'
-    - 'spec/dnsimple/client/tlds_spec.rb'
-    - 'spec/dnsimple/client/zones_records_spec.rb'
-    - 'spec/dnsimple/client/zones_spec.rb'
-    - 'spec/dnsimple/client_spec.rb'

data/CHANGELOG.md

@@ -2,7 +2,19 @@
 
 This project uses [Semantic Versioning 2.0.0](http://semver.org/), the format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
-## 12.1.0 - 2025-02-26
+## 12.2.0 - 2026-03-23
+
+### Added
+
+- Added support for trustee service. (#459)
+
+## 12.1.1 - 2026-03-11
+
+### Changed
+
+- Migrated the test suite from RSpec to Minitest. This is an internal change with no public API impact. (#448)
+
+## 12.1.0 - 2026-02-26
 
 ### Added
 

data/Gemfile

@@ -9,6 +9,6 @@ gem "bigdecimal", "~> 4.0"
 gem "coveralls", "~> 0.8", require: false
 gem "csv", "~> 3.2"
 gem "rubocop", "~> 1.75", require: false
+gem "rubocop-minitest", "~> 0.36", require: false
 gem "rubocop-performance", "~> 1.25", require: false
 gem "rubocop-rake", "~> 0.7", require: false
-gem "rubocop-rspec", "~> 3.6", require: false

data/Rakefile

@@ -1,12 +1,14 @@
 require 'bundler/gem_tasks'
 
 # By default, run tests and linter.
-task default: [:spec, :rubocop]
+task default: [:test, :rubocop]
 
 
-require 'rspec/core/rake_task'
+require 'rake/testtask'
 
-RSpec::Core::RakeTask.new do |t|
+Rake::TestTask.new do |t|
+  t.libs << "test"
+  t.test_files = FileList["test/**/*_test.rb"]
   t.verbose = !ENV["VERBOSE"].nil?
 end
 

data/dnsimple.gemspec

@@ -21,7 +21,8 @@ Gem::Specification.new do |s|
   s.add_dependency 'httparty'
 
   s.add_development_dependency 'rake'
-  s.add_development_dependency 'rspec'
+  s.add_development_dependency 'minitest'
+  s.add_development_dependency 'minitest-mock'
   s.add_development_dependency 'yard'
   s.add_development_dependency 'webmock'
 end

data/lib/dnsimple/struct/domain.rb

@@ -27,6 +27,9 @@ module Dnsimple
       # @return [Bool] True if the domain WHOIS privacy is enabled, false otherwise.
       attr_accessor :private_whois
 
+      # @return [Bool] True if the domain Trustee service is enabled, false otherwise.
+      attr_accessor :trustee_service
+
       # @return [String] The timestamp when domain will expire.
       attr_accessor :expires_at
 

data/lib/dnsimple/struct/domain_price.rb

@@ -17,6 +17,9 @@ module Dnsimple
 
       # @return [Float] The price for transfer
       attr_accessor :transfer_price
+
+      # @return [Float] The Trustee service price
+      attr_accessor :trustee_service_price
     end
   end
 end

data/lib/dnsimple/struct/domain_registration.rb

@@ -24,6 +24,9 @@ module Dnsimple
       # @return [Bool] True if the domain WHOIS privacy was requested.
       attr_accessor :whois_privacy
 
+      # @return [Bool] True if the domain Trustee service was requested.
+      attr_accessor :trustee_service
+
       # @return [String] When the domain renewal was created in DNSimple.
       attr_accessor :created_at
 

data/lib/dnsimple/struct/tld.rb

@@ -30,6 +30,12 @@ module Dnsimple
       # @return [Boolean] True if DNSimple supports inbound transfers for this TLD.
       attr_accessor :transfer_enabled
 
+      # @return [Boolean] True if Trustee service is enabled for this TLD.
+      attr_accessor :trustee_service_enabled
+
+      # @return [Boolean] True if Trustee service is required for this TLD.
+      attr_accessor :trustee_service_required
+
       # @return [String, nil] Type of data interface required for DNSSEC for this TLD.
       attr_accessor :dnssec_interface_type
     end

data/lib/dnsimple/version.rb

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

data/test/dnsimple/client/accounts_test.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+class AccountsTest < Minitest::Test
+  def setup
+    @subject = Dnsimple::Client.new(base_url: "https://api.dnsimple.test", access_token: "a1b2c3").accounts
+  end
+
+  test "builds the correct request" do
+    stub_request(:get, %r{/v2/accounts$})
+        .to_return(read_http_fixture("listAccounts/success-user.http"))
+
+    @subject.accounts
+
+    assert_requested(:get, "https://api.dnsimple.test/v2/accounts",
+                     headers: { "Accept" => "application/json" })
+  end
+
+  test "returns the accounts" do
+    stub_request(:get, %r{/v2/accounts$})
+        .to_return(read_http_fixture("listAccounts/success-user.http"))
+
+    response = @subject.accounts
+
+    assert_kind_of(Dnsimple::Response, response)
+
+    result = response.data
+
+    assert_kind_of(Dnsimple::Struct::Account, result.first)
+    assert_kind_of(Dnsimple::Struct::Account, result.last)
+  end
+end

data/test/dnsimple/client/billing_test.rb

@@ -0,0 +1,106 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+require "bigdecimal/util"
+
+class BillingTest < Minitest::Test
+  def setup
+    @subject = Dnsimple::Client.new(base_url: "https://api.dnsimple.test", access_token: "a1b2c3").billing
+    @account_id = 1010
+  end
+
+  test "builds the correct request" do
+    stub_request(:get, %r{/v2/#{@account_id}/billing/charges})
+        .to_return(read_http_fixture("listCharges/success.http"))
+
+    @subject.charges(@account_id)
+
+    assert_requested(:get, "https://api.dnsimple.test/v2/#{@account_id}/billing/charges",
+                     headers: { "Accept" => "application/json" })
+  end
+
+  test "exposes the pagination information" do
+    stub_request(:get, %r{/v2/#{@account_id}/billing/charges})
+        .to_return(read_http_fixture("listCharges/success.http"))
+
+    response = @subject.charges(@account_id)
+
+    assert_respond_to(response, :page)
+    assert_equal(1, response.page)
+    assert_kind_of(Integer, response.per_page)
+    assert_kind_of(Integer, response.total_entries)
+    assert_kind_of(Integer, response.total_pages)
+  end
+
+  test "returns the charges" do
+    stub_request(:get, %r{/v2/#{@account_id}/billing/charges})
+        .to_return(read_http_fixture("listCharges/success.http"))
+
+    response = @subject.charges(@account_id)
+
+    assert_kind_of(Dnsimple::PaginatedResponse, response)
+    assert_kind_of(Array, response.data)
+    assert_equal(3, response.data.size)
+
+    response.data.each do |result|
+      assert_kind_of(Dnsimple::Struct::Charge, result)
+      assert_kind_of(BigDecimal, result.balance_amount)
+      assert_kind_of(String, result.reference)
+      assert_kind_of(Array, result.items)
+      assert_kind_of(Dnsimple::Struct::Charge::ChargeItem, result.items[0])
+    end
+
+    assert_kind_of(BigDecimal, response.data[0].total_amount)
+    assert_equal("14.5", response.data[0].total_amount.to_s("F"))
+    assert_kind_of(BigDecimal, response.data[0].items[0].amount)
+    assert_equal("14.5", response.data[0].items[0].amount.to_s("F"))
+  end
+
+  test "supports filters" do
+    stub_request(:get, %r{/v2/#{@account_id}/billing/charges})
+        .to_return(read_http_fixture("listCharges/success.http"))
+
+    @subject.charges(@account_id, filter: { start_date: "2023-01-01", end_date: "2023-08-31" })
+
+    assert_requested(:get, "https://api.dnsimple.test/v2/#{@account_id}/billing/charges?start_date=2023-01-01&end_date=2023-08-31")
+  end
+
+  test "supports pagination" do
+    stub_request(:get, %r{/v2/#{@account_id}/billing/charges})
+        .to_return(read_http_fixture("listCharges/success.http"))
+
+    @subject.charges(@account_id, page: 2)
+
+    assert_requested(:get, "https://api.dnsimple.test/v2/#{@account_id}/billing/charges?page=2")
+  end
+
+  test "supports sorting" do
+    stub_request(:get, %r{/v2/#{@account_id}/billing/charges})
+        .to_return(read_http_fixture("listCharges/success.http"))
+
+    @subject.charges(@account_id, sort: "invoiced:asc")
+
+    assert_requested(:get, "https://api.dnsimple.test/v2/#{@account_id}/billing/charges?sort=invoiced:asc")
+  end
+
+  test "raises error when using a bad filter" do
+    stub_request(:get, %r{/v2/#{@account_id}/billing/charges})
+        .to_return(read_http_fixture("listCharges/fail-400-bad-filter.http"))
+
+    error = assert_raises(Dnsimple::RequestError) do
+      @subject.charges(@account_id, filter: { start_date: "01-01-2023" })
+    end
+    assert_equal("Invalid date format must be ISO8601 (YYYY-MM-DD)", error.message)
+  end
+
+  test "raises error when account is not authorized" do
+    stub_request(:get, %r{/v2/#{@account_id}/billing/charges})
+        .to_return(read_http_fixture("listCharges/fail-403.http"))
+
+    error = assert_raises(Dnsimple::RequestError) do
+      @subject.charges(@account_id)
+    end
+    assert_equal("Permission Denied. Required Scope: billing:*:read", error.message)
+  end
+end