Skip to content

Commit b65ba60

Browse files
committed
Polish 'Simplify some code'
See gh-18438
1 parent e6b45de commit b65ba60

File tree

8 files changed

+11
-14
lines changed

8 files changed

+11
-14
lines changed

spring-boot-project/spring-boot-test-autoconfigure/src/main/java/org/springframework/boot/test/autoconfigure/OverrideAutoConfigurationContextCustomizerFactory.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ public void customizeContext(ConfigurableApplicationContext context, MergedConte
5656

5757
@Override
5858
public boolean equals(Object obj) {
59-
return (obj != null && obj.getClass() == getClass());
59+
return (obj != null) && (obj.getClass() == getClass());
6060
}
6161

6262
@Override

spring-boot-project/spring-boot-test-autoconfigure/src/main/java/org/springframework/boot/test/autoconfigure/filter/TypeExcludeFiltersContextCustomizer.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,8 @@ private TypeExcludeFilter instantiateTypeExcludeFilter(Class<?> testClass, Class
7171

7272
@Override
7373
public boolean equals(Object obj) {
74-
return (obj != null && getClass() == obj.getClass()
75-
&& this.filters.equals(((TypeExcludeFiltersContextCustomizer) obj).filters));
74+
return (obj != null) && (getClass() == obj.getClass())
75+
&& this.filters.equals(((TypeExcludeFiltersContextCustomizer) obj).filters);
7676
}
7777

7878
@Override

spring-boot-project/spring-boot-test-autoconfigure/src/main/java/org/springframework/boot/test/autoconfigure/properties/PropertyMappingContextCustomizer.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,8 +57,8 @@ public void customizeContext(ConfigurableApplicationContext context,
5757

5858
@Override
5959
public boolean equals(Object obj) {
60-
return (obj != null && getClass() == obj.getClass()
61-
&& this.propertySource.equals(((PropertyMappingContextCustomizer) obj).propertySource));
60+
return (obj != null) && (getClass() == obj.getClass())
61+
&& this.propertySource.equals(((PropertyMappingContextCustomizer) obj).propertySource);
6262
}
6363

6464
@Override

spring-boot-project/spring-boot-test/src/main/java/org/springframework/boot/test/context/filter/ExcludeFilterContextCustomizer.java

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,7 @@ public void customizeContext(ConfigurableApplicationContext context,
3737

3838
@Override
3939
public boolean equals(Object obj) {
40-
if (obj == this) {
41-
return true;
42-
}
43-
return obj != null && getClass() == obj.getClass();
40+
return (obj != null) && (getClass() == obj.getClass());
4441
}
4542

4643
@Override

spring-boot-project/spring-boot-test/src/main/java/org/springframework/boot/test/json/DuplicateJsonObjectContextCustomizerFactory.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ private void logDuplicateJsonObjectsWarning(List<URL> jsonObjects) {
8080

8181
@Override
8282
public boolean equals(Object obj) {
83-
return obj != null && obj.getClass() == getClass();
83+
return (obj != null) && (getClass() == obj.getClass());
8484
}
8585

8686
@Override

spring-boot-project/spring-boot-test/src/main/java/org/springframework/boot/test/system/OutputCapture.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -208,8 +208,8 @@ PrintStream getParent() {
208208
}
209209

210210
private static PrintStream getSystemStream(PrintStream printStream) {
211-
if (printStream instanceof PrintStreamCapture) {
212-
return ((PrintStreamCapture) printStream).getParent();
211+
while (printStream instanceof PrintStreamCapture) {
212+
printStream = ((PrintStreamCapture) printStream).getParent();
213213
}
214214
return printStream;
215215
}

spring-boot-project/spring-boot-test/src/main/java/org/springframework/boot/test/web/client/TestRestTemplateContextCustomizer.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ private void registerTestRestTemplate(BeanDefinitionRegistry registry) {
7878

7979
@Override
8080
public boolean equals(Object obj) {
81-
return obj != null && obj.getClass() == getClass();
81+
return (obj != null) && (obj.getClass() == getClass());
8282
}
8383

8484
@Override

spring-boot-project/spring-boot-test/src/main/java/org/springframework/boot/test/web/reactive/server/WebTestClientContextCustomizer.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ private void registerWebTestClient(BeanDefinitionRegistry registry) {
7979

8080
@Override
8181
public boolean equals(Object obj) {
82-
return (obj != null && obj.getClass() == getClass());
82+
return (obj != null) && (obj.getClass() == getClass());
8383
}
8484

8585
@Override

0 commit comments

Comments
 (0)