OR Multiple _typeFilters (#54)

* OR Multiple _typeFilters

* Comment update

src/services/export.service.js

@@ -229,7 +229,9 @@ function validateExportParams(parameters, reply) {
   }
 
   if (parameters._typeFilter) {
-    const typeFilterArray = parameters._typeFilter.split(',');
+    const typeFilterArray = Array.isArray(parameters._typeFilter)
+      ? parameters._typeFilter
+      : parameters._typeFilter.split(',');
     const unsupportedTypeFilterTypes = [];
     typeFilterArray.forEach(line => {
       const resourceType = line.substring(0, line.indexOf('?'));

src/util/exportToNDJson.js

@@ -77,8 +77,8 @@ const exportToNDJson = async jobOptions => {
     const searchParameterQueries = {};
     const valueSetQueries = {};
     if (typeFilter) {
-      // subqueries may be joined together with a comma for a logical "or"
-      const tyq = typeFilter.split(',');
+      // subqueries may be joined together with a comma or as an array for a logical "or"
+      const tyq = Array.isArray(typeFilter) ? typeFilter : typeFilter.split(',');
       // loop over each subquery and extract all search params, which are joined via the "&" operator
       // each subquery is of the format <resource type>?<query>
       tyq.forEach(query => {
@@ -104,6 +104,7 @@ const exportToNDJson = async jobOptions => {
               valueSetQueries[resourceType] = { [property]: [propertyValue] };
             }
           } else {
+            // TODO: this gets overwritten for subqueries with "&" operators
             subqueries[property] = propertyValue;
           }
         });

test/util/exportToNDJson.test.js

@@ -27,6 +27,10 @@ const mockTypeFilter = 'Patient?maritalStatus:in=http://example.com/example-valu
 
 const complexMockTypeFilter =
   'Patient?maritalStatus:in=http://example.com/example-valueset-1,Encounter?type:in=http://example.com/example-valueset-1,ServiceRequest?code:in=http://example.com/example-valueset-1';
+const mockOrTypeFilter = [
+  'Patient?maritalStatus:in=http://example.com/example-valueset-1',
+  'Encounter?type:in=http://example.com/example-valueset-1'
+];
 const expectedFileNameEncounter = './tmp/123456/Encounter.ndjson';
 const expectedFileNameServiceRequest = './tmp/123456/ServiceRequest.ndjson';
 const typeFilterWOValueSet = 'Procedure?type:in=http';
@@ -82,6 +86,11 @@ describe('check export logic', () => {
       expect(fs.existsSync(expectedFileNameEncounter)).toBe(true);
       expect(fs.existsSync(expectedFileNameServiceRequest)).toBe(true);
     });
+    test('Expect folder created and export successful when Array _typeFilter parameter is retrieved from request', async () => {
+      await exportToNDJson({ clientEntry: clientId, type: mockType, typeFilter: mockOrTypeFilter });
+      expect(fs.existsSync(expectedFileName)).toBe(true);
+      expect(fs.existsSync(expectedFileNameEncounter)).toBe(true);
+    });
     test('Expect folder created and export to fail when _typeFilter parameter is retrieved from request and contains an invalid param', async () => {
       // Note: invalid types are checked in the export service
       await exportToNDJson({ clientEntry: clientId, type: mockType, typeFilter: typeFilterWithInvalidType });