Skip to content

Commit d655a31

Browse files
committed
fix: resolve remaining GuardLogStatement and ControlStatementBraces smells
Signed-off-by: AleGTorres <[email protected]>
1 parent 6886389 commit d655a31

File tree

2 files changed

+7
-4
lines changed

2 files changed

+7
-4
lines changed

src/test/java/org/springframework/samples/petclinic/PostgresIntegrationTests.java

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -129,12 +129,14 @@ public void printProperties() {
129129

130130
String value = sourceProperty.toString();
131131

132-
if(log.isInfoEnabled()) {
133-
if (resolved.equals(value)) {
132+
if (resolved.equals(value)) {
133+
if (log.isInfoEnabled()) {
134134
log.info(name + "=" + resolved);
135135
}
136136
else {
137-
log.info(name + "=" + value + " OVERRIDDEN to " + resolved);
137+
if (log.isInfoEnabled()) {
138+
log.info(name + "=" + value + " OVERRIDDEN to " + resolved);
139+
}
138140
}
139141
}
140142
}

src/test/java/org/springframework/samples/petclinic/system/I18nPropertiesSyncTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,8 +117,9 @@ public void checkI18nPropertyFilesAreInSync() throws IOException {
117117
String fileName = entry.getKey();
118118
// We use fallback logic to include english strings, hence messages_en is not
119119
// populated.
120-
if (fileName.equals(baseFile) || fileName.equals("messages_en.properties"))
120+
if (fileName.equals(baseFile) || fileName.equals("messages_en.properties")) {
121121
continue;
122+
}
122123

123124
Properties props = entry.getValue();
124125
Set<String> missingKeys = new TreeSet<>(baseKeys);

0 commit comments

Comments
 (0)