change_health 6.1.1 → 6.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c48e5a833ac4455745b3bc9a78e4f928a225a094cfc22b905495a00a6bd59a60
-  data.tar.gz: 0fffec97226198d5a9d778a4431ebad2ee4323f403ce2c5bf3f8f2b408f3cb68
+  metadata.gz: 5b3aeaa4ef2ca260c28862838a67cfb270d8350725af6aed958c519ba9371c3e
+  data.tar.gz: 0f27cc0e7f26fdb1a1fca6099dfcb06d903d202c725bd1c1ac3cebe3e3315d40
 SHA512:
-  metadata.gz: 4aaddd07aa48dad10b0fd70d6b75cbe0894d14f05b33ee4f0281826b5b4a676ad0160da7a07b6b0cd9c6f201b1bdf689d766904b248c5f601ea768bc03937b97
-  data.tar.gz: '0999b19cdb5eb21c0d1c5c647bef7d38748049d82a8cf67080d90235319e311ecf83e8237e755af664e00868d5b940355f1ca86b46a626cfb15bdd57c58bdc65'
+  metadata.gz: 49d4bbcf780a360836f34792728be99048f1c65988909793c5a0d01ab399aa6e28a044a913f9c813c903d1df5d9e487907b2a09f5f663a56246df262d334185d
+  data.tar.gz: 83af98e96baa90258668886c8ca92417670e43d629a2731af7de1223d19830a2d1a43e58aa862bacac97d667a70f98149b020539c9544ac7cf674a51f296b73a

data/CHANGELOG.md

@@ -5,6 +5,10 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+# [6.1.2] - 2025-07-02
+
+Updates ChangeHealth Response error message to return description if field is not found instead of previous behaviour returning nil.
+
 # [6.1.1] - 2025-01-30
 
 ### Bug Fix
@@ -780,6 +784,7 @@ Added the ability to hit professional claim submission API. For more details, se
 * Authentication
 * Configuration
 
+[6.1.2]: https://github.com/WeInfuse/change_health/compare/v6.1.1...v6.1.2
 [6.1.1]: https://github.com/WeInfuse/change_health/compare/v6.1.0...v6.1.1
 [6.1.0]: https://github.com/WeInfuse/change_health/compare/v6.0.0...v6.1.0
 [6.0.0]: https://github.com/WeInfuse/change_health/compare/v5.18.0...v6.0.0

data/Gemfile

@@ -1,4 +1,16 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in change_health.gemspec
 gemspec
+
+gem 'bundler', '>=1', '<3'
+gem 'byebug', '~> 11'
+gem 'minitest', '~> 5.0'
+gem 'rake', '~> 13.0'
+gem 'rubocop'
+gem 'rubocop-minitest'
+gem 'rubocop-performance'
+gem 'webmock', '~> 3.1'
+gem 'yard', '~> 0.9'

data/Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'bundler/gem_tasks'
 require 'rake/testtask'
 

data/change_health.gemspec

@@ -1,8 +1,12 @@
+# frozen_string_literal: true
+
 lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'change_health/version'
 
 Gem::Specification.new do |spec|
+  spec.required_ruby_version = '>= 2.7.8'
+
   spec.name          = 'change_health'
   spec.version       = ChangeHealth::VERSION
   spec.authors       = ['Mike Crockett']
@@ -14,13 +18,14 @@ Gem::Specification.new do |spec|
   # Prevent pushing this gem to RubyGems.org.
   if spec.respond_to?(:metadata)
     spec.metadata['allowed_push_host'] = 'https://rubygems.org'
+    spec.metadata['rubygems_mfa_required'] = 'true'
   else
     raise 'RubyGems 2.0 or newer is required to protect against ' \
-      'public gem pushes.'
+          'public gem pushes.'
   end
 
   spec.files = `git ls-files -z`.split("\x0").reject do |f|
-    f.match?(%r{^(test|spec|features|bin|helpers|)/}) || f.match?(%r{^(\.[[:alnum:]]+)})
+    f.match?(%r{^(test|spec|features|bin|helpers|)/}) || f.match?(/^(\.[[:alnum:]]+)/)
   end
 
   spec.bindir        = 'exe'
@@ -28,12 +33,6 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
   spec.licenses      = ['MIT']
 
-  spec.add_dependency 'httparty', '~> 0.17'
   spec.add_dependency 'hashie', '~> 3.5'
-  spec.add_development_dependency 'bundler', '>=1', '<3'
-  spec.add_development_dependency 'byebug', '~> 11'
-  spec.add_development_dependency 'minitest', '~> 5.0'
-  spec.add_development_dependency 'rake', '~> 13.0'
-  spec.add_development_dependency 'webmock', '~> 3.1'
-  spec.add_development_dependency 'yard', '~> 0.9'
+  spec.add_dependency 'httparty', '~> 0.17'
 end

data/lib/change_health/authentication.rb

@@ -1,8 +1,10 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   class Authentication
     attr_accessor :response
 
-    AUTH_ENDPOINT    = '/apip/auth/v2/token'.freeze
+    AUTH_ENDPOINT = '/apip/auth/v2/token'
 
     def initialize
       @response     = nil
@@ -10,16 +12,20 @@ module ChangeHealth
     end
 
     def authenticate(base_uri: nil)
-      if (self.expires?)
+      if expires?
         base_uri ||= Connection.base_uri
         request = {
-          body: { client_id: ChangeHealth.configuration.client_id, client_secret: ChangeHealth.configuration.client_secret, grant_type: ChangeHealth.configuration.grant_type },
+          body: {
+            client_id: ChangeHealth.configuration.client_id,
+            client_secret: ChangeHealth.configuration.client_secret,
+            grant_type: ChangeHealth.configuration.grant_type
+          },
           endpoint: AUTH_ENDPOINT
         }
 
         response = Connection.new.request(**request, auth: false, base_uri: base_uri)
 
-        if (false == response.ok?)
+        if false == response.ok?
           @response = nil
           raise ChangeHealthException.from_response(response, msg: 'Authentication')
         else
@@ -28,36 +34,34 @@ module ChangeHealth
         end
       end
 
-      return self
+      self
     end
 
     def access_token
-      return @response['access_token'] if @response
+      @response['access_token'] if @response
     end
 
     def expires_in
-      return @response['expires_in'].to_i if @response
+      @response['expires_in'].to_i if @response
     end
 
     def token_type
-      return @response['token_type'] if @response
+      @response['token_type'] if @response
     end
 
     def expiry
-      @request_time + self.expires_in if @request_time && self.expires_in
+      @request_time + expires_in if @request_time && expires_in
     end
 
     def expires?(seconds_from_now = 60)
-      if (self.expiry)
-        return self.expiry.utc <= (Time.now + seconds_from_now).utc
-      else
-        return true
-      end
+      return expiry.utc <= (Time.now + seconds_from_now).utc if expiry
+
+      true
     end
 
     def access_header
-      return {
-        'Authorization' => "Bearer #{self.access_token}",
+      {
+        'Authorization' => "Bearer #{access_token}"
       }
     end
 

data/lib/change_health/change_health_exception.rb

@@ -1,22 +1,25 @@
+# frozen_string_literal: true
+
 module ChangeHealth
-  class ChangeHealthException < Exception
+  class ChangeHealthException < StandardError
     def self.from_response(response, msg: nil)
-      exception_msg = "Failed #{msg}:"
-      exception_msg << " HTTP code: #{response.code} MSG: "
+      error_msg = nil
 
       begin
-        error_response = response.parsed_response
-
-        if (error_response.is_a?(Hash) && error_response.include?("error_description"))
-          exception_msg << error_response["error_description"]
-        else
-          exception_msg << error_response
-        end
+        error_msg = parse_error_msg(response.parsed_response)
       rescue JSON::ParserError
-        exception_msg << response.body
+        error_msg = response.body
       end
 
-      return ChangeHealthException.new(exception_msg)
+      ChangeHealthException.new("Failed #{msg}: HTTP code: #{response&.code} MSG: #{error_msg}")
+    end
+
+    def self.parse_error_msg(error_response)
+      if error_response.is_a?(Hash) && error_response.include?('error_description')
+        error_response['error_description']
+      else
+        error_response
+      end
     end
   end
 end

data/lib/change_health/connection.rb

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   class Connection
-    URI_BUILDER = ->(host) { "https://#{host}apigw.changehealthcare.com/".freeze }
+    URI_BUILDER = ->(host) { "https://#{host}apigw.changehealthcare.com/" }
 
     QA_ENDPOINT   = URI_BUILDER.call('sandbox.')
     PROD_ENDPOINT = URI_BUILDER.call('')
@@ -13,6 +15,7 @@ module ChangeHealth
 
     format :json
 
+    # rubocop:disable Metrics/ParameterLists
     def request(
       endpoint:,
       query: nil,
@@ -30,6 +33,7 @@ module ChangeHealth
 
       self.class.send(verb.to_s, endpoint, query: query, body: body, headers: headers, base_uri: base_uri)
     end
+    # rubocop:enable Metrics/ParameterLists
 
     def self.endpoint_for(klass, default_endpoint: nil)
       endpoint_options = ChangeHealth.configuration.endpoints || {}

data/lib/change_health/extensions.rb

@@ -1,14 +1,16 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Extensions
     module InNetworkW
       def in_network
-        self.where(inPlanNetworkIndicatorCode: 'Y') + self.where(inPlanNetworkIndicatorCode: 'W')
+        where(inPlanNetworkIndicatorCode: 'Y') + where(inPlanNetworkIndicatorCode: 'W')
       end
     end
 
     module InNetworkMissing
       def in_network
-        self.where_not(inPlanNetworkIndicatorCode: 'N')
+        where_not(inPlanNetworkIndicatorCode: 'N')
       end
     end
 
@@ -32,7 +34,7 @@ module ChangeHealth
 
     module IndividualsAllNonFamily
       def individuals
-        self.where_not(coverageLevelCode: 'FAM')
+        where_not(coverageLevelCode: 'FAM')
       end
     end
   end

data/lib/change_health/models/claim/submission/address.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/billing_pay_to_address_name.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/claim_code_information.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/claim_date_information.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/claim_information.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/claim_supplemental_information.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/contact_information.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/dependent.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/diagnosis.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/drug_identification.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/institutional_service.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/line_adjudication_information.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/other_payer_name.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/other_subscriber_information.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/other_subscriber_name.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/professional_service.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/provider.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/receiver.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/report_information.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/service_facility_location.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/service_line.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/submitter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/claim/submission/subscriber.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Claim

data/lib/change_health/models/eligibility/encounter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Eligibility
@@ -8,9 +10,9 @@ module ChangeHealth
         property :endDateOfService, from: :end_date_of_service, required: false
         property :serviceTypeCodes, from: :service_type_codes, required: false
 
-        alias_method :dateRange?, :dateRange
-        alias_method :date_range?, :dateRange
-        alias_method :service_type_codes, :serviceTypeCodes
+        alias dateRange? dateRange
+        alias date_range? dateRange
+        alias service_type_codes serviceTypeCodes
 
         def add_service_type_code(code)
           self[:serviceTypeCodes] ||= []

data/lib/change_health/models/eligibility/provider.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Eligibility
@@ -16,7 +18,7 @@ module ChangeHealth
         property :serviceProviderNumber, from: :service_provider_number, required: false
         property :taxId, from: :tax_id, required: false
 
-        alias_method :name?, :name
+        alias name? name
       end
     end
   end

data/lib/change_health/models/eligibility/subscriber.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     module Eligibility
@@ -14,7 +16,8 @@ module ChangeHealth
         property :idCard, from: :id_card, required: false
         property :idCardIssueDate, from: :id_card_issued_date, required: false
         property :lastName, from: :last_name, required: false
-        property :medicaidRecipientIdentificationNumber, from: :medicaid_recipient_identification_number, required: false
+        property :medicaidRecipientIdentificationNumber, from: :medicaid_recipient_identification_number,
+                                                         required: false
         property :memberId, from: :member_id, required: false
         property :providerCode, from: :provider_code, required: false
         property :providerIdentifier, from: :provider_identification, required: false

data/lib/change_health/models/model.rb

@@ -1,8 +1,10 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
-    DATE_FORMAT = '%Y%m%d'.freeze
-    DATE_HOUR_FORMAT = '%Y%m%d%H%M'.freeze
-    HOUR_FORMAT = '%H%M'.freeze
+    DATE_FORMAT = '%Y%m%d'
+    DATE_HOUR_FORMAT = '%Y%m%d%H%M'
+    HOUR_FORMAT = '%H%M'
 
     # Deprecated should use date_formatter instead
     DATE_FORMATTER = lambda { |date|
@@ -21,6 +23,7 @@ module ChangeHealth
       time_formatter(hour, HOUR_FORMAT)
     end
 
+    # rubocop:disable Lint/SuppressedException
     def self.time_formatter(time, format)
       begin
         time = Time.parse(time) if time.is_a?(String)
@@ -31,7 +34,9 @@ module ChangeHealth
 
       time
     end
+    # rubocop:enable Lint/SuppressedException
 
+    # rubocop:disable Lint/SuppressedException
     PARSE_DATE = lambda { |d|
       begin
         d = Date.strptime(d.tr('-', ''), ChangeHealth::Models::DATE_FORMAT)
@@ -40,7 +45,9 @@ module ChangeHealth
 
       d
     }
+    # rubocop:enable Lint/SuppressedException
 
+    # rubocop:disable Lint/SuppressedException
     POSTAL_CODE_FORMATTER = lambda { |postal_code|
       begin
         formatted_postal_code = postal_code&.to_s&.tr('-', '')
@@ -48,14 +55,16 @@ module ChangeHealth
       end
       formatted_postal_code || postal_code
     }
+    # rubocop:enable Lint/SuppressedException
 
-    CONTROL_NUMBER = -> { '%09d' % rand(1_000_000_000) }
+    CONTROL_NUMBER = -> { format('%09d', rand(1_000_000_000)) }
 
     class Model < Hashie::Trash
       def to_h
         self.class.hashify(self)
       end
 
+      # rubocop:disable Style/MapToHash
       def self.hashify(model)
         model.map do |key, value|
           formatted_value = case value
@@ -75,17 +84,20 @@ module ChangeHealth
           [key, formatted_value]
         end.to_h
       end
+      # rubocop:enable Style/MapToHash
 
       def self.format_value(key, value)
         return nil if value == ''
 
-        return ChangeHealth::Models.date_hour_formatter(value) if key.to_s.downcase.include?('dateandhour')
+        downcased_key = key.to_s.downcase
+
+        return ChangeHealth::Models.date_hour_formatter(value) if downcased_key.include?('dateandhour')
 
-        return ChangeHealth::Models.hour_formatter(value) if key.to_s.downcase.include?('hour')
+        return ChangeHealth::Models.hour_formatter(value) if downcased_key.include?('hour')
 
-        return ChangeHealth::Models.date_formatter(value) if key.to_s.downcase.include?('date')
+        return ChangeHealth::Models.date_formatter(value) if downcased_key.include?('date')
 
-        return ChangeHealth::Models::POSTAL_CODE_FORMATTER.call(value) if key.to_s.downcase.include?('postalcode')
+        return ChangeHealth::Models::POSTAL_CODE_FORMATTER.call(value) if downcased_key.include?('postalcode')
 
         value
       end

data/lib/change_health/models/trading_partner/trading_partner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Models
     class TradingPartner < Hashie::Trash

data/lib/change_health/request/eligibility.rb

@@ -1,8 +1,10 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Request
     class Eligibility < Hashie::Trash
-      ENDPOINT = '/medicalnetwork/eligibility/v3'.freeze
-      HEALTH_CHECK_ENDPOINT = ENDPOINT + '/healthcheck'.freeze
+      ENDPOINT = '/medicalnetwork/eligibility/v3'
+      HEALTH_CHECK_ENDPOINT = "#{ENDPOINT}/healthcheck"
 
       property :controlNumber, from: :control_number, required: true, default: ChangeHealth::Models::CONTROL_NUMBER
       property :dependents, required: false
@@ -16,8 +18,8 @@ module ChangeHealth
       property :tradingPartnerId, from: :trading_partner_id, required: false
       property :tradingPartnerServiceId, from: :trading_partner_service_id, required: false
 
-      alias_method :partnerId?, :partnerId
-      alias_method :partner_id?, :partnerId
+      alias partnerId? partnerId
+      alias partner_id? partnerId
 
       def add_dependent(dependent)
         self[:dependents] ||= []
@@ -27,7 +29,8 @@ module ChangeHealth
       def query
         endpoint = ChangeHealth::Connection.endpoint_for(self.class)
 
-        ChangeHealth::Response::EligibilityData.new(response: ChangeHealth::Connection.new.request(endpoint: endpoint, body: self.to_h))
+        ChangeHealth::Response::EligibilityData.new(response: ChangeHealth::Connection.new.request(endpoint: endpoint,
+                                                                                                   body: to_h))
       end
 
       def self.health_check
@@ -35,7 +38,7 @@ module ChangeHealth
       end
 
       def self.ping
-        self.health_check
+        health_check
       end
     end
   end

data/lib/change_health/request/report.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 module ChangeHealth
   module Request
     module Claim
       class Report
-        ENDPOINT = '/medicalnetwork/reports/v2'.freeze
-        HEALTH_CHECK_ENDPOINT = ENDPOINT + '/healthcheck'.freeze
+        ENDPOINT = '/medicalnetwork/reports/v2'
+        HEALTH_CHECK_ENDPOINT = "#{ENDPOINT}/healthcheck"
 
         def self.report_list(headers: nil, more_url: nil, base_uri: nil, endpoint: nil, auth_headers: nil)
           endpoint ||= ChangeHealth::Connection.endpoint_for(self)
@@ -18,6 +20,10 @@ module ChangeHealth
           ))
         end
 
+        # rubocop:disable Metrics/CyclomaticComplexity
+        # rubocop:disable Metrics/MethodLength
+        # rubocop:disable Metrics/PerceivedComplexity
+        # rubocop:disable Metrics/ParameterLists
         def self.get_report(
           report_name,
           as_json_report: true,
@@ -67,6 +73,10 @@ module ChangeHealth
                    response: response)
           end
         end
+        # rubocop:enable Metrics/CyclomaticComplexity
+        # rubocop:enable Metrics/MethodLength
+        # rubocop:enable Metrics/PerceivedComplexity
+        # rubocop:enable Metrics/ParameterLists
 
         def self.delete_report(report_name, headers: nil, base_uri: nil, endpoint: nil, auth_headers: nil)
           return if report_name.nil? || report_name.empty?