all tests except file and exchange IO

Author: Katsute

README.md

@@ -1,4 +1,7 @@
 <div id="top" align="center">
+    <a href="https://github.com/KatsuteDev/simplehttpserver">
+        <img src="https://raw.githubusercontent.com/KatsuteDev/simplehttpserver/main/assets/icon.png" alt="icon" width="100" height="100">
+    </a>
     <h3>SimpleHttpServer</h3>
     <h5>A simple and efficient HTTP server for Java</h5>
     <div>
@@ -20,8 +23,7 @@
 
 <br>
 
-> **Warning**:
-> simplehttpserver5 is not compatible with any previous version of [simplehttpserver](https://github.com/Ktt-Development/simplehttpserver)
+> ⚠️ simplehttpserver5 is not compatible with any previous version of [simplehttpserver](https://github.com/Ktt-Development/simplehttpserver)
 
 🚧 WIP
 

assets/icon.png

@@ -247,6 +247,7 @@
                         <exclude/>
                     </excludes>
                     <trimStackTrace>false</trimStackTrace>
+                    <reuseForks>false</reuseForks>
                 </configuration>
             </plugin>
         </plugins>

pom.xml

@@ -0,0 +1,7 @@
+package dev.katsute.simplehttpserver.exchange;
+
+final class ExchangeIOTests {
+
+    // https://github.com/Ktt-Development/simplehttpserver/tree/main/src/test/java/com/kttdevelopment/simplehttpserver/simplehttpexchange/io
+
+}

src/test/java/dev/katsute/simplehttpserver/exchange/ExchangeIOTests.java

@@ -0,0 +1,7 @@
+package dev.katsute.simplehttpserver.handler;
+
+final class FileHandlerTests {
+
+    // https://github.com/Ktt-Development/simplehttpserver/tree/main/src/test/java/com/kttdevelopment/simplehttpserver/handlers/file
+
+}

src/test/java/dev/katsute/simplehttpserver/handler/FileHandlerTests.java

@@ -0,0 +1,37 @@
+package dev.katsute.simplehttpserver.server;
+
+import dev.katsute.simplehttpserver.SimpleHttpServer;
+import org.junit.jupiter.api.*;
+
+import java.io.IOException;
+import java.net.BindException;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+final class ServerBindTests {
+
+    @Test
+    final void testBind() throws IOException{
+        final SimpleHttpServer server = SimpleHttpServer.create();
+        assertNull(server.getAddress());
+        assertDoesNotThrow(() -> server.bind(8080));
+        assertTrue(server.getAddress().getAddress().isAnyLocalAddress());
+
+        // required to unbind
+        server.start();
+        server.stop();
+    }
+
+    @Test
+    final void testOccupied() throws IOException{
+        final SimpleHttpServer server = SimpleHttpServer.create(8080);
+        server.start();
+
+        Assertions.assertThrows(BindException.class, () -> SimpleHttpServer.create(8080));
+
+        server.stop();
+
+        Assertions.assertDoesNotThrow(() -> SimpleHttpServer.create(8080));
+    }
+
+}

src/test/java/dev/katsute/simplehttpserver/server/ServerBindTests.java

@@ -0,0 +1,46 @@
+package dev.katsute.simplehttpserver.server;
+
+import dev.katsute.simplehttpserver.SimpleHttpServer;
+import dev.katsute.simplehttpserver.SimpleHttpsServer;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+
+import java.io.IOException;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+final class ServerCreateTests {
+
+    @Test
+    final void testHttpCreate() throws IOException{
+        final SimpleHttpServer server = SimpleHttpServer.create();
+
+        assertThrows(IllegalStateException.class, server::start);
+
+        server.bind(8080);
+        assertEquals(8080, server.getAddress().getPort());
+
+        assertDoesNotThrow(server::start);
+        assertThrows(IllegalStateException.class, server::start);
+
+        assertDoesNotThrow(() -> server.stop());
+        assertDoesNotThrow(() -> server.stop(), "Second stop should not throw an exception");
+    }
+
+    @Test
+    final void testHttpsCreate() throws IOException{
+        final SimpleHttpsServer server = SimpleHttpsServer.create();
+
+        assertThrows(IllegalStateException.class, server::start);
+
+        server.bind(8080);
+        assertEquals(8080, server.getAddress().getPort());
+
+        assertDoesNotThrow(server::start);
+        assertThrows(IllegalStateException.class, server::start);
+
+        assertDoesNotThrow(() -> server.stop());
+        assertDoesNotThrow(() -> server.stop(), "Second stop should not throw an exception");
+    }
+
+}

src/test/java/dev/katsute/simplehttpserver/server/ServerCreateTests.java

@@ -1,12 +1,11 @@
-package dev.katsute.simplehttpserver;
+package dev.katsute.simplehttpserver.server;
 
 import com.sun.net.httpserver.HttpContext;
 import com.sun.net.httpserver.HttpExchange;
 import dev.katsute.simplehttpserver.*;
-import org.junit.jupiter.api.*;
+import org.junit.jupiter.api.Test;
 
 import java.io.IOException;
-import java.net.BindException;
 
 import static org.junit.jupiter.api.Assertions.*;
 
@@ -26,86 +25,9 @@ final void testProp() throws IOException{
         assertTrue(server.getContexts().isEmpty());
     }
 
-    @Nested
-    final class BindTests {
+    //
 
-        @BeforeEach
-        final void beforeEach() throws InterruptedException{
-            Thread.sleep(500);
-        }
-
-        @Test
-        final void testBind() throws IOException{
-            final SimpleHttpServer server = SimpleHttpServer.create();
-            assertNull(server.getAddress());
-            assertDoesNotThrow(() -> server.bind(8080));
-            assertTrue(server.getAddress().getAddress().isAnyLocalAddress());
-
-            // required to unbind
-            server.start();
-            server.stop();
-        }
-
-        @Test
-        final void testOccupied() throws IOException{
-            final SimpleHttpServer server = SimpleHttpServer.create(8080);
-            server.start();
-
-            Assertions.assertThrows(BindException.class, () -> SimpleHttpServer.create(8080));
-
-            server.stop();
-
-            Assertions.assertDoesNotThrow(() -> SimpleHttpServer.create(8080));
-        }
-
-    }
-
-    @Nested
-    final class CreateTests {
-
-        @BeforeEach
-        final void beforeEach() throws InterruptedException{
-            Thread.sleep(500);
-        }
-
-        @Test
-        final void testHttpCreate() throws IOException{
-            final SimpleHttpServer server = SimpleHttpServer.create();
-
-            assertThrows(IllegalStateException.class, server::start);
-
-            server.bind(8080);
-            assertEquals(8080, server.getAddress().getPort());
-
-            assertDoesNotThrow(server::start);
-            assertThrows(IllegalStateException.class, server::start);
-
-            assertDoesNotThrow(() -> server.stop());
-            assertDoesNotThrow(() -> server.stop(), "Second stop should not throw an exception");
-        }
-
-        @Test
-        final void testHttpsCreate() throws IOException{
-            final SimpleHttpsServer server = SimpleHttpsServer.create();
-
-            assertThrows(IllegalStateException.class, server::start);
-
-            server.bind(8080);
-            assertEquals(8080, server.getAddress().getPort());
-
-            assertDoesNotThrow(server::start);
-            assertThrows(IllegalStateException.class, server::start);
-
-            assertDoesNotThrow(() -> server.stop());
-            assertDoesNotThrow(() -> server.stop(), "Second stop should not throw an exception");
-        }
-
-    }
-
-    @Nested
-    final class ContextTests {
-
-        @Test
+    @Test
         final void testRandomContext() throws IOException{
             final SimpleHttpServer server = SimpleHttpServer.create();
 
@@ -192,6 +114,4 @@ final void testDuplicateContext() throws IOException{
             server.createContext("", HttpExchange::close); // supposed to throw an exception, docs are invalid
         }
 
-    }
-
 }