Updates software version to 1.3.3

This fixes a bug where players with underscores in their names won't be added to the player list

.gitignore

@@ -28,4 +28,5 @@ hs_err_pid*
 
 bin/
 out/
+.idea
 *.txt

pom.xml

@@ -126,9 +126,9 @@
             <version>2.8.5</version>
         </dependency>
         <dependency>
-            <groupId>junit</groupId>
+            <groupId>org.junit.jupiter</groupId>
-            <artifactId>junit</artifactId>
+            <artifactId>junit-jupiter-api</artifactId>
-            <version>4.11</version>
+            <version>5.7.2</version>
             <scope>test</scope>
         </dependency>
     </dependencies>

src/main/java/net/knarcraft/minecraftserverlauncher/Main.java

@@ -1,15 +1,16 @@
 package net.knarcraft.minecraftserverlauncher;
 
 import net.knarcraft.minecraftserverlauncher.profile.Collection;
-import net.knarcraft.minecraftserverlauncher.profile.Controller;
+import net.knarcraft.minecraftserverlauncher.profile.ServerLauncherController;
+import net.knarcraft.minecraftserverlauncher.server.Server;
 import net.knarcraft.minecraftserverlauncher.userinterface.ServerConsoles;
 import net.knarcraft.minecraftserverlauncher.userinterface.ServerLauncherGUI;
+import net.knarcraft.minecraftserverlauncher.utility.CommonFunctions;
 import net.knarcraft.minecraftserverlauncher.utility.Updater;
 
 import java.awt.*;
 import java.io.File;
 import java.io.IOException;
-import java.io.PrintWriter;
 import java.net.URISyntaxException;
 import java.util.concurrent.Executors;
 import java.util.concurrent.ScheduledExecutorService;
@@ -27,21 +28,17 @@ import java.util.concurrent.TimeUnit;
 public class Main {
     private static final String updateChannel = "beta";
     private static final String updateURL = "https://api.knarcraft.net/minecraftserverlauncher";
-    private static final Controller controller = Controller.getInstance();
+    private static final ServerLauncherController controller = ServerLauncherController.getInstance();
     private static String applicationWorkDirectory;
     private static boolean serversAreRunning = false;
     private static ServerLauncherGUI gui;
 
     public static void main(String[] args) throws IOException {
         Updater.checkForUpdate(updateURL, updateChannel);
-        try (PrintWriter file = new PrintWriter(Main.getApplicationWorkDirectory() + File.separator + "latestrun.log")) {
+        CommonFunctions.writeFile(Main.getApplicationWorkDirectory() + File.separator + "latestrun.log", "");
-            file.print("");
-        } catch (IOException e) {
-            e.printStackTrace();
-        }
         EventQueue.invokeLater(() -> {
             try {
-                new ServerConsoles();
+                ServerConsoles.instantiate();
                 controller.loadState();
                 gui = controller.getGUI();
                 ScheduledExecutorService exec = Executors.newSingleThreadScheduledExecutor();
@@ -57,7 +54,7 @@ public class Main {
      *
      * @return <p>The controller used by the software</p>
      */
-    public static Controller getController() {
+    public static ServerLauncherController getController() {
         return controller;
     }
 
@@ -85,6 +82,7 @@ public class Main {
         boolean runningNew = serversRunning();
         if (serversAreRunning && !runningNew) {
             //Servers stopped running
+            Server.serversStopped();
             gui.updateGUIElementsWhenServersStartOrStop(false);
             gui.setStatus("Servers are stopped");
         } else if (!serversAreRunning && runningNew) {

src/main/java/net/knarcraft/minecraftserverlauncher/profile/Profile.java

@@ -8,7 +8,7 @@ import net.knarcraft.minecraftserverlauncher.utility.CommonFunctions;
 
 import javax.naming.ConfigurationException;
 import javax.swing.*;
-import java.io.*;
+import java.io.IOException;
 import java.util.ArrayList;
 
 /**
@@ -230,7 +230,9 @@ public class Profile {
      */
     public static Profile fromString(String profileString) throws ConfigurationException {
         Profile profile;
-        if (profileString.contains("?")) {
+        if (profileString.equals("")) {
+            return null;
+        } else if (profileString.contains("?")) {
             String[] data = profileString.split("\\?");
             String[] profileData = data[0].split(";", -1);
             profile = parseProfile(profileData);

src/main/java/net/knarcraft/minecraftserverlauncher/profile/ServerLauncherController.java

@@ -9,16 +9,20 @@ import net.knarcraft.minecraftserverlauncher.utility.CommonFunctions;
 import net.knarcraft.minecraftserverlauncher.utility.JarDownloader;
 
 import javax.naming.ConfigurationException;
-import java.io.*;
+import java.io.File;
+import java.io.FileNotFoundException;
+import java.io.IOException;
 import java.util.ArrayList;
+import java.util.HashMap;
 import java.util.List;
+import java.util.Map;
 import java.util.Scanner;
 import java.util.concurrent.Executors;
 
 /**
  * This class handles profiles, GUI creation and session restoration
  */
-public class Controller {
+public class ServerLauncherController {
 
     private final String workingDirectory = Main.getApplicationWorkDirectory() + File.separator + "files";
     private final String profilesFile = workingDirectory + File.separator + "Profiles.txt";
@@ -28,12 +32,14 @@ public class Controller {
     private ServerLauncherGUI serverLauncherGUI;
     private Profile currentProfile;
     private boolean downloadAllJars;
-    private static Controller controller;
+    private static ServerLauncherController controller;
+    private String javaCommand = "java";
+    private String oldJavaCommand = "java";
 
     /**
      * Instantiates a new controller
      */
-    private Controller() {
+    private ServerLauncherController() {
         this.profileList = new ArrayList<>();
     }
 
@@ -42,9 +48,9 @@ public class Controller {
      *
      * @return <p>An instance of the controller</p>
      */
-    public static Controller getInstance() {
+    public static ServerLauncherController getInstance() {
         if (controller == null) {
-            controller = new Controller();
+            controller = new ServerLauncherController();
         }
         return controller;
     }
@@ -85,6 +91,47 @@ public class Controller {
         return currentProfile.getDelayStartup();
     }
 
+    /**
+     * Gets the command for running java
+     *
+     * @return <p>The command for running Java</p>
+     */
+    public String getJavaCommand() {
+        return javaCommand;
+    }
+
+    /**
+     * Gets the command for running older versions of Java
+     *
+     * <p>The command used to run older minecraft server versions. JRE 8 should probably work.
+     * Can be just "java" or a file path.</p>
+     *
+     * @return <p>The command for running older versions of Java</p>
+     */
+    public String getOldJavaCommand() {
+        return oldJavaCommand;
+    }
+
+    /**
+     * Sets the command for running Java
+     *
+     * <p>To play on the newest version of Minecraft, this needs to be JDK 16. Can be just "java" or a file path.</p>
+     *
+     * @param javaCommand <p>The command used for running Java</p>
+     */
+    public void setJavaCommand(String javaCommand) {
+        this.javaCommand = javaCommand;
+    }
+
+    /**
+     * Sets the command for running older versions of java
+     *
+     * @param oldJavaCommand <p>The command used for running older versions of Java</p>
+     */
+    public void setOldJavaCommand(String oldJavaCommand) {
+        this.oldJavaCommand = oldJavaCommand;
+    }
+
     @Override
     public String toString() {
         int guiWidth;
@@ -96,7 +143,8 @@ public class Controller {
             guiWidth = serverLauncherGUI.getSize().width;
             guiHeight = serverLauncherGUI.getSize().height;
         }
-        return String.format("%s;%d;%d;%b", currentProfile.getName(), guiWidth, guiHeight, downloadAllJars);
+        return String.format("selectedProfile;%s\nguiWidth;%d\nguiHeight;%d\ndownloadAllJars;%b\njavaCommand;%s\noldJavaCommand;%s",
+                currentProfile.getName(), guiWidth, guiHeight, downloadAllJars, javaCommand, oldJavaCommand);
     }
 
     /**
@@ -171,37 +219,135 @@ public class Controller {
     }
 
     /**
-     * Loads data about the controller from a save string
+     * Sets the download all jars option
      *
-     * @param data <p>The save string to load</p>
+     * @param downloadAllJars <p>Whether to download all jars</p>
-     * @return <p>The currently selected profile</p>
-     * @throws IOException <p>If unable to create the GUI</p>
      */
-    private String fromString(String data) throws IOException {
+    public void setDownloadAllJars(boolean downloadAllJars) {
-        String[] dataList = data.split(";");
+        this.downloadAllJars = downloadAllJars;
-        this.downloadAllJars = Boolean.parseBoolean(dataList[3]);
+    }
-        this.serverLauncherGUI = new ServerLauncherGUI(Integer.parseInt(dataList[1]), Integer.parseInt(dataList[2]));
+
-        return dataList[0];
+    /**
+     * Sets the currently selected profile
+     *
+     * @param profileName <p>The name of the currently selected profile</p>
+     */
+    public void setCurrentProfile(String profileName) {
+        this.currentProfile = getProfileByName(profileName);
+    }
+
+    /**
+     * Gets the directory containing .jar files
+     *
+     * @return <p>The directory containing .jar files</p>
+     */
+    public String getJarDirectory() {
+        return this.jarDirectory;
+    }
+
+    /**
+     * Saves the state of the entire application to disk
+     */
+    public void saveState() {
+        saveGUIStateToServer();
+        try {
+            CommonFunctions.createAllFolders();
+            CommonFunctions.writeFile(mainFile, this.toString());
+            StringBuilder builder = new StringBuilder();
+            for (Profile profile : profileList) {
+                builder.append(profile.toString()).append("\n");
+            }
+            CommonFunctions.writeFile(profilesFile, builder.toString());
+        } catch (IOException e) {
+            serverLauncherGUI.showError("Unable to save data.");
+            e.printStackTrace();
+        }
     }
 
     /**
      * Reads profiles and servers from a text file
      */
     public void loadState() throws ConfigurationException, IOException {
+        loadState(false);
+    }
+
+    /**
+     * Reads profiles and servers from a text file
+     *
+     * @param silent <p>If silent, no GUI will be created</p>
+     */
+    public void loadState(boolean silent) throws ConfigurationException, IOException {
         //Loads data regarding this controller
         String currentProfile = null;
         if (new File(mainFile).exists()) {
-            String controllerData = CommonFunctions.readBufferedReader(new BufferedReader(new InputStreamReader(new FileInputStream(mainFile))));
+            currentProfile = this.fromString(silent);
-            currentProfile = this.fromString(controllerData);
+        } else if (!silent) {
-        } else {
             this.serverLauncherGUI = new ServerLauncherGUI();
         }
+        if (silent) {
+            return;
+        }
         //Loads all saved profiles
         loadProfiles(currentProfile);
         //Makes the GUI show the loaded data
         executeGUILoadingTasks();
     }
 
+    /**
+     * Loads data about the controller from a save string
+     *
+     * @param silent <p>If silent, no GUI will be created</p>
+     * @return <p>The currently selected profile</p>
+     */
+    private String fromString(boolean silent) {
+        Map<String, String> loadedData = new HashMap<>();
+
+        try {
+            String currentData = CommonFunctions.readFile(mainFile);
+            for (String line : currentData.split("\n")) {
+                parseSaveLine(loadedData, line);
+            }
+            this.downloadAllJars = Boolean.parseBoolean(loadedData.get("downloadAllJars"));
+
+            if (!silent) {
+                if (loadedData.get("guiWidth") != null && loadedData.get("guiHeight") != null) {
+                    this.serverLauncherGUI = new ServerLauncherGUI(Integer.parseInt(loadedData.get("guiWidth")),
+                            Integer.parseInt(loadedData.get("guiHeight")));
+                } else {
+                    this.serverLauncherGUI = new ServerLauncherGUI();
+                }
+            } else {
+                this.serverLauncherGUI = new ServerLauncherGUI(true);
+            }
+
+            if (loadedData.get("javaCommand") != null) {
+                this.javaCommand = loadedData.get("javaCommand");
+            }
+            if (loadedData.get("oldJavaCommand") != null) {
+                this.oldJavaCommand = loadedData.get("oldJavaCommand");
+            }
+            return loadedData.get("selectedProfile");
+        } catch (IOException e) {
+            e.printStackTrace();
+            return null;
+        }
+    }
+
+    /**
+     * Saves one config line to the provided map
+     *
+     * @param loadedData <p>The map to save the loaded configuration to</p>
+     * @param line       <p>The line to read</p>
+     */
+    private void parseSaveLine(Map<String, String> loadedData, String line) {
+        String[] lineData = line.split(";");
+        if (lineData.length != 2) {
+            return;
+        }
+
+        loadedData.put(lineData[0], lineData[1]);
+    }
+
     /**
      * Loads all saved profiles
      *
@@ -213,6 +359,9 @@ public class Controller {
             while (in.hasNextLine()) {
                 String profileData = in.nextLine();
                 Profile loadedProfile = Profile.fromString(profileData);
+                if (loadedProfile == null) {
+                    continue;
+                }
                 profileList.add(loadedProfile);
             }
         } catch (FileNotFoundException e) {
@@ -248,45 +397,6 @@ public class Controller {
         }
     }
 
-    /**
-     * Sets the download all jars option
-     *
-     * @param downloadAllJars <p>Whether to download all jars</p>
-     */
-    public void setDownloadAllJars(boolean downloadAllJars) {
-        this.downloadAllJars = downloadAllJars;
-    }
-
-    /**
-     * Sets the currently selected profile
-     *
-     * @param profileName <p>The name of the currently selected profile</p>
-     */
-    public void setCurrentProfile(String profileName) {
-        this.currentProfile = getProfileByName(profileName);
-    }
-
-    /**
-     * Saves the state of the entire application to disk
-     */
-    public void saveState() {
-        saveGUIStateToServer();
-        try {
-            CommonFunctions.createAllFolders();
-            PrintWriter mFile = new PrintWriter(mainFile);
-            mFile.println(this.toString());
-            mFile.close();
-            PrintWriter pFile = new PrintWriter(profilesFile);
-            for (Profile profile : profileList) {
-                pFile.println(profile.toString());
-            }
-            pFile.close();
-        } catch (FileNotFoundException e) {
-            serverLauncherGUI.showError("Unable to save data.");
-            e.printStackTrace();
-        }
-    }
-
     /**
      * Updates the server object with the current state of the GUI server tab
      */

src/main/java/net/knarcraft/minecraftserverlauncher/server/Server.java

@@ -2,7 +2,7 @@ package net.knarcraft.minecraftserverlauncher.server;
 
 import net.knarcraft.minecraftserverlauncher.Main;
 import net.knarcraft.minecraftserverlauncher.profile.Collection;
-import net.knarcraft.minecraftserverlauncher.profile.Controller;
+import net.knarcraft.minecraftserverlauncher.profile.ServerLauncherController;
 import net.knarcraft.minecraftserverlauncher.server.servertypes.ServerType;
 import net.knarcraft.minecraftserverlauncher.utility.CommonFunctions;
 
@@ -19,8 +19,8 @@ import java.util.List;
 import java.util.concurrent.Executors;
 import java.util.concurrent.ScheduledExecutorService;
 import java.util.concurrent.TimeUnit;
-
+import java.util.regex.Matcher;
-import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.stringBetween;
+import java.util.regex.Pattern;
 
 
 /**
@@ -51,6 +51,7 @@ public class Server {
     private BufferedReader reader;
     private boolean started;
     private ScheduledExecutorService consoleOutputExecutor;
+    private static boolean stoppingServers = false;
 
     /**
      * Initializes a new server with default values
@@ -108,12 +109,27 @@ public class Server {
         return this.reader;
     }
 
+    /**
+     * Marks the servers as finished stopping when a stop is confirmed
+     */
+    public static void serversStopped() {
+        if (stoppingServers) {
+            stoppingServers = false;
+        }
+    }
+
     /**
      * Tries to stop all enabled servers
      *
      * @throws IOException <p>If a writer's process is already closed but not null</p>
      */
-    public static void stop() throws IOException {
+    public static void stop() throws IOException, InterruptedException {
+        if (stoppingServers) {
+            killServers();
+            return;
+        }
+
+        stoppingServers = true;
         for (Collection collection : Main.getController().getCurrentProfile().getCollections()) {
             Server server = collection.getServer();
             if (server.writer != null) {
@@ -129,18 +145,46 @@ public class Server {
         }
     }
 
+    /**
+     * Kills all server processes
+     *
+     * @throws InterruptedException <p>If interrupted waiting for any of the servers to stop</p>
+     */
+    private static void killServers() throws InterruptedException {
+        for (Collection collection : Main.getController().getCurrentProfile().getCollections()) {
+            Server server = collection.getServer();
+            killServer(server);
+        }
+    }
+
+    /**
+     * Kills the given server after waiting 30 seconds for it to terminate normally
+     *
+     * @param server <p>The server to kill</p>
+     * @throws InterruptedException <p>If interrupted waiting for the server to stop</p>
+     */
+    private static void killServer(Server server) throws InterruptedException {
+        if (server.process != null) {
+            if (!server.process.waitFor(30, TimeUnit.SECONDS)) {
+                server.process.destroyForcibly();
+                server.process.waitFor();
+            }
+        }
+    }
+
     /**
      * Runs all enabled servers with their settings
      */
     public static void startServers() {
-        Controller controller = Main.getController();
+        ServerLauncherController controller = Main.getController();
         controller.getGUI().setStatus("Starting servers");
+        int serverNum = 0;
         for (Collection collection : controller.getCurrentProfile().getCollections()) {
-            if (!collection.getServer().runServer()) {
+            if (!collection.getServer().runServer(serverNum++ == 0)) {
-                controller.getGUI().setStatus("An error occurred. Start aborted");
+                controller.getGUI().showError("An error occurred. Start aborted. Please check the BuildTools log.");
                 try {
                     Server.stop();
-                } catch (IOException e) {
+                } catch (IOException | InterruptedException e) {
                     e.printStackTrace();
                 }
                 controller.getGUI().updateGUIElementsWhenServersStartOrStop(false);
@@ -280,9 +324,9 @@ public class Server {
     }
 
     /**
-     * Checks whether this server is fully stopped
+     * Removes all information about the server's process, writer and reader
      */
-    private void stopped() {
+    private void cleanStoppedServerValues() {
         consoleOutputExecutor.shutdown();
         process = null;
         writer = null;
@@ -357,17 +401,23 @@ public class Server {
      *
      * @return <p>True if nothing went wrong</p>
      */
-    private boolean runServer() {
+    private boolean runServer(boolean isFirstServer) {
+        if (stoppingServers) {
+            return false;
+        }
         //Ignore a disabled server
         if (!this.enabled) {
             this.started = false;
             return true;
         }
         //Tries to do necessary pre-start work
-        if (!initializeJarDownload() || !delayStartup()) {
+        if (!initializeJarDownload() || (!isFirstServer && !delayStartup())) {
             this.started = false;
             return false;
         }
+        if (stoppingServers) {
+            return false;
+        }
         //Starts the server if possible
         try {
             startServerProcess();
@@ -381,6 +431,23 @@ public class Server {
         }
     }
 
+    /**
+     * Gets the correct java command to use for the selected server version
+     *
+     * @return <p>The java version to run</p>
+     */
+    private String getJavaCommand() {
+        ServerLauncherController controller = ServerLauncherController.getInstance();
+        if (serverVersion.toLowerCase().contains("latest")) {
+            return controller.getJavaCommand();
+        } else if (serverVersion.contains(".") && serverVersion.split("\\.").length >= 2 &&
+                Integer.parseInt(serverVersion.split("\\.")[1]) >= 17) {
+            return controller.getJavaCommand();
+        } else {
+            return controller.getOldJavaCommand();
+        }
+    }
+
     /**
      * Starts the process running this server
      *
@@ -395,7 +462,7 @@ public class Server {
         } else {
             serverPath = jarDirectory + this.type.getName() + serverVersion + ".jar";
         }
-        builder = new ProcessBuilder("java", "-Xmx" + this.maxRam, "-Xms512M",
+        builder = new ProcessBuilder(getJavaCommand(), "-Xmx" + this.maxRam, "-Xms512M",
                 "-Djline.terminal=jline.UnsupportedTerminal", "-Dcom.mojang.eula.agree=true", "-jar", serverPath,
                 "nogui");
         builder.directory(new File(this.path));
@@ -427,7 +494,7 @@ public class Server {
             updatePlayerList(readText);
         }
         if (!getProcess().isAlive()) {
-            stopped();
+            cleanStoppedServerValues();
         }
     }
 
@@ -461,23 +528,44 @@ public class Server {
      */
     private String getPlayer(String text, boolean joined) {
         String playerName;
+
+        String loginPattern1 = " ([A-Z0-9a-z_]+)\\[/[0-9]+\\.[0-9]+\\.[0-9]+\\.[0-9]+:[0-9]+] logged in";
+        String loginPattern2 = "UUID of player ([A-Z0-9a-z_]+) is";
+
+        String logoutPattern1 = "INFO]: ([A-Z0-9a-z_]+) lost connection";
+        String logoutPattern2 = " ([A-Z0-9a-z_]+) left the game";
+
         if (joined) {
-            playerName = stringBetween(text, "[Server thread/INFO]: ", " joined the game");
+            playerName = getFirstRegexCaptureGroup(loginPattern1, text);
             if (playerName.equals("")) {
-                playerName = stringBetween(text, "UUID of player ", " is ");
+                playerName = getFirstRegexCaptureGroup(loginPattern2, text);
             }
         } else {
-            playerName = stringBetween(text, "INFO]: ", " lost connection");
+            playerName = getFirstRegexCaptureGroup(logoutPattern1, text);
             if (playerName.equals("")) {
-                playerName = stringBetween(text, "[Server thread/INFO]: ", " left the game");
+                playerName = getFirstRegexCaptureGroup(logoutPattern2, text);
-            }
-            if (playerName.equals("")) {
-                playerName = stringBetween(text, "INFO]: ", " left the game");
             }
         }
         return playerName;
     }
 
+    /**
+     * Returns the first regex capture group found in a pattern
+     *
+     * @param pattern <p>The regex pattern to use</p>
+     * @param text    <p>The string to execute the pattern on</p>
+     * @return <p>The first capture group if a match is found. An empty string otherwise</p>
+     */
+    private String getFirstRegexCaptureGroup(String pattern, String text) {
+        Pattern compiledPattern = Pattern.compile(pattern);
+        Matcher patternMatcher = compiledPattern.matcher(text);
+        if (patternMatcher.find()) {
+            return patternMatcher.group(1);
+        } else {
+            return "";
+        }
+    }
+
     /**
      * Delays the server's startup for the given amount of time
      *
@@ -501,7 +589,7 @@ public class Server {
      * @return <p>True if nothing went wrong</p>
      */
     private boolean initializeJarDownload() {
-        Controller controller = Main.getController();
+        ServerLauncherController controller = Main.getController();
         if (!controller.getDownloadAllJars()) {
             try {
                 controller.getGUI().setStatus("Downloading jar...");

src/main/java/net/knarcraft/minecraftserverlauncher/server/ServerTypeHandler.java

@@ -1,6 +1,17 @@
 package net.knarcraft.minecraftserverlauncher.server;
 
-import net.knarcraft.minecraftserverlauncher.server.servertypes.*;
+import net.knarcraft.minecraftserverlauncher.server.servertypes.BungeeCord;
+import net.knarcraft.minecraftserverlauncher.server.servertypes.CraftBukkit;
+import net.knarcraft.minecraftserverlauncher.server.servertypes.Custom;
+import net.knarcraft.minecraftserverlauncher.server.servertypes.MCPCPlus;
+import net.knarcraft.minecraftserverlauncher.server.servertypes.Paper;
+import net.knarcraft.minecraftserverlauncher.server.servertypes.ServerType;
+import net.knarcraft.minecraftserverlauncher.server.servertypes.Spigot;
+import net.knarcraft.minecraftserverlauncher.server.servertypes.SpongeForge;
+import net.knarcraft.minecraftserverlauncher.server.servertypes.SpongeVanilla;
+import net.knarcraft.minecraftserverlauncher.server.servertypes.Travertine;
+import net.knarcraft.minecraftserverlauncher.server.servertypes.Vanilla;
+import net.knarcraft.minecraftserverlauncher.server.servertypes.Waterfall;
 
 import javax.naming.ConfigurationException;
 import java.io.FileNotFoundException;
@@ -76,8 +87,13 @@ public class ServerTypeHandler {
             throw new ConfigurationException("Server type configuration file is missing.");
         }
         while (file.hasNextLine()) {
+            String nextLine = file.nextLine();
+            if (nextLine.startsWith("#")) {
+                continue;
+            }
+
             //Splits the next file line into arguments
-            String[] serverTypeInfo = file.nextLine().split(";", -1);
+            String[] serverTypeInfo = nextLine.split(";", -1);
 
             //Gets list of server versions from file line
             String[] serverVersions;

src/main/java/net/knarcraft/minecraftserverlauncher/server/ServerVersionContainer.java

@@ -3,7 +3,10 @@ package net.knarcraft.minecraftserverlauncher.server;
 import net.knarcraft.minecraftserverlauncher.Main;
 import net.knarcraft.minecraftserverlauncher.utility.CommonFunctions;
 
-import java.io.*;
+import java.io.File;
+import java.io.FileNotFoundException;
+import java.io.IOException;
+import java.io.PrintWriter;
 import java.util.HashMap;
 import java.util.Map;
 
@@ -21,6 +24,7 @@ public class ServerVersionContainer {
     private Map<String, String> travertineVersions;
     private Map<String, String> spongeVanillaVersions;
     private Map<String, String> spongeForgeVersions;
+    private String downloadedBuildToolsVersion;
 
     /**
      * Initializes a new server version container
@@ -56,6 +60,7 @@ public class ServerVersionContainer {
         this.travertineVersions = new HashMap<>();
         this.spongeVanillaVersions = new HashMap<>();
         this.spongeForgeVersions = new HashMap<>();
+        this.downloadedBuildToolsVersion = null;
     }
 
     @Override
@@ -66,7 +71,8 @@ public class ServerVersionContainer {
                 "waterfallVersions;" + mapToString(waterfallVersions) + "\n" +
                 "travertineVersions;" + mapToString(travertineVersions) + "\n" +
                 "spongeVanillaVersions;" + mapToString(spongeVanillaVersions) + "\n" +
-                "spongeForgeVersions;" + mapToString(spongeForgeVersions);
+                "spongeForgeVersions;" + mapToString(spongeForgeVersions) + "\n" +
+                "downloadedBuildToolsVersion;" + downloadedBuildToolsVersion;
     }
 
     /**
@@ -105,7 +111,7 @@ public class ServerVersionContainer {
                 }
             }
             file = new PrintWriter(versionFile);
-            file.println(this.toString());
+            file.println(this);
             file.close();
         } catch (IOException e) {
             e.printStackTrace();
@@ -119,8 +125,8 @@ public class ServerVersionContainer {
         if (!new File(versionFile).exists()) {
             return;
         }
-        try (BufferedReader reader = new BufferedReader(new InputStreamReader(new FileInputStream(versionFile)))) {
+        try {
-            String currentData = CommonFunctions.readBufferedReader(reader);
+            String currentData = CommonFunctions.readFile(versionFile);
             for (String line : currentData.split("\n")) {
                 parseSaveLine(line);
             }
@@ -163,6 +169,9 @@ public class ServerVersionContainer {
             case "spongeForgeVersions":
                 parseVersionsToMap(spongeForgeVersions, variableValue);
                 break;
+            case "downloadedBuildToolsVersion":
+                downloadedBuildToolsVersion = variableValue;
+                break;
             default:
                 throw new IllegalArgumentException("Invalid key encountered in the server version file.");
         }
@@ -323,5 +332,25 @@ public class ServerVersionContainer {
         spongeForgeVersions.put(mapKey, newValue);
         saveState();
     }
+
+    /**
+     * Gets the version of the downloaded BuildTools file
+     *
+     * @return <p>The version of the downloaded BuildTools file</p>
+     */
+    public String getDownloadedBuildToolsVersion() {
+        return this.downloadedBuildToolsVersion;
+    }
+
+    /**
+     * Sets the version of the downloaded BuildTools file
+     *
+     * @param newValue <p>The new version</p></p>
+     */
+    public void setDownloadedBuildToolsVersion(String newValue) {
+        this.downloadedBuildToolsVersion = newValue;
+        saveState();
+    }
+
 }
 

src/main/java/net/knarcraft/minecraftserverlauncher/server/servertypes/BungeeCord.java

@@ -7,7 +7,7 @@ import java.io.IOException;
 import java.nio.file.Paths;
 
 import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.downloadFile;
-import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.readFile;
+import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.readRemoteFile;
 import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.stringBetween;
 
 /**
@@ -65,7 +65,7 @@ public class BungeeCord extends AbstractServerType {
     private boolean downloadLatestJar(File filePath) throws IOException {
         ServerVersionContainer versionContainer = ServerVersionContainer.getInstance();
 
-        String newestVersion = stringBetween(readFile(versionURL), srcStart, srcEnd);
+        String newestVersion = stringBetween(readRemoteFile(versionURL), srcStart, srcEnd);
         String oldVersion = versionContainer.getBungeeVersion();
         //The file is already the newest version
         if (filePath.isFile() && newestVersion.equals(oldVersion)) {

src/main/java/net/knarcraft/minecraftserverlauncher/server/servertypes/CraftBukkit.java

@@ -1,5 +1,10 @@
 package net.knarcraft.minecraftserverlauncher.server.servertypes;
 
+import net.knarcraft.minecraftserverlauncher.Main;
+import net.knarcraft.minecraftserverlauncher.profile.ServerLauncherController;
+import net.knarcraft.minecraftserverlauncher.userinterface.GUI;
+import net.knarcraft.minecraftserverlauncher.utility.JarBuilder;
+
 import java.io.File;
 import java.nio.file.Paths;
 
@@ -31,7 +36,31 @@ public class CraftBukkit extends AbstractServerType {
         String targetFile = this.getName() + version + ".jar";
         String file = downloadURLPart + version + ".jar";
         File filePath = new File(folder + targetFile);
+
+        if (version.equals("Latest")) {
+            return filePath.isFile() || buildJar(folder);
+        } else {
             return filePath.isFile() || downloadFile(downloadURL + file, Paths.get(filePath.toURI()));
         }
+    }
+
+    /**
+     * Builds the necessary .jar file
+     *
+     * @param folder <p>The folder the build file should be moved to</p>
+     * @return <p>True if the build was successful</p>
+     */
+    private boolean buildJar(String folder) {
+        String buildToolsDirectory = Main.getApplicationWorkDirectory() + File.separator + "files" + File.separator +
+                "BuildTools" + File.separator;
+
+        GUI gui = ServerLauncherController.getInstance().getGUI();
+        JarBuilder jarBuilder = new JarBuilder(buildToolsDirectory, folder, gui);
+        if (this instanceof Spigot) {
+            return jarBuilder.buildSpigotJar();
+        } else {
+            return jarBuilder.buildBukkitJar();
+        }
+    }
 
 }

src/main/java/net/knarcraft/minecraftserverlauncher/server/servertypes/SpongeVanilla.java

@@ -53,7 +53,7 @@ public class SpongeVanilla extends AbstractServerType {
         String file = this.getName() + version + ".jar";
         File filePath = new File(folder + file);
 
-        String versionText = CommonFunctions.readFile(versionURL + version);
+        String versionText = CommonFunctions.readRemoteFile(versionURL + version);
         String newestVersion = CommonFunctions.stringBetween(versionText, srcStart, srcEnd);
 
         String jarURL = downloadURL + newestVersion + downloadURLPart + newestVersion + ".jar";

src/main/java/net/knarcraft/minecraftserverlauncher/server/servertypes/Vanilla.java

@@ -12,7 +12,7 @@ import java.io.IOException;
 import java.nio.file.Paths;
 
 import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.downloadFile;
-import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.readFile;
+import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.readRemoteFile;
 
 /**
  * This class represents the regular vanilla Minecraft server type
@@ -90,7 +90,7 @@ public class Vanilla extends AbstractServerType {
      * @throws IOException <p>If the remote resource cannot be readFromServer</p>
      */
     private String[] getLatestFile(String releaseType) throws IOException {
-        String versionText = readFile(versionURL);
+        String versionText = readRemoteFile(versionURL);
         JsonObject jsonObject = new JsonParser().parse(versionText).getAsJsonObject();
         String latest = jsonObject.getAsJsonObject("latest").get(releaseType).getAsString();
         String versionURL = getServerFileVersionURL(latest);
@@ -106,7 +106,7 @@ public class Vanilla extends AbstractServerType {
      * @throws IOException <p>If the remote resource cannot be readFromServer</p>
      */
     private String getVanillaDownloadURL(String versionURL) throws IOException {
-        String versionText = readFile(versionURL);
+        String versionText = readRemoteFile(versionURL);
         JsonObject jsonObject = new JsonParser().parse(versionText).getAsJsonObject();
         return jsonObject.getAsJsonObject("downloads").getAsJsonObject("server").get("url").getAsString();
     }
@@ -119,7 +119,7 @@ public class Vanilla extends AbstractServerType {
      * @throws IOException <p>If the file cannot be downloaded</p>
      */
     private String getServerFileVersionURL(String targetVersion) throws IOException {
-        String versionText = readFile(versionURL);
+        String versionText = readRemoteFile(versionURL);
         JsonObject jsonObject = new JsonParser().parse(versionText).getAsJsonObject();
         JsonArray availableVersions = jsonObject.getAsJsonArray("versions");
         for (JsonElement availableVersion : availableVersions) {

src/main/java/net/knarcraft/minecraftserverlauncher/server/servertypes/Waterfall.java

@@ -9,7 +9,7 @@ import java.util.function.BiConsumer;
 import java.util.function.Function;
 
 import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.downloadFile;
-import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.readFile;
+import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.readRemoteFile;
 import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.stringBetween;
 
 /**
@@ -50,7 +50,7 @@ public class Waterfall extends AbstractServerType {
     public boolean downloadJar(String folder, String version) throws IOException {
         String file = this.getName() + version + ".jar";
         File filePath = new File(folder + file);
-        String newestVersion = stringBetween(readFile(versionURL + version), srcStart, srcEnd);
+        String newestVersion = stringBetween(readRemoteFile(versionURL + version), srcStart, srcEnd);
         String fullURL = downloadURL + version + "/" + newestVersion + "/download";
         String oldVersion = oldVersionFunction.apply(version);
         //The file is already the newest version

src/main/java/net/knarcraft/minecraftserverlauncher/userinterface/Console.java

@@ -5,7 +5,11 @@ import net.knarcraft.minecraftserverlauncher.Main;
 import javax.swing.*;
 import javax.swing.text.DefaultCaret;
 import java.awt.*;
-import java.awt.event.*;
+import java.awt.event.ActionEvent;
+import java.awt.event.ActionListener;
+import java.awt.event.KeyAdapter;
+import java.awt.event.KeyEvent;
+import java.awt.event.KeyListener;
 import java.util.ArrayList;
 
 import static javax.swing.text.DefaultCaret.ALWAYS_UPDATE;
@@ -26,6 +30,7 @@ public class Console extends KeyAdapter implements ActionListener, KeyListener {
     private final JPanel panel;
     private final ArrayList<String> commands = new ArrayList<>();
     private int commandIndex;
+    private final int maxConsoleLines = 1000;
 
     /**
      * Instantiates a new console
@@ -67,9 +72,33 @@ public class Console extends KeyAdapter implements ActionListener, KeyListener {
      * @param text <p>The string to print</p>
      */
     public void output(String text) {
+        int outputLines = this.textOutput.getLineCount();
+        if (outputLines > maxConsoleLines) {
+            truncateConsole(outputLines);
+        }
         this.textOutput.setText(this.textOutput.getText() + "\n" + text);
     }
 
+    /**
+     * Truncates the first 50 lines if the console output has reached the max limit
+     *
+     * @param outputLines <p>The currently readable lines in the console output field</p>
+     */
+    private void truncateConsole(int outputLines) {
+        String oldText = this.textOutput.getText();
+        String[] oldTextList = oldText.split("\n");
+        for (int i = 0; i < outputLines - maxConsoleLines + 50; i++) {
+            oldTextList[i] = "";
+        }
+        StringBuilder newTextBuilder = new StringBuilder();
+        for (String line : oldTextList) {
+            if (!line.equals("")) {
+                newTextBuilder.append(line);
+            }
+        }
+        this.textOutput.setText(newTextBuilder.toString());
+    }
+
     @Override
     public void actionPerformed(ActionEvent e) {
         //Sends the command from the input to the server with the same name.
@@ -99,7 +128,7 @@ public class Console extends KeyAdapter implements ActionListener, KeyListener {
      * Shows the previously executed command in the input field
      *
      * <p>Shows the previously executed command if a command was just executed.
-     * Shows the command executed earlier if already showing a previously executed command.</p>
+     * Shows the command executed earlier if already showing a previously executed command</p>
      */
     private void showPreviousCommand() {
         if (commands.size() > 0 && commandIndex > 0) {
@@ -111,7 +140,7 @@ public class Console extends KeyAdapter implements ActionListener, KeyListener {
      * Shows the next previously executed command or clears the input field
      *
      * <p>Shows the next previously executed command if such a command exists.
-     * Clears the input field if no next used command exists.</p>
+     * Clears the input field if no next used command exists</p>
      */
     private void showNextCommand() {
         if (commands.size() > 0) {

src/main/java/net/knarcraft/minecraftserverlauncher/userinterface/GUI.java

@@ -44,6 +44,20 @@ public interface GUI {
      */
     void showMessage(String message);
 
+    /**
+     * Logs a message to the logfile
+     *
+     * @param message <p>The message to log</p>
+     */
+    void logMessage(String message);
+
+    /**
+     * Logs an error to the logfile
+     *
+     * @param error <p>The error to log</p>
+     */
+    void logError(String error);
+
     /**
      * Asks the user for a directory as a file object
      *

src/main/java/net/knarcraft/minecraftserverlauncher/userinterface/MessageHandler.java

@@ -1,6 +1,13 @@
 package net.knarcraft.minecraftserverlauncher.userinterface;
 
+import net.knarcraft.minecraftserverlauncher.Main;
+import net.knarcraft.minecraftserverlauncher.utility.CommonFunctions;
+
 import javax.swing.*;
+import java.io.BufferedWriter;
+import java.io.File;
+import java.io.IOException;
+import java.io.OutputStreamWriter;
 
 /**
  * This class handles displaying messages to the user
@@ -8,6 +15,8 @@ import javax.swing.*;
 public abstract class MessageHandler implements GUI {
 
     private final boolean silent;
+    private final BufferedWriter writer;
+    private final String logFile = Main.getApplicationWorkDirectory() + File.separator + "latestrun.log";
 
     /***
      * Initializes a new message handler
@@ -16,15 +25,42 @@ public abstract class MessageHandler implements GUI {
      */
     public MessageHandler(boolean silent) {
         this.silent = silent;
+        this.writer = new BufferedWriter(new OutputStreamWriter(System.out));
+    }
+
+    @Override
+    public void logMessage(String message) {
+        try {
+            CommonFunctions.appendFile(logFile, "[Info]: " + message);
+        } catch (IOException e) {
+            e.printStackTrace();
+        }
+    }
+
+    @Override
+    public void logError(String message) {
+        try {
+            CommonFunctions.appendFile(logFile, "[Error]: " + message);
+        } catch (IOException e) {
+            e.printStackTrace();
+        }
     }
 
     @Override
     public void showError(String title, String message) {
         if (silent) {
+            try {
+                writer.write("[Error]: ");
+                writer.write(message);
+                writer.newLine();
+                writer.flush();
+            } catch (IOException e) {
                 System.out.println(message);
+            }
         } else {
             showJOptionPane(title, message, JOptionPane.ERROR_MESSAGE);
         }
+        logError(message);
     }
 
     @Override
@@ -35,10 +71,17 @@ public abstract class MessageHandler implements GUI {
     @Override
     public void showMessage(String title, String message) {
         if (silent) {
+            try {
+                writer.write("[Info]: " + message);
+                writer.newLine();
+                writer.flush();
+            } catch (IOException e) {
                 System.out.println(message);
+            }
         } else {
             showJOptionPane(title, message, JOptionPane.INFORMATION_MESSAGE);
         }
+        logMessage(message);
     }
 
     @Override

src/main/java/net/knarcraft/minecraftserverlauncher/userinterface/ServerConsoles.java

@@ -17,16 +17,22 @@ public class ServerConsoles {
     private static JFrame frame;
     private static JTabbedPane consolesTabbedPane;
 
+    private ServerConsoles() {
+
+    }
+
     /**
      * Initializes the server consoles frame
      */
-    public ServerConsoles() {
+    public static void instantiate() {
+        if (frame == null || consolesTabbedPane == null) {
             frame = new JFrame();
             frame.setBounds(100, 100, 450, 300);
             frame.setDefaultCloseOperation(JFrame.HIDE_ON_CLOSE);
             consolesTabbedPane = new JTabbedPane(JTabbedPane.TOP);
             frame.getContentPane().add(consolesTabbedPane, BorderLayout.CENTER);
         }
+    }
 
     /**
      * Adds a new console tab

src/main/java/net/knarcraft/minecraftserverlauncher/userinterface/ServerControlTab.java

@@ -1,7 +1,7 @@
 package net.knarcraft.minecraftserverlauncher.userinterface;
 
 import net.knarcraft.minecraftserverlauncher.profile.Collection;
-import net.knarcraft.minecraftserverlauncher.profile.Controller;
+import net.knarcraft.minecraftserverlauncher.profile.ServerLauncherController;
 
 import javax.swing.*;
 import java.awt.event.ActionEvent;
@@ -25,7 +25,7 @@ public class ServerControlTab implements ActionListener {
     private JButton showConsolesButton;
     private JTextField customCommandTextField;
 
-    private Controller controller = Controller.getInstance();
+    private final ServerLauncherController controller = ServerLauncherController.getInstance();
     private final ArrayList<String> globalPlayers;
 
     /**
@@ -149,7 +149,7 @@ public class ServerControlTab implements ActionListener {
     public void update() {
         this.targetServerCombo.removeAllItems();
         this.targetServerCombo.addItem("All");
-        for (Collection collection : Controller.getInstance().getCurrentProfile().getCollections()) {
+        for (Collection collection : ServerLauncherController.getInstance().getCurrentProfile().getCollections()) {
             this.targetServerCombo.addItem(collection.getName());
         }
     }

src/main/java/net/knarcraft/minecraftserverlauncher/userinterface/ServerLauncherGUI.java

@@ -2,7 +2,7 @@ package net.knarcraft.minecraftserverlauncher.userinterface;
 
 import net.knarcraft.minecraftserverlauncher.Main;
 import net.knarcraft.minecraftserverlauncher.profile.Collection;
-import net.knarcraft.minecraftserverlauncher.profile.Controller;
+import net.knarcraft.minecraftserverlauncher.profile.ServerLauncherController;
 import net.knarcraft.minecraftserverlauncher.server.Server;
 import net.knarcraft.minecraftserverlauncher.utility.CommonFunctions;
 
@@ -12,15 +12,14 @@ import javax.swing.*;
 import java.awt.*;
 import java.awt.event.ActionEvent;
 import java.awt.event.ActionListener;
-import java.awt.event.MouseAdapter;
+import java.io.File;
-import java.awt.event.MouseEvent;
+import java.io.FileNotFoundException;
-import java.awt.event.WindowAdapter;
+import java.io.IOException;
-import java.awt.event.WindowEvent;
+import java.util.HashMap;
-import java.io.*;
+import java.util.Map;
-import java.util.*;
+import java.util.Scanner;
 import java.util.concurrent.Executors;
 
-import static java.awt.Frame.NORMAL;
 import static javax.swing.WindowConstants.DO_NOTHING_ON_CLOSE;
 import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.getResourceAsScanner;
 import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.getResourceAsStream;
@@ -35,7 +34,7 @@ import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.getR
 public class ServerLauncherGUI extends MessageHandler implements ActionListener, GUI {
 
     private final JLabel lblStatuslabel = new JLabel("Servers are stopped");
-    private Controller controller;
+    private final ServerLauncherController controller;
     private Map<String, String> textStrings;
     private Tray applicationTray;
 
@@ -64,6 +63,20 @@ public class ServerLauncherGUI extends MessageHandler implements ActionListener,
         this.controller = Main.getController();
     }
 
+    /**
+     * Creates the application window
+     *
+     * @param silent <p>Whether to make the GUI silent (hidden, for testing)</p>
+     */
+    public ServerLauncherGUI(boolean silent) throws IOException {
+        super(silent);
+        if (!silent) {
+            initialize(440, 170);
+        }
+        loadMessages();
+        this.controller = Main.getController();
+    }
+
     /**
      * Creates the application window with a preferred width and height
      *
@@ -89,12 +102,7 @@ public class ServerLauncherGUI extends MessageHandler implements ActionListener,
     @Override
     public void setStatus(String text) {
         this.lblStatuslabel.setText(text);
-        try (PrintWriter file = new PrintWriter(new FileWriter(Main.getApplicationWorkDirectory() +
+        this.logMessage(text);
-                File.separator + "latestrun.log", true))) {
-            file.println(text);
-        } catch (IOException e) {
-            e.printStackTrace();
-        }
     }
 
     @Override
@@ -146,7 +154,7 @@ public class ServerLauncherGUI extends MessageHandler implements ActionListener,
      * Updates ServerLauncherGUI according to current profile settings
      */
     public void updateWithSavedProfileData() {
-        Controller controller = Main.getController();
+        ServerLauncherController controller = Main.getController();
         serversPane.removeAll();
         serverLauncherMenu.update();
         serverControlTab.update();
@@ -311,7 +319,6 @@ public class ServerLauncherGUI extends MessageHandler implements ActionListener,
     }
 
 
-
     /**
      * Handles buttons and the combo on the main tab
      *
@@ -401,6 +408,9 @@ public class ServerLauncherGUI extends MessageHandler implements ActionListener,
         } catch (IOException e1) {
             showError("Could not stop server.");
             e1.printStackTrace();
+        } catch (InterruptedException e) {
+            showError("Could not kill server.");
+            e.printStackTrace();
         }
     }
 

src/main/java/net/knarcraft/minecraftserverlauncher/userinterface/ServerLauncherMenu.java

@@ -1,12 +1,13 @@
 package net.knarcraft.minecraftserverlauncher.userinterface;
 
-import net.knarcraft.minecraftserverlauncher.profile.Controller;
 import net.knarcraft.minecraftserverlauncher.profile.Profile;
+import net.knarcraft.minecraftserverlauncher.profile.ServerLauncherController;
 import net.knarcraft.minecraftserverlauncher.utility.CommonFunctions;
 
 import javax.swing.*;
 import java.awt.event.ActionEvent;
 import java.awt.event.ActionListener;
+import java.io.File;
 import java.util.Objects;
 
 /**
@@ -14,11 +15,14 @@ import java.util.Objects;
  */
 public class ServerLauncherMenu implements ActionListener {
 
-    private JMenuBar menuBar;
+    private final JMenuBar menuBar;
     //Options
     private JCheckBoxMenuItem runInBackgroundCheckBoxMenuItem;
     private JCheckBoxMenuItem delayStartupCheckBoxMenuItem;
     private JCheckBoxMenuItem downloadJarsCheckBoxMenuItem;
+    private JMenuItem javaCommandMenuItem;
+    private JMenuItem oldJavaCommandMenuItem;
+    private JMenuItem deleteBuiltJarsMenuItem;
     //Help
     private JMenuItem errorsMenuItem;
     private JMenuItem setupMenuItem;
@@ -27,12 +31,15 @@ public class ServerLauncherMenu implements ActionListener {
     private JMenuItem runInBackgroundMenuItem;
     private JMenuItem delayStartupMenuItem;
     private JMenuItem downloadJarsMenuItem;
+    private JMenuItem javaCommandInfoMenuItem;
+    private JMenuItem oldJavaCommandInfoMenuItem;
+    private JMenuItem deleteBuiltJarsInfoMenuItem;
     //Info/about
     private JMenuItem aboutMenuItem;
     private JMenuItem storyMenuItem;
 
-    private Controller controller;
+    private final ServerLauncherController controller;
-    private ServerLauncherGUI serverLauncherGUI;
+    private final ServerLauncherGUI serverLauncherGUI;
 
     /**
      * Initializes a new server launcher menu
@@ -41,7 +48,7 @@ public class ServerLauncherMenu implements ActionListener {
      * @param serverLauncherGUI <p>The server launcher GUI to use</p>
      */
     public ServerLauncherMenu(JMenuBar menuBar, ServerLauncherGUI serverLauncherGUI) {
-        this.controller = Controller.getInstance();
+        this.controller = ServerLauncherController.getInstance();
         this.menuBar = menuBar;
         this.serverLauncherGUI = serverLauncherGUI;
         initialize();
@@ -53,64 +60,23 @@ public class ServerLauncherMenu implements ActionListener {
     public void initialize() {
         JMenu mnOptions = new JMenu("Options");
         menuBar.add(mnOptions);
-
+        generateOptionsMenuItems(mnOptions);
-        runInBackgroundCheckBoxMenuItem = new JCheckBoxMenuItem("Run in background on exit");
-        mnOptions.add(runInBackgroundCheckBoxMenuItem);
-        runInBackgroundCheckBoxMenuItem.addActionListener(this);
-
-        delayStartupCheckBoxMenuItem = new JCheckBoxMenuItem("Delay Startup");
-        mnOptions.add(delayStartupCheckBoxMenuItem);
-        delayStartupCheckBoxMenuItem.addActionListener(this);
-
-        downloadJarsCheckBoxMenuItem = new JCheckBoxMenuItem("Download jars");
-        mnOptions.add(downloadJarsCheckBoxMenuItem);
-        downloadJarsCheckBoxMenuItem.addActionListener(this);
 
         JMenu mnHelp = new JMenu("Help");
         menuBar.add(mnHelp);
 
-        errorsMenuItem = new JMenuItem("Errors");
+        errorsMenuItem = createMenuItem("Errors", mnHelp);
-        mnHelp.add(errorsMenuItem);
+        setupMenuItem = createMenuItem("Setup", mnHelp);
-        errorsMenuItem.addActionListener(this);
+        manualUpdateMenuItem = createMenuItem("Manual update", mnHelp);
-
-        setupMenuItem = new JMenuItem("Setup");
-        mnHelp.add(setupMenuItem);
-        setupMenuItem.addActionListener(this);
-
-        manualUpdateMenuItem = new JMenuItem("Manual update");
-        mnHelp.add(manualUpdateMenuItem);
-        manualUpdateMenuItem.addActionListener(this);
 
         JMenu mnInfo = new JMenu("Info");
         menuBar.add(mnInfo);
-
+        generateInfoMenuItems(mnInfo);
-        JMenu mnOptionsInfo = new JMenu("Options");
-        mnInfo.add(mnOptionsInfo);
-
-        runInBackgroundMenuItem = new JMenuItem("Run in background on exit");
-        mnOptionsInfo.add(runInBackgroundMenuItem);
-        runInBackgroundMenuItem.addActionListener(this);
-
-        delayStartupMenuItem = new JMenuItem("Delay Startup");
-        mnOptionsInfo.add(delayStartupMenuItem);
-        delayStartupMenuItem.addActionListener(this);
-
-        downloadJarsMenuItem = new JMenuItem("Download jars");
-        mnOptionsInfo.add(downloadJarsMenuItem);
-        downloadJarsMenuItem.addActionListener(this);
-
-        JMenu mnAbout = new JMenu("About");
-        mnInfo.add(mnAbout);
-
-        aboutMenuItem = new JMenuItem("About");
-        mnAbout.add(aboutMenuItem);
-        aboutMenuItem.addActionListener(this);
-
-        storyMenuItem = new JMenuItem("Story");
-        mnAbout.add(storyMenuItem);
-        storyMenuItem.addActionListener(this);
     }
 
+    /**
+     * Updates the state of checkboxes based on whether options are enabled
+     */
     public void update() {
         runInBackgroundCheckBoxMenuItem.setState(controller.getRunInBackground());
         delayStartupCheckBoxMenuItem.setState(controller.getDelayStartup() > 0);
@@ -126,6 +92,12 @@ public class ServerLauncherMenu implements ActionListener {
             delay();
         } else if (actionSource == downloadJarsCheckBoxMenuItem) {
             downloadJars();
+        } else if (actionSource == javaCommandMenuItem) {
+            configureJava(false);
+        } else if (actionSource == oldJavaCommandMenuItem) {
+            configureJava(true);
+        } else if (actionSource == deleteBuiltJarsMenuItem) {
+            deleteBuiltJars();
         } else if (actionSource == errorsMenuItem) {
             CommonFunctions.goToURL(serverLauncherGUI.getMessage("infoURL"));
         } else if (actionSource == setupMenuItem) {
@@ -138,6 +110,12 @@ public class ServerLauncherMenu implements ActionListener {
             serverLauncherGUI.showMessage("Delay startup", serverLauncherGUI.getMessage("delayStartupText"));
         } else if (actionSource == downloadJarsMenuItem) {
             serverLauncherGUI.showMessage("Download jars", serverLauncherGUI.getMessage("downloadJarsText"));
+        } else if (actionSource == javaCommandInfoMenuItem) {
+            serverLauncherGUI.showMessage("Java command", serverLauncherGUI.getMessage("javaCommandText"));
+        } else if (actionSource == oldJavaCommandInfoMenuItem) {
+            serverLauncherGUI.showMessage("Old Java command", serverLauncherGUI.getMessage("oldJavaCommandText"));
+        } else if (actionSource == deleteBuiltJarsInfoMenuItem) {
+            serverLauncherGUI.showMessage("Delete built jar files", serverLauncherGUI.getMessage("deleteBuiltJarFilesText"));
         } else if (actionSource == aboutMenuItem) {
             serverLauncherGUI.showMessage("About", serverLauncherGUI.getMessage("aboutText"));
         } else if (actionSource == storyMenuItem) {
@@ -145,6 +123,116 @@ public class ServerLauncherMenu implements ActionListener {
         }
     }
 
+    /**
+     * Generates the children of the options menu
+     *
+     * @param mnOptions <p>A reference to the options menu</p>
+     */
+    private void generateOptionsMenuItems(JMenu mnOptions) {
+        runInBackgroundCheckBoxMenuItem = createCheckBoxMenuItem("Run in background on exit", mnOptions);
+        delayStartupCheckBoxMenuItem = createCheckBoxMenuItem("Delay Startup", mnOptions);
+        downloadJarsCheckBoxMenuItem = createCheckBoxMenuItem("Download jars", mnOptions);
+        javaCommandMenuItem = createMenuItem("Java command", mnOptions);
+        oldJavaCommandMenuItem = createMenuItem("Old Java command", mnOptions);
+        deleteBuiltJarsMenuItem = createMenuItem("Delete built jar files", mnOptions);
+    }
+
+    /**
+     * Generates the children of the info menu
+     *
+     * @param mnInfo <p>A reference to the info menu</p>
+     */
+    private void generateInfoMenuItems(JMenu mnInfo) {
+        JMenu mnOptionsInfo = new JMenu("Options");
+        mnInfo.add(mnOptionsInfo);
+
+        runInBackgroundMenuItem = createMenuItem("Run in background on exit", mnOptionsInfo);
+        delayStartupMenuItem = createMenuItem("Delay Startup", mnOptionsInfo);
+        downloadJarsMenuItem = createMenuItem("Download jars", mnOptionsInfo);
+        javaCommandInfoMenuItem = createMenuItem("Java command", mnOptionsInfo);
+        oldJavaCommandInfoMenuItem = createMenuItem("Old Java command", mnOptionsInfo);
+        deleteBuiltJarsInfoMenuItem = createMenuItem("Delete built jar files", mnOptionsInfo);
+
+        JMenu mnAbout = new JMenu("About");
+        mnInfo.add(mnAbout);
+
+        aboutMenuItem = createMenuItem("About", mnAbout);
+        storyMenuItem = createMenuItem("Story", mnAbout);
+    }
+
+    /**
+     * Creates a checkbox menu item
+     *
+     * @param itemName <p>The name of the new checkbox item</p>
+     * @param parent <p>The parent menu the item belongs to</p>
+     * @return <p>The created checkbox menu item</p>
+     */
+    private JCheckBoxMenuItem createCheckBoxMenuItem(String itemName, JMenu parent) {
+        JCheckBoxMenuItem menuItem = new JCheckBoxMenuItem(itemName);
+        parent.add(menuItem);
+        menuItem.addActionListener(this);
+        return menuItem;
+    }
+
+    /**
+     * Creates a menu item
+     *
+     * @param itemName <p>The name of the new item</p>
+     * @param parent <p>The parent menu the item belongs to</p>
+     * @return <p>The created menu item</p>
+     */
+    private JMenuItem createMenuItem(String itemName, JMenu parent) {
+        JMenuItem menuItem = new JMenuItem(itemName);
+        parent.add(menuItem);
+        menuItem.addActionListener(this);
+        return menuItem;
+    }
+
+    /**
+     * Asks the user for the new Java path
+     * @param old <p>Whether asking for the path to the old java version</p>
+     */
+    private void configureJava(boolean old) {
+        if (old) {
+            String response = JOptionPane.showInputDialog("Command or path to Java: ", controller.getOldJavaCommand());
+            if (response != null) {
+                controller.setOldJavaCommand(response);
+            }
+        } else {
+            String response = JOptionPane.showInputDialog("Command or path to Java: ", controller.getJavaCommand());
+            if (response != null) {
+                controller.setJavaCommand(response);
+            }
+        }
+    }
+
+    /**
+     * Deletes build Spigot and Bukkit .jar files if the user accepts
+     */
+    private void deleteBuiltJars() {
+        int answer = JOptionPane.showConfirmDialog(null, "This will delete built .jar files, causing them " +
+                        "to be rebuilt on the next run. Do you want to continue?", "Delete built .jar files",
+                JOptionPane.YES_NO_OPTION
+        );
+        if (answer == JOptionPane.YES_NO_OPTION) {
+            String jarDirectory = controller.getJarDirectory();
+            File spigotFile = new File(jarDirectory + "SpigotLatest.jar");
+            File bukkitFile = new File(jarDirectory + "BukkitLatest.jar");
+            boolean success = true;
+            if (spigotFile.exists() && !spigotFile.delete()) {
+                serverLauncherGUI.showError("Unable to delete latest spigot .jar");
+                success = false;
+            }
+            if (bukkitFile.exists() && !bukkitFile.delete()) {
+                serverLauncherGUI.showError("Unable to delete latest bukkit .jar");
+                success = false;
+            }
+            if (success) {
+                serverLauncherGUI.showMessage("Deletion successful", "Deleted built .jar files");
+            }
+        }
+    }
+
     /**
      * Asks the user for a delay if checked, and sets the value to the current profile
      */

src/main/java/net/knarcraft/minecraftserverlauncher/userinterface/Tray.java

@@ -1,13 +1,17 @@
 package net.knarcraft.minecraftserverlauncher.userinterface;
 
 import net.knarcraft.minecraftserverlauncher.Main;
-import net.knarcraft.minecraftserverlauncher.profile.Controller;
+import net.knarcraft.minecraftserverlauncher.profile.ServerLauncherController;
 import net.knarcraft.minecraftserverlauncher.utility.CommonFunctions;
 
 import javax.imageio.ImageIO;
 import javax.swing.*;
 import java.awt.*;
-import java.awt.event.*;
+import java.awt.event.ActionListener;
+import java.awt.event.MouseAdapter;
+import java.awt.event.MouseEvent;
+import java.awt.event.WindowAdapter;
+import java.awt.event.WindowEvent;
 import java.io.IOException;
 import java.io.InputStream;
 
@@ -20,9 +24,9 @@ public class Tray {
 
     private SystemTray tray;
     private TrayIcon trayIcon;
-    private Controller controller;
+    private final ServerLauncherController controller;
-    private ServerLauncherGUI serverLauncherGUI;
+    private final ServerLauncherGUI serverLauncherGUI;
-    private JFrame mainFrame;
+    private final JFrame mainFrame;
 
     /**
      * Instantiates a new tray

src/main/java/net/knarcraft/minecraftserverlauncher/userinterface/WebBrowser.java

@@ -12,7 +12,8 @@ public class WebBrowser {
     private static JFrame browserFrame;
     private static JTextPane editorPane;
 
-    private WebBrowser() {}
+    private WebBrowser() {
+    }
 
     /**
      * Instantiates a new web browser
@@ -44,7 +45,8 @@ public class WebBrowser {
             editorPane.setContentType("text/html");
             editorPane.addHyperlinkListener(hyperlinkEvent -> {
                 if (hyperlinkEvent.getEventType().equals(HyperlinkEvent.EventType.ACTIVATED)) {
-                displayPage(hyperlinkEvent.getURL().toString()); }
+                    displayPage(hyperlinkEvent.getURL().toString());
+                }
             });
         } catch (IOException e) {
             editorPane.setContentType("text/html");

src/main/java/net/knarcraft/minecraftserverlauncher/utility/CommonFunctions.java

@@ -6,7 +6,18 @@ import net.knarcraft.minecraftserverlauncher.server.Server;
 import net.knarcraft.minecraftserverlauncher.userinterface.GUI;
 import net.knarcraft.minecraftserverlauncher.userinterface.WebBrowser;
 
-import java.io.*;
+import java.io.BufferedReader;
+import java.io.BufferedWriter;
+import java.io.File;
+import java.io.FileInputStream;
+import java.io.FileNotFoundException;
+import java.io.FileOutputStream;
+import java.io.FileWriter;
+import java.io.IOException;
+import java.io.InputStream;
+import java.io.InputStreamReader;
+import java.io.OutputStream;
+import java.io.OutputStreamWriter;
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.net.URL;
@@ -16,7 +27,7 @@ import java.nio.file.StandardCopyOption;
 import java.util.Scanner;
 
 /**
- * A holding class for methods shared between classes.
+ * A holding class for methods shared between classes
  *
  * @author Kristian Knarvik <kristian.knarvik@knett.no>
  * @version 1.0.0
@@ -34,7 +45,6 @@ public final class CommonFunctions {
     public static void createAllFolders() throws FileNotFoundException {
         createFolder(new File(filesDirectory));
         createFolder(new File(filesDirectory + File.separator + "Jars"));
-        createFolder(new File(filesDirectory + File.separator + "testjars"));
     }
 
     /**
@@ -43,7 +53,7 @@ public final class CommonFunctions {
      * @param folder <p>The folder to create</p>
      * @throws FileNotFoundException <p>If unable to create the folder</p>
      */
-    private static void createFolder(File folder) throws FileNotFoundException {
+    protected static void createFolder(File folder) throws FileNotFoundException {
         if (!folder.exists()) {
             if (!folder.mkdirs()) {
                 throw new FileNotFoundException("Cannot create necessary directory.");
@@ -100,12 +110,88 @@ public final class CommonFunctions {
      *
      * @param path <p>The full url of the file to readFromServer</p>
      * @return <p>True if successful. False otherwise</p>
+     * @throws IOException <p>If unable to find or read the file</p>
      */
-    public static String readFile(String path) throws IOException {
+    public static String readRemoteFile(String path) throws IOException {
         URL url = new URL(path);
         return new Scanner(url.openStream()).useDelimiter("\\Z").next();
     }
 
+    /**
+     * Gets a buffered reader for reading a given file
+     *
+     * @param path <p>The path of the file to read</p>
+     * @return <p>A buffered reader for reading the file</p>
+     * @throws FileNotFoundException <p>If the file does not exist</p>
+     */
+    public static BufferedReader getFileReader(String path) throws FileNotFoundException {
+        return new BufferedReader(new InputStreamReader(new FileInputStream(path)));
+    }
+
+    /**
+     * Gets a buffered writer for writing to a given file
+     * @param path <p>The path to the file to write to</p>
+     * @return <p>A buffered writer for writing to the file</p>
+     * @throws FileNotFoundException <p>If the file does not exist</p>
+     */
+    public static BufferedWriter getFileWriter(String path) throws FileNotFoundException {
+        return new BufferedWriter(new OutputStreamWriter(new FileOutputStream(path)));
+    }
+
+    /**
+     * Reads a file from disk
+     *
+     * @param path <p>The path of the file to read</p>
+     * @return <p>The contents of the file</p>
+     * @throws IOException <p>If unable to find or read the file</p>
+     */
+    public static String readFile(String path) throws IOException {
+        return CommonFunctions.readBufferedReader(getFileReader(path));
+    }
+
+
+    /**
+     * Writes text to a file and adds a trailing newline
+     *
+     * @param path               <p>The path of the file to write to</p>
+     * @param text               <p>The text to write</p>
+     * @throws IOException <p>If unable to write to the file</p>
+     */
+    public static void writeFile(String path, String text) throws IOException {
+        writeFile(path, text, true);
+    }
+
+    /**
+     * Writes text to a file
+     *
+     * @param path               <p>The path of the file to write to</p>
+     * @param text               <p>The text to write</p>
+     * @param addTrailingNewline <p>Whether to add a new line at the end of the file</p>
+     * @throws IOException <p>If unable to write to the file</p>
+     */
+    public static void writeFile(String path, String text, Boolean addTrailingNewline) throws IOException {
+        BufferedWriter writer = getFileWriter(path);
+        writer.write(text);
+        if (addTrailingNewline) {
+            writer.newLine();
+        }
+        writer.close();
+    }
+
+    /**
+     * Appends text to a file
+     *
+     * @param path <p>The path to the file to append to</p>
+     * @param text <p>The text to append</p>
+     * @throws IOException <p>If unable to append to the file</p>
+     */
+    public static void appendFile(String path, String text) throws IOException {
+        BufferedWriter writer = new BufferedWriter(new FileWriter(path, true));
+        writer.write(text);
+        writer.newLine();
+        writer.close();
+    }
+
     /**
      * Downloads a file from a website and replaces the target file
      *
@@ -199,9 +285,12 @@ public final class CommonFunctions {
         StringBuilder text = new StringBuilder();
         char[] readCharacters = new char[1000];
         while (reader.ready()) {
-            reader.read(readCharacters);
+            if (reader.read(readCharacters) > 0) {
                 text.append(readCharacters);
                 readCharacters = new char[1000];
+            } else {
+                return text.toString().trim();
+            }
         }
         return text.toString().trim();
     }
@@ -244,4 +333,32 @@ public final class CommonFunctions {
     public static boolean nameIsValid(String name) {
         return !name.equals("") && name.matches("[^!?;,]+");
     }
+
+    /**
+     * Removes all files within a folder
+     *
+     * @param target <p>The folder to delete from</p>
+     */
+    static void removeFilesRecursively(File target) {
+        File[] oldFiles = target.listFiles();
+        if (oldFiles == null) {
+            throw new IllegalArgumentException("Unable to list files in directory");
+        }
+        for (File file : oldFiles) {
+            if (file.isFile()) {
+                if (!file.delete()) {
+                    throw new IllegalArgumentException("Unable to delete a file from the directory");
+                }
+            }
+        }
+        for (File file : oldFiles) {
+            if (file.isDirectory()) {
+                removeFilesRecursively(file);
+                if (!file.delete()) {
+                    throw new IllegalArgumentException("Unable to delete a file from the directory");
+                }
+            }
+        }
+    }
+
 }

src/main/java/net/knarcraft/minecraftserverlauncher/utility/JarBuilder.java

@@ -0,0 +1,179 @@
+package net.knarcraft.minecraftserverlauncher.utility;
+
+import net.knarcraft.minecraftserverlauncher.Main;
+import net.knarcraft.minecraftserverlauncher.profile.ServerLauncherController;
+import net.knarcraft.minecraftserverlauncher.server.ServerVersionContainer;
+import net.knarcraft.minecraftserverlauncher.userinterface.GUI;
+import net.knarcraft.minecraftserverlauncher.userinterface.ServerLauncherGUI;
+
+import java.io.BufferedReader;
+import java.io.BufferedWriter;
+import java.io.File;
+import java.io.FileNotFoundException;
+import java.io.IOException;
+import java.io.InputStreamReader;
+import java.io.OutputStreamWriter;
+import java.nio.file.Paths;
+
+/**
+ * The jar builder is responsible for building the newest Spigot and CraftBukkit .jar files using BuildTools
+ */
+public class JarBuilder {
+
+    private final String buildDirectory;
+    private final String jarDirectory;
+    private final ServerVersionContainer versionContainer;
+    private final GUI gui;
+    private final String javaCommand;
+
+    /**
+     * Instantiates a new jar builder
+     *
+     * @param buildDirectory <p>The directory containing BuildTool files</p>
+     * @param jarDirectory   <p>The directory containing downloaded .jar files</p>
+     * @param gui            <p>The GUI to write messages to</p>
+     */
+    public JarBuilder(String buildDirectory, String jarDirectory, GUI gui) {
+        this.buildDirectory = buildDirectory;
+        this.jarDirectory = jarDirectory;
+        this.gui = gui;
+        this.versionContainer = ServerVersionContainer.getInstance();
+        this.javaCommand = ServerLauncherController.getInstance().getJavaCommand();
+        try {
+            CommonFunctions.createFolder(new File(buildDirectory));
+        } catch (FileNotFoundException e) {
+            e.printStackTrace();
+        }
+    }
+
+    /**
+     * Builds the latest version of the spigot .jar file
+     */
+    public boolean buildSpigotJar() {
+        gui.setStatus("Building Spigot jar ...");
+        downloadBuildTools();
+        ProcessBuilder processBuilder = new ProcessBuilder(javaCommand, "-jar", "BuildTools.jar", "--rev",
+                "latest", "--output-dir", jarDirectory);
+        if (executeBuildProcess(processBuilder) && moveBuiltJar("spigot-", "SpigotLatest.jar")) {
+            gui.setStatus("Finished moving spigot.jar");
+            return true;
+        } else {
+            return false;
+        }
+    }
+
+    /**
+     * Builds the latest version of the craftbukkit .jar file
+     */
+    public boolean buildBukkitJar() {
+        gui.setStatus("Building Bukkit jar ...");
+        downloadBuildTools();
+        ProcessBuilder processBuilder = new ProcessBuilder(javaCommand, "-jar", "BuildTools.jar", "--compile",
+                "craftbukkit", "--rev", "latest", "--output-dir", jarDirectory);
+        if (executeBuildProcess(processBuilder) && moveBuiltJar("craftbukkit-", "BukkitLatest.jar")) {
+            gui.setStatus("Finished moving craftbukkit.jar");
+            return true;
+        } else {
+            return false;
+        }
+    }
+
+    /**
+     * Downloads the latest BuildTools version
+     */
+    public void downloadBuildTools() {
+        ServerLauncherGUI gui = Main.getController().getGUI();
+        try {
+            String latestVersion = getLatestBuildToolsVersion();
+            boolean exists = new File(buildDirectory + "BuildTools.jar").exists();
+            boolean isUpdated = latestVersion.equals(versionContainer.getDownloadedBuildToolsVersion());
+
+            if (!exists || !isUpdated) {
+                String buildToolsURL = "https://hub.spigotmc.org/jenkins/job/BuildTools/lastSuccessfulBuild/artifact/target/BuildTools.jar";
+                boolean success = CommonFunctions.downloadFile(buildToolsURL, Paths.get(buildDirectory + "BuildTools.jar"));
+                if (!success) {
+                    gui.setStatus("Unable to download the latest BuildTools");
+                    gui.logError("Target: " + buildDirectory + "BuildTools.jar");
+                    gui.logError("URL: " + buildToolsURL);
+                } else {
+                    versionContainer.setDownloadedBuildToolsVersion(latestVersion);
+                }
+            }
+        } catch (IOException e) {
+            gui.setStatus("Unable to download the latest BuildTools");
+            gui.logMessage(e.getMessage());
+        }
+    }
+
+    /**
+     * Moves a built .jar file to its target file
+     *
+     * @param searchString <p>The start of the name of the built file used to find it</p>
+     * @param newName      <p>The new name of the built file</p>
+     * @return <p>True if the .jar file was moved successfully</p>
+     */
+    private boolean moveBuiltJar(String searchString, String newName) {
+        File[] foundFiles = new File(jarDirectory).listFiles((file) -> file.isFile() && file.getName().startsWith(searchString));
+        if (foundFiles != null && foundFiles.length == 1) {
+            File newFileLocation = new File(jarDirectory + File.separator + newName);
+            if (newFileLocation.exists()) {
+                if (!newFileLocation.delete()) {
+                    gui.showError("Unable to delete previously built .jar");
+                }
+            }
+            if (!foundFiles[0].renameTo(newFileLocation)) {
+                gui.showError("Unable to move built .jar");
+            } else {
+                return true;
+            }
+        }
+        return false;
+    }
+
+    /**
+     * Starts the build process and initializes
+     *
+     * @param processBuilder <p>The process builder to execute</p>
+     * @return <p>True if the process exited successfully</p>
+     */
+    private boolean executeBuildProcess(ProcessBuilder processBuilder) {
+        processBuilder.directory(new File(buildDirectory));
+        processBuilder.redirectErrorStream(true);
+        try {
+            Process process = processBuilder.start();
+            BufferedWriter writer = new BufferedWriter(new OutputStreamWriter(System.out));
+            BufferedReader reader = new BufferedReader(new InputStreamReader(process.getInputStream()));
+
+            do {
+                writer.write(CommonFunctions.readBufferedReader(reader));
+            } while (process.isAlive());
+
+            writer.newLine();
+            writer.flush();
+
+            if (process.exitValue() == 0) {
+                gui.setStatus("Jar building process finished successfully.");
+                return true;
+            } else {
+                gui.showError("Jar building process failed with exit code " + process.exitValue() +
+                        ". Please check the BuildTools log for errors.");
+            }
+        } catch (IOException e) {
+            e.printStackTrace();
+        }
+        return false;
+    }
+
+    /**
+     * Gets the latest build tools version available
+     *
+     * @return <p>The latest build tools version available</p>
+     * @throws IOException <p>If unable to read the version file</p>
+     */
+    String getLatestBuildToolsVersion() throws IOException {
+        String versionDocument;
+        versionDocument = CommonFunctions.readRemoteFile("https://hub.spigotmc.org/jenkins/job/BuildTools/lastSuccessfulBuild/");
+        return CommonFunctions.stringBetween(versionDocument, "BuildTools #", " [");
+    }
+
+}

src/main/java/net/knarcraft/minecraftserverlauncher/utility/JarDownloader.java

@@ -60,6 +60,9 @@ public final class JarDownloader {
                 continue;
             }
             for (String version : type.getVersions()) {
+                if ((type.getName().equals("Spigot") || type.getName().equals("Bukkit")) && version.equals("Latest")) {
+                    continue;
+                }
                 gui.setStatus("Downloading " + type.getName() + version + "...");
                 boolean success = type.downloadJar(jarDirectory, version);
                 if (!success) {

src/main/java/net/knarcraft/minecraftserverlauncher/utility/Updater.java

@@ -5,16 +5,18 @@ import com.google.gson.JsonElement;
 import com.google.gson.JsonObject;
 import com.google.gson.JsonParser;
 import net.knarcraft.minecraftserverlauncher.Main;
-import net.knarcraft.minecraftserverlauncher.userinterface.GUI;
+import net.knarcraft.minecraftserverlauncher.profile.ServerLauncherController;
 
 import javax.swing.*;
-import java.io.*;
+import java.io.File;
+import java.io.FileNotFoundException;
+import java.io.IOException;
 import java.nio.file.Paths;
 import java.util.Scanner;
 
 import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.downloadFile;
 import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.getResourceAsScanner;
-import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.readFile;
+import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.readRemoteFile;
 
 /**
  * A utility used for updating the software
@@ -45,11 +47,11 @@ public final class Updater {
         String oldVer = file.nextLine();
         file.close();
 
-        String data = readFile(updateURL);
+        String data = readRemoteFile(updateURL);
         JsonObject jsonObject = new JsonParser().parse(data).getAsJsonObject();
         String latest = jsonObject.getAsJsonObject("latest").get(updateChannel).getAsString();
 
-        if (!oldType.equals(updateChannel) || !oldVer.equals(latest)) {
+        if (!oldType.equals(updateChannel) || isVersionHigher(oldVer, latest)) {
             JsonArray versionList = jsonObject.getAsJsonArray("versions");
             String url = "";
             for (JsonElement elem : versionList) {
@@ -75,6 +77,27 @@ public final class Updater {
         }
     }
 
+    /**
+     * Decides whether one version number is higher than another
+     *
+     * @param oldVersion <p>The old version to check</p>
+     * @param newVersion <p>The new version to check</p>
+     * @return <p>True if the new version is higher than the old one</p>
+     */
+    public static boolean isVersionHigher(String oldVersion, String newVersion) {
+        String[] oldVersionParts = oldVersion.split("\\.");
+        String[] newVersionParts = newVersion.split("\\.");
+        int versionLength = Math.max(oldVersionParts.length, newVersionParts.length);
+        for (int i = 0; i < versionLength; i++) {
+            int oldVersionNumber = oldVersionParts.length > i ? Integer.parseInt(oldVersionParts[i]) : 0;
+            int newVersionNumber = newVersionParts.length > i ? Integer.parseInt(newVersionParts[i]) : 0;
+            if (newVersionNumber != oldVersionNumber) {
+                return newVersionNumber > oldVersionNumber;
+            }
+        }
+        return false;
+    }
+
     /**
      * Updates the software
      *
@@ -82,8 +105,9 @@ public final class Updater {
      * @throws IOException <p>If unable to run the updater</p>
      */
     private static void runUpdater(String url) throws IOException {
+        String javaCommand = ServerLauncherController.getInstance().getJavaCommand();
         ProcessBuilder builder;
-        builder = new ProcessBuilder("java", "-jar", "Updater.jar", url, "yes", targetFile, "5", "nogui");
+        builder = new ProcessBuilder(javaCommand, "-jar", "Updater.jar", url, "yes", targetFile, "5", "nogui");
         builder.directory(new File(Main.getApplicationWorkDirectory()));
         builder.redirectErrorStream(true);
         builder.start();

src/main/resources/currentversion.csv

@@ -1,2 +1,2 @@
 beta
-1.2.3
+1.3.3

src/main/resources/menumsg.csv

@@ -6,3 +6,6 @@ aboutText=This software was created to start and manage several servers simultan
 infoURL=https://archive.knarcraft.net/Scripts/BungeeMinecraftServerLauncherInfo/
 manualUpdateURL=https://git.knarcraft.net/KnarCraft/Minecraft-Server-Launcher/releases
 storyURL=https://archive.knarcraft.net/Scripts/BungeeMinecraftServerLauncherStory/
+javaCommandText=This option allows you to set a custom command/path to the Java executable used for Minecraft 1.17 and above._BREAK_If "java" is currently pointing to Java 8, you can use this to set a custom one for running new Minecraft servers and BuildTools.
+oldJavaCommandText=This option allows you to set a custom command/path to the Java executable used for Minecraft 1.16 and below._BREAK_If "java" is currently pointing to Java 16, you can use this to set a custom one for running old Minecraft servers.
+deleteBuiltJarFilesText=This option allows you to easily delete built Spigot and Bukkit .jar files._BREAK_You should occasionally run this option to update the built .jar files to ensure you're running the latest update.

src/main/resources/servertypes.csv

@@ -1,11 +1,11 @@
-Vanilla;Latest,Snapshot,1.16.3,1.15.2,1.14.4,1.13.2,1.12.2,1.11.2,1.10.2,1.9.4,1.8.9,1.7.10,1.6.4,1.5.2,1.4.7,1.3.2,1.2.5;https://launchermeta.mojang.com/mc/game/version_manifest.json;"release":";";https://s3.amazonaws.com/Minecraft.Download/versions/
+Vanilla;Latest,Snapshot,1.17.1,1.16.5,1.15.2,1.14.4,1.13.2,1.12.2,1.11.2,1.10.2,1.9.4,1.8.9,1.7.10,1.6.4,1.5.2,1.4.7,1.3.2,1.2.5;https://launchermeta.mojang.com/mc/game/version_manifest.json;"release":";";https://s3.amazonaws.com/Minecraft.Download/versions/
-Spigot;1.16.3,1.15.2,1.14.4,1.13.2,1.12.2,1.11.2,1.10.2,1.9.4,1.9.4,1.8.8,1.7.10,1.6.4-R2.1,1.5.2-R1.1,1.4.7-R1.1;https://static.knarcraft.net/archive/downloads/minecraftserverjars/Spigot/;spigot-
+Spigot;Latest,1.17.1,1.16.5,1.15.2,1.14.4,1.13.2,1.12.2,1.11.2,1.10.2,1.9.4,1.9.4,1.8.8,1.7.10,1.6.4-R2.1,1.5.2-R1.1,1.4.7-R1.1;https://static.knarcraft.net/archive/downloads/minecraftserverjars/Spigot/;spigot-
-Paper;1.16.3,1.15.2,1.14.4,1.13.2,1.12.2,1.11.2,1.10.2,1.9.4,1.8.8;https://static.knarcraft.net/archive/downloads/minecraftserverjars/Paper/;Paper-
+Paper;1.17.1,1.16.5,1.15.2,1.14.4,1.13.2,1.12.2,1.11.2,1.10.2,1.9.4,1.8.8;https://static.knarcraft.net/archive/downloads/minecraftserverjars/Paper/;Paper-
-SpongeVanilla;1.12.2,1.11.2,1.10.2,1.8.9;https://dl-api.spongepowered.org/v1/org.spongepowered/spongevanilla/downloads?type=stable&minecraft=;"version":";",;https://repo.spongepowered.org/maven/org/spongepowered/spongevanilla/;/spongevanilla-
+#SpongeVanilla;1.12.2,1.11.2,1.10.2,1.8.9;https://dl-api.spongepowered.org/v1/org.spongepowered/spongevanilla/downloads?type=stable&minecraft=;"version":";",;https://repo.spongepowered.org/maven/org/spongepowered/spongevanilla/;/spongevanilla-
-Craftbukkit;1.16.3,1.15.2,1.14.4,1.13.2,1.12.2,1.11.2,1.10.2,1.9.4,1.8.8,1.7.10-R0.1,1.6.4-R2.0,1.5.2-R1.0,1.4.6-R0.3,1.3.2-R3.0,1.2.5-R2.0,1.1-R6,1.0.1-R1,b1.8.1,b1.7.3;https://static.knarcraft.net/archive/downloads/minecraftserverjars/Bukkit/;craftbukkit-
+Craftbukkit;Latest,1.17.1,1.16.5,1.15.2,1.14.4,1.13.2,1.12.2,1.11.2,1.10.2,1.9.4,1.8.8,1.7.10-R0.1,1.6.4-R2.0,1.5.2-R1.0,1.4.6-R0.3,1.3.2-R3.0,1.2.5-R2.0,1.1-R6,1.0.1-R1,b1.8.1,b1.7.3;https://static.knarcraft.net/archive/downloads/minecraftserverjars/Bukkit/;craftbukkit-
-SpongeForge;1.12.2,1.11.2,1.10.2;https://dl-api.spongepowered.org/v1/org.spongepowered/spongeforge/downloads?type=stable&minecraft=;"version":";",;https://repo.spongepowered.org/maven/org/spongepowered/spongeforge/;/spongeforge-
+#SpongeForge;1.12.2,1.11.2,1.10.2;https://dl-api.spongepowered.org/v1/org.spongepowered/spongeforge/downloads?type=stable&minecraft=;"version":";",;https://repo.spongepowered.org/maven/org/spongepowered/spongeforge/;/spongeforge-
 MCPCplus;1.6.4,1.6.2,1.5.2,1.4.7;https://static.knarcraft.net/archive/downloads/minecraftserverjars/MCPC+/;mcpcplus
 Bungee;Latest,1.7.10,1.6.4,1.5.2,1.4.7;https://ci.md-5.net/job/BungeeCord/lastSuccessfulBuild/artifact/bootstrap/target/;Artifacts of BungeeCord #; ;http://ci.md-5.net/job/BungeeCord/lastSuccessfulBuild/artifact/bootstrap/target/BungeeCord.jar;https://static.knarcraft.net/archive/downloads/minecraftserverjars/BungeeCord/;BungeeCord-
-Waterfall;1.16,1.15,1.14,1.13,1.12,1.11;https://papermc.io/api/v1/waterfall/;"latest":";";https://papermc.io/api/v1/waterfall/
+Waterfall;1.17,1.16,1.15,1.14,1.13,1.12,1.11;https://papermc.io/api/v1/waterfall/;"latest":;,;https://papermc.io/api/v1/waterfall/
-Travertine;1.16,1.15,1.14,1.13,1.12;https://papermc.io/api/v1/travertine/;"latest":";";https://papermc.io/api/v1/travertine/
+Travertine;1.16,1.15,1.14,1.13,1.12;https://papermc.io/api/v1/travertine/;"latest":;,;https://papermc.io/api/v1/travertine/
 Custom;

src/test/java/net/knarcraft/minecraftserverlauncher/server/ServerTest.java

@@ -1,11 +1,11 @@
 package net.knarcraft.minecraftserverlauncher.server;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 import javax.naming.ConfigurationException;
 
-import static junit.framework.TestCase.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
-import static junit.framework.TestCase.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertFalse;
 
 public class ServerTest {
 

src/test/java/net/knarcraft/minecraftserverlauncher/server/ServerTypeHandlerTest.java

@@ -1,13 +1,12 @@
 package net.knarcraft.minecraftserverlauncher.server;
 
 import net.knarcraft.minecraftserverlauncher.server.servertypes.ServerType;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 import javax.naming.ConfigurationException;
-
 import java.util.Arrays;
 
-import static junit.framework.TestCase.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
 public class ServerTypeHandlerTest {
 

src/test/java/net/knarcraft/minecraftserverlauncher/server/ServerVersionContainerTest.java

@@ -2,13 +2,16 @@ package net.knarcraft.minecraftserverlauncher.server;
 
 import net.knarcraft.minecraftserverlauncher.Main;
 import net.knarcraft.minecraftserverlauncher.utility.CommonFunctions;
-import org.junit.Before;
+import org.junit.jupiter.api.BeforeEach;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
-import java.io.*;
+import java.io.BufferedReader;
+import java.io.File;
+import java.io.FileNotFoundException;
+import java.io.IOException;
 
-import static junit.framework.TestCase.assertEquals;
 import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.createAllFolders;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
 public class ServerVersionContainerTest {
 
@@ -16,7 +19,7 @@ public class ServerVersionContainerTest {
     private final String versionFile = filesDirectory + File.separator + "versions.csv";
     private ServerVersionContainer serverVersionContainer;
 
-    @Before
+    @BeforeEach
     public void setUp() {
         try {
             createAllFolders();
@@ -35,14 +38,14 @@ public class ServerVersionContainerTest {
         serverVersionContainer.reset();
         System.out.println(serverVersionContainer.toString());
         assertEquals("vanillaVersion;null\nsnapshotVersion;null\nbungeeVersion;null\nwaterfallVersions;\n" +
-                        "travertineVersions;\nspongeVanillaVersions;\nspongeForgeVersions;",
+                        "travertineVersions;\nspongeVanillaVersions;\nspongeForgeVersions;\ndownloadedBuildToolsVersion;null",
                 serverVersionContainer.toString());
     }
 
     @Test
     public void saveStateTest() throws IOException {
         serverVersionContainer.saveState();
-        BufferedReader reader = new BufferedReader(new InputStreamReader(new FileInputStream(versionFile)));
+        BufferedReader reader = CommonFunctions.getFileReader(versionFile);
         String savedData = CommonFunctions.readBufferedReader(reader);
         reader.close();
         assertEquals(serverVersionContainer.toString(), savedData);

src/test/java/net/knarcraft/minecraftserverlauncher/userinterface/FakeGUI.java

@@ -1,20 +1,32 @@
 package net.knarcraft.minecraftserverlauncher.userinterface;
 
+import java.io.BufferedWriter;
 import java.io.File;
+import java.io.IOException;
+import java.io.OutputStreamWriter;
 
 public class FakeGUI extends MessageHandler implements GUI {
 
+    private final BufferedWriter writer;
+
     /***
      * Initializes a new fake gui
      */
     public FakeGUI() {
         super(true);
+        this.writer = new BufferedWriter(new OutputStreamWriter(System.out));
     }
 
     @Override
     public void setStatus(String message) {
+        try {
+            writer.write(message);
+            writer.newLine();
+            writer.flush();
+        } catch (IOException e) {
             System.out.println(message);
         }
+    }
 
     @Override
     public File askForDirectory(String prompt) {

src/test/java/net/knarcraft/minecraftserverlauncher/utility/CommonFunctionsTest.java

@@ -1,19 +1,12 @@
 package net.knarcraft.minecraftserverlauncher.utility;
 
-import net.knarcraft.minecraftserverlauncher.Main;
+import org.junit.jupiter.api.Test;
-import org.junit.Test;
 
 import static net.knarcraft.minecraftserverlauncher.utility.CommonFunctions.stringBetween;
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
 public class CommonFunctionsTest {
 
-    @Test
-    public void saveProfileTest() {
-        Main.getController().addProfile("Test");
-        Main.getController().saveState();
-    }
-
     @Test
     public void stringBetweenTest() {
         String substring = stringBetween("fish'nchips", "f", "'");

src/test/java/net/knarcraft/minecraftserverlauncher/utility/JarBuilderTest.java

@@ -0,0 +1,105 @@
+package net.knarcraft.minecraftserverlauncher.utility;
+
+import net.knarcraft.minecraftserverlauncher.Main;
+import net.knarcraft.minecraftserverlauncher.profile.ServerLauncherController;
+import net.knarcraft.minecraftserverlauncher.userinterface.FakeGUI;
+import org.junit.jupiter.api.AfterAll;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.MethodOrderer;
+import org.junit.jupiter.api.Order;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.TestMethodOrder;
+
+import javax.naming.ConfigurationException;
+import java.io.File;
+import java.io.IOException;
+
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+@TestMethodOrder(MethodOrderer.OrderAnnotation.class)
+public class JarBuilderTest {
+
+    private static JarBuilder jarBuilder;
+    private static String targetDirectory;
+    private static String jarDirectory;
+
+    @BeforeAll
+    public static void preSetUp() {
+        try {
+            ServerLauncherController.getInstance().loadState(true);
+        } catch (ConfigurationException | IOException e) {
+            e.printStackTrace();
+        }
+    }
+
+    @BeforeEach
+    public void setUp() {
+        targetDirectory = Main.getApplicationWorkDirectory() + File.separator + "files" + File.separator +
+                "BuildTools" + File.separator;
+        jarDirectory = Main.getApplicationWorkDirectory() + File.separator + "files" + File.separator +
+                "testjars" + File.separator;
+        jarBuilder = new JarBuilder(targetDirectory, jarDirectory, new FakeGUI());
+        removeBuildToolsFiles();
+    }
+
+    @AfterAll
+    public static void tearDown() {
+        removeBuildToolsFiles();
+    }
+
+    @Test
+    @Order(3)
+    public void buildLatestSpigotJarTest() {
+        File spigotFile = new File(jarDirectory + "SpigotLatest.jar");
+        if (spigotFile.exists() && !spigotFile.delete()) {
+            throw new IllegalArgumentException("Unable to remove existing spigot .jar");
+        }
+        jarBuilder.buildSpigotJar();
+        assertTrue(spigotFile.exists());
+    }
+
+    @Test
+    @Order(4)
+    public void buildLatestBukkitJarTest() {
+        File bukkitFile = new File(jarDirectory + "BukkitLatest.jar");
+        if (bukkitFile.exists() && !bukkitFile.delete()) {
+            throw new IllegalArgumentException("Unable to remove existing bukkit .jar");
+        }
+        jarBuilder.buildBukkitJar();
+        assertTrue(new File(jarDirectory + "BukkitLatest.jar").exists());
+    }
+
+    @Test
+    @Order(2)
+    public void downloadLatestBuildToolsJarTest() {
+        jarBuilder.downloadBuildTools();
+        assertTrue(new File(targetDirectory + "BuildTools.jar").exists());
+    }
+
+    @Test
+    @Order(1)
+    public void getLatestBuildToolsVersionTest() {
+        try {
+            String latestVersion = jarBuilder.getLatestBuildToolsVersion();
+            assertNotEquals("", latestVersion);
+            int newVersion = Integer.parseInt(latestVersion);
+            assertNotEquals(newVersion, 0);
+        } catch (IOException e) {
+            e.printStackTrace();
+        }
+    }
+
+    /**
+     * Removes build tools files
+     */
+    private static void removeBuildToolsFiles() {
+        File target = new File(targetDirectory);
+        if (!target.exists() && !target.mkdirs()) {
+            throw new IllegalArgumentException("Unable to create the test files directory");
+        }
+        CommonFunctions.removeFilesRecursively(target);
+    }
+
+}

src/test/java/net/knarcraft/minecraftserverlauncher/utility/JarDownloaderTest.java

@@ -4,28 +4,35 @@ import net.knarcraft.minecraftserverlauncher.Main;
 import net.knarcraft.minecraftserverlauncher.server.ServerTypeHandler;
 import net.knarcraft.minecraftserverlauncher.server.servertypes.ServerType;
 import net.knarcraft.minecraftserverlauncher.userinterface.FakeGUI;
-import org.junit.AfterClass;
+import org.junit.jupiter.api.AfterAll;
-import org.junit.BeforeClass;
+import org.junit.jupiter.api.BeforeAll;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 import javax.naming.ConfigurationException;
 import java.io.File;
+import java.io.FileNotFoundException;
 import java.io.IOException;
 
-import static junit.framework.TestCase.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
-import static junit.framework.TestCase.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
 
 public class JarDownloaderTest {
     private static String targetDirectory;
 
-    @BeforeClass
+    @BeforeAll
     public static void setUp() {
-        targetDirectory = Main.getApplicationWorkDirectory() + File.separator + "files" + File.separator +
+        String filesDirectory = Main.getApplicationWorkDirectory() + File.separator + "files";
-                "testjars" + File.separator;
+        try {
+            CommonFunctions.createFolder(new File(filesDirectory + File.separator + "testjars"));
+        } catch (FileNotFoundException e) {
+            e.printStackTrace();
+        }
+        targetDirectory = filesDirectory + File.separator + "testjars" + File.separator;
         removeDownloadedFiles();
     }
 
-    @AfterClass
+    @AfterAll
     public static void cleanUp() {
         removeDownloadedFiles();
     }
@@ -36,7 +43,7 @@ public class JarDownloaderTest {
         downloader.downloadJars();
     }
 
-    @Test
+    /*@Test
     public void spongeVanillaDownloadTest() throws IOException, ConfigurationException {
         singleDownloadTest(ServerTypeHandler.getByName("SpongeVanilla"));
     }
@@ -44,7 +51,7 @@ public class JarDownloaderTest {
     @Test
     public void spongeForgeDownloadTest() throws IOException, ConfigurationException {
         singleDownloadTest(ServerTypeHandler.getByName("SpongeForge"));
-    }
+    }*/
 
     @Test
     public void spigotDownloadTest() throws ConfigurationException, IOException {
@@ -57,7 +64,7 @@ public class JarDownloaderTest {
     }
 
     @Test
-    public void craftbukkitDownloadTest() throws ConfigurationException, IOException {
+    public void craftBukkitDownloadTest() throws ConfigurationException, IOException {
         singleDownloadTest(ServerTypeHandler.getByName("Bukkit"));
     }
 
@@ -95,6 +102,10 @@ public class JarDownloaderTest {
     private void singleDownloadTest(ServerType serverType) throws IOException {
         assertNotNull(serverType);
         for (String serverVersion : serverType.getVersions()) {
+            if ((serverType.getName().equals("Spigot") || serverType.getName().equals("Bukkit"))
+                    && serverVersion.equals("Latest")) {
+                continue;
+            }
             System.out.println("Downloading " + serverType.getName() + serverVersion + ".jar");
             serverType.downloadJar(targetDirectory, serverVersion);
             assertTrue(new File(targetDirectory + serverType.getName() + serverVersion + ".jar").exists());
@@ -109,12 +120,6 @@ public class JarDownloaderTest {
         if (!target.exists() && !target.mkdirs()) {
             throw new IllegalArgumentException("Unable to create the test files directory");
         }
-        File[] oldFiles = target.listFiles();
+        CommonFunctions.removeFilesRecursively(target);
-        if (oldFiles == null) {
-            throw new IllegalArgumentException("Unable to list files in jar test directory");
-        }
-        for (File file : oldFiles) {
-            assertTrue(file.delete());
-        }
     }
 }

src/test/java/net/knarcraft/minecraftserverlauncher/utility/UpdaterTest.java

@@ -0,0 +1,23 @@
+package net.knarcraft.minecraftserverlauncher.utility;
+
+import org.junit.jupiter.api.Test;
+
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+public class UpdaterTest {
+
+    @Test
+    public void isVersionHigherTest() {
+        assertTrue(Updater.isVersionHigher("1.1.1", "1.1.2"));
+        assertFalse(Updater.isVersionHigher("1.1.1", "1.1.1"));
+        assertFalse(Updater.isVersionHigher("1.1.1", "1.1.0"));
+        assertTrue(Updater.isVersionHigher("1.1.1", "1.2.1"));
+        assertFalse(Updater.isVersionHigher("1.2.1", "1.1.6"));
+        assertTrue(Updater.isVersionHigher("1.2.1", "2.1.6"));
+        assertTrue(Updater.isVersionHigher("1.2.1", "2.0.0"));
+        assertTrue(Updater.isVersionHigher("1.2", "1.2.1"));
+        assertTrue(Updater.isVersionHigher("1.1.2", "1.2"));
+    }
+
+}