dhs 1.4.0 → 1.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c31dbca2778853f9bafb3a74e2c7ef830d3897c0caa78b66e5bb58203fc8bb11
-  data.tar.gz: 8e5968134c395c0b4e6196506ef5502f5941f68754423bc885bc536a44c2ba33
+  metadata.gz: 690186fffb1221bbc432b5a502bdfecb895b1f117e42602e6bef6b2c55cd7bd1
+  data.tar.gz: 7f544c63db8a4db092a6c3980b3cb2f437879876db145a6ac7d8c6cfc7647a05
 SHA512:
-  metadata.gz: 54e1c9ec3c0b71566a8b6831690250d0eecc00f00155617dc39873c20d49a26ef57051ae04d7d3a1bbb3733f75fc3ccc0024b96c3b4bd086dafe48a1634cb8cf
-  data.tar.gz: 1bea7f70ccd0cba8dc488604d5f4a668d1bc602a2cb30d19058b195a17b2d9f1aaa5d72e21286e910570a89be0e78c25adac4bb45186a6f3f11ffcc81128b39c
+  metadata.gz: '087dacf286f24b6860419759c5ea7cd443d69e2cd8f4900b2ee17fdf93d2d9f55918534ec0aff8294e6a4906c2dad51d3c4eb2b6ad8b3f19111a4133ed15f6c5'
+  data.tar.gz: f14b0dad8dffb72c65cd6b785469a59c590a99187639e554865e99bcf8e696434f262c4e95d64b1241bfd997180ef278ce2f0f0f5b91c6e5f11a673466f7eff8

data/lib/dhs/concerns/record/request.rb

@@ -204,7 +204,7 @@ class DHS::Record
       end
 
       def load_and_merge_not_paginated_collection!(data, options)
-        return if data.length.zero?
+        return if data.empty?
         options = options.is_a?(Hash) ? options : {}
         limit = options.dig(:params, limit_key(:parameter)) || pagination_class::DEFAULT_LIMIT
         offset = options.dig(:params, pagination_key(:parameter)) || pagination_class::DEFAULT_OFFSET
@@ -230,7 +230,7 @@ class DHS::Record
       end
 
       def load_and_merge_paginated_collection!(data, options)
-        set_nested_data(data._raw, limit_key(:body), data.length) if data._raw.dig(*limit_key(:body)).blank? && !data.length.zero?
+        set_nested_data(data._raw, limit_key(:body), data.length) if data._raw.dig(*limit_key(:body)).blank? && !data.empty?
         pagination = data._record.pagination(data)
         return data unless pagination.pages_left?
         record = data._record
@@ -256,7 +256,7 @@ class DHS::Record
           page_data = if next_value.is_a?(String) && next_value.match(/^http/)
              record.request(options.except(:all).merge(url: next_value))
           else
-            record.request(options.except(:all).merge(params: (options.dig(:params) || {}).merge(next_value) ))
+            record.request(options.except(:all).merge(params: (options.dig(:params) || {}).merge(next_value)))
           end
           next_value = pagination.next(page_data._raw)
           merge_batch_data_with_parent!(page_data, data, pagination)

data/lib/dhs/pagination/next_offset.rb

@@ -3,7 +3,7 @@
 class DHS::Pagination::NextOffset < DHS::Pagination::Base
 
   DEFAULT_OFFSET = 0
-  
+
   def total
     data._raw.dig(*_record.items_key).count || 0
   end

data/lib/dhs/problems/errors.rb

@@ -54,7 +54,7 @@ module DHS::Problems
     end
 
     def messages_from_response(response = nil)
-      return {} if !response || !response.body.is_a?(String) || response.body.length.zero?
+      return {} if !response || !response.body.is_a?(String) || response.body.empty?
       json = JSON.parse(response.body)
       parse_messages(json)
     end

data/lib/dhs/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DHS
-  VERSION = '1.4.0'
+  VERSION = '1.4.1'
 end

data/spec/concerns/record/request_spec.rb

@@ -19,15 +19,15 @@ describe DHS::Record::Request do
 
     it 'calls correct prepare method for a Hash' do
       expect(subject).to receive(:prepare_option_for_include_all_request!)
-        .with(abc: 'def').and_return('ignore')
-      expect(subject.send(:prepare_options_for_include_all_request!, abc: 'def')).to eq(abc: 'def')
+        .with({ abc: 'def' }).and_return('ignore')
+      expect(subject.send(:prepare_options_for_include_all_request!, { abc: 'def' })).to eq(abc: 'def')
     end
 
     it 'calls correct prepare method for a Hash (collection)' do
       expect(subject).to receive(:prepare_option_for_include_all_request!)
-        .with(abc: 'def').and_return('ignore')
+        .with({ abc: 'def' }).and_return('ignore')
       expect(subject).to receive(:prepare_option_for_include_all_request!)
-        .with(hij: 'kel').and_return('ignore')
+        .with({ hij: 'kel' }).and_return('ignore')
       expect(subject.send(:prepare_options_for_include_all_request!, [{ abc: 'def' }, { hij: 'kel' }]))
         .to eq([{ abc: 'def' }, { hij: 'kel' }])
     end

data/spec/pagination/next_offset_spec.rb

@@ -3,11 +3,9 @@
 require 'rails_helper'
 
 describe DHS::Record do
-
   context 'pagination' do
-
-    def stub_api_request(items: [], offset: nil, next_offset:)
-      stub_request(:get, ["http://depay.fi/v2/transactions?limit=100", offset ? "offset=#{offset}" : nil].compact.join('&'))
+    def stub_api_request(next_offset:, items: [], offset: nil)
+      stub_request(:get, ['http://depay.fi/v2/transactions?limit=100', offset ? "offset=#{offset}" : nil].compact.join('&'))
         .to_return(body: { items: items, next_offset: next_offset }.to_json)
     end
 

data/spec/record/endpoint_options_spec.rb

@@ -16,12 +16,12 @@ describe DHS::Record do
 
     it 'uses the options that are configured for an endpoint' do
       expect(DHC).to receive(:request)
-        .with(
-          cache_expires_in: 1.day,
-          retry: 2,
-          cache: true,
-          url: 'backend/v2/feedbacks/1'
-        ).and_call_original
+        .with({
+                cache_expires_in: 1.day,
+                retry: 2,
+                cache: true,
+                url: 'backend/v2/feedbacks/1'
+              }).and_call_original
 
       stub_request(:get, 'http://backend/v2/feedbacks/1').to_return(status: 200)
       Record.find(1)

data/spec/record/includes_spec.rb

@@ -214,12 +214,12 @@ describe DHS::Record do
 
       it 'overwrites existing pagination paramters if they are already contained in a string' do
         expect(DHC).to receive(:request)
-          .with(url: 'http://datastore/customers/1').and_call_original
+          .with({ url: 'http://datastore/customers/1' }).and_call_original
 
         expect(DHC).to receive(:request)
-          .with(url: 'http://datastore/customers/1/contracts',
-                all: true,
-                params: { limit: 100 }).and_call_original
+          .with({ url: 'http://datastore/customers/1/contracts',
+                  all: true,
+                  params: { limit: 100 } }).and_call_original
 
         expect(DHC).to receive(:request)
           .with([{ url: 'http://datastore/customers/1/contracts',

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dhs
 version: !ruby/object:Gem::Version
-  version: 1.4.0
+  version: 1.4.1
 platform: ruby
 authors:
 - https://github.com/DePayFi/dhs/graphs/contributors
@@ -580,7 +580,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements:
 - Ruby >= 2.7.2
-rubygems_version: 3.2.33
+rubygems_version: 3.2.3
 signing_key:
 specification_version: 4
 summary: 'REST services accelerator: Rails gem providing an easy, active-record-like