DATAMONGO-2228 - Polishing.

Favor AssertJ over hamcrest.

Original Pull Request: #661

Author: christophstrobl

spring-data-mongodb/src/test/java/org/springframework/data/mongodb/repository/support/SpringDataMongodbSerializerUnitTests.java

@@ -18,8 +18,7 @@
 import static com.querydsl.core.types.ExpressionUtils.path;
 import static com.querydsl.core.types.ExpressionUtils.predicate;
 import static com.querydsl.core.types.dsl.Expressions.*;
-import static org.hamcrest.Matchers.*;
-import static org.junit.Assert.*;
+import static org.assertj.core.api.Assertions.*;
 
 import java.util.Collections;
 
@@ -78,14 +77,14 @@ public void setUp() {
 	public void uses_idAsKeyForIdProperty() {
 
 		StringPath path = QPerson.person.id;
-		assertThat(serializer.getKeyForPath(path, path.getMetadata()), is("_id"));
+		assertThat(serializer.getKeyForPath(path, path.getMetadata())).isEqualTo("_id");
 	}
 
 	@Test
 	public void buildsNestedKeyCorrectly() {
 
 		StringPath path = QPerson.person.address.street;
-		assertThat(serializer.getKeyForPath(path, path.getMetadata()), is("street"));
+		assertThat(serializer.getKeyForPath(path, path.getMetadata())).isEqualTo("street");
 	}
 
 	@Test
@@ -98,18 +97,17 @@ public void convertsComplexObjectOnSerializing() {
 		Document document = serializer.asDocument("foo", address);
 
 		Object value = document.get("foo");
-		assertThat(value, is(notNullValue()));
-		assertThat(value, is(instanceOf(Document.class)));
+		assertThat(value).isNotNull().isInstanceOf(Document.class);
 
 		Object reference = converter.convertToMongoType(address);
-		assertThat(value, is(reference));
+		assertThat(value).isEqualTo(reference);
 	}
 
 	@Test // DATAMONGO-376
 	public void returnsEmptyStringIfNoPathExpressionIsGiven() {
 
 		QAddress address = QPerson.person.shippingAddresses.any();
-		assertThat(serializer.getKeyForPath(address, address.getMetadata()), is(""));
+		assertThat(serializer.getKeyForPath(address, address.getMetadata())).isEmpty();
 	}
 
 	@Test // DATAMONGO-467, DATAMONGO-1798
@@ -121,9 +119,7 @@ public void retainsIdPropertyType() {
 		StringPath idPath = builder.getString("id");
 
 		Document result = (Document) serializer.visit((BooleanOperation) idPath.eq(id.toString()), null);
-		assertThat(result.get("_id"), is(notNullValue()));
-		assertThat(result.get("_id"), is(instanceOf(String.class)));
-		assertThat(result.get("_id"), is(id.toString()));
+		assertThat(result.get("_id")).isNotNull().isInstanceOf(String.class).isEqualTo(id.toString());
 	}
 
 	@Test // DATAMONGO-761
@@ -133,7 +129,7 @@ public void looksUpKeyForNonPropertyPath() {
 		SimplePath<Object> firstElementPath = builder.getArray("foo", String[].class).get(0);
 		String path = serializer.getKeyForPath(firstElementPath, firstElementPath.getMetadata());
 
-		assertThat(path, is("0"));
+		assertThat(path).isEqualTo("0");
 	}
 
 	@Test // DATAMONGO-1485
@@ -148,10 +144,10 @@ public void takesCustomConversionForEnumsIntoAccount() {
 		this.converter = converter;
 		this.serializer = new SpringDataMongodbSerializer(this.converter);
 
-		Object mappedPredicate = this.serializer.handle(QPerson.person.sex.eq(Sex.FEMALE));
+		Object mappedPredicate = serializer.handle(QPerson.person.sex.eq(Sex.FEMALE));
 
-		assertThat(mappedPredicate, is(instanceOf(Document.class)));
-		assertThat(((Document) mappedPredicate).get("sex"), is("f"));
+		assertThat(mappedPredicate).isInstanceOf(Document.class);
+		assertThat(((Document) mappedPredicate).get("sex")).isEqualTo("f");
 	}
 
 	@Test // DATAMONGO-1848, DATAMONGO-1943
@@ -160,9 +156,8 @@ public void shouldRemarshallListsAndDocuments() {
 		BooleanExpression criteria = QPerson.person.lastname.isNotEmpty()
 				.and(QPerson.person.firstname.containsIgnoreCase("foo")).not();
 
-		assertThat(this.serializer.handle(criteria),
-				is(equalTo(Document.parse("{ \"$or\" : [ { \"lastname\" : { \"$not\" : { "
-						+ "\"$ne\" : \"\"}}} , { \"firstname\" : { \"$not\" : { \"$regex\" : \".*\\\\Qfoo\\\\E.*\" , \"$options\" : \"i\"}}}]}"))));
+		assertThat(serializer.handle(criteria)).isEqualTo(Document.parse("{ \"$or\" : [ { \"lastname\" : { \"$not\" : { "
+				+ "\"$ne\" : \"\"}}} , { \"firstname\" : { \"$not\" : { \"$regex\" : \".*\\\\Qfoo\\\\E.*\" , \"$options\" : \"i\"}}}]}"));
 	}
 
 	@Test // DATAMONGO-2228
@@ -174,9 +169,7 @@ public void retainsOpsInAndExpression() {
 				predicate(Ops.OR, predicate(Ops.EQ, path(Object.class, "lastname"), constant("Smith")),
 						predicate(Ops.EQ, path(Object.class, "lastname"), constant("Connor"))));
 
-		Document result = (Document) serializer.visit(testExpression, null);
-
-		assertThat(result.toJson(), is(
+		assertThat(serializer.handle(testExpression)).isEqualTo(Document.parse(
 				"{\"$and\": [{\"$or\": [{\"firstname\": \"John\"}, {\"firstname\": \"Sarah\"}]}, {\"$or\": [{\"lastname\": \"Smith\"}, {\"lastname\": \"Connor\"}]}]}"));
 	}