Skip to content

Commit 398d305

Browse files
Merge pull request #36 from bcgsc/release/v4.1.1
Release/v4.1.1
2 parents 0f62355 + 5968d81 commit 398d305

File tree

4 files changed

+40
-8
lines changed

4 files changed

+40
-8
lines changed

package-lock.json

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@bcgsc-pori/graphkb-schema",
3-
"version": "4.1.0",
3+
"version": "4.1.1",
44
"description": "Shared package between the API and GUI for GraphKB which holds the schema definitions and schema-related functions",
55
"bugs": {
66
"email": "graphkb@bcgsc.ca"

src/sentenceTemplates.ts

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,12 +46,14 @@ const addEvidence = (
4646
record: StatementRecord,
4747
keys = TEMPLATE_KEYS,
4848
) => {
49-
// remove preexisting evidence info, if any
50-
let updated = template.replace(' (${keys.evidenceLevel})', '');
51-
updated = template.replace(' (${keys.evidence})', '');
52-
updated = template.replace(' ${keys.preclinicalWarning}', '');
49+
// remove preexisting evidence info, if any, e.g. from default template
50+
let updated = template
51+
.replace(` (${keys.evidenceLevel})`, '')
52+
.replace(` (${keys.evidence})`, '')
53+
.replace(` ${keys.preclinicalWarning}`, '');
5354

5455
// preclinical warning
56+
// eslint-disable-next-line one-var
5557
let isPreclinical = false;
5658

5759
if (record.evidenceLevel) {
@@ -368,6 +370,7 @@ const generateStatementSentence = (
368370
};
369371

370372
export {
373+
addEvidence,
371374
generateStatementSentence,
372375
chooseDefaultTemplate,
373376
DEFAULT_TEMPLATE,

test/sentenceTemplates.test.ts

Lines changed: 30 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,10 @@
11
import { schema } from '../src';
2-
import { DEFAULT_TEMPLATE, generateStatementSentence, PRECLINICAL_WARNING } from '../src/sentenceTemplates';
2+
import {
3+
addEvidence,
4+
DEFAULT_TEMPLATE,
5+
generateStatementSentence,
6+
PRECLINICAL_WARNING,
7+
} from '../src/sentenceTemplates';
38

49
import examples from './testData/statementExamples.json';
510

@@ -332,3 +337,27 @@ describe('generateStatementSentence', () => {
332337
});
333338
});
334339
});
340+
341+
describe('addEvidence', () => {
342+
test('evidence', () => {
343+
const updatedTemplate = addEvidence(DEFAULT_TEMPLATE, examples['evidence']);
344+
const expected = 'Given {conditions}, {relevance} applies to {subject} ({evidence})'
345+
expect(updatedTemplate).toEqual(expected);
346+
});
347+
test('evidenceLevel', () => {
348+
const updatedTemplate = addEvidence(DEFAULT_TEMPLATE, examples['evidenceLevel']);
349+
const expected = 'Given {conditions}, {relevance} applies to {subject} ({evidence}) ({evidenceLevel})'
350+
expect(updatedTemplate).toEqual(expected);
351+
});
352+
test('preclinical warning', () => {
353+
const updatedTemplate = addEvidence(DEFAULT_TEMPLATE, examples['preclinicalWarning']);
354+
const expected = 'Given {conditions}, {relevance} applies to {subject} {preclinicalWarning} ({evidence}) ({evidenceLevel})'
355+
expect(updatedTemplate).toEqual(expected);
356+
});
357+
test('remove preexisting evidence info', () => {
358+
const template = '... {preclinicalWarning} ({evidence}) ({evidenceLevel})'
359+
const updatedTemplate = addEvidence(template, examples['evidence']);
360+
const expected = '... ({evidence})'
361+
expect(updatedTemplate).toEqual(expected);
362+
});
363+
});

0 commit comments

Comments
 (0)