-
Notifications
You must be signed in to change notification settings - Fork 99
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
test(StockExitFormHelper): add unit tests
Adds unit tests for the description rendering of the StockExitFormHelper. Note the testing $translate is a little finicky, but this could be improved later.
- Loading branch information
Showing
7 changed files
with
216 additions
and
131 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
152 changes: 152 additions & 0 deletions
152
client/src/modules/stock/StockExitFormHelper.service.js
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,152 @@ | ||
| angular.module('bhima.services') | ||
| .service('StockExitFormHelperService', StockExitFormHelperService); | ||
|
|
||
| StockExitFormHelperService.$inject = [ | ||
| 'moment', '$q', '$translate', 'bhConstants', | ||
| 'PatientService', 'PatientInvoiceService', 'ServiceService', | ||
| ]; | ||
|
|
||
| /** | ||
| * @class StockExitFormHelperService | ||
| * | ||
| * @description | ||
| * This form powers the stock exit form in BHIMA. | ||
| */ | ||
| function StockExitFormHelperService(moment, $q, $translate, bhConstants, Patients, Invoices, Services) { | ||
| const service = {}; | ||
|
|
||
| const { formatDB } = bhConstants.dates; | ||
|
|
||
| /** | ||
| * @function getDescriptionForPatient | ||
| * | ||
| * @description | ||
| * Fetches all the components to create a proper translation | ||
| * for the stock exit to patient. | ||
| * | ||
| * @returns {Promise<String>} description | ||
| */ | ||
| function getDescriptionForPatient(details, i18nKeys) { | ||
|
|
||
| // load the required information for the patient description | ||
| const queries = $q.all([ | ||
| Patients.read(null, { uuid : details.entity_uuid }), | ||
| Invoices.read(null, { uuid : details.invoice_uuid }), | ||
| ]); | ||
|
|
||
| return queries | ||
| .then(([patients, invoices]) => { | ||
| const [patient] = patients; | ||
| const [invoice] = invoices; | ||
|
|
||
| Object.assign(i18nKeys, { | ||
| patient : patient.display_name.concat(` (${patient.reference})`), | ||
| invoice : invoice.reference, | ||
| }); | ||
|
|
||
| return $translate.instant('STOCK.EXIT_PATIENT_WITH_INVOICE', i18nKeys); | ||
| }); | ||
| } | ||
|
|
||
| /** | ||
| * @function getDescriptionForService | ||
| * | ||
| * @description | ||
| * Fetches all the components to create a proper translation | ||
| * for the stock exit to service. | ||
| * | ||
| * @returns {Promise<String>} description | ||
| */ | ||
| function getDescriptionForService(details, i18nKeys) { | ||
| // load the required information for the service description | ||
| Services.read(null, { uuid : details.entity_uuid }) | ||
| .then(([{ name }]) => { | ||
|
|
||
| Object.assign(i18nKeys, { | ||
| service : name, | ||
| }); | ||
|
|
||
| return $translate.instant('STOCK.EXIT_SERVICE_ADVANCED', i18nKeys); | ||
| }); | ||
| } | ||
|
|
||
| // depots do not require any special translation | ||
| // TODO(@jniles) - should this say "exit from depot X to depot Y of Z items"? | ||
| function getDescriptionForDepot(details, i18nKeys) { | ||
| return $q.resolve($translate.instant('STOCK.EXIT_DEPOT', i18nKeys)); | ||
| } | ||
|
|
||
| // losses do not require any special translation | ||
| // TODO(@jniles) - should this say "Loss of Z items from Depot Y"? | ||
| function getDescriptionForLoss(details, i18nKeys) { | ||
| return $q.resolve($translate.instant('STOCK.EXIT_LOSS', i18nKeys)); | ||
| } | ||
|
|
||
| /** | ||
| * @function getI18nKeys | ||
| * | ||
| * @description | ||
| * Gets the i18nKeys to render the description. Note, not all data is | ||
| * cached on the client so this function is async, looking up data from | ||
| * the server. | ||
| * | ||
| * It requires that an exit type be set before calling it. | ||
| * | ||
| * @returns {Promise<String>} description | ||
| */ | ||
| service.getDescription = function getDescription(depot, details) { | ||
| const i18nKeys = { depot : depot.text }; | ||
|
|
||
| let promise; | ||
|
|
||
| switch (details.exit_type) { | ||
| case 'patient': | ||
| promise = getDescriptionForPatient(details, i18nKeys); | ||
| break; | ||
| case 'service': | ||
| promise = getDescriptionForService(details, i18nKeys); | ||
| break; | ||
| case 'depot': | ||
| promise = getDescriptionForDepot(details, i18nKeys); | ||
| break; | ||
| case 'loss': | ||
| promise = getDescriptionForLoss(details, i18nKeys); | ||
| break; | ||
| default: | ||
| throw new Error('Unrecognized Exit Type'); | ||
| } | ||
|
|
||
| return promise | ||
| .then(description => { | ||
| return description.concat(' - ', details.description); | ||
| }); | ||
| }; | ||
|
|
||
| /** | ||
| * @function formatRowsForExport | ||
| * | ||
| * @description this function will be apply to grid columns as filter for getting new columns | ||
| * | ||
| * @param {array} rows - refer to the grid data array | ||
| * @return {array} - return an array of array with value as an object in this format : { value : ... } | ||
| */ | ||
| service.formatRowsForExport = function formatRowsForExport(rows = []) { | ||
| return rows.map(row => { | ||
| const code = row.inventory?.code; | ||
| const description = row.inventory?.text; | ||
| const lot = row.lot?.label; | ||
| const price = row.inventory?.unit_cost; | ||
| const quantity = row.quantity?.quantity; | ||
| const type = row.quantity?.unit_type; | ||
| const available = row.inventory?.quantity; | ||
| const amount = (price && quantity) ? price * quantity : 0; | ||
| const expiration = (row.lot && row.lot.expiration_date) | ||
| ? moment(row.lot.expiration_date).format(formatDB) : null; | ||
|
|
||
| return [code, description, lot, price, quantity, type, available, amount, expiration].map(value => ({ value })); | ||
| }); | ||
| }; | ||
|
|
||
| return service; | ||
|
|
||
| } |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.