Skip to content

Commit 4b09026

Browse files
committed
Remove sonar code smells.
1 parent eb1b70b commit 4b09026

File tree

3 files changed

+14
-14
lines changed

3 files changed

+14
-14
lines changed

spring-annotated-data-binder-core/src/test/java/com/mattbertolini/spring/web/bind/introspect/DefaultAnnotatedRequestBeanIntrospectorTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ void returnsResolversForType() {
5151
FakeResolver resolver = new FakeResolver(RequestParameter.class);
5252
registry.addResolver(resolver);
5353
Collection<ResolvedPropertyData> resolvers = introspector.getResolversFor(SimpleType.class);
54-
assertThat(resolvers.size()).isEqualTo(1);
54+
assertThat(resolvers).hasSize(1);
5555
ResolvedPropertyData data = resolvers.iterator().next();
5656
assertThat(data.getPropertyName()).isEqualTo("data");
5757
}
@@ -61,7 +61,7 @@ void returnsResolversUsingNestedTypes() {
6161
FakeResolver resolver = new FakeResolver(RequestParameter.class);
6262
registry.addResolver(resolver);
6363
Collection<ResolvedPropertyData> resolvers = introspector.getResolversFor(OuterBean.class);
64-
assertThat(resolvers.size()).isEqualTo(1);
64+
assertThat(resolvers).hasSize(1);
6565
ResolvedPropertyData data = resolvers.iterator().next();
6666
assertThat(data.getPropertyName()).isEqualTo("innerBean.inner");
6767
}

spring-webflux-annotated-data-binder/src/test/java/com/mattbertolini/spring/web/reactive/bind/PropertyResolverRegistryTest.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,25 +23,25 @@ void setUp() {
2323

2424
@Test
2525
void addsResolversFromSet() {
26-
assertThat(registry.getPropertyResolvers().size()).isZero();
26+
assertThat(registry.getPropertyResolvers()).isEmpty();
2727
registry.addResolvers(Collections.singleton(new FakeResolver()));
28-
assertThat(registry.getPropertyResolvers().size()).isEqualTo(1);
28+
assertThat(registry.getPropertyResolvers()).hasSize(1);
2929
}
3030

3131
@Test
3232
void addsSingleResolver() {
33-
assertThat(registry.getPropertyResolvers().size()).isZero();
33+
assertThat(registry.getPropertyResolvers()).isEmpty();
3434
registry.addResolver(new FakeResolver());
35-
assertThat(registry.getPropertyResolvers().size()).isEqualTo(1);
35+
assertThat(registry.getPropertyResolvers()).hasSize(1);
3636
}
3737

3838
@Test
3939
void addsResolversFromRegistry() {
4040
PropertyResolverRegistry anotherRegistry = new PropertyResolverRegistry();
4141
anotherRegistry.addResolver(new FakeResolver());
42-
assertThat(registry.getPropertyResolvers().size()).isZero();
42+
assertThat(registry.getPropertyResolvers()).isEmpty();
4343
registry.addResolvers(anotherRegistry);
44-
assertThat(registry.getPropertyResolvers().size()).isEqualTo(1);
44+
assertThat(registry.getPropertyResolvers()).hasSize(1);
4545
}
4646

4747
private static class FakeResolver implements RequestPropertyResolver {

spring-webmvc-annotated-data-binder/src/test/java/com/mattbertolini/spring/web/servlet/mvc/bind/PropertyResolverRegistryTest.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -22,25 +22,25 @@ void setUp() {
2222

2323
@Test
2424
void addsResolversFromSet() {
25-
assertThat(registry.getPropertyResolvers().size()).isZero();
25+
assertThat(registry.getPropertyResolvers()).isEmpty();
2626
registry.addResolvers(Collections.singleton(new FakeResolver()));
27-
assertThat(registry.getPropertyResolvers().size()).isEqualTo(1);
27+
assertThat(registry.getPropertyResolvers()).hasSize(1);
2828
}
2929

3030
@Test
3131
void addsSingleResolver() {
32-
assertThat(registry.getPropertyResolvers().size()).isZero();
32+
assertThat(registry.getPropertyResolvers()).isEmpty();
3333
registry.addResolver(new FakeResolver());
34-
assertThat(registry.getPropertyResolvers().size()).isEqualTo(1);
34+
assertThat(registry.getPropertyResolvers()).hasSize(1);
3535
}
3636

3737
@Test
3838
void addsResolversFromRegistry() {
3939
PropertyResolverRegistry anotherRegistry = new PropertyResolverRegistry();
4040
anotherRegistry.addResolver(new FakeResolver());
41-
assertThat(registry.getPropertyResolvers().size()).isZero();
41+
assertThat(registry.getPropertyResolvers()).isEmpty();
4242
registry.addResolvers(anotherRegistry);
43-
assertThat(registry.getPropertyResolvers().size()).isEqualTo(1);
43+
assertThat(registry.getPropertyResolvers()).hasSize(1);
4444
}
4545

4646
private static class FakeResolver implements RequestPropertyResolver {

0 commit comments

Comments
 (0)