Add ServerControllerTests#ensureUnknownPort

src/main/java/me/braydon/mc/controller/ServerController.java

@@ -65,7 +65,7 @@ public final class ServerController {
      * @param hostname the hostname of the server
      * @param port     the port of the server, null for default
      * @return the server
-     * @throws BadRequestException       if the hostname or platform is invalid
+     * @throws BadRequestException       if the hostname, platform, or port is invalid
      * @throws ResourceNotFoundException if the server isn't found
      */
     @GetMapping("/{platform}/{hostname}")

src/main/java/me/braydon/mc/service/MojangService.java

@@ -347,7 +347,7 @@ public final class MojangService {
      * @param hostname     the hostname of the server
      * @param portString   the port of the server, null for default
      * @return the resolved Minecraft server
-     * @throws BadRequestException       if the hostname or platform is invalid
+     * @throws BadRequestException       if the hostname, platform, or port is invalid
      * @throws ResourceNotFoundException if the server isn't found
      */
     @NonNull

src/test/java/me/braydon/mc/test/controller/ServerControllerTests.java

@@ -120,6 +120,21 @@ public final class ServerControllerTests {
                 .andReturn();
     }
 
+    /**
+     * Run a test to ensure looking up
+     * an invalid port results in a 400.
+     *
+     * @throws Exception if the test fails
+     */
+    @Test
+    void ensureUnknownPort() throws Exception {
+        mockMvc.perform(get("/server/java/hypixel.net?port=A")
+                        .accept(MediaType.APPLICATION_JSON) // Accept JSON
+                        .contentType(MediaType.APPLICATION_JSON) // Content type is JSON
+                ).andExpect(status().isBadRequest()) // Expect 400 (Bad Request)
+                .andReturn();
+    }
+
     /**
      * Run a test to ensure checking if
      * a server is banned is successful.