Merge branch 'tempBranch' into 'master'

Miscellaneous changes

See merge request oysteikt/h20-tdt4100-project!7

src/main/java/app/MainController.java

@@ -13,7 +13,6 @@ import app.controllers.*;
 import app.events.ExitApplicationEvent;
 import app.events.LanguageChangedEvent;
 import app.events.OpenLinkInBrowserEvent;
-import app.events.SaveFileEvent;
 import app.events.ThemeChangedEvent;
 import app.model.Model;
 import javafx.application.HostServices;
@@ -21,6 +20,9 @@ import javafx.application.Platform;
 import javafx.fxml.FXML;
 import javafx.fxml.Initializable;
 
+/**
+ * An FXML controller that controls the application and all subcontrollers
+ */
 public class MainController implements Initializable {
 
   @FXML
@@ -67,7 +69,9 @@ public class MainController implements Initializable {
     return hostServices;
   }
 
-  // TODO: Document
+  /**
+   * @return All subcontrollers of this controller
+   */
   public List<Controller> getInnerControllers() {
     return List.of(editorController, filetreeController, modelineController, menubarController);
   }
@@ -95,13 +99,17 @@ public class MainController implements Initializable {
     Model.getScene().getStylesheets().set(position, nextStyleSheet);
   }
 
+  /* ------------------------------------------------------------------------ */
+  /*                            EVENT BUS LISTENERS                           */
+  /* ------------------------------------------------------------------------ */
+
   /**
    * Change the CSS according to which language is being used
    * 
    * @param event
    */
   @Subscribe
-  private void handle(LanguageChangedEvent event) {
+  public void handle(LanguageChangedEvent event) {
     this.setCSSAt(1, "/styling/languages/" + event.getLanguage().toLowerCase() + ".css");
   }
 
@@ -111,7 +119,7 @@ public class MainController implements Initializable {
    * @param event
    */
   @Subscribe
-  private void handle(ThemeChangedEvent event) {
+  public void handle(ThemeChangedEvent event) {
     this.setCSSAt(0, "/styling/themes/" + event.getTheme().toLowerCase().replace(" ", "-") + ".css");
   }
 
@@ -121,7 +129,7 @@ public class MainController implements Initializable {
    * @param event
    */
   @Subscribe
-  private void handle(OpenLinkInBrowserEvent event) {
+  public void handle(OpenLinkInBrowserEvent event) {
     this.getHostServices().showDocument(event.getLink());
   }
 
@@ -133,18 +141,14 @@ public class MainController implements Initializable {
    * @param event
    */
   @Subscribe
-  private void handle(ExitApplicationEvent event) {
+  public void handle(ExitApplicationEvent event) {
     if (!Model.getFileIsSaved()) {
       int g = JOptionPane.showConfirmDialog(null, "Your files are not saved.\nSave before exit?", "Exit",
           JOptionPane.YES_NO_OPTION);
 
-      if (g == JOptionPane.YES_OPTION) {
-        this.eventBus.post(new SaveFileEvent());
-        Platform.exit();
-      }
-    } else {
-      Platform.exit();
+      if (g == JOptionPane.YES_OPTION)
+        this.editorController.saveCodeArea(Model.getActiveFilePath().isEmpty());
     }
-
+    Platform.exit();
   }
 }

src/main/java/app/MainLauncher.java

@@ -1,6 +1,14 @@
 package app;
 
+/**
+ * A launcher class to point towards as the start point for a packaged JAR
+ */
 public class MainLauncher {
+  /**
+   * The root function of the call stack
+   * 
+   * @param args Commandline arguments
+   */
     public static void main(String[] args) {
         Main.main(args);
     }

src/main/java/app/controllers/Controller.java

@@ -3,12 +3,12 @@ package app.controllers;
 import com.google.common.eventbus.EventBus;
 
 /**
- * Interface describing a controller that contains an EventBus
+ * Interface describing a JavaFX controller that contains an EventBus
  */
 public interface Controller {  
   /**
    * Registers the main EventBus into the controller.
-   * @param eventBus The EventBus
+   * @param eventBus
    */
   public void setEventBus(EventBus eventBus);
 }

src/main/java/app/controllers/EditorController.java

@@ -34,7 +34,7 @@ import javafx.fxml.Initializable;
 import javafx.stage.Stage;
 
 /**
- * A FXML controller that controls the editor component of the UI
+ * An FXML controller that controls the CodeArea
  */
 public class EditorController implements Initializable, Controller {
 
@@ -60,15 +60,10 @@ public class EditorController implements Initializable, Controller {
     this.eventBus.register(this);
   }
 
-  // TODO: document
-  public CodeArea getEditor() {
-    return editor;
-  }
-
   /**
    * Applies highlighting to the editor.
    * 
-   * @param highlighting highlighting data
+   * @param highlighting Syntax highlighting data
    */
   private void setHighlighting(StyleSpans<Collection<String>> highlighting) {
     this.editor.setStyleSpans(0, highlighting);
@@ -90,7 +85,6 @@ public class EditorController implements Initializable, Controller {
    *      ProgrammingLanguage.commentLine(line)
    */
   private void toggleComment() {
-    // TODO: This logic might need to be moved to LanguageOperations
     if (editor.getSelectedText().equals("")) {
       String currentLine = editor.getText(editor.getCurrentParagraph());
 
@@ -118,7 +112,7 @@ public class EditorController implements Initializable, Controller {
   /**
    * Updates the wraptext setting of the code area
    * 
-   * @param isWrapText The new value for the setting
+   * @param isWrapText The updated setting value
    */
   private void setWrapText(boolean isWrapText) {
     this.editor.setWrapText(isWrapText);
@@ -143,14 +137,16 @@ public class EditorController implements Initializable, Controller {
    * 
    * @param newContent The String to be inserted into the editor
    */
-  public void setEditorContent(String newContent) {
+  private void setEditorContent(String newContent) {
       editor.clear();
       editor.appendText(newContent);
   }
 
   /**
-   * Saving/Writing to the file based on the spesific filepath. Otherwise it will
-   * open an error dialog to give the user feedback about what has happened.
+   * Saving/Writing to the file based on the active filepath in {@link app.model.Model Model}
+   * if it is a new File. Otherwise it will open a dialog to ask the user where to save the file.
+   * 
+   * @param isNewFile Whether or not the file already has a path
    */
   public void saveCodeArea(boolean isNewFile) {
     Stage stage = (Stage) editor.getScene().getWindow();
@@ -164,17 +160,14 @@ public class EditorController implements Initializable, Controller {
     }    
   }
 
-  /**
-   * Checking if all is saved before closing the app. The user can either choose
-   * to exit or go back to the application and save.
-   */
-
   /* ------------------------------------------------------------------------ */
-  /* SUBSCRIPTIONS */
+  /*                            EVENT BUS LISTENERS                           */
   /* ------------------------------------------------------------------------ */
 
   /**
-   * Updates Code Area (read from file) whenever the FileSelected is changed
+   * Updates the CodeArea whenever a new file is opened.
+   * 
+   * @param event
    */
   @Subscribe
   public void handle(OpenFileEvent event) {
@@ -187,54 +180,96 @@ public class EditorController implements Initializable, Controller {
   }
 
   /**
-   * Save file (write to file) whenever the save in the menubare is selected
+   * Saves the editor content to a file
+   * 
+   * @param event
    */
   @Subscribe
-  private void handle(SaveFileEvent event) {
+  public void handle(SaveFileEvent event) {
     this.saveCodeArea(event.getIsNewFile());
   }
 
+  /**
+   * Refreshes the syntax highlighting when the Programming language is changed
+   * 
+   * @param event
+   */
   @Subscribe
-  private void handle(LanguageChangedEvent event) {
+  public void handle(LanguageChangedEvent event) {
     this.refreshHighlighting();
   }
 
+  /**
+   * Toggles a comment based on the editor state
+   * 
+   * @param event
+   */
   @Subscribe
   public void handle(ToggleCommentEvent event) {
     this.toggleComment();
   }
 
+  /**
+   * Toggles the WrapText setting
+   * 
+   * @param event
+   */
   @Subscribe
-  private void handle(ToggleWrapTextEvent event) {
+  public void handle(ToggleWrapTextEvent event) {
     this.setWrapText(event.getIsWrapped());
   }
 
+  /**
+   * Undo if focused
+   * 
+   * @param event
+   */
   @Subscribe
-  private void handle(UndoEvent event) {
+  public void handle(UndoEvent event) {
     if (this.editor.isFocused())
       this.editor.undo();
   }
 
+  /**
+   * Redo if focused
+   * 
+   * @param event
+   */
   @Subscribe
-  private void handle(RedoEvent event) {
+  public void handle(RedoEvent event) {
     if (this.editor.isFocused())
       this.editor.redo();
   }
 
+  /**
+   * Copy selected content if focused
+   * 
+   * @param event
+   */
   @Subscribe
-  private void handle(CopyEvent event) {
+  public void handle(CopyEvent event) {
     if (this.editor.isFocused())
       this.editor.copy();
   }
 
+  /**
+   * Cut selected content if focused
+   * 
+   * @param event
+   */
   @Subscribe
-  private void handle(CutEvent event) {
+  public void handle(CutEvent event) {
     if (this.editor.isFocused())
       this.editor.cut();
   }
 
+  /**
+   * Paste from clipboard if focused
+   * 
+   * @param event
+   */
   @Subscribe
-  private void handle(PasteEvent event) {
+  public void handle(PasteEvent event) {
     if (this.editor.isFocused())
       this.editor.paste();
   }

src/main/java/app/controllers/FiletreeController.java

@@ -26,7 +26,7 @@ import app.service.FiletreeOperations;
 import javafx.fxml.Initializable;
 
 /**
- * A FXML controller that controls the filetree component of the UI
+ * An FXML controller that controls the Filetree
  */
 public class FiletreeController implements Initializable, Controller {
 
@@ -44,19 +44,16 @@ public class FiletreeController implements Initializable, Controller {
     this.eventBus.register(this);
   }
 
-  /* ------------------------------------------------------------------------ */
-  /* FILETREE */
-  /* ------------------------------------------------------------------------ */
-
   /**
-   * The displaying of the fileTree. The inputChosen(the path) is aquired from the
-   * eventBus (OpeFileProjectEvent). The root is created as a CheckBoxItems and
-   * sends it to generateTree, and after that setting it to the root.
+   * Generate a tree structure of a directory, and set the filetree to
+   * show the new tree
+   * 
+   * @param rootDir Path to the directory to be the root of the tree
    */
-  private void showTree(String inputChosen) {
-    CheckBoxTreeItem<String> root = new CheckBoxTreeItem<>(inputChosen);
+  private void showTree(Path rootDir) {
+    CheckBoxTreeItem<String> root = new CheckBoxTreeItem<>(rootDir.getFileName().toString());
     filetree.setShowRoot(false);
-    File fileInputChosen = new File(inputChosen);
+    File fileInputChosen = rootDir.toFile();
 
     try {
       FiletreeOperations.generateTree(fileInputChosen, root);
@@ -67,18 +64,13 @@ public class FiletreeController implements Initializable, Controller {
       DialogBoxes.showErrorMessage(
         "Could not open folder.\n\n"
         + "Do you have the right permissions for this folder?\n"
-        + "Or does the folder contain any shortcut to somewhere within itself?");
+        + "Or does the folder contain any shortcut to somewhere within itself?"
+      );
     }
   }
 
-  /* ------------------------------------------------------------------------ */
-  /* MouseClick */
-  /* ------------------------------------------------------------------------ */
-
   /**
-   * Handles whenever a filetree item is clicked twice. A while loop to create the
-   * correct filepath.
-   */
+   * Handles opening a file whenever a filetree item is clicked twice. */
   @FXML
   private void handleMouseClick(MouseEvent event) {
     if (event.getClickCount() == 2) {
@@ -98,27 +90,33 @@ public class FiletreeController implements Initializable, Controller {
   }
 
   /* ------------------------------------------------------------------------ */
-  /* SUBSCRIPTIONS */
+  /*                            EVENT BUS LISTENERS                           */
   /* ------------------------------------------------------------------------ */
 
   /**
-   * Updates the filetree whenever a new ProjectPath is selected.
+   * Updates the filetree whenever a new project is opened
+   * 
+   * @param event
    */
-
   @Subscribe
   private void handle(OpenProjectEvent event) {
     event.getPath().ifPresentOrElse(
-      path -> this.showTree(path.toString()),
+      path -> this.showTree(path),
       () -> System.err.println("[ERROR] OpenProjectEvent was empty")
     );
   }
 
+  /**
+   * Updates the filetree whenever a new file gets saved
+   * 
+   * @param event
+   */
   @Subscribe
   private void handle(SaveFileEvent event) {
     if (event.getIsNewFile())
       Model
         .getProjectPath()
-        .ifPresent(path -> this.showTree(path.toString()));
+        .ifPresent(path -> this.showTree(path));
   }
 
 }

src/main/java/app/controllers/MenubarController.java

@@ -60,28 +60,25 @@ public class MenubarController implements Initializable, Controller {
     this.eventBus.register(this);
   }
 
-  /* ------------------------------------------------------------------------ */
-  /* CREATE FILE/DIRECTORY */
-  /* ------------------------------------------------------------------------ */
+  /* ---------------------------------- File ---------------------------------- */
+
+  /**
+   * Handles whenever the New File button is pressed in the menubar
+   * 
+   * @param event
+   */
   @FXML
   private void handleNewFile() {
     this.eventBus.post(new OpenFileEvent(Optional.empty()));
   }
 
-  @FXML
-  private void handleNewFolder() {
-
-  }
-
-  /* ------------------------------------------------------------------------ */
-  /* OPEN FILE/PROJECT */
-  /* ------------------------------------------------------------------------ */
-
   /**
    * Handles whenever the Open File button is pressed in the menubar
+   * 
+   * @param event
    */
   @FXML
-  public void handleOpenFile() {
+  private void handleOpenFile() {
     Stage stage = (Stage) menubar.getScene().getWindow();
 
     try {
@@ -95,6 +92,8 @@ public class MenubarController implements Initializable, Controller {
 
   /**
    * Handles whenever the Open Project button is pressed in the menubar
+   * 
+   * @param event
    */
   @FXML
   private void handleOpenProject() {
@@ -107,12 +106,11 @@ public class MenubarController implements Initializable, Controller {
     } catch (FileNotFoundException e) {}
   }
 
-  /* ------------------------------------------------------------------------ */
-  /* SAVE FILE */
-  /* ------------------------------------------------------------------------ */
 
   /**
    * Handles whenever the Save button is pressed in the menubar
+   * 
+   * @param event
    */
   @FXML
   private void handleSaveFile() {
@@ -121,6 +119,8 @@ public class MenubarController implements Initializable, Controller {
 
   /**
    * Handles whenever the Save as button is pressed in the menubar
+   * 
+   * @param event
    */
   @FXML
   private void handleSaveAsFile() {
@@ -129,6 +129,8 @@ public class MenubarController implements Initializable, Controller {
 
   /**
    * Handles whenever the programming language is changed from the menubar.
+   * 
+   * @param event
    */
   @FXML
   private void handleLanguageChange(ActionEvent event) {
@@ -137,6 +139,8 @@ public class MenubarController implements Initializable, Controller {
 
   /**
    * Handles whenever the wraptext togglebutton is pressed in the menubar
+   * 
+   * @param event
    */
   @FXML
   private void handleToggleWraptext(ActionEvent event) {
@@ -146,6 +150,8 @@ public class MenubarController implements Initializable, Controller {
 
   /**
    * Handles whenever the theme is changed from the menubar
+   * 
+   * @param event
    */
   @FXML
   private void handleThemeChange(ActionEvent event) {
@@ -154,18 +160,20 @@ public class MenubarController implements Initializable, Controller {
 
   /**
    * Handles whenever the exit button is pressed in the menubar
+   * 
+   * @param event
    */
   @FXML
   private void handleExitApplication(ActionEvent event) {
     this.eventBus.post(new ExitApplicationEvent());
   }
 
-  /* ------------------------------------------------------------------------ */
-  /* EDIT */
-  /* ------------------------------------------------------------------------ */
+  /* ---------------------------------- Edit ---------------------------------- */
 
   /**
    * Handles whenever the undo button is pressed in the menubar
+   * 
+   * @param event
    */
   @FXML
   private void handleUndo(ActionEvent event) {
@@ -174,6 +182,8 @@ public class MenubarController implements Initializable, Controller {
 
   /**
    * Handles whenever the redo button is pressed in the menubar
+   * 
+   * @param event
    */
   @FXML
   private void handleRedo(ActionEvent event) {
@@ -182,6 +192,8 @@ public class MenubarController implements Initializable, Controller {
 
   /**
    * Handles whenever the copy button is pressed in the menubar
+   * 
+   * @param event
    */
   @FXML
   private void handleCopy(ActionEvent event) {
@@ -190,6 +202,8 @@ public class MenubarController implements Initializable, Controller {
 
   /**
    * Handles whenever the cut button is pressed in the menubar
+   * 
+   * @param event
    */
   @FXML
   private void handleCut(ActionEvent event) {
@@ -198,6 +212,8 @@ public class MenubarController implements Initializable, Controller {
 
   /**
    * Handles whenever the paste button is pressed in the menubar
+   * 
+   * @param event
    */
   @FXML
   private void handlePaste(ActionEvent event) {
@@ -206,18 +222,20 @@ public class MenubarController implements Initializable, Controller {
 
   /**
    * Handles whenever the Toggle Comment button is pressed in the menubar
+   * 
+   * @param event
    */
   @FXML
   private void handleToggleComment(ActionEvent event) {
     this.eventBus.post(new ToggleCommentEvent());
   }
 
-  /* ------------------------------------------------------------------------ */
-  /* ABOUT */
-  /* ------------------------------------------------------------------------ */
+  /* ---------------------------------- About --------------------------------- */
 
   /**
    * Handles whenever the About button is pressed in the menubar
+   * 
+   * @param event
    */
   @FXML
   private void handleAbout(ActionEvent event) {
@@ -231,22 +249,37 @@ public class MenubarController implements Initializable, Controller {
 
   /**
    * Updates menubuttons whenever the language is changed
+   * 
+   * @param event
    */
   @Subscribe
-  private void handle(LanguageChangedEvent event) {
-    this.languageToggleGroup.getToggles().stream().map(RadioMenuItem.class::cast)
-        .filter(t -> t.getId().equals("toggle" + event.getLanguage())).findFirst().orElseThrow().setSelected(true);
+  public void handle(LanguageChangedEvent event) {
+    this.languageToggleGroup
+      .getToggles()
+      .stream()
+      .map(RadioMenuItem.class::cast)
+      .filter(t -> t.getId().equals("toggle" + event.getLanguage()))
+      .findFirst()
+      // This should never happen!
+      .orElseThrow(() -> new IllegalStateException("Language button missing: " + event.getLanguage()))
+      .setSelected(true);
   }
 
   /**
    * Updates menubuttons whenever the theme is changed
+   * 
+   * @param event
    */
-
   @Subscribe
-  private void handle(ThemeChangedEvent event) {
-    this.themeToggleGroup.getToggles().stream().map(RadioMenuItem.class::cast)
-        .filter(t -> t.getId().equals("toggle" + event.getTheme().replace(" ", "_"))).findFirst().orElseThrow().setSelected(true);
-
+  public void handle(ThemeChangedEvent event) {
+    this.themeToggleGroup
+      .getToggles()
+      .stream()
+      .map(RadioMenuItem.class::cast)
+      .filter(t -> t.getId().equals("toggle" + event.getTheme().replace(" ", "_")))
+      .findFirst()
+      // This should never happen!
+      .orElseThrow(() -> new IllegalStateException("Theme button missing: " + event.getTheme()))
+      .setSelected(true);
   }
-
 }

src/main/java/app/controllers/ModelineController.java

@@ -53,34 +53,48 @@ public class ModelineController implements Initializable, Controller {
     this.columnrow.setText(String.format("[%d:%d]", row, column));
   }
 
+  /* ------------------------------------------------------------------------ */
+  /* SUBSCRIPTIONS */
+  /* ------------------------------------------------------------------------ */
+
   /**
    * Updates the column-row number display whenever the editor cursor
    * changes position.
+   * 
+   * @param event
    */
   @Subscribe
-  private void handle(EditorChangedEvent event) {
+  public void handle(EditorChangedEvent event) {
     this.setColumnRow(event.getColumn(), event.getLine());
   }
 
   /**
    * Updates the saveState label whenever the file either is saved or modified
+   * 
+   * @param event
    */
   @Subscribe
-  private void handle(FileSaveStateChangedEvent event) {
+  public void handle(FileSaveStateChangedEvent event) {
     // TODO: Add CSS styleclass for coloring the saveState label
     //       whenever it changes
     this.saveState.setText(event.getIsSaved() ? "Saved!" : "Modified");
   }
 
   /**
-   * Updates the modeline to display a new language
-   * whenever it is changed.
+   * Updates the modeline to display a new language when changed.
+   * 
+   * @param event
    */
   @Subscribe
   private void handle(LanguageChangedEvent event) {
     this.language.setText(event.getLanguage());
   }
 
+  /**
+   * Updates the modeline to display the name of the current file when changed
+   * 
+   * @param event
+   */
   @Subscribe
   private void handle(OpenFileEvent event) {
     this.filename.setText(

src/main/java/app/model/ProgrammingLanguage.java

@@ -5,22 +5,21 @@ import java.util.regex.Pattern;
 
 /**
  * An interface describing functions required for a class to
- * provide language specific details and functionality to the 
- * editor
+ * provide language specific details and functionality.
  */
 public interface ProgrammingLanguage {
   /**
-   * The name of the programming language
+   * @return The name of the programming language
    */
   public String getName();
   
   /**
-   * The map containing the regex and corresponding style-classes to be used for syntax highlighting
+   * @return The map containing the regexes and corresponding style-classes to be used for syntax highlighting
    */
   public Map<Pattern,String> getPatternMap();
 
   /**
-   * The pattern containing all regexes for syntax highlighting
+   * @return A combined regex for syntax highlighting
    */
   public Pattern getPattern();
 
@@ -39,8 +38,8 @@ public interface ProgrammingLanguage {
   public String unCommentLine(String line);
 
   /**
-   * Whether or not a line is commented
    * @param line The text of the line
+   * @return Whether or not a line is commented
    */
   public boolean isCommentedLine(String line);
 
@@ -52,15 +51,15 @@ public interface ProgrammingLanguage {
   public String commentSelection(String selection);
 
   /**
-   * Uncomment a line
-   * @param selection The text of the line to uncomment
+   * Uncomment an area of text
+   * @param selection The text of the area to uncomment
    * @return The uncommented area
    */
   public String unCommentSelection(String selection);
 
   /**
-   * Whether or not an area of text is commented
    * @param selection The content of the area
+   * @return Whether or not an area of text is commented
    */
   public boolean isCommentedSelection(String selection);
 

src/main/java/app/model/languages/Java.java

@@ -15,6 +15,7 @@ import app.model.ProgrammingLanguage;
 public class Java implements ProgrammingLanguage {
 
   private String name = "Java";
+  private static Map<Pattern, String> pattern;
 
   private static final String[] keywords = new String[] {
     "abstract", "assert", "boolean", "break", "byte",
@@ -52,9 +53,11 @@ public class Java implements ProgrammingLanguage {
       e("(?://.*)|/\\*(?:\\n|.)*?\\*/", "comment")
     );
   
-  private static Map<Pattern, String> pattern;
-
   public Java() {
+    this.initializePatternMap();
+  }
+
+  private void initializePatternMap() {
     pattern = new LinkedHashMap<>();
     patternList
       .forEach(e -> pattern.put(e.getKey(), e.getValue()));

src/main/java/app/model/languages/Markdown.java

@@ -14,6 +14,7 @@ import app.model.ProgrammingLanguage;
 public class Markdown implements ProgrammingLanguage {
 
   private String name = "Markdown";
+  private static Map<Pattern, String> pattern;
 
   private static Entry<Pattern, String> e(String k, String v) {
     return new AbstractMap.SimpleEntry<>(Pattern.compile(k), v);
@@ -38,9 +39,12 @@ public class Markdown implements ProgrammingLanguage {
       e("\\[\\d+\\]: .*", "source")
     );
 
-  private static Map<Pattern, String> pattern;
 
   public Markdown() {
+    this.initializePatternMap();
+  }
+
+  private void initializePatternMap() {
     pattern = new LinkedHashMap<>();
     patternList
       .forEach(e -> pattern.put(e.getKey(), e.getValue()));

src/main/java/app/service/DialogBoxes.java

@@ -1,16 +1,51 @@
 package app.service;
 
+import java.io.File;
+
+import app.model.Model;
 import javafx.scene.control.Alert;
 import javafx.scene.control.Alert.AlertType;
+import javafx.stage.DirectoryChooser;
+import javafx.stage.FileChooser;
+import javafx.stage.Stage;
 
 public class DialogBoxes {
 
   private DialogBoxes() {}
+  private static FileChooser fc = new FileChooser();
+  private static DirectoryChooser dc = new DirectoryChooser();
+  private static Alert error = new Alert(AlertType.ERROR);
 
   public static void showErrorMessage(String errorMessage) {
-      Alert error = new Alert(AlertType.ERROR);
       error.setContentText(errorMessage);
       error.showAndWait();
   }
+
+  public static File showopenFileWithDialog(Stage stage) {
+    fc.setTitle("Open File");
+    File chosenFile = fc.showOpenDialog(stage);
+
+    return chosenFile; 
+  }
+
+  public static File showOpenFolderWithDialog(Stage stage) {
+    dc.setTitle("Open Project");
+    File dir = dc.showDialog(stage);
+
+    return dir;
+  }
+
+  public static File showSaveFileWithDialog(Stage stage) {
+    FileChooser fc = new FileChooser();
+    fc.setTitle("Save as");
+
+    Model
+      .getProjectPath()
+      .ifPresent(path -> fc.setInitialDirectory(path.toFile()));
+
+    File chosenLocation = fc.showSaveDialog(stage);
+
+    return chosenLocation;
+  }
   
 }

src/main/java/app/service/FileOperations.java

@@ -4,12 +4,11 @@ import java.io.File;
 import java.io.FileNotFoundException;
 import java.io.PrintWriter;
 import java.nio.file.Path;
+import java.util.NoSuchElementException;
 import java.util.Optional;
 import java.util.Scanner;
 
 import app.model.Model;
-import javafx.stage.DirectoryChooser;
-import javafx.stage.FileChooser;
 import javafx.stage.Stage;
 
 public class FileOperations {
@@ -20,11 +19,7 @@ public class FileOperations {
   // TODO: This class needs to be extensively error checked
 
   public static File openFileWithDialog(Stage stage) throws FileNotFoundException {
-
-    FileChooser fc = new FileChooser();
-    fc.setTitle("Open File");
-
-    File chosenFile = fc.showOpenDialog(stage);
+    File chosenFile = DialogBoxes.showopenFileWithDialog(stage);
 
     if (chosenFile == null)
       throw new FileNotFoundException();
@@ -34,11 +29,7 @@ public class FileOperations {
   }
 
   public static File openFolderWithDialog(Stage stage) throws FileNotFoundException {
-
-    DirectoryChooser dc = new DirectoryChooser();
-    dc.setTitle("Open Project");
-
-    File dir = dc.showDialog(stage);
+    File dir = DialogBoxes.showOpenFolderWithDialog(stage);
 
     if (dir == null)
       throw new FileNotFoundException();
@@ -58,16 +49,15 @@ public class FileOperations {
   }
 
   public static boolean saveFileWithDialog(Stage stage, String content) {
-    FileChooser fc = new FileChooser();
-    fc.setTitle("Save as");
+    File chosenLocation;
 
-    Model
-      .getProjectPath()
-      .ifPresent(path -> fc.setInitialDirectory(path.toFile()));
-
-    File chosenLocation = fc.showSaveDialog(stage);
-    if (chosenLocation == null)
+    try {
+      chosenLocation = DialogBoxes.showSaveFileWithDialog(stage);
+    } catch (NoSuchElementException e) {
       return false;
+    }
+
+    if (chosenLocation == null) return false;
 
     if (saveFile(chosenLocation.toPath(), content)) {
       Model.setActiveFilePath(Optional.of(chosenLocation.toPath()));

src/main/java/app/settings/SettingsProvider.java

@@ -17,15 +17,21 @@ import app.model.Model;
 
 public class SettingsProvider implements SettingsProviderI {
 
-    private static EventBus eventBus;
+    private EventBus eventBus;
 
-    private static final String SETTINGS_PATH =
+    private String settingsPath =
       (System.getProperty("os.name").startsWith("Windows"))
       ? System.getProperty("user.home") + "\\AppData\\Roaming\\/BNNsettings.dat"
       : System.getProperty("user.home") + System.getProperty("file.separator") + ".BNNsettings.dat";
 
-    private static List<String> legalSettings = 
+    private List<String> legalSettings = 
        Arrays.asList("Java", "Markdown", "Monokai", "Solarized Light");
+    
+
+    // Only for testing purposes
+    protected void setSettingsPath(String settingsPath) {
+      this.settingsPath = settingsPath;
+    }
 
 
     public SettingsProvider(EventBus eB) {
@@ -35,13 +41,13 @@ public class SettingsProvider implements SettingsProviderI {
 
     public void setEventBus(EventBus eB) {
         eventBus = eB;
-        SettingsProvider.eventBus.register(this);
+        eventBus.register(this);
     }
 
     @Override
     public void loadSettings() {
         List<String> settings = new ArrayList<>();
-        try (Scanner sc = new Scanner(new File(SETTINGS_PATH))) {
+        try (Scanner sc = new Scanner(new File(settingsPath))) {
 
             while (sc.hasNextLine()) {
                 var nextLine = sc.nextLine().trim();
@@ -69,7 +75,7 @@ public class SettingsProvider implements SettingsProviderI {
 
     @Override
     public void saveSettings() {
-        try (PrintWriter writer = new PrintWriter(new File(SETTINGS_PATH))) {
+        try (PrintWriter writer = new PrintWriter(new File(settingsPath))) {
             writer.println("- Settings:");
             writer.println("Programming Language = " + Model.getLanguage().getName());
             writer.println("Theme = " + Model.getTheme());

src/main/java/app/settings/SettingsProviderI.java

@@ -2,8 +2,14 @@ package app.settings;
 
 public interface SettingsProviderI {
 
-    void loadSettings();
+  /**
+   * Load settings from disk, and fire events to update the program state
+   */
+  void loadSettings();
 
-    void saveSettings();
+  /**
+   * Save the state from {@link app.model.Model Model} to disk.
+   */
+  void saveSettings();
 
 }

src/main/resources/fxml/components/Menubar.fxml

@@ -19,7 +19,6 @@
     <Menu mnemonicParsing="false" text="File">
       <items>
         <MenuItem mnemonicParsing="false" text="New File" accelerator="Shortcut+n" onAction="#handleNewFile"/>
-        <MenuItem mnemonicParsing="false" text="New Folder" accelerator="Shortcut+Shift+N" onAction="#handleNewFolder"/>
         <SeparatorMenuItem/>
         <MenuItem mnemonicParsing="false" text="Open File" accelerator="Shortcut+o" onAction="#handleOpenFile"/>
         <MenuItem mnemonicParsing="false" text="Open Project" accelerator="Shortcut+Shift+O" onAction="#handleOpenProject"/>