From 2c59b130689fb2722ca444374b4998dca85733f7 Mon Sep 17 00:00:00 2001 From: bnguyen-bcgsc Date: Wed, 16 Sep 2026 15:46:26 -0700 Subject: [PATCH 1/2] - DEVSU-2990 - Make variant text header font size consistent - Add space break after comma of each cancer type in variant text header --- pori_python/ipr/summary.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pori_python/ipr/summary.py b/pori_python/ipr/summary.py index 6038eb0..05025c2 100644 --- a/pori_python/ipr/summary.py +++ b/pori_python/ipr/summary.py @@ -372,11 +372,11 @@ def prep_single_ipr_variant_comment(variant_text): Returns: section: html-formatted string """ - cancer_type = ','.join(variant_text['cancerType']) + cancer_type = ', '.join(variant_text['cancerType']) if not cancer_type: cancer_type = 'no specific cancer types' cancer_type = f' ({cancer_type})' - section = [f'

{variant_text["variantName"]}{cancer_type}

'] + section = [f'

{variant_text["variantName"]}{cancer_type}

'] section.append(f'

{variant_text["text"]}

') return section From c47359cec451556cb95a1f1eeb1a8699e2ee4308 Mon Sep 17 00:00:00 2001 From: bnguyen-bcgsc Date: Thu, 17 Sep 2026 08:55:16 -0700 Subject: [PATCH 2/2] - Update asserted mock variant text summary lines in test summary --- tests/test_ipr/test_summary.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/test_ipr/test_summary.py b/tests/test_ipr/test_summary.py index a2f0009..c3e128f 100644 --- a/tests/test_ipr/test_summary.py +++ b/tests/test_ipr/test_summary.py @@ -218,7 +218,7 @@ def test_gets_fully_matched_output_when_possible(self): include_nonspecific_template=True, ) summary_lines = ipr_summary.split('\n') - assert summary_lines[1] == '

ERBB2 amplification (test1,test)

' + assert summary_lines[1] == '

ERBB2 amplification (test1, test)

' assert summary_lines[2] == '

normal

' assert len(summary_lines) == 3 @@ -321,7 +321,7 @@ def test_includes_nonspecific_disease_matches_when_specified(self): include_nonspecific_template=False, ) summary_lines = ipr_summary.split('\n') - assert summary_lines[1] == '

ERBB2 amplification (no specific cancer types)

' + assert summary_lines[1] == '

ERBB2 amplification (no specific cancer types)

' assert summary_lines[2] == '

no cancerType

' assert len(summary_lines) == 3 @@ -340,7 +340,7 @@ def test_includes_all_graphkb_disease_matches(self): include_nonspecific_template=False, ) summary_lines = ipr_summary.split('\n') - assert summary_lines[1] == '

ERBB2 amplification (test1,test)

' + assert summary_lines[1] == '

ERBB2 amplification (test1, test)

' assert summary_lines[2] == '

normal

' assert len(summary_lines) == 3 @@ -363,7 +363,7 @@ def test_prepare_section_for_multiple_variants(self): assert len(summary_lines) == 5 assert ( '\n'.join(summary_lines[1:]) - == '

ERBB2 amplification (test1,test)

\n

normal

\n

second variant (test1,test)

\n

normal, second variant

' + == '

ERBB2 amplification (test1, test)

\n

normal

\n

second variant (test1, test)

\n

normal, second variant

' ) def test_empty_section_when_no_variant_match(self):