Skip to content
This repository was archived by the owner on Oct 30, 2023. It is now read-only.

Commit e7995c0

Browse files
committed
Add unit tests for NewOrderResponse.
1 parent 2da9453 commit e7995c0

2 files changed

Lines changed: 50 additions & 0 deletions

File tree

src/main/java/com/binance/api/client/domain/account/NewOrderResponse.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@ public class NewOrderResponse {
5353

5454
private OrderSide side;
5555

56+
// @JsonSetter(nulls = Nulls.AS_EMPTY)
5657
private List<Trade> fills;
5758

5859
/**
Lines changed: 49 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,49 @@
1+
package com.binance.api.client.domain.account;
2+
3+
import org.junit.Before;
4+
import org.junit.Test;
5+
6+
import java.util.Arrays;
7+
import java.util.Collections;
8+
import java.util.List;
9+
10+
import static org.hamcrest.CoreMatchers.containsString;
11+
import static org.hamcrest.CoreMatchers.is;
12+
import static org.junit.Assert.assertThat;
13+
14+
/**
15+
* @see NewOrderResponse
16+
*/
17+
public class NewOrderResponseTest {
18+
19+
private NewOrderResponse newOrderResponse;
20+
private Trade trade;
21+
22+
@Before
23+
public void setUp() {
24+
newOrderResponse = new NewOrderResponse();
25+
trade = new Trade();
26+
trade.setId(123L);
27+
}
28+
29+
@Test
30+
public void shouldHandleToStringWithNullFills() {
31+
assertThat(newOrderResponse.toString(), containsString(",fills="));
32+
}
33+
34+
@Test
35+
public void shouldHandleToStringWithNoFills() {
36+
newOrderResponse.setFills(Collections.emptyList());
37+
assertThat(newOrderResponse.toString(), containsString(",fills="));
38+
}
39+
40+
@Test
41+
public void shouldHandleToStringWithFills() {
42+
newOrderResponse.setFills(trades(trade));
43+
assertThat(newOrderResponse.toString(), containsString(",fills=Trade[id=123,"));
44+
}
45+
46+
private static List<Trade> trades(final Trade... trades) {
47+
return Arrays.asList(trades);
48+
}
49+
}

0 commit comments

Comments
 (0)