Merge pull request #69 from sashirestela/67-multipart-convert-collect…

…ions-to-bytearray

Fixing race conditions on unit tests

src/test/java/io/github/sashirestela/cleverclient/http/HttpProcessorTest.java

@@ -4,6 +4,7 @@
 import io.github.sashirestela.cleverclient.support.Configurator;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Order;
 import org.junit.jupiter.api.Test;
 
 import java.io.ByteArrayInputStream;
@@ -30,6 +31,7 @@
 import static org.mockito.Mockito.when;
 
 @SuppressWarnings("unchecked")
+@Order(2)
 class HttpProcessorTest {
 
     HttpProcessor httpProcessor;

src/test/java/io/github/sashirestela/cleverclient/support/CleverClientSSETest.java

@@ -2,12 +2,14 @@
 
 import io.github.sashirestela.cleverclient.support.CleverClientSSE.LineRecord;
 import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Order;
 import org.junit.jupiter.api.Test;
 
 import java.util.Set;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
+@Order(1)
 class CleverClientSSETest {
 
     Set<String> events = Set.of("process", "process2");
@@ -36,7 +38,8 @@ void shouldReturnExpectedValueWhenRawDataMeetsConditions() {
             var event = (CleverClientSSE) data[0];
             var actualCondition = event.isActualData();
             var expectedCondition = (boolean) data[1];
-            assertEquals(expectedCondition, actualCondition);
+            assertEquals(expectedCondition, actualCondition, "For data '" + event.getActualData()
+                    + "' was expecting " + expectedCondition + " but was " + actualCondition);
         }
     }
 

src/test/resources/junit-platform.properties

@@ -0,0 +1,2 @@
+# ClassOrderer$OrderAnnotation sorts classes based on their @Order annotation
+junit.jupiter.testclass.order.default=org.junit.jupiter.api.ClassOrderer$OrderAnnotation