his_emr_api_lab 1.1.13 → 1.1.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f7b7bee82670948e0315bde7a3d0f9cab91476bcb27646ec7c234e0c96d11bc9
-  data.tar.gz: 127d70d6a044a8ddc5dcc25ed2460d1fcb463d8f27f3817b54fca7d5ada6d88a
+  metadata.gz: f655e84314f6a25a8ecce1668132e93bbd1a175efd35cb832792b94cc9d33e42
+  data.tar.gz: 298f226847e19dd868e7caf487b00b49dc306bed4ca2c2a0f2c07765278598e5
 SHA512:
-  metadata.gz: 421db4bdc5cb32b994d9329c87ebe8c79eae681765abe980bd87c6fab83c70de74961d78845d2602af31aea018b480ddcad9f97de4dfa01a8cd975c8c9a49db8
-  data.tar.gz: 4d1c1df19df7b297e66d44055bcbe88b410f25e744a56f0bb46a99f8fd06af9e25ad332499cbd34e8ea10f7879c14455f8b16d376e24c7a50bf7518156ce3646
+  metadata.gz: 132e7e46b9c9c6468651b1ebde2ce182852cc17992a54928b49b3605191c53f81ab324e21657d1b421ac317d48c51cd2a73c24d11f94cd5ca4f0d47714c04d9b
+  data.tar.gz: c49d8d971697158bf1646027935ed1e95cb825a161fef18a9d658affb7e1493b24b0f70748d8c61d4a9e69d81e00d5d3f3395eec7ea4c4189401b992a27d97f5

data/app/jobs/lab/push_order_job.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+module Lab
+  ##
+  # Push an order to LIMS.
+  class PushOrderJob < ApplicationJob
+    def perform(order_id)
+      push_worker = Lab::Lims::PushWorker.new(Lab::Lims::ApiFactory.create_api)
+      push_worker.push_order_by_id(order_id)
+    end
+  end
+end

data/app/jobs/lab/update_patient_orders_job.rb

@@ -20,8 +20,7 @@ module Lab
           break false
         end
 
-        lims_api = Lab::Lims::Api::RestApi.new(Lab::Lims::Config.rest_api)
-        worker = Lab::Lims::PullWorker.new(lims_api)
+        worker = Lab::Lims::PullWorker.new(Lab::Lims::ApiFactory.create_api)
         worker.pull_orders(patient_id: patient_id)
 
         true

data/app/jobs/lab/void_order_job.rb

@@ -10,8 +10,7 @@ module Lab
       User.current = Lab::Lims::Utils.lab_user
       Location.current = Location.find_by_name('ART clinic')
 
-      lims_api = Lab::Lims::Api::RestApi.new
-      worker = Lab::Lims::Worker.new(lims_api)
+      worker = Lab::Lims::PushWorker.new(Lab::Lims::ApiFactory.create_api)
       worker.push_order(Lab::LabOrder.unscoped.find(order_id))
     end
   end

data/app/models/lab/lab_order.rb

@@ -42,14 +42,21 @@ module Lab
     end
 
     scope :drawn, -> { where.not(concept_id: ConceptName.where(name: 'Unknown').select(:concept_id)) }
-
     scope :not_drawn, -> { where(concept_id: ConceptName.where(name: 'Unknown').select(:concept_id)) }
 
+    after_save :queue_lims_push
+
     def self.prefetch_relationships
       includes(:reason_for_test,
                :requesting_clinician,
                :target_lab,
                tests: [:result])
     end
+
+    private
+
+    def queue_lims_push
+      Lab::PushOrderJob.perform_later(order_id)
+    end
   end
 end

data/app/serializers/lab/lab_order_serializer.rb

@@ -6,7 +6,7 @@ module Lab
       tests ||= order.voided == 1 ? voided_tests(order) : order.tests
       requesting_clinician ||= order.requesting_clinician
       reason_for_test ||= order.reason_for_test
-      target_lab = target_lab || order.target_lab || Location.current_health_center.name
+      target_lab = target_lab&.value_text || order.target_lab&.value_text || Location.current_health_center&.name
 
       ActiveSupport::HashWithIndifferentAccess.new(
         {
@@ -21,7 +21,7 @@ module Lab
             name: concept_name(order.concept_id)
           },
           requesting_clinician: requesting_clinician&.value_text,
-          target_lab: target_lab&.value_text,
+          target_lab: target_lab,
           reason_for_test: {
             concept_id: reason_for_test&.value_coded,
             name: concept_name(reason_for_test&.value_coded)

data/app/services/lab/lims/api/blackhole_api.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module Lab
+  module Lims
+    module Api
+      ##
+      # A LIMS Api wrappper that does nothing really.
+      #
+      # Primarily meant as a dummy for testing environments.
+      class BlackholeApi
+        def create_order(order_dto); end
+
+        def update_order(order_dto); end
+
+        def void_order(order_dto); end
+
+        def consume_orders(&_block); end
+      end
+    end
+  end
+end

data/app/services/lab/lims/api_factory.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module Lab
+  module Lims
+    ##
+    # Creates LIMS Apis based on current configuration
+    module ApiFactory
+      def self.create_api
+        return Lab::Lims::Api::BlackholeApi.new if Rails.env.casecmp?('test')
+
+        case Lab::Lims::Config.preferred_api
+        when /rest/i then Lab::Lims::Api::RestApi.new(Lab::Lims::Config.rest_api)
+        when /couchdb/ then Lab::Lims::Api::CouchDbApi.new(config: Lab::Lims::Config.couchdb_api)
+        else raise "Invalid lims_api configuration: #{Lab::Lims::Config.preferred_api}"
+        end
+      end
+    end
+  end
+end

data/app/services/lab/lims/migrator.rb

@@ -46,9 +46,20 @@ require_relative 'utils'
 
 module Lab
   module Lims
+    ##
+    # Tools for performing a bulk import of data from LIMS' databases to local OpenMRS database.
+    #
+    # Migration sources supported:
+    #   - MySQL
+    #   - CouchDB
+    #
+    # The sources above can be changed by setting the environment various MIGRATION_SOURCE to
+    # either mysql or couchdb.
     module Migrator
       MAX_THREADS = ENV.fetch('MIGRATION_WORKERS', 6).to_i
 
+      ##
+      # A Lab::Lims::Api object that supports crawling of a LIMS CouchDB instance.
       class CouchDbMigratorApi < Lab::Lims::Api::CouchDbApi
         def initialize(*args, processes: 1, on_merge_processes: nil, **kwargs)
           super(*args, **kwargs)
@@ -91,6 +102,12 @@ module Lab
         end
       end
 
+      ##
+      # Extends the PullWorker to provide pause/resume capabilities.
+      #
+      # Migrations can be take a long time to complete, in cases where something
+      # went wrong you wouldn't to start all over. This worker thus saves
+      # progress and allows for the process to continue from whether it stopped.
       class MigrationWorker < PullWorker
         LOG_FILE_PATH = Utils::LIMS_LOG_PATH.join('migration-last-id.dat')
 

data/app/services/lab/lims/pull_worker.rb

@@ -2,6 +2,8 @@
 
 module Lab
   module Lims
+    ##
+    # Pulls orders from a Lims API object and saves them to the local database.
     class PullWorker
       attr_reader :lims_api
 
@@ -100,9 +102,7 @@ module Lab
                           .distinct(:patient_id)
                           .all
 
-        if patients.size > 1
-          raise DuplicateNHID, "Duplicate National Health ID: #{nhid}"
-        end
+        raise DuplicateNHID, "Duplicate National Health ID: #{nhid}" if patients.size > 1
 
         patients.first
       end
@@ -166,9 +166,7 @@ module Lab
       def create_order(patient, order_dto)
         logger.debug("Creating order ##{order_dto['_id']}")
         order = OrdersService.order_test(order_dto.to_order_service_params(patient_id: patient.patient_id))
-        unless order_dto['test_results'].empty?
-          update_results(order, order_dto['test_results'])
-        end
+        update_results(order, order_dto['test_results']) unless order_dto['test_results'].empty?
 
         order
       end
@@ -177,9 +175,7 @@ module Lab
         logger.debug("Updating order ##{order_dto['_id']}")
         order = OrdersService.update_order(order_id, order_dto.to_order_service_params(patient_id: patient.patient_id)
                                                               .merge(force_update: 'true'))
-        unless order_dto['test_results'].empty?
-          update_results(order, order_dto['test_results'])
-        end
+        update_results(order, order_dto['test_results']) unless order_dto['test_results'].empty?
 
         order
       end
@@ -283,9 +279,7 @@ module Lab
         mapping = Lab::LimsOrderMapping.find_by(lims_id: lims_id)
         return nil unless mapping
 
-        if Lab::LabOrder.where(order_id: mapping.order_id).exists?
-          return mapping
-        end
+        return mapping if Lab::LabOrder.where(order_id: mapping.order_id).exists?
 
         mapping.destroy
         nil

data/app/services/lab/lims/push_worker.rb

@@ -2,6 +2,8 @@
 
 module Lab
   module Lims
+    ##
+    # Pushes all local orders to a LIMS Api object.
     class PushWorker
       attr_reader :lims_api
 
@@ -33,7 +35,9 @@ module Lab
       end
 
       def push_order_by_id(order_id)
-        order = Lab::LabOrder.unscoped.find(order_id)
+        order = Lab::LabOrder.joins(order_type: { name: 'Lab' })
+                             .unscoped
+                             .find(order_id)
         push_order(order)
       end
 

data/app/services/lab/lims/worker.rb

@@ -79,11 +79,7 @@ module Lab
       end
 
       def self.lims_api
-        case Lims::Config.preferred_api
-        when /couchdb/i then Api::CouchDbApi.new(config: Lab::Lims::Config.couchdb)
-        when /rest/i then Api::RestApi.new(Lab::Lims::Config.rest_api)
-        else raise "Invalid LIMS API in application.yml, expected 'rest' or 'couchdb'"
-        end
+        Lab::Lims::ApiFactory.create_api
       end
     end
   end

data/lib/lab/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Lab
-  VERSION = '1.1.13'
+  VERSION = '1.1.14'
 end

data/lib/tasks/loaders/data/test-measures.csv

@@ -68,6 +68,7 @@
 "Hepatitis B Test","Hepatitis B"
 "Hepatitis C Test","Hepatitis C"
 "Rheumatoid Factor Test","Rheumatoid Factor"
+"CrAg","CrAg"
 "Cryptococcus Antigen Test","CrAg"
 "Anti Streptolysis O","ASO"
 "C-reactive protein","CRP"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: his_emr_api_lab
 version: !ruby/object:Gem::Version
-  version: 1.1.13
+  version: 1.1.14
 platform: ruby
 authors:
 - Elizabeth Glaser Pediatric Foundation Malawi
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-09-06 00:00:00.000000000 Z
+date: 2021-09-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: couchrest
@@ -248,6 +248,7 @@ files:
 - app/controllers/lab/test_types_controller.rb
 - app/controllers/lab/tests_controller.rb
 - app/jobs/lab/application_job.rb
+- app/jobs/lab/push_order_job.rb
 - app/jobs/lab/update_patient_orders_job.rb
 - app/jobs/lab/void_order_job.rb
 - app/mailers/lab/application_mailer.rb
@@ -265,10 +266,12 @@ files:
 - app/services/lab/accession_number_service.rb
 - app/services/lab/concepts_service.rb
 - app/services/lab/labelling_service/order_label.rb
+- app/services/lab/lims/api/blackhole_api.rb
 - app/services/lab/lims/api/couchdb_api.rb
 - app/services/lab/lims/api/mysql_api.rb
 - app/services/lab/lims/api/rest_api.rb
 - app/services/lab/lims/api/ws_api.rb
+- app/services/lab/lims/api_factory.rb
 - app/services/lab/lims/config.rb
 - app/services/lab/lims/exceptions.rb
 - app/services/lab/lims/migrator.rb