Skip to content

Commit 07d4e28

Browse files
committed
Fix test code
AssertJ changed some APIs so make tests compliant with new APIs.
1 parent d00d8b2 commit 07d4e28

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

core/sdk-core/src/test/java/software/amazon/awssdk/core/SdkNumberTest.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -148,8 +148,8 @@ public void numberFromString() {
148148

149149
@Test
150150
public void numberFromNaNDouble() {
151-
final SdkNumber sdkNan = SdkNumber.fromDouble(Double.longBitsToDouble(0x7ff8000000000000L));
152-
final Double nanDouble = Double.longBitsToDouble(0x7ff8000000000000L);
151+
final SdkNumber sdkNan = SdkNumber.fromDouble(Double.NaN);
152+
final Double nanDouble = Double.NaN;
153153
assertThat(nanDouble.isNaN()).isTrue();
154154
assertThatThrownBy(() -> sdkNan.bigDecimalValue()).isInstanceOf(NumberFormatException.class);
155155
assertEqualitySDKNumberWithNumber(sdkNan, nanDouble);
@@ -207,7 +207,8 @@ private void assertEqualitySDKNumberWithNumber(SdkNumber sdkNan, Number nanDoubl
207207
assertThat(sdkNan.intValue()).isEqualTo(nanDouble.intValue());
208208
assertThat(sdkNan).hasToString(nanDouble.toString());
209209
assertThat(sdkNan.byteValue()).isEqualTo(nanDouble.byteValue());
210-
assertThat(sdkNan.doubleValue()).isEqualTo(nanDouble.doubleValue());
210+
// convert to nullable Double to prevent comparison of primitives because Double.NaN == Double.NaN is false
211+
assertThat(Double.valueOf(sdkNan.doubleValue())).isEqualTo(Double.valueOf(nanDouble.doubleValue()));
211212
assertThat(sdkNan.byteValue()).isEqualTo(nanDouble.byteValue());
212213
}
213214

services-custom/dynamodb-enhanced/src/test/java/software/amazon/awssdk/enhanced/dynamodb/mocktests/BatchGetItemTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ public void responseWithUnprocessedKeys_iteratePage_shouldFetchUnprocessedKeys()
114114

115115
BatchGetResultPage secondPage = iterator.next();
116116
assertThat(secondPage.resultsForTable(table).size()).isEqualTo(1);
117-
assertThat(iterator).isEmpty();
117+
assertThat(iterator).isExhausted();
118118
}
119119

120120
@Test

0 commit comments

Comments
 (0)