Skip to content

Commit fe17ca4

Browse files
committed
test(server/ClientXmlReadTest): simplify assertions
1 parent bfe52a4 commit fe17ca4

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

server/test/sc/server/client/MessageListener.kt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ package sc.server.client
33
import io.kotest.matchers.collections.shouldNotBeEmpty
44
import io.kotest.matchers.should
55
import io.kotest.matchers.types.beInstanceOf
6-
import io.kotest.matchers.types.shouldBeInstanceOf
76
import kotlinx.coroutines.runBlocking
87
import sc.server.network.await
98
import java.util.ArrayDeque
@@ -13,7 +12,7 @@ import kotlin.reflect.cast
1312
import kotlin.time.ExperimentalTime
1413

1514
open class MessageListener<T: Any> {
16-
private val messages: Queue<T> = ArrayDeque()
15+
protected val messages: Queue<T> = ArrayDeque()
1716

1817
fun addMessage(message: T) =
1918
messages.add(message)

server/test/sc/server/network/ClientXmlReadTest.java

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
package sc.server.network;
22

3-
import org.junit.jupiter.api.Assertions;
43
import org.junit.jupiter.api.Test;
54
import org.junit.jupiter.api.Timeout;
65
import sc.networking.clients.XStreamClient;
@@ -9,6 +8,8 @@
98

109
import java.io.IOException;
1110

11+
import static org.junit.jupiter.api.Assertions.*;
12+
1213
public class ClientXmlReadTest {
1314
private static class StupidClientListener implements IClientRequestListener {
1415
public Object lastPacket = null;
@@ -33,9 +34,9 @@ public void clientReceivePacketTest() throws IOException, InterruptedException {
3334
client.setRequestHandler(clientListener);
3435
client.start();
3536

36-
Assertions.assertNotNull(client.receive());
37-
Assertions.assertNotNull(clientListener.lastPacket);
38-
Assertions.assertTrue(clientListener.lastPacket instanceof ExamplePacket);
37+
assertNotNull(client.receive());
38+
assertNotNull(clientListener.lastPacket);
39+
assertTrue(clientListener.lastPacket instanceof ExamplePacket);
3940
}
4041

4142
@Test
@@ -47,7 +48,7 @@ public void clientSendPacketTest() throws IOException {
4748
client.start();
4849
client.send(new ExamplePacket());
4950
String data = stringInterface.getData();
50-
Assertions.assertTrue(data.startsWith("<protocol>\n <example"));
51+
assertEquals(data, "<protocol>\n <example/>");
5152
}
5253

5354
private void aliasExamplePacket(XStreamClient client) {

0 commit comments

Comments
 (0)