usps-imis-api 0.6.14 → 0.6.15

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 05b1b663fab7b4f81c95f105c0c7f667bc2f5b9855b5fd48e5f384ce0adba8dc
-  data.tar.gz: '02529c44a577d5e239b3bd306cec0f9f91e53413ef3d921c5564352e8dca3d5f'
+  metadata.gz: 1ece5a19e925e1ef85b9e54b1d8fe7139c4015b0a1fe7ddcb602e172fbb50cab
+  data.tar.gz: 723eb4ccbf3b0c86c3f309c83d32d2b892420c688fd1f8aa2bfedcc47d61d2a9
 SHA512:
-  metadata.gz: e81f01c677aa5a22b1c7c5cfeb6e3119cc1f4387dc2076fec0001032879378f5be3cd562c3ccdd689a5e26e7becb4381a891249c175b025014d78638a36dd03e
-  data.tar.gz: 3e3f53303d8316f814cefffb3473ccb8e91c5fbd5146bc94b1701532c006fbf8b0ab905a156bd880e3fe6a2f1c2c4f79528c591df8b6ad6e26cb8cee7ab88346
+  metadata.gz: fbca9aeda6a7a4e7fc6701c2767da349ab49e6b9126b36ea0a6225b4c18b2c7970dc22390ddabfbf571411d76b86ab2b37bbde79f471f4b5a3ceba9d271fdd5c
+  data.tar.gz: 3b09c75a050ad32e8077d9bc07772d91bb37a7e4628734c0159cb05a206f7f9128059532e22e90f014f07041f2ac87d0d37c2d7878845069c1220ec221cb4e69

data/.rubocop.yml

@@ -37,6 +37,8 @@ Lint/UnusedMethodArgument:
   Enabled: true
 Lint/UselessAssignment:
   Enabled: true
+Lint/ItWithoutArgumentsInBlock:
+  Enabled: false
 
 Metrics/MethodLength:
   Enabled: true

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    usps-imis-api (0.6.14)
+    usps-imis-api (0.6.15)
       activesupport (~> 8.0)
 
 GEM

data/Readme.md

@@ -237,7 +237,7 @@ api.with(31092) do
   # These requests are identical:
 
   on('ABC_ASC_Individual_Demog') do
-    get['Properties']['$values'].find { |hash| hash['Name'] == 'TotMMS' }['Value']['$value']
+    get['Properties']['$values'].find { it['Name'] == 'TotMMS' }['Value']['$value']
   end
 
   on('ABC_ASC_Individual_Demog') { get_field('TotMMS') }

data/lib/usps/imis/business_object.rb

@@ -49,7 +49,7 @@ module Usps
       #
       def get_field(name)
         values = get['Properties']['$values']
-        value = values.find { |hash| hash['Name'] == name }['Value']
+        value = values.find { it['Name'] == name }['Value']
 
         value.is_a?(String) ? value : value['$value']
       end
@@ -69,6 +69,8 @@ module Usps
 
       # Update a business object for the current member
       #
+      # Any properties not included will be left unmodified
+      #
       # @param body [Hash] Full raw API object data
       #
       # @return [Hash] Response data from the API
@@ -133,22 +135,21 @@ module Usps
         existing = get
 
         JSON.parse(JSON.dump(existing)).tap do |updated|
-          # Wipe the array on the duped object
-          #
-          # The first property is always the iMIS ID again
-          #
-          updated['Properties']['$values'] = []
+          # Preserve the iMIS ID, as well as the Ordinal (if present)
+          updated['Properties']['$values'], properties =
+            existing['Properties']['$values'].partition { %w[ID Ordinal].include?(it['Name']) }
 
           # Iterate through all existing fields
-          existing['Properties']['$values'].each do |value|
-            if fields.keys.include?(value['Name'])
-              # Strings are not wrapped in the type definition structure
-              new_value = fields[value['Name']]
-              if new_value.is_a?(String)
-                value['Value'] = new_value
-              else
-                value['Value']['$value'] = new_value
-              end
+          properties.each do |value|
+            # Skip unmodified fields
+            next unless fields.keys.include?(value['Name'])
+
+            # Strings are not wrapped in the type definition structure
+            new_value = fields[value['Name']]
+            if new_value.is_a?(String)
+              value['Value'] = new_value
+            else
+              value['Value']['$value'] = new_value
             end
 
             # Add the completed field with the updated value

data/lib/usps/imis/error/response_error.rb

@@ -43,7 +43,7 @@ module Usps
         # @return [Hash]
         #
         def bugsnag_meta_data
-          base_metadata.tap { |m| m[:api].merge!(metadata) }
+          base_metadata.tap { it[:api].merge!(metadata) }
         end
 
         # Auto-formatted exception message, based on the provided API response

data/lib/usps/imis/requests.rb

@@ -20,7 +20,7 @@ module Usps
       #
       def authorize(request)
         authenticate if token_expiration < Time.now
-        request.tap { |r| r.add_field('Authorization', "Bearer #{token}") }
+        request.tap { it.add_field('Authorization', "Bearer #{token}") }
       end
 
       def submit(uri, request)

data/lib/usps/imis/version.rb

@@ -2,6 +2,6 @@
 
 module Usps
   module Imis
-    VERSION = '0.6.14'
+    VERSION = '0.6.15'
   end
 end

data/spec/lib/usps/imis/business_object_spec.rb

@@ -10,7 +10,7 @@ describe Usps::Imis::BusinessObject do
     {
       'Properties' => {
         '$values' => [
-          { 'Name' => 'Stub iMIS ID', 'Value' => { '$value' => '31092' } },
+          { 'Name' => 'ID', 'Value' => { '$value' => '31092' } },
           { 'Name' => 'Stub Integer', 'Value' => { '$value' => 43 } },
           { 'Name' => 'Stub String', 'Value' => 'other' }
         ]
@@ -22,7 +22,7 @@ describe Usps::Imis::BusinessObject do
     allow(business_object).to receive(:get).and_return({
       'Properties' => {
         '$values' => [
-          { 'Name' => 'Stub iMIS ID', 'Value' => { '$value' => '31092' } },
+          { 'Name' => 'ID', 'Value' => { '$value' => '31092' } },
           { 'Name' => 'Stub Integer', 'Value' => { '$value' => 42 } },
           { 'Name' => 'Stub String', 'Value' => 'something' }
         ]

data/spec/lib/usps/imis/config_spec.rb

@@ -24,7 +24,7 @@ describe Usps::Imis::Config do
       subject { config.environment }
 
       let(:config) do
-        described_class.new { |c| c.environment = 'test' }
+        described_class.new { it.environment = 'test' }
       end
 
       it { is_expected.to be_test }

data/spec/lib/usps/imis/panel/education_spec.rb

@@ -14,7 +14,6 @@ describe Usps::Imis::Panel::Education do
         source: 'Online Exams System',
         code: 'MN',
         type_code: 'CRS'
-
       }
     end
 

data/spec/lib/usps/imis/panel/vsc_spec.rb

@@ -29,7 +29,7 @@ describe Usps::Imis::Panel::Vsc do
     ordinal = new_record['Identity']['IdentityElements']['$values'][1]
 
     update_result = vsc.update(details.merge(count: 43, ordinal:))
-    updated = update_result['Properties']['$values'].find { |v| v['Name'] == 'Quantity' }
+    updated = update_result['Properties']['$values'].find { it['Name'] == 'Quantity' }
     expect(updated['Value']['$value']).to eq(43)
 
     expect(vsc.destroy(ordinal)).to eq('')

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: usps-imis-api
 version: !ruby/object:Gem::Version
-  version: 0.6.14
+  version: 0.6.15
 platform: ruby
 authors:
 - Julian Fiander