Skip to content

Commit

Permalink
Merge pull request #85 from MeasureAuthoringTool/MAT-7388_transferred…
Browse files Browse the repository at this point in the history
…MeasureStratNone

Mat 7388 transferred measure strat none
  • Loading branch information
sb-cecilialiu authored Jul 24, 2024
2 parents f372ffb + 9b262ed commit a228f00
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 1 deletion.
2 changes: 1 addition & 1 deletion src/main/java/gov/cms/madie/util/HumanReadableUtil.java
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ public static String getStratification(Measure measure) {
.append("\n");
}
}
if (!allDescriptions.isEmpty()) {
if (!StringUtils.isBlank(allDescriptions)) {
return HumanReadableUtil.escapeHtmlString(allDescriptions.toString().trim());
} else {
return "None";
Expand Down
14 changes: 14 additions & 0 deletions src/test/java/gov/cms/madie/util/HumanReadableUtilTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -270,4 +270,18 @@ void testGetVersionNumberMeasureIsVersioned() {
String version = HumanReadableUtil.getVersionNumber(measure);
assertThat(version, is(equalTo("1.2.003")));
}

@Test
void testGetFunctionSignatureCQLDefinitionNull() {
String result = HumanReadableUtil.getFunctionSignature(null);
assertNull(result);
}

@Test
void testGetFunctionSignatureFunctionArgumentsNull() {
CQLDefinition definition =
CQLDefinition.builder().definitionName("test definitioin name").build();
String result = HumanReadableUtil.getFunctionSignature(definition);
assertThat(result, is(equalTo("test definitioin name")));
}
}

0 comments on commit a228f00

Please sign in to comment.