laa-fee-calculator-client 1.3.0 → 1.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a6720e8d64a0423da427ccaec75a0bbd5bacca65634fca113ff4863a277b0065
-  data.tar.gz: 8a1e9ad3aca5a391316c5825e8c47296b98fc1d15a96a75cd3ec93087e40c870
+  metadata.gz: 846930f03ac36c9166dcba3a25cc3b2b552f32b31861043fa1908741912da754
+  data.tar.gz: 3b5ec2435a93a7584eff62facedf12a8ab3c754f6f83bcabc625b1ef8fc96c07
 SHA512:
-  metadata.gz: a10006fdb40c4c1749031e500848c349fdc4e24cdc66c71384c1729a2d8ae63a7eaad15ac5ec32888df7ee17c0dc7bd9390a914f14b959f272adc106656cc7e5
-  data.tar.gz: 85b5c5aeb923f0014fee35043938fea0e239db55334f65bbfda20910202407a4a97ffd4a0cfda8f709e7c21cd28b0e6af6377846d82e6d07684da69309adaf25
+  metadata.gz: 953bba0d471745c2502a7dc9b0d7cc21c1a58fae009ce09777d419b4a9cc7177b25593d90476ed3536b2f17092b2f124336090dd6c600b7567021dc0f2d26113
+  data.tar.gz: 44e1ca22ff932959a6ddfab3046120b8d619647178e9e37957b23539176cee281916e24f23cc6b4fb409b0c58862c0650bf460eb2ee92032c0dbd0a0472e4e25

data/CHANGELOG.md

@@ -12,8 +12,12 @@ Add latest change log entries at top using template:
   -
 ### Removed
   -
-
 ```
+## Version 1.3.1
+
+### Removed
+  - `addressable` gem dependency
+
 ## Version 1.3.0
 
 ### Modified

data/Gemfile

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-source "https://rubygems.org"
+source 'https://rubygems.org'
 
 git_source(:github) { |repo_name| "https://github.com/#{repo_name}" }
 

data/README.md

@@ -177,13 +177,11 @@ Bug reports and pull requests are welcome on GitHub at [ministryofjustice/laa-fe
 
 3. Update the VERSION in `lib/laa/fee_calculator/version` using [semantic versioning](https://guides.rubygems.org/patterns/#semantic-versioning).
 
-4. Update the VERSION_RELEASED in `lib/laa/fee_calculator/version` to the date you intend to publish/release the version.
+4. Update the `CHANGELOG.md` - format/documentation standards specified at top of file.
 
-5. Update the `CHANGELOG.md` - format/documentation standards specified at top of file.
+5. PR the change, code-review, merge.
 
-6. PR the change, code-review, merge.
-
-7. Pull master and run rake task below to publish. Note, you will need a [rubygems](https://rubygems.org/) account with gem owner privileges and have configured your `~/.gem/credentials` in order for this task to complete successfully.
+6. Pull master and run rake task below to publish. Note, you will need a [rubygems](https://rubygems.org/) account with gem owner privileges and have configured your `~/.gem/credentials` in order for this task to complete successfully.
 
 ```bash
 $ rake release

data/Rakefile

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
 require 'rubocop/rake_task'
 
 RSpec::Core::RakeTask.new(:spec)

data/bin/console

@@ -1,8 +1,8 @@
 #!/usr/bin/env ruby
 # frozen_string_literal: true
 
-require "bundler/setup"
-require "laa/fee_calculator"
+require 'bundler/setup'
+require 'laa/fee_calculator'
 
 # You can add fixtures and/or initialization code here to make experimenting
 # with your gem easier. You can also use a different console, if you like.
@@ -16,5 +16,5 @@ LAA::FeeCalculator.configure do |config|
   config.host = 'http://localhost:8000/api/v1'
 end
 
-require "irb"
+require 'irb'
 IRB.start(__FILE__)

data/laa-fee-calculator-client.gemspec

@@ -9,12 +9,11 @@ Gem::Specification.new do |spec|
   spec.version       = LAA::FeeCalculator::VERSION
   spec.authors       = ['Katharine Ahern', 'Ministry of Justice']
   spec.email         = ['katharine.ahern@digital.justice.gov.uk', 'joel.sugarman@digital.justice.gov.uk', 'tools@digital.justice.gov.uk']
-  spec.date          = LAA::FeeCalculator::VERSION_RELEASED
   spec.summary       = 'Ruby client for the Legal Aid Agency fee calculator API'
-  spec.description   = "Ruby client for the Ministry of Justices LAA fee calculator API. A simple interface for transparent calling of the API endpoints to query data and return calculated fee amounts."
+  spec.description   = 'Ruby client for the Ministry of Justices LAA fee calculator API. A simple interface for transparent calling of the API endpoints to query data and return calculated fee amounts.'
   spec.homepage      = 'https://github.com/ministryofjustice/laa-fee-calculator-client'
   spec.license       = 'MIT'
-  spec.extra_rdoc_files = ["LICENSE", "README.md"]
+  spec.extra_rdoc_files = ['LICENSE', 'README.md']
 
   spec.files = `git ls-files -z`.split("\x0").reject do |f|
     f.match(%r{^(test/|spec/|features/|\..*)})
@@ -26,14 +25,16 @@ Gem::Specification.new do |spec|
   # see bin/ruby_version_test
   spec.required_ruby_version = '>= 2.6.0'
 
-  spec.add_runtime_dependency 'addressable', '~> 2.3', '>= 2.3.7'
   spec.add_runtime_dependency 'faraday', '~> 1.0'
 
   spec.add_development_dependency 'awesome_print'
   spec.add_development_dependency 'rake', '~> 13.0'
   spec.add_development_dependency 'rb-readline'
   spec.add_development_dependency 'rspec', '~> 3.0'
-  spec.add_development_dependency 'rubocop', '~> 0.78'
+  spec.add_development_dependency 'rubocop', '~> 1.28'
+  spec.add_development_dependency 'rubocop-performance', '~> 1.13'
+  spec.add_development_dependency 'rubocop-rake', '~> 0.6'
+  spec.add_development_dependency 'rubocop-rspec', '~> 2.10'
   spec.add_development_dependency 'vcr'
   spec.add_development_dependency 'webmock'
 end

data/lib/laa/fee_calculator/calculator.rb

@@ -6,25 +6,18 @@ module LAA
       extend Forwardable
       def_delegators :connection, :get
 
-      attr_reader :scheme_pk, :options
-
       def initialize(scheme_pk, options)
-        @scheme_pk = scheme_pk
-        @options = options
+        @uri = "fee-schemes/#{scheme_pk}/calculate/"
+        @filtered_params = options.reject { |k, _v| k.eql?(:id) }
       end
 
       def call
-        uri.query_values = options.reject { |k, _v| k.eql?(:id) }
-        json = get(uri).body
+        json = get(@uri, @filtered_params).body
         JSON.parse(json)['amount']
       end
 
       private
 
-      def uri
-        @uri ||= Addressable::URI.parse("fee-schemes/#{scheme_pk}/calculate/")
-      end
-
       def connection
         @connection ||= Connection.instance
       end

data/lib/laa/fee_calculator/client.rb

@@ -1,4 +1,4 @@
-require "addressable/uri"
+# frozen_string_literal: true
 
 module LAA
   module FeeCalculator
@@ -17,13 +17,11 @@ module LAA
       end
 
       def fee_schemes(id = nil, **options)
-        uri = 'fee-schemes/'
-        id ||= options.fetch(:id, nil)
-        uri.concat(id.to_s, "/") if id
-        uri = Addressable::URI.parse(uri)
-        uri.query_values = options.reject { |k, _v| k.eql?(:id) }
+        uri = fee_schemes_uri(id || options[:id])
+        filtered_params = options.reject { |k, _| k.eql?(:id) }
+        # Ruby 3.0+; filtered_params = options.except(:id)
 
-        json = get(uri).body
+        json = get(uri, filtered_params).body
 
         fstruct = JSON.parse(json, object_class: FeeScheme)
         return fstruct unless fstruct.respond_to?(:results)
@@ -31,6 +29,14 @@ module LAA
 
         fstruct.results
       end
+
+      private
+
+      def fee_schemes_uri(id)
+        return "fee-schemes/#{id}/" if id
+
+        'fee-schemes/'
+      end
     end
   end
 end

data/lib/laa/fee_calculator/has_manyable.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'json'
 
 module LAA
@@ -17,18 +19,11 @@ module LAA
         end
 
         def has_many(association)
-          define_method("#{association}_uri".to_sym) do |scheme_pk = nil, id = nil|
-            uri = scheme_pk.nil? ? "#{association.to_s.tr('_', '-')}/" : "fee-schemes/#{scheme_pk}/#{association.to_s.tr('_', '-')}/"
-            uri = uri.concat(id.to_s, '/') unless id.nil?
-            Addressable::URI.parse(uri)
-          end
-
           define_method(association) do |id = nil, **options|
-            id ||= options&.fetch(:id, nil)
-            uri = self.send("#{association}_uri", self.id, id)
-            uri.query_values = options.reject { |k, _v| k.eql?(:id) }
+            uri = uri_for(association, id: id || options[:id])
+            filtered_params = options.reject { |k, _v| k.eql?(:id) }
 
-            json = get(uri).body
+            json = get(uri, filtered_params).body
 
             ostruct = JSON.parse(json, object_class: OpenStruct)
             return ostruct unless ostruct.respond_to?(:results)
@@ -37,6 +32,16 @@ module LAA
           end
         end
       end
+
+      def uri_for(association, id: nil)
+        return "#{base_fee_scheme_uri}#{association.to_s.tr('_', '-')}/" if id.nil?
+
+        "#{base_fee_scheme_uri}#{association.to_s.tr('_', '-')}/#{id}/"
+      end
+
+      def base_fee_scheme_uri
+        id ? "fee-schemes/#{id}/" : ''
+      end
     end
   end
 end

data/lib/laa/fee_calculator/version.rb

@@ -2,8 +2,7 @@
 
 module LAA
   module FeeCalculator
-    VERSION = "1.3.0"
-    VERSION_RELEASED = "2022-04-21"
+    VERSION = '1.3.1'
     USER_AGENT = "laa-fee-calculator-client/#{VERSION}"
   end
 end

data/lib/laa/fee_calculator.rb

@@ -12,7 +12,7 @@ module LAA
   module FeeCalculator
     class << self
       def root
-        spec = Gem::Specification.find_by_name("laa-fee-calculator-client")
+        spec = Gem::Specification.find_by_name('laa-fee-calculator-client')
         spec.gem_dir
       end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: laa-fee-calculator-client
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.3.1
 platform: ruby
 authors:
 - Katharine Ahern
@@ -9,28 +9,8 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-04-21 00:00:00.000000000 Z
+date: 2022-04-29 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: addressable
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.3'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 2.3.7
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.3'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 2.3.7
 - !ruby/object:Gem::Dependency
   name: faraday
   requirement: !ruby/object:Gem::Requirement
@@ -107,14 +87,56 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.78'
+        version: '1.28'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.28'
+- !ruby/object:Gem::Dependency
+  name: rubocop-performance
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.13'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.13'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.10'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.78'
+        version: '2.10'
 - !ruby/object:Gem::Dependency
   name: vcr
   requirement: !ruby/object:Gem::Requirement
@@ -195,7 +217,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: Ruby client for the Legal Aid Agency fee calculator API