polish assertj assertions

etrandafir93 · etrandafir93 · commit 987404f66ec2 · 2025-05-10T16:06:12.000+03:00
Issue #1058 Signed-off-by: etrandafir93 <emanueltrandafir1993@gmail.com>
diff --git a/test-support/src/test/java/org/springframework/ldap/test/EmbeddedLdapServerFactoryBeanTests.java b/test-support/src/test/java/org/springframework/ldap/test/EmbeddedLdapServerFactoryBeanTests.java
@@ -39,7 +39,7 @@ void serverStartup() throws Exception {
 
 		List<String> list = ldapTemplate.search(LdapQueryBuilder.query().where("objectclass").is("person"),
 				this::mapFromAttributes);
-		assertThat(5).isEqualTo(list.size());
+		assertThat(list).hasSize(5);
 	}
 
 	private String mapFromAttributes(Attributes attrs) throws NamingException {
diff --git a/test-support/src/test/java/org/springframework/ldap/test/unboundid/EmbeddedLdapServerFactoryBeanTests.java b/test-support/src/test/java/org/springframework/ldap/test/unboundid/EmbeddedLdapServerFactoryBeanTests.java
@@ -49,7 +49,7 @@ void serverStartup() throws Exception {
 
 		List<String> list = ldapTemplate.search(LdapQueryBuilder.query().where("objectclass").is("person"),
 				this::mapFromAttributes);
-		assertThat(list.size()).isEqualTo(5);
+		assertThat(list).hasSize(5);
 	}
 
 	private String mapFromAttributes(Attributes attrs) throws NamingException {
diff --git a/test-support/src/test/java/org/springframework/ldap/test/unboundid/TestContextSourceFactoryBeanTests.java b/test-support/src/test/java/org/springframework/ldap/test/unboundid/TestContextSourceFactoryBeanTests.java
@@ -49,7 +49,7 @@ void serverStartup() throws Exception {
 
 		List<String> list = ldapTemplate.search(LdapQueryBuilder.query().where("objectclass").is("person"),
 				this::mapFromAttributes);
-		assertThat(list.size()).isEqualTo(5);
+		assertThat(list).hasSize(5);
 	}
 
 	private String mapFromAttributes(Attributes attrs) throws NamingException {

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ void serverStartup() throws Exception {`
`39`	`39`
`40`	`40`	`List<String> list = ldapTemplate.search(LdapQueryBuilder.query().where("objectclass").is("person"),`
`41`	`41`	`this::mapFromAttributes);`
`42`		`- assertThat(5).isEqualTo(list.size());`
	`42`	`+ assertThat(list).hasSize(5);`
`43`	`43`	`}`
`44`	`44`
`45`	`45`	`private String mapFromAttributes(Attributes attrs) throws NamingException {`
Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ void serverStartup() throws Exception {`
`49`	`49`
`50`	`50`	`List<String> list = ldapTemplate.search(LdapQueryBuilder.query().where("objectclass").is("person"),`
`51`	`51`	`this::mapFromAttributes);`
`52`		`- assertThat(list.size()).isEqualTo(5);`
	`52`	`+ assertThat(list).hasSize(5);`
`53`	`53`	`}`
`54`	`54`
`55`	`55`	`private String mapFromAttributes(Attributes attrs) throws NamingException {`