@@ -119,7 +119,8 @@ public void getMutationCountsByType() {
119119 List <GenomicDataCountItem > actualMutationCountsByType =
120120 mapper .getMutationCountsByType (
121121 StudyViewFilterFactory .make (studyViewFilter , null , studyViewFilter .getStudyIds (), null ),
122- List .of (genomicDataFilterMutation ), false );
122+ List .of (genomicDataFilterMutation ),
123+ false );
123124 List <GenomicDataCountItem > expectedMutationCountsByType =
124125 List .of (
125126 new GenomicDataCountItem (
@@ -134,28 +135,29 @@ public void getMutationCountsByType() {
134135 .isEqualTo (expectedMutationCountsByType );
135136 }
136137
137- @ Test
138- public void getMutationCountsByTypeAddSampleId () {
139- StudyViewFilter studyViewFilter = new StudyViewFilter ();
140- studyViewFilter .setStudyIds (List .of (STUDY_TCGA_PUB ));
141-
142- GenomicDataFilter genomicDataFilterMutation = new GenomicDataFilter ("AKT1" , "mutation" );
143- List <GenomicDataCountItem > mutationCountsByType =
144- mapper .getMutationCountsByType (
145- StudyViewFilterFactory .make (studyViewFilter , null , studyViewFilter .getStudyIds (), null ),
146- List .of (genomicDataFilterMutation ), true );
147-
148- assertThat (mutationCountsByType )
149- .flatExtracting (GenomicDataCountItem ::getCounts )
150- .extracting (GenomicDataCount ::getSampleIds )
151- .allSatisfy (sampleIds ->
138+ @ Test
139+ public void getMutationCountsByTypeAddSampleId () {
140+ StudyViewFilter studyViewFilter = new StudyViewFilter ();
141+ studyViewFilter .setStudyIds (List .of (STUDY_TCGA_PUB ));
142+
143+ GenomicDataFilter genomicDataFilterMutation = new GenomicDataFilter ("AKT1" , "mutation" );
144+ List <GenomicDataCountItem > mutationCountsByType =
145+ mapper .getMutationCountsByType (
146+ StudyViewFilterFactory .make (studyViewFilter , null , studyViewFilter .getStudyIds (), null ),
147+ List .of (genomicDataFilterMutation ),
148+ true );
149+
150+ assertThat (mutationCountsByType )
151+ .flatExtracting (GenomicDataCountItem ::getCounts )
152+ .extracting (GenomicDataCount ::getSampleIds )
153+ .allSatisfy (
154+ sampleIds ->
152155 assertThat (sampleIds )
153156 .as ("sampleIds should be populated when includeSampleIds=true" )
154157 .isNotNull ()
155- .isNotEmpty ()
156- );
157- }
158-
158+ .isNotEmpty ());
159+ }
160+
159161 @ Test
160162 public void getProteinExpressionCounts () {
161163 // Testing combined study missing samples when one lacks a relevant genomic profile
0 commit comments