Skip to content

Commit 398eafe

Browse files
christophstroblmp911de
authored andcommitted
DATAMONGO-2113 - Polishing.
Use AssertJ in tests. Original pull request: spring-projects#615.
1 parent 4673525 commit 398eafe

File tree

1 file changed

+16
-18
lines changed

1 file changed

+16
-18
lines changed

spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/convert/MongoConvertersUnitTests.java

+16-18
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,7 @@
1515
*/
1616
package org.springframework.data.mongodb.core.convert;
1717

18-
import static org.hamcrest.Matchers.*;
19-
import static org.junit.Assert.*;
18+
import static org.assertj.core.api.Assertions.*;
2019

2120
import java.math.BigDecimal;
2221
import java.time.Instant;
@@ -44,7 +43,6 @@
4443
import org.springframework.data.mongodb.core.convert.MongoConverters.StringToBigDecimalConverter;
4544
import org.springframework.data.mongodb.core.convert.MongoConverters.StringToCurrencyConverter;
4645
import org.springframework.data.mongodb.core.geo.Sphere;
47-
import org.springframework.data.mongodb.test.util.Assertions;
4846

4947
/**
5048
* Unit tests for {@link MongoConverters}.
@@ -60,10 +58,10 @@ public void convertsBigDecimalToStringAndBackCorrectly() {
6058

6159
BigDecimal bigDecimal = BigDecimal.valueOf(254, 1);
6260
String value = BigDecimalToStringConverter.INSTANCE.convert(bigDecimal);
63-
assertThat(value, is("25.4"));
61+
assertThat(value).isEqualTo("25.4");
6462

6563
BigDecimal reference = StringToBigDecimalConverter.INSTANCE.convert(value);
66-
assertThat(reference, is(bigDecimal));
64+
assertThat(reference).isEqualTo(bigDecimal);
6765
}
6866

6967
@Test // DATAMONGO-858
@@ -74,7 +72,7 @@ public void convertsBoxToDocumentAndBackCorrectly() {
7472
Document document = GeoConverters.BoxToDocumentConverter.INSTANCE.convert(box);
7573
Shape shape = GeoConverters.DocumentToBoxConverter.INSTANCE.convert(document);
7674

77-
assertThat(shape, is((org.springframework.data.geo.Shape) box));
75+
assertThat(shape).isEqualTo(box);
7876
}
7977

8078
@Test // DATAMONGO-858
@@ -85,7 +83,7 @@ public void convertsCircleToDocumentAndBackCorrectly() {
8583
Document document = GeoConverters.CircleToDocumentConverter.INSTANCE.convert(circle);
8684
Shape shape = GeoConverters.DocumentToCircleConverter.INSTANCE.convert(document);
8785

88-
assertThat(shape, is((org.springframework.data.geo.Shape) circle));
86+
assertThat(shape).isEqualTo(circle);
8987
}
9088

9189
@Test // DATAMONGO-858
@@ -96,7 +94,7 @@ public void convertsPolygonToDocumentAndBackCorrectly() {
9694
Document document = GeoConverters.PolygonToDocumentConverter.INSTANCE.convert(polygon);
9795
Shape shape = GeoConverters.DocumentToPolygonConverter.INSTANCE.convert(document);
9896

99-
assertThat(shape, is((org.springframework.data.geo.Shape) polygon));
97+
assertThat(shape).isEqualTo(polygon);
10098
}
10199

102100
@Test // DATAMONGO-858
@@ -107,7 +105,7 @@ public void convertsSphereToDocumentAndBackCorrectly() {
107105
Document document = GeoConverters.SphereToDocumentConverter.INSTANCE.convert(sphere);
108106
org.springframework.data.geo.Shape shape = GeoConverters.DocumentToSphereConverter.INSTANCE.convert(document);
109107

110-
assertThat(shape, is((org.springframework.data.geo.Shape) sphere));
108+
assertThat(shape).isEqualTo(sphere);
111109
}
112110

113111
@Test // DATAMONGO-858
@@ -118,43 +116,43 @@ public void convertsPointToListAndBackCorrectly() {
118116
Document document = GeoConverters.PointToDocumentConverter.INSTANCE.convert(point);
119117
org.springframework.data.geo.Point converted = GeoConverters.DocumentToPointConverter.INSTANCE.convert(document);
120118

121-
assertThat(converted, is((org.springframework.data.geo.Point) point));
119+
assertThat(converted).isEqualTo(point);
122120
}
123121

124122
@Test // DATAMONGO-1372
125123
public void convertsCurrencyToStringCorrectly() {
126-
assertThat(CurrencyToStringConverter.INSTANCE.convert(Currency.getInstance("USD")), is("USD"));
124+
assertThat(CurrencyToStringConverter.INSTANCE.convert(Currency.getInstance("USD"))).isEqualTo("USD");
127125
}
128126

129127
@Test // DATAMONGO-1372
130128
public void convertsStringToCurrencyCorrectly() {
131-
assertThat(StringToCurrencyConverter.INSTANCE.convert("USD"), is(Currency.getInstance("USD")));
129+
assertThat(StringToCurrencyConverter.INSTANCE.convert("USD")).isEqualTo(Currency.getInstance("USD"));
132130
}
133131

134132
@Test // DATAMONGO-1416
135133
public void convertsAtomicLongToLongCorrectly() {
136-
assertThat(AtomicLongToLongConverter.INSTANCE.convert(new AtomicLong(100L)), is(100L));
134+
assertThat(AtomicLongToLongConverter.INSTANCE.convert(new AtomicLong(100L))).isEqualTo(100L);
137135
}
138136

139137
@Test // DATAMONGO-1416
140138
public void convertsAtomicIntegerToIntegerCorrectly() {
141-
assertThat(AtomicIntegerToIntegerConverter.INSTANCE.convert(new AtomicInteger(100)), is(100));
139+
assertThat(AtomicIntegerToIntegerConverter.INSTANCE.convert(new AtomicInteger(100))).isEqualTo(100);
142140
}
143141

144142
@Test // DATAMONGO-1416
145143
public void convertsLongToAtomicLongCorrectly() {
146-
assertThat(LongToAtomicLongConverter.INSTANCE.convert(100L), is(instanceOf(AtomicLong.class)));
144+
assertThat(LongToAtomicLongConverter.INSTANCE.convert(100L)).isInstanceOf(AtomicLong.class);
147145
}
148146

149147
@Test // DATAMONGO-1416
150148
public void convertsIntegerToAtomicIntegerCorrectly() {
151-
assertThat(IntegerToAtomicIntegerConverter.INSTANCE.convert(100), is(instanceOf(AtomicInteger.class)));
149+
assertThat(IntegerToAtomicIntegerConverter.INSTANCE.convert(100)).isInstanceOf(AtomicInteger.class);
152150
}
153151

154152
@Test // DATAMONGO-2113
155153
public void convertsBsonTimestampToInstantCorrectly() {
156-
157-
Assertions.assertThat(BsonTimestampToInstantConverter.INSTANCE.convert(new BsonTimestamp(6615900307735969796L)))
154+
155+
assertThat(BsonTimestampToInstantConverter.INSTANCE.convert(new BsonTimestamp(6615900307735969796L)))
158156
.isCloseTo(Instant.ofEpochSecond(1540384327), new TemporalUnitLessThanOffset(100, ChronoUnit.MILLIS));
159157
}
160158

0 commit comments

Comments
 (0)