Kickoff import

[lmd59]

Summary

Creates a new endpoint on the bulk-export-server to kickoff $import against a specified endpoint on an import server

New behavior

Server now supports endpoint: POST [fhir base]/bulkstatus/[client id]/kickoff-import

Code changes

• Update README to describe new endpoint
• Update package.json to make axios play nice with jest
• Add new endpoint to app.js and bulkstatus.service.js, including pulling data from the existing bulk status to create the import manifest, appropriate error handling, and appropriate pass-through of headers
• Pull gatherParams out of export.service.js into its own shared serviceUtils

Testing guidance

• npm run check
• Start bulk-export-server on port 3001 (and change to port 3001 for both environment variables)
• Start deqm-test-server (on default port 3000, make sure you have the latest with the new $import workflow)
• The Insomnia collection below has a basic set of requests that go through the workflow as follows:
  • Do an $export request on bulk-export-server
  • Check the status of that request at the bulk-export-server bulkstatus endpoint (from the $export response content-location header)
  • Add /kickoff-import to the end of the bulkstatus request and use that endpoint to do a POST with a receiver parameter in the body. Example:
    { "resourceType": "Parameters", "parameter": [ { "name": "receiver", "valueString": "http://localhost:3000/4_0_1/$import" } ] }
  • Check the status of the $import request using the deqm-test-server bulkstatus endpoint (from the kickoff-import response content location header)
  • Repeat this workflow for other export options and for error options to make sure the kickoff-import responses makes sense

kickoff-import-insomnia.json

[github-actions]

Coverage report

St.❔	Category	Percentage	Covered / Total
🟡	Statements	72.58% (-4% 🔻)	503/693
🟡	Branches	60% (-2.58% 🔻)	180/300
🟡	Functions	71.3% (-4.14% 🔻)	77/108
🟡	Lines	72.83% (-4.07% 🔻)	496/681

Show new covered files 🐣

St.❔	File	Statements	Branches	Functions	Lines
🟢	... / serviceUtils.js	94.44%	80%	100%	94.44%

Show files with reduced coverage 🔻

St.❔	File	Statements	Branches	Functions	Lines
🟢	... / export.service.js	85% (-0.9% 🔻)	79.27% (-0.17% 🔻)	93.75% (-0.69% 🔻)	84.67% (-0.95% 🔻)
🟢	... / mongo.controller.js	91.07% (-3.57% 🔻)	85.71%	78.57% (-7.14% 🔻)	91.07% (-3.57% 🔻)
🟡	... / bulkstatus.service.js	63.64% (-33.92% 🔻)	63.33% (-23.03% 🔻)	66.67% (-33.33% 🔻)	63.64% (-33.92% 🔻)
🟢	... / group.service.js	95.65% (-0.9% 🔻)	83.33% (-4.17% 🔻)	100%	95.65% (-0.9% 🔻)
🔴	... / bundleUtils.js	12.5% (-6.73% 🔻)	0%	0% (-28.57% 🔻)	13.04% (-6.96% 🔻)

Test suite run success

78 tests passing in 7 suites.

Report generated by 🧪jest coverage report action from 614dbe8

[elsaperelli]

Just one small comment! I think we can wait to think about unit testing for this when we do the unit testing task for the bulk-export-server.

[elsaperelli]

This link does not appear to be working for me

[elsaperelli]

Lgtm!