Removing the use of MRN in the parametrize function; separate branch or the sake of readability

Author: Dtphelan1

src/extractors/BaseFHIRExtractor.js

@@ -19,8 +19,8 @@ class BaseFHIRExtractor extends Extractor {
   /* eslint-disable class-methods-use-this */
   // Use context to get PatientId by default; common need across almost all extractors
   // NOTE: Async because other extractors that extend this may need to make async lookups in the future
-  async parametrizeArgsForFHIRModule({ mrn, context }) {
-    const patient = getPatientFromContext(mrn, context);
+  async parametrizeArgsForFHIRModule({ context }) {
+    const patient = getPatientFromContext(context);
     return { patient: patient.id };
   }
   /* eslint-enable class-methods-use-this */

src/extractors/FHIRAdverseEventExtractor.js

@@ -10,8 +10,8 @@ class FHIRAdverseEventExtractor extends BaseFHIRExtractor {
   }
 
   // In addition to default parametrization, add study if specified
-  async parametrizeArgsForFHIRModule({ mrn, context }) {
-    const paramsWithID = await super.parametrizeArgsForFHIRModule({ mrn, context });
+  async parametrizeArgsForFHIRModule({ context }) {
+    const paramsWithID = await super.parametrizeArgsForFHIRModule({ context });
     // The patient is referenced in the 'subject' field of an AdverseEvent
     paramsWithID.subject = paramsWithID.patient;
     delete paramsWithID.patient;

src/extractors/FHIRAllergyIntoleranceExtractor.js

@@ -10,8 +10,8 @@ class FHIRAllergyIntoleranceExtractor extends BaseFHIRExtractor {
   }
 
   // In addition to default parametrization, add clinical status
-  async parametrizeArgsForFHIRModule({ mrn, context }) {
-    const paramsWithID = await super.parametrizeArgsForFHIRModule({ mrn, context });
+  async parametrizeArgsForFHIRModule({ context }) {
+    const paramsWithID = await super.parametrizeArgsForFHIRModule({ context });
     return {
       ...paramsWithID,
       'clinical-status': this.clinicalStatus,

src/extractors/FHIRConditionExtractor.js

@@ -10,8 +10,8 @@ class FHIRConditionExtractor extends BaseFHIRExtractor {
   }
 
   // In addition to default parametrization, add category
-  async parametrizeArgsForFHIRModule({ mrn, context }) {
-    const paramsWithID = await super.parametrizeArgsForFHIRModule({ mrn, context });
+  async parametrizeArgsForFHIRModule({ context }) {
+    const paramsWithID = await super.parametrizeArgsForFHIRModule({ context });
     return {
       ...paramsWithID,
       category: this.category,

src/extractors/FHIRMedicationRequestExtractor.js

@@ -10,8 +10,8 @@ class FHIRMedicationRequestExtractor extends BaseFHIRExtractor {
   }
 
   // In addition to default parametrization, add status if specified
-  async parametrizeArgsForFHIRModule({ mrn, context }) {
-    const paramsWithID = await super.parametrizeArgsForFHIRModule({ mrn, context });
+  async parametrizeArgsForFHIRModule({ context }) {
+    const paramsWithID = await super.parametrizeArgsForFHIRModule({ context });
     // Only add status to parameters if it has been specified
     return {
       ...paramsWithID,

src/extractors/FHIRObservationExtractor.js

@@ -10,8 +10,8 @@ class FHIRObservationExtractor extends BaseFHIRExtractor {
   }
 
   // In addition to default parametrization, add category
-  async parametrizeArgsForFHIRModule({ mrn, context }) {
-    const paramsWithID = await super.parametrizeArgsForFHIRModule({ mrn, context });
+  async parametrizeArgsForFHIRModule({ context }) {
+    const paramsWithID = await super.parametrizeArgsForFHIRModule({ context });
     return {
       ...paramsWithID,
       category: this.category,

src/helpers/contextUtils.js

@@ -7,7 +7,7 @@ const { getBundleEntriesByResourceType, getBundleResourcesByType } = require('./
 * @param {Object} context - Context object consisting of a FHIR Bundle
 * @return {Object} The first Patient resource found in the bundle
 */
-function getPatientFromContext(mrn, context) {
+function getPatientFromContext(context) {
   logger.debug('Getting patient from context');
   const patientResourceInContext = getBundleResourcesByType(context, 'Patient', {}, true);
   if (!patientResourceInContext) {

test/extractors/BaseFHIRExtractor.test.js

@@ -43,15 +43,15 @@ describe('BaseFhirExtractor', () => {
 
   test('parametrizeArgsForFHIRModule parses data off of context if available', async () => {
     baseFHIRModuleSearchSpy.mockClear();
-    const paramsBasedOnContext = await baseFHIRExtractor.parametrizeArgsForFHIRModule({ mrn: MOCK_PATIENT_MRN, context: MOCK_CONTEXT });
+    const paramsBasedOnContext = await baseFHIRExtractor.parametrizeArgsForFHIRModule({ context: MOCK_CONTEXT });
     expect(baseFHIRModuleSearchSpy).not.toHaveBeenCalled();
     expect(paramsBasedOnContext).toHaveProperty('patient');
     expect(paramsBasedOnContext.patient).toEqual(MOCK_CONTEXT.entry[0].resource.id);
   });
 
   test('parametrizeArgsForFHIRModule throws an error if context has no relevant ID', async () => {
     baseFHIRModuleSearchSpy.mockClear();
-    await expect(baseFHIRExtractor.parametrizeArgsForFHIRModule({ mrn: MOCK_PATIENT_MRN, context: {} })).rejects.toThrow();
+    await expect(baseFHIRExtractor.parametrizeArgsForFHIRModule({ context: {} })).rejects.toThrow();
     expect(baseFHIRModuleSearchSpy).not.toHaveBeenCalled();
   });
 

test/extractors/FHIRAdverseEventExtractor.test.js

@@ -36,19 +36,19 @@ describe('FHIRAdverseEventExtractor', () => {
 
   describe('parametrizeArgsForFHIRModule', () => {
     test('should not add study when not set to param values', async () => {
-      const params = await extractor.parametrizeArgsForFHIRModule({ mrn: MOCK_MRN, context: MOCK_CONTEXT });
+      const params = await extractor.parametrizeArgsForFHIRModule({ context: MOCK_CONTEXT });
       expect(params).not.toHaveProperty('study');
     });
 
     describe('pass in optional study parameter', () => {
       test('should add study when set to param values', async () => {
-        const params = await extractorWithStudy.parametrizeArgsForFHIRModule({ mrn: MOCK_MRN, context: MOCK_CONTEXT });
+        const params = await extractorWithStudy.parametrizeArgsForFHIRModule({ context: MOCK_CONTEXT });
         expect(params).toHaveProperty('study');
         expect(params.study).toEqual(extractorWithStudy.study);
       });
 
       test('should delete patient after its value is moved to subject', async () => {
-        const params = await extractorWithStudy.parametrizeArgsForFHIRModule({ mrn: MOCK_MRN, context: MOCK_CONTEXT });
+        const params = await extractorWithStudy.parametrizeArgsForFHIRModule({ context: MOCK_CONTEXT });
         expect(params).not.toHaveProperty('patient');
       });
     });

test/extractors/FHIRAllergyIntoleranceExtractor.test.js

@@ -37,7 +37,7 @@ describe('FHIRAllergyIntoleranceExtractor', () => {
 
   describe('parametrizeArgsForFHIRModule', () => {
     test('should add category to param values', async () => {
-      const params = await extractor.parametrizeArgsForFHIRModule({ mrn: MOCK_MRN, context: MOCK_CONTEXT });
+      const params = await extractor.parametrizeArgsForFHIRModule({ context: MOCK_CONTEXT });
       expect(params).toHaveProperty('clinical-status');
       expect(params['clinical-status']).toEqual(baseClinicalStatus);
     });