formatting changes, fixing some casing errors, adding tests for csvValidator and csvModule

Author: dmendelowitz

src/extractors/CSVPatientExtractor.js

@@ -23,7 +23,9 @@ function joinAndReformatData(patientData) {
     ethnicity,
     language,
     addressline: addressLine,
-    city, state, zip,
+    city,
+    state,
+    zip,
   } = patientData;
 
   if (!mrn) {

src/extractors/CSVStagingExtractor.js

@@ -10,7 +10,10 @@ function formatTNMCategoryData(stagingData, patientId) {
   const formattedData = [];
   const {
     conditionid: conditionId,
-    t, n, m, type,
+    t,
+    n,
+    m,
+    type,
     stagingsystem: stagingSystem,
     stagingcodesystem: stagingCodeSystem,
     effectivedate: effectiveDate,

test/extractors/CSVCancerDiseaseStatusExtractor.test.js

@@ -35,7 +35,7 @@ describe('CSVCancerDiseaseStatusExtractor', () => {
       expect(csvCancerDiseaseStatusExtractor.joinAndReformatData(exampleCSVDiseaseStatusModuleResponse, patientId)).toEqual(expect.anything());
 
       localData[0].evidence = ''; // Evidence is not required and will not throw an error
-      localData[0].observationStatus = ''; // Observation Status is not required and will not throw an error
+      localData[0].observationstatus = ''; // Observation Status is not required and will not throw an error
 
       // Only including required properties is valid
       expect(csvCancerDiseaseStatusExtractor.joinAndReformatData(localData, patientId)).toEqual(expect.anything());

test/extractors/CSVObservationExtractor.test.js

@@ -38,7 +38,7 @@ describe('CSVObservationExtractor', () => {
       expect(formatData(exampleCSVObservationModuleResponse, patientId)).toEqual(expect.anything());
 
       // Test that deleting an optional value works fine
-      delete localData[0].bodySite;
+      delete localData[0].bodysite;
       expect(formatData(exampleCSVObservationModuleResponse, patientId)).toEqual(expect.anything());
 
       // Test that deleting a mandatory value throws an error

test/extractors/CSVProcedureExtractor.test.js

@@ -37,7 +37,7 @@ describe('CSVProcedureExtractor', () => {
       expect(formatData(localData, patientId)).toEqual(expect.anything());
 
       // Test that removing an optional value works
-      delete localData[0].bodySite;
+      delete localData[0].bodysite;
       expect(formatData(localData, patientId)).toEqual(expect.anything());
 
       // Test that removing a required value throws

test/helpers/csvValidator.test.js

@@ -65,6 +65,14 @@ const SIMPLE_DATA_MISSING_OPTIONAL_COLUMN = [
   },
 ];
 
+const SIMPLE_DATA_DIFFERENT_CASING = [
+  {
+    Header1: '1',
+    hEaDeR2: '2',
+    header3: '3',
+  },
+];
+
 const schema = {
   headers: [
     { name: 'header1', required: true },
@@ -93,4 +101,8 @@ describe('csvValidator', () => {
   test('data missing an optional column should still validate', () => {
     expect(validateCSV('', schema, SIMPLE_DATA_MISSING_OPTIONAL_COLUMN)).toBe(true);
   });
+
+  test('data with different casing in the column header should still validate', () => {
+    expect(validateCSV('', schema, SIMPLE_DATA_DIFFERENT_CASING)).toBe(true);
+  });
 });

test/modules/CSVModule.test.js

@@ -41,3 +41,8 @@ test('Should return an empty array when key-value pair does not exist', async ()
   const data = await csvModule.get('mrn', INVALID_MRN);
   expect(data).toEqual([]);
 });
+
+test('Should return proper value regardless of key casing', async () => {
+  const data = await csvModule.get('mRN', 'example-mrn-1');
+  expect(data).toEqual(exampleResponse);
+});