delete comments

Author: nglmercer

src/test/java/ServerAPITest.java

@@ -1,15 +1,15 @@
 import io.swagger.v3.core.util.Json;
-import org.json.JSONArray; // Add this import
-import org.json.JSONObject; // Add this import
+import org.json.JSONArray;  
+import org.json.JSONObject;  
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
 
 import java.io.IOException;
 import java.net.HttpURLConnection;
 
-import static org.junit.jupiter.api.Assertions.assertEquals; // Add this import
-import static org.junit.jupiter.api.Assertions.assertTrue; // Add this import
-import static org.junit.jupiter.api.Assertions.assertNotNull; // Add this import
+import static org.junit.jupiter.api.Assertions.assertEquals;  
+import static org.junit.jupiter.api.Assertions.assertTrue;  
+import static org.junit.jupiter.api.Assertions.assertNotNull;  
 
 public class ServerAPITest extends ApiTestHelper {
     @Test
@@ -159,13 +159,12 @@ public void testServerLogEndpoint() throws IOException {
 
 @Test
 void testExecMultipleCommands() throws Exception {
-    // Prepare JSON payload
     JSONObject payload = new JSONObject();
     JSONArray commands = new JSONArray();
     commands.put("say Hello from testExecMultipleCommands1");
     commands.put("time set day");
-    commands.put("nonexistentcommandtest"); // A command expected to fail
-    commands.put(""); // An empty command
+    commands.put("nonexistentcommandtest"); 
+    commands.put("");
     payload.put("commands", commands);
 
     // Perform POST request
@@ -184,36 +183,19 @@ void testExecMultipleCommands() throws Exception {
     JSONObject result1 = resultsArray.getJSONObject(0);
     assertEquals("say Hello from testExecMultipleCommands1", result1.getString("command"));
     assertTrue(result1.getBoolean("success"));
-    // Output for 'say' command can vary, so we check if it's not empty or if it contains a known part.
-    // For testing purposes, we'll assume it contains the message.
-    // Depending on server setup, 'say' might not produce direct string output here in the same way console commands do.
-    // If Bukkit.dispatchCommand for 'say' doesn't populate outputCapture in this test environment, this might need adjustment.
-    // For now, let's assume it might be empty or specific.
-    //assertTrue(result1.getString("output").contains("Hello from testExecMultipleCommands1") || result1.getString("output").isEmpty());
-
 
     // Command 2: time set day
     JSONObject result2 = resultsArray.getJSONObject(1);
     assertEquals("time set day", result2.getString("command"));
     assertTrue(result2.getBoolean("success"));
-    // Similar to 'say', 'time set day' output might be minimal or environment-dependent.
-    //assertTrue(result2.getString("output").contains("Set the time to") || result2.getString("output").isEmpty());
-
-
+ 
     // Command 3: nonexistentcommandtest
     JSONObject result3 = resultsArray.getJSONObject(2);
     assertEquals("nonexistentcommandtest", result3.getString("command"));
-    // This depends on how the server handles unknown commands.
-    // Bukkit.dispatchCommand usually returns false for unknown commands.
-    // assertTrue(!result3.getBoolean("success")); // This might be true or false depending on server behavior for unknown commands.
-    // The output usually contains "Unknown command".
-    // assertTrue(result3.getString("output").toLowerCase().contains("unknown command"));
-
 
     // Command 4: Empty command
     JSONObject result4 = resultsArray.getJSONObject(3);
-    //assertEquals(JSONObject.NULL, result4.get("command")); // This was in the plan, but JSONObject.NULL might not be directly comparable like this.
-                                                       // Let's check for the output message instead.
+
     assertTrue(result4.isNull("command") || "".equals(result4.optString("command"))); // Check if command is null or empty string
     assertEquals(false, result4.getBoolean("success"));
     assertEquals("Empty command string provided.", result4.getString("output"));