Remove schedule test buttons

Task scheduler entry for Windows, cron entry for *nix

src/main/java/name/nathanmcrae/numbersstation/AboutController.java

@@ -48,7 +48,7 @@ public class AboutController {
         try {
             properties.load(getClass().getClassLoader().getResourceAsStream("git.properties"));
         } catch (IOException e) {
-            logger.log(Level.SEVERE, "Failed to load main view", e);
+            logger.log(Level.SEVERE, "Failed to load git information", e);
         }
         versionTextField.setText(String.valueOf(properties.get("git.commit.id.full")));
 

src/main/java/name/nathanmcrae/numbersstation/HelpController.java

@@ -13,9 +13,7 @@ public class HelpController {
 
     @FXML
     public void initialize() throws Exception {
-        logger.info("20250407T222451");
         String url = HelpController.class.getResource("/help-index.html").toExternalForm();
         webView.getEngine().load(url);
-        logger.info("20250407T222456");
     }
 }

src/main/java/name/nathanmcrae/numbersstation/LinuxScheduler.java

@@ -27,8 +27,7 @@ public class LinuxScheduler {
         try {
             String taskName = "numbers-station-main_" + settings.getName();
 
-            // TODO: assume it's on the PATH
-            Process listProcess = new ProcessBuilder("/usr/bin/crontab", "-l").start();
+            Process listProcess = new ProcessBuilder("crontab", "-l").start();
             if (!listProcess.waitFor(5, TimeUnit.SECONDS)) {
                 String message = "Failed to query " + taskName + " task: process timed out";
                 logger.log(Level.SEVERE, message);
@@ -61,6 +60,10 @@ public class LinuxScheduler {
                     matcher.appendReplacement(sb, "");
                 }
 
+                if (foundMultiple) {
+                    logger.log(Level.WARNING, "Found multiple instances of '" + taskName + "' prior to replacement");
+                }
+
                 matcher.appendTail(sb);
             } else {
                 sb.append(currentCrontab);
@@ -69,8 +72,7 @@ public class LinuxScheduler {
 
             String newCrontab = sb.toString();
 
-            // TODO: assume it's on the PATH
-            Process addProcess = new ProcessBuilder("/usr/bin/crontab", "-").start();
+            Process addProcess = new ProcessBuilder("crontab", "-").start();
             Writer w = new OutputStreamWriter(addProcess.getOutputStream(), "UTF-8");
             System.out.println(newCrontab);
             w.write(newCrontab);
@@ -94,22 +96,104 @@ public class LinuxScheduler {
             return Results.failure(message);
         }
 
-        // Path cronDPath = Paths.get("/etc/cron.d");
-        // if (!Files.exists(cronDPath)) {
-        //     String message =  "/etc/cron.d does not exist, cannot create cron entry. Select 'Manage schedule externally' and set up scheduling as desired.";
-        //     logger.log(Level.SEVERE, message);
-        //     return Results.failure(message);
-        // }
+        return Results.success(true);
+    }
 
-        // Path cronPath = cronDPath.resolve(settings.safeName());
+    public static Result<Boolean, String> scheduleExists(StationSettings settings) {
+        try {
+            String taskName = "numbers-station-main_" + settings.getName();
 
-        // try {
-        //     Files.write(cronPath, cronEntry(settings).getBytes(StandardCharsets.UTF_8));
-        // } catch (Exception e) {
-        //     String message = "Failed to write cron file at '" + cronPath.toString() + "'";
-        //     logger.log(Level.SEVERE, message, e);
-        //     return Results.failure(message);
-        // }
+            Process listProcess = new ProcessBuilder("crontab", "-l").start();
+            if (!listProcess.waitFor(5, TimeUnit.SECONDS)) {
+                String message = "Failed to query " + taskName + " task: process timed out";
+                logger.log(Level.SEVERE, message);
+                return Results.failure(message);
+            }
+
+            Pair<String, String> output = WindowsScheduler.captureProcessOutput(listProcess);
+            if(listProcess.exitValue() != 0) {
+                String message = "Failed to get user id. Exit code: " + listProcess.exitValue() + ". stdout: " + output.getKey() + "\n\tstderr: " + output.getValue();
+                logger.log(Level.SEVERE, message);
+                return Results.failure(message);
+            }
+
+            String currentCrontab = output.getKey();
+
+            String cronEntry = "# " + taskName + "\n" + cronExpression(settings) + "\n\n";
+
+            Pattern pattern = Pattern.compile("# " + taskName + "\\n[^\\n]+\\n\\n");
+
+            Matcher matcher = pattern.matcher(currentCrontab);
+
+            if (matcher.find()) {
+                return Results.success(true);
+            } else {
+                return Results.success(true);
+            }
+        } catch (CronExpressionException | IOException | InterruptedException e) {
+            String message = "Exception while checking for schedule";
+            logger.log(Level.SEVERE, message, e);
+            return Results.failure(message);
+        }
+    }
+
+    public static Result<Boolean, String> removeSchedule(StationSettings settings) {
+        try {
+            String taskName = "numbers-station-main_" + settings.getName();
+
+            Process listProcess = new ProcessBuilder("crontab", "-l").start();
+            if (!listProcess.waitFor(5, TimeUnit.SECONDS)) {
+                String message = "Failed to query " + taskName + " task: process timed out";
+                logger.log(Level.SEVERE, message);
+                return Results.failure(message);
+            }
+
+            Pair<String, String> output = WindowsScheduler.captureProcessOutput(listProcess);
+            if(listProcess.exitValue() != 0) {
+                String message = "Failed to get user id. Exit code: " + listProcess.exitValue() + ". stdout: " + output.getKey() + "\n\tstderr: " + output.getValue();
+                logger.log(Level.SEVERE, message);
+                return Results.failure(message);
+            }
+
+            String currentCrontab = output.getKey();
+
+            String cronEntry = "# " + taskName + "\n" + cronExpression(settings) + "\n\n";
+
+            Pattern pattern = Pattern.compile("# " + taskName + "\\n[^\\n]+\\n\\n");
+
+            Matcher matcher = pattern.matcher(currentCrontab);
+
+            StringBuilder sb = new StringBuilder();
+
+            while (matcher.find()) {
+                matcher.appendReplacement(sb, "");
+            }
+
+            String newCrontab = sb.toString();
+
+            Process addProcess = new ProcessBuilder("crontab", "-").start();
+            Writer w = new OutputStreamWriter(addProcess.getOutputStream(), "UTF-8");
+            System.out.println(newCrontab);
+            w.write(newCrontab);
+            w.flush();
+            w.close();
+            if (!addProcess.waitFor(5, TimeUnit.SECONDS)) {
+                String message = "Failed to register " + taskName + " task: process timed out";
+                logger.log(Level.SEVERE, message);
+                return Results.failure(message);
+            }
+
+            if(addProcess.exitValue() != 0) {
+                Pair<String, String> addOutput = WindowsScheduler.captureProcessOutput(addProcess);
+                String message = "Failed to get user id. Exit code: " + addProcess.exitValue() + ". stdout: " + addOutput.getKey() + ". stderr: " + addOutput.getValue();
+                logger.log(Level.SEVERE, message);
+                return Results.failure(message);
+            }
+        } catch (CronExpressionException | IOException | InterruptedException e) {
+            String message = "Exception while removing schedule";
+            logger.log(Level.SEVERE, message, e);
+            return Results.failure(message);
+        }
 
         return Results.success(true);
     }

src/main/java/name/nathanmcrae/numbersstation/Main.java

@@ -36,6 +36,7 @@ import java.util.logging.FileHandler;
 import java.util.logging.Logger;
 import java.util.logging.Level;
 import java.util.Optional;
+import java.util.Properties;
 import javafx.application.Application;
 import javafx.application.Platform;
 import javafx.fxml.FXMLLoader;
@@ -50,8 +51,6 @@ public class Main extends Application {
     public record StartParameters (Optional<NotificationController.NotificationParameters> notification) {}
 
     private static final Logger logger = Logger.getLogger(Main.class.getName());
-    // TODO: get git info
-    private static final String VERSION = "0.0.1";
     private static Path configPath = null;
     private static Path statePath = null;
 
@@ -130,6 +129,14 @@ public class Main extends Application {
                 primaryStage.titleProperty().bindBidirectional(controller.windowTitle);
                 primaryStage.show();
                 logger.info("Application started");
+
+                Properties properties = new Properties();
+                try {
+                    properties.load(getClass().getClassLoader().getResourceAsStream("git.properties"));
+                    logger.info("Application version: " + String.valueOf(properties.get("git.commit.id.full")));
+                } catch (IOException e) {
+                    logger.log(Level.SEVERE, "Failed to load git information", e);
+                }
             }
         } catch (IOException e) {
             logger.log(Level.SEVERE, "Failed to load main view", e);
@@ -167,7 +174,13 @@ public class Main extends Application {
         }
 
         if (parsedArgs.getVersionFlag()) {
-            System.out.println("Numbers Station version " + VERSION);
+            Properties properties = new Properties();
+            try {
+                properties.load(Main.class.getClassLoader().getResourceAsStream("git.properties"));
+                System.out.println(String.valueOf(properties.get("git.commit.id.full")));
+            } catch (IOException e) {
+                System.out.println("Failed to get git information");
+            }
             System.exit(0);
         }
 
@@ -270,8 +283,8 @@ public class Main extends Application {
             switch (loadedStation.getMessageMethod()) {
                 case StationSettings.MessageMethod.EXTERNAL_PROGRAM:
                     Process process = new ProcessBuilder(loadedStation.getExternalProgramCommand(), nextMessagePath.toString()).start();
-                    // TODO: make timeout a setting
-                    if (!process.waitFor(5, TimeUnit.SECONDS)) {
+
+                    if (!process.waitFor(loadedStation.getExternalProgramTimeout().getSeconds(), TimeUnit.SECONDS)) {
                         throw new StationRunException("Timeout while running external program " + loadedStation.getExternalProgramCommand());
                     }
 

src/main/java/name/nathanmcrae/numbersstation/MainController.java

@@ -25,6 +25,7 @@ import javafx.fxml.FXMLLoader;
 import javafx.fxml.Initializable;
 import javafx.scene.control.Alert;
 import javafx.scene.control.Alert.AlertType;
+import javafx.scene.control.Button;
 import javafx.scene.control.ButtonType;
 import javafx.scene.control.Label;
 import javafx.scene.control.TextArea;
@@ -181,22 +182,37 @@ public class MainController implements Initializable {
         stationNameField.textProperty().bindBidirectional(selectedStationName);
         nextSendTimeLabel.textProperty().bind(nextSendTime);
 
-        nextSendTime.set("sup brah");
-
         Result<MainSettings, Exception> result = MainSettings.load();
         if (!result.hasSuccess()) {
-            // TODO: on failure, prompt user to re-initialize settings
-            Alert alert = new Alert(Alert.AlertType.ERROR);
+            String logPath = Main.getStatePath().resolve("main.log").toString();
+            String message = "Unable to load settings file. See log file at " + logPath + " for details. Reinitialize settings? This may overwrite existing settings.";
+            Alert alert = new Alert(Alert.AlertType.ERROR, message, ButtonType.NO, ButtonType.YES);
             alert.setTitle("Settings load error");
             alert.setHeaderText(null);
-            alert.setContentText("Unable to load settings file");
-            alert.showAndWait();
 
-            return;
+            //Deactivate Defaultbehavior for yes-Button:
+            Button yesButton = (Button) alert.getDialogPane().lookupButton( ButtonType.YES );
+            yesButton.setDefaultButton( false );
+
+            //Activate Defaultbehavior for no-Button:
+            Button noButton = (Button) alert.getDialogPane().lookupButton( ButtonType.NO );
+            noButton.setDefaultButton( true );
+
+            Optional<ButtonType> promptResult = alert.showAndWait();
+
+            if (!promptResult.isPresent()) {
+                System.exit(1);
+            }
+
+            if (promptResult.get() == ButtonType.NO) {
+                System.exit(1);
+            }
+
+            settings = new MainSettings();
+        } else {
+            settings = result.getSuccess().get();
         }
 
-        settings = result.getSuccess().get();
-
         selectedStationName.set(settings.getSelectedStationName());
 
         if (selectedStationName.get() == null || selectedStationName.get() == "") {
@@ -209,8 +225,6 @@ public class MainController implements Initializable {
             .findFirst()
             .orElse(null);
 
-        // TODO: if there are no stations, then create a default station and select that
-
         if (selectedStation == null) {
             logger.log(Level.SEVERE, "Selected station '" + selectedStationName.get() + "' not found");
             selectedStation = settings.getStations().get(0);
@@ -315,8 +329,6 @@ public class MainController implements Initializable {
 
         settings.save();
 
-        // TODO: Load message from file
-        // If the message we're overwriting is different from its file, then prompt to confirm
         Path nextMessagePath = selectedStation.stationPath().resolve("next-message.txt");
 
         String messageText;

src/main/java/name/nathanmcrae/numbersstation/MainSettings.java

@@ -100,6 +100,7 @@ public class MainSettings {
         XmlMapper xmlMapper = new XmlMapper();
         xmlMapper.registerModule(new JavaTimeModule());
         xmlMapper.configure(SerializationFeature.WRITE_DATES_AS_TIMESTAMPS, false);
+        xmlMapper.configure(SerializationFeature.WRITE_DURATIONS_AS_TIMESTAMPS, false);
         xmlMapper.enable(SerializationFeature.INDENT_OUTPUT);
         try {
             Path filePath = getSettingsFilePath();

src/main/java/name/nathanmcrae/numbersstation/StationSelectionController.java

@@ -138,14 +138,12 @@ public class StationSelectionController {
         while (change.next()) {
             if (change.wasAdded()) {
                 for (StationSettings addedStation : change.getAddedSubList()) {
-                    System.out.println("Added: " + addedStation.getName());
                 }
             }
             if (change.wasRemoved()) {
                 for (StationSettings removedStation : change.getRemoved()) {
-                    System.out.println("Removed: " + removedStation.getName());
-
                     try {
+                        StationSettings.deleteStationData(removedStation.getName());
                         removedStation.deleteDir();
                     } catch (IOException e) {
                         logger.log(Level.SEVERE, "Exception when removing station directory", e);
@@ -170,7 +168,6 @@ public class StationSelectionController {
 
         if (selectedStation != null) {
             stage.setUserData(selectedStation.getName());
-            System.out.println("Selected Station: " + selectedStation.getName());
         }
         stage.close();
     }

src/main/java/name/nathanmcrae/numbersstation/StationSettings.java

@@ -10,6 +10,7 @@ import java.nio.file.Path;
 import java.nio.file.Paths;
 import java.nio.file.SimpleFileVisitor;
 import java.security.SecureRandom;
+import java.time.Duration;
 import java.time.LocalDate;
 import java.time.LocalDateTime;
 import java.time.ZonedDateTime;
@@ -27,6 +28,7 @@ public class StationSettings {
     private String address;
     private int digitsPerGroup;
     private String externalProgramCommand;
+    private Duration externalProgramTimeout;
     private boolean manageScheduleExternally;
     private int messageLength;
     private MessageMethod messageMethod;
@@ -53,6 +55,7 @@ public class StationSettings {
         prefixes = new ArrayList<String>();
         digitsPerGroup = 4;
         messageLength = 100;
+        externalProgramTimeout = Duration.ofSeconds(5);
         messageMethod = MessageMethod.SFTP;
         messagePeriod = MessagePeriod.DAILY;
         scheduleStart = ZonedDateTime.now();
@@ -62,6 +65,7 @@ public class StationSettings {
         name = newName;
         prefixes = new ArrayList<String>();
         digitsPerGroup = 4;
+        externalProgramTimeout = Duration.ofSeconds(5);
         messageLength = 100;
         messageMethod = MessageMethod.SFTP;
         messagePeriod = MessagePeriod.DAILY;
@@ -77,8 +81,7 @@ public class StationSettings {
         if (osName.contains("win")) {
             WindowsScheduler.removeSchedule(temp);
         } else if (osName.contains("nix") || osName.contains("nux") || osName.contains("aix")) {
-            // TODO: linux
-            logger.log(Level.SEVERE, "Unsupported OS " + osName);
+            LinuxScheduler.removeSchedule(temp);
         } else {
             logger.log(Level.SEVERE, "Unsupported OS " + osName);
         }
@@ -97,8 +100,10 @@ public class StationSettings {
                 return true;
             }
         } else if (osName.contains("nix") || osName.contains("nux") || osName.contains("aix")) {
-            // TODO: linux
-            logger.log(Level.SEVERE, "Unsupported OS " + osName);
+            Result<Boolean, String> result = LinuxScheduler.scheduleExists(temp);
+            if (result.hasSuccess() && result.getSuccess().get()) {
+                return true;
+            }
         } else {
             logger.log(Level.SEVERE, "Unsupported OS " + osName);
         }
@@ -107,6 +112,10 @@ public class StationSettings {
     }
 
     public void deleteDir() throws IOException {
+        if (!Files.exists(stationPath())) {
+            return;
+        }
+
         Files.walkFileTree(stationPath(), new SimpleFileVisitor<Path>() {
             @Override
             public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {
@@ -250,6 +259,14 @@ public class StationSettings {
         externalProgramCommand = newExternalProgramCommand;
     }
 
+    public Duration getExternalProgramTimeout() {
+        return externalProgramTimeout;
+    }
+
+    public void setExternalProgramTimeout(Duration newExternalProgramTimeout) {
+        externalProgramTimeout = newExternalProgramTimeout;
+    }
+
     public boolean getManageScheduleExternally() {
         return manageScheduleExternally;
     }

src/main/java/name/nathanmcrae/numbersstation/StationSettingsController.java

@@ -462,8 +462,14 @@ public class StationSettingsController {
         } else if (osName.contains("nix") || osName.contains("nux") || osName.contains("aix")) {
             LinuxScheduler.registerSchedule(settings);
         } else {
-            logger.log(Level.SEVERE, "Unsupported OS " + osName);
-            // TODO: Alert
+            String message = "Unsupported OS " + osName;
+            logger.log(Level.SEVERE, message);
+
+            Alert alert = new Alert(Alert.AlertType.ERROR);
+            alert.setTitle(message);
+            alert.setHeaderText(null);
+            alert.setContentText(message);
+            alert.showAndWait();
         }
 
         Node node = (Node) e.getSource();
@@ -527,16 +533,6 @@ public class StationSettingsController {
         }
     }
 
-    @FXML
-    private void handleTestRegisterScheduleButtonPress() {
-        WindowsScheduler.registerSchedule(settings);
-    }
-
-    @FXML
-    private void handleTestRunScheduleButtonPress() {
-        WindowsScheduler.runSchedule(settings);
-    }
-
     public StringProperty stationAddressProperty() {
         return stationAddress;
     }
@@ -559,7 +555,6 @@ public class StationSettingsController {
         messagePeriod.set(settings.getMessagePeriod());
         username.set(settings.getUsername());
         password.set(settings.getPassword());
-        System.out.println(settings.getPrefixes());
         prefixListView.getItems().addAll(settings.getPrefixes());
         scheduleStartDate.set(settings.getScheduleStart().withZoneSameInstant(ZoneId.systemDefault()).toLocalDate());
         DateTimeFormatter formatter = DateTimeFormatter.ofPattern("HH:mm:ss");

src/main/java/name/nathanmcrae/numbersstation/WindowsScheduler.java

@@ -228,7 +228,6 @@ public class WindowsScheduler {
         logger.info("Executable Path: " + executablePath);
 
         Element command = doc.createElement("Command");
-        // TODO: need to figure out the real invocation
         command.appendChild(doc.createTextNode(executablePath.toString()));
         exec.appendChild(command);
 
@@ -438,7 +437,7 @@ public class WindowsScheduler {
 
     /**
      * @return (stdout contents, stderr contents)
-     * TODO: don't assume that process has exited yet. If it does we don't want to hang.
+     * Note that caller should check that process has exited
      */
     public static Pair<String, String> captureProcessOutput(Process process) throws IOException {
         StringBuilder output = new StringBuilder();

src/main/resources/AboutView.fxml

@@ -12,15 +12,15 @@
             <Font size="18.0" />
          </font>
       </Label>
-      <Label layoutX="20.0" layoutY="110.0" text="Application executable: " AnchorPane.leftAnchor="20.0" AnchorPane.topAnchor="110.0" />
       <Label layoutX="20.0" layoutY="50.0" text="Homepage:" AnchorPane.leftAnchor="20.0" AnchorPane.topAnchor="50.0" />
-      <Label layoutX="20.0" layoutY="140.0" text="Config directory: " AnchorPane.leftAnchor="20.0" AnchorPane.topAnchor="140.0" />
+      <TextField fx:id="homepageURLTextField" accessibleText="Homepage URL field" editable="false" layoutX="89.0" layoutY="46.0" prefHeight="25.0" prefWidth="501.0" text="asefa" AnchorPane.leftAnchor="89.0" AnchorPane.rightAnchor="10.0" />
       <Label layoutX="20.0" layoutY="80.0" text="Version:" AnchorPane.leftAnchor="20.0" AnchorPane.topAnchor="80.0" />
-      <TextField fx:id="versionTextField" editable="false" layoutX="72.0" layoutY="76.0" text="asefa" />
-      <TextField fx:id="executableTextField" editable="false" layoutX="153.0" layoutY="106.0" prefHeight="25.0" prefWidth="403.0" text="sflejk" AnchorPane.leftAnchor="153.0" AnchorPane.rightAnchor="10.0" />
-      <TextField fx:id="directoryTextField" editable="false" layoutX="118.0" layoutY="136.0" prefHeight="25.0" prefWidth="437.0" text="afseijl" AnchorPane.leftAnchor="118.0" AnchorPane.rightAnchor="10.0" />
+      <TextField fx:id="versionTextField" accessibleText="Application version field" editable="false" layoutX="72.0" layoutY="76.0" text="asefa" AnchorPane.leftAnchor="89.0" AnchorPane.rightAnchor="10.0" />
+      <Label layoutX="20.0" layoutY="110.0" text="Application executable: " AnchorPane.leftAnchor="20.0" AnchorPane.topAnchor="110.0" />
+      <TextField fx:id="executableTextField" accessibleText="Application executable path field" editable="false" layoutX="153.0" layoutY="106.0" prefHeight="25.0" prefWidth="403.0" text="sflejk" AnchorPane.leftAnchor="153.0" AnchorPane.rightAnchor="10.0" />
+      <Label layoutX="20.0" layoutY="140.0" text="Config directory: " AnchorPane.leftAnchor="20.0" AnchorPane.topAnchor="140.0" />
+      <TextField fx:id="directoryTextField" accessibleText="Configuration directory path field" editable="false" layoutX="118.0" layoutY="136.0" prefHeight="25.0" prefWidth="437.0" text="afseijl" AnchorPane.leftAnchor="118.0" AnchorPane.rightAnchor="10.0" />
       <Label layoutX="20.0" layoutY="168.0" text="Log file: " AnchorPane.leftAnchor="20.0" AnchorPane.topAnchor="170.0" />
-      <TextField fx:id="logPathTextField" editable="false" layoutX="70.0" layoutY="166.0" prefHeight="25.0" prefWidth="485.0" text="afseijl" AnchorPane.leftAnchor="70.0" AnchorPane.rightAnchor="10.0" />
-      <TextField fx:id="homepageURLTextField" editable="false" layoutX="89.0" layoutY="46.0" prefHeight="25.0" prefWidth="501.0" text="asefa" AnchorPane.leftAnchor="89.0" AnchorPane.rightAnchor="10.0" />
+      <TextField fx:id="logPathTextField" accessibleText="Log file path field" editable="false" layoutX="70.0" layoutY="166.0" prefHeight="25.0" prefWidth="485.0" text="afseijl" AnchorPane.leftAnchor="70.0" AnchorPane.rightAnchor="10.0" />
    </children>
 </AnchorPane>

src/main/resources/AddPrefixView.fxml

@@ -11,13 +11,13 @@
    <children>
       <AnchorPane prefHeight="200.0" prefWidth="312.0">
          <children>
-            <Button fx:id="okButton" layoutX="89.0" layoutY="67.0" mnemonicParsing="false" prefHeight="25.0" prefWidth="65.0" text="OK" AnchorPane.bottomAnchor="15.0" AnchorPane.rightAnchor="89.0" />
-            <Button fx:id="cancelButton" layoutX="169.0" layoutY="67.0" mnemonicParsing="false" prefHeight="25.0" prefWidth="65.0" text="Cancel" AnchorPane.bottomAnchor="15.0" AnchorPane.rightAnchor="9.0" />
             <Label layoutX="14.0" layoutY="14.0" text="Enter a message prefix / identifier:">
                <font>
                   <Font name="Liberation Sans" size="13.0" />
                </font></Label>
-            <TextField fx:id="prefixField" layoutX="216.0" layoutY="10.0" prefHeight="25.0" prefWidth="86.0" AnchorPane.leftAnchor="216.0" AnchorPane.rightAnchor="10.0" />
+            <TextField fx:id="prefixField" accessibleText="Prefix field" layoutX="216.0" layoutY="10.0" prefHeight="25.0" prefWidth="86.0" AnchorPane.leftAnchor="216.0" AnchorPane.rightAnchor="10.0" />
+            <Button fx:id="okButton" layoutX="89.0" layoutY="67.0" mnemonicParsing="false" prefHeight="25.0" prefWidth="65.0" text="OK" AnchorPane.bottomAnchor="15.0" AnchorPane.rightAnchor="89.0" />
+            <Button fx:id="cancelButton" layoutX="169.0" layoutY="67.0" mnemonicParsing="false" prefHeight="25.0" prefWidth="65.0" text="Cancel" AnchorPane.bottomAnchor="15.0" AnchorPane.rightAnchor="9.0" />
          </children>
       </AnchorPane>
    </children>

src/main/resources/HelpView.fxml

@@ -3,4 +3,4 @@
 <?import javafx.scene.web.WebView?>
 
 
-<WebView fx:id="webView" prefHeight="391.0" prefWidth="440.0" xmlns="http://javafx.com/javafx/23.0.1" xmlns:fx="http://javafx.com/fxml/1" fx:controller="name.nathanmcrae.numbersstation.HelpController" />
+<WebView fx:id="webView" prefHeight="597.0" prefWidth="652.0" xmlns="http://javafx.com/javafx/23.0.1" xmlns:fx="http://javafx.com/fxml/1" fx:controller="name.nathanmcrae.numbersstation.HelpController" />

src/main/resources/MainView.fxml

@@ -65,7 +65,7 @@
                <children>
                   <Button layoutX="228.0" layoutY="1.0" mnemonicParsing="false" onAction="#handleSelectStationButtonPress" text="Select Station" />
                   <Label alignment="CENTER_RIGHT" layoutY="5.0" prefHeight="17.0" prefWidth="60.0" text="Station: " textAlignment="RIGHT" />
-                  <TextField fx:id="stationNameField" editable="false" layoutX="68.0" prefHeight="25.0" prefWidth="153.0" style="-fx-background-color: #EEEEEE; -fx-border-color: #AAAAAA;" text="My Station" />
+                  <TextField id="teststationnamefield" fx:id="stationNameField" accessibleText="Station name field" editable="false" layoutX="68.0" prefHeight="25.0" prefWidth="153.0" style="-fx-background-color: #EEEEEE; -fx-border-color: #AAAAAA;" text="My Station" />
                   <Button layoutX="602.0" layoutY="1.0" mnemonicParsing="false" onAction="#handleHelpButtonPress" text="Help" AnchorPane.rightAnchor="0.0" />
                   <Button layoutX="516.0" layoutY="1.0" mnemonicParsing="false" onAction="#handleAboutButtonPress" text="About" AnchorPane.rightAnchor="52.0" />
                </children>
@@ -90,7 +90,7 @@
               <items>
                 <AnchorPane minHeight="0.0" minWidth="0.0" prefHeight="160.0" prefWidth="100.0">
                      <children>
-                        <TextArea fx:id="messageTextArea" prefHeight="270.0" text="0239 0480 2938 0928&#10;3093 2298 3923 8933" wrapText="true" AnchorPane.bottomAnchor="0.0" AnchorPane.leftAnchor="0.0" AnchorPane.rightAnchor="0.0" AnchorPane.topAnchor="0.0">
+                        <TextArea fx:id="messageTextArea" accessibleText="Next message editor" prefHeight="270.0" text="0239 0480 2938 0928&#10;3093 2298 3923 8933" wrapText="true" AnchorPane.bottomAnchor="0.0" AnchorPane.leftAnchor="0.0" AnchorPane.rightAnchor="0.0" AnchorPane.topAnchor="0.0">
                            <font>
                               <Font name="Liberation Mono" size="30.0" />
                            </font>

src/main/resources/NotificationView.fxml

@@ -40,18 +40,18 @@
 
 <AnchorPane maxHeight="-Infinity" maxWidth="-Infinity" minHeight="-Infinity" minWidth="-Infinity" prefHeight="250.0" prefWidth="395.0" stylesheets="@css/application.css" xmlns="http://javafx.com/javafx/23.0.1" xmlns:fx="http://javafx.com/fxml/1" fx:controller="name.nathanmcrae.numbersstation.NotificationController">
    <children>
-      <Label fx:id="titleLabel" layoutX="86.0" layoutY="24.0" text="Notification title (this sure is a title)">
-         <font>
-            <Font name="Liberation Sans Bold" size="16.0" />
-         </font>
-      </Label>
-      <Button fx:id="dismissButton" layoutX="325.0" layoutY="139.0" mnemonicParsing="false" onAction="#handleDismissButtonPressed" text="Dismiss" AnchorPane.bottomAnchor="14.0" AnchorPane.rightAnchor="14.0" />
-      <Button fx:id="openButton" layoutX="270.0" layoutY="139.0" mnemonicParsing="false" onAction="#handleOpenButtonPressed" text="Open" visible="false" AnchorPane.bottomAnchor="14.0" AnchorPane.rightAnchor="79.5" />
       <Label fx:id="iconLabel" alignment="CENTER" contentDisplay="CENTER" layoutX="14.0" layoutY="5.0" prefHeight="60.0" prefWidth="65.0" text="⚠" textAlignment="CENTER" textFill="RED">
          <font>
             <Font size="42.0" />
          </font>
       </Label>
-      <TextArea fx:id="descriptionTextArea" editable="false" layoutX="14.0" layoutY="67.0" prefHeight="62.0" prefWidth="367.0" style="-fx-background-color: EEEEEE;" text="Quidem repellendus perferendis quos ipsum. Nihil nobis esse aperiam. Modi optio quia repudiandae quaerat fugit accusamus. Qui sint qui cupiditate rerum quisquam aspernatur illum. Deleniti voluptatum nam alias fugiat.Ab in iste dolores. Ad nostrum quia quisquam deleniti dolorum sequi. Est ut id et enim.Repellat nemo velit modi sunt provident neque neque. Est consequatur est beatae voluptates tempore commodi voluptas. Dicta asperiores deserunt veritatis quia officia. Reiciendis et qui placeat est minus." wrapText="true" AnchorPane.bottomAnchor="49.0" AnchorPane.leftAnchor="14.0" AnchorPane.rightAnchor="14.0" AnchorPane.topAnchor="67.0" />
+      <Label fx:id="titleLabel" layoutX="86.0" layoutY="24.0" text="Notification title (this sure is a title)">
+         <font>
+            <Font name="Liberation Sans Bold" size="16.0" />
+         </font>
+      </Label>
+      <TextArea fx:id="descriptionTextArea" accessibleText="Notification text field" editable="false" layoutX="14.0" layoutY="67.0" prefHeight="62.0" prefWidth="367.0" style="-fx-background-color: EEEEEE;" text="Quidem repellendus perferendis quos ipsum. Nihil nobis esse aperiam. Modi optio quia repudiandae quaerat fugit accusamus. Qui sint qui cupiditate rerum quisquam aspernatur illum. Deleniti voluptatum nam alias fugiat.Ab in iste dolores. Ad nostrum quia quisquam deleniti dolorum sequi. Est ut id et enim.Repellat nemo velit modi sunt provident neque neque. Est consequatur est beatae voluptates tempore commodi voluptas. Dicta asperiores deserunt veritatis quia officia. Reiciendis et qui placeat est minus." wrapText="true" AnchorPane.bottomAnchor="49.0" AnchorPane.leftAnchor="14.0" AnchorPane.rightAnchor="14.0" AnchorPane.topAnchor="67.0" />
+      <Button fx:id="openButton" layoutX="270.0" layoutY="139.0" mnemonicParsing="false" onAction="#handleOpenButtonPressed" text="Open" visible="false" AnchorPane.bottomAnchor="14.0" AnchorPane.rightAnchor="79.5" />
+      <Button fx:id="dismissButton" layoutX="325.0" layoutY="139.0" mnemonicParsing="false" onAction="#handleDismissButtonPressed" text="Dismiss" AnchorPane.bottomAnchor="14.0" AnchorPane.rightAnchor="14.0" />
    </children>
 </AnchorPane>

src/main/resources/StationSelectionView.fxml

@@ -13,9 +13,9 @@
             <Font size="20.0" />
          </font>
       </Label>
-      <Button layoutX="358.0" layoutY="17.0" mnemonicParsing="false" onAction="#handleAddButtonPress" prefHeight="25.0" prefWidth="26.0" text="+" AnchorPane.rightAnchor="55.0" AnchorPane.topAnchor="17.0" />
-      <Button layoutX="391.0" layoutY="17.0" mnemonicParsing="false" onAction="#handleRemoveButtonPress" prefHeight="25.0" prefWidth="25.0" text="-" AnchorPane.rightAnchor="21.0" AnchorPane.topAnchor="17.0" />
-      <ListView fx:id="stationListView" layoutX="14.0" layoutY="57.0" prefHeight="301.0" prefWidth="409.0" AnchorPane.bottomAnchor="42.0" AnchorPane.leftAnchor="14.0" AnchorPane.rightAnchor="14.0" AnchorPane.topAnchor="57.0" />
+      <Button accessibleText="Add new station button" layoutX="358.0" layoutY="17.0" mnemonicParsing="false" onAction="#handleAddButtonPress" prefHeight="25.0" prefWidth="26.0" text="+" AnchorPane.rightAnchor="55.0" AnchorPane.topAnchor="17.0" />
+      <Button accessibleText="Remove selected station button" layoutX="391.0" layoutY="17.0" mnemonicParsing="false" onAction="#handleRemoveButtonPress" prefHeight="25.0" prefWidth="25.0" text="-" AnchorPane.rightAnchor="21.0" AnchorPane.topAnchor="17.0" />
+      <ListView fx:id="stationListView" accessibleText="Station selection list" layoutX="14.0" layoutY="57.0" prefHeight="301.0" prefWidth="409.0" AnchorPane.bottomAnchor="42.0" AnchorPane.leftAnchor="14.0" AnchorPane.rightAnchor="14.0" AnchorPane.topAnchor="57.0" />
       <Button layoutX="293.0" layoutY="366.0" mnemonicParsing="false" onAction="#handleSelectButtonPress" text="Select" AnchorPane.bottomAnchor="10.0" AnchorPane.rightAnchor="85.0" />
       <Button layoutX="370.0" layoutY="365.0" mnemonicParsing="false" onAction="#handleCancelButtonPress" text="Cancel" AnchorPane.bottomAnchor="10.0" AnchorPane.rightAnchor="15.5" />
    </children>

src/main/resources/StationSettingsView.fxml

@@ -27,7 +27,7 @@
                   <FlowPane alignment="CENTER_LEFT" prefHeight="41.0" prefWidth="640.0">
                      <children>
                         <Label alignment="CENTER_RIGHT" prefHeight="17.0" prefWidth="58.0" text="Station: " />
-                        <TextField fx:id="stationNameField" style="-fx-border-color: #AAAAAA;" text="My Station">
+                        <TextField fx:id="stationNameField" accessibleText="Station name field" style="-fx-border-color: #AAAAAA;" text="My Station">
                            <FlowPane.margin>
                               <Insets right="10.0" />
                            </FlowPane.margin>
@@ -47,7 +47,7 @@
                                     <Insets right="10.0" />
                                  </FlowPane.margin>
                               </Label>
-                              <Spinner fx:id="digitsPerGroupSpinner" editable="true" prefHeight="25.0" prefWidth="96.0" />
+                              <Spinner fx:id="digitsPerGroupSpinner" accessibleText="Digit group size field" editable="true" prefHeight="25.0" prefWidth="96.0" />
                            </children>
                         </FlowPane>
                         <FlowPane alignment="CENTER_LEFT" layoutX="244.0" layoutY="7.0" prefHeight="63.0" prefWidth="216.0">
@@ -57,7 +57,7 @@
                                     <Insets right="10.0" />
                                  </FlowPane.margin>
                               </Label>
-                              <Spinner fx:id="messageLengthSpinner" editable="true" prefHeight="25.0" prefWidth="96.0" />
+                              <Spinner fx:id="messageLengthSpinner" accessibleText="Message length field" editable="true" prefHeight="25.0" prefWidth="96.0" />
                            </children>
                         </FlowPane>
                      </children>
@@ -70,8 +70,8 @@
                               <Font size="14.0" />
                            </font>
                         </Label>
-                        <Label layoutX="170.0" layoutY="39.0" text="Numbers station address:" />
-                        <TextField fx:id="stationAddressField" layoutX="188.0" layoutY="63.0" prefHeight="25.0" prefWidth="390.0" AnchorPane.leftAnchor="188.0" AnchorPane.rightAnchor="0.0" />
+                        <Label fx:id="connectionTestStatusLabel" accessibleText="Test connection result" alignment="CENTER_RIGHT" layoutX="136.0" layoutY="11.0" prefHeight="17.0" prefWidth="317.0" textAlignment="RIGHT" AnchorPane.leftAnchor="136.0" AnchorPane.rightAnchor="125.0" />
+                        <Button fx:id="testConnectionButton" layoutX="464.0" layoutY="7.0" mnemonicParsing="false" onAction="#handleTestConnectionButtonPress" text="Test connection" AnchorPane.rightAnchor="0.0" />
                         <RadioButton fx:id="sftpRadioButton" layoutX="16.0" layoutY="42.0" mnemonicParsing="false" text="SFTP">
                             <toggleGroup>
                               <ToggleGroup fx:id="messageMethodGroup" />
@@ -87,13 +87,13 @@
                                <fx:reference source="messageMethodGroup" />
                             </toggleGroup>
                         </RadioButton>
+                        <Label layoutX="170.0" layoutY="39.0" text="Numbers station address:" />
+                        <TextField fx:id="stationAddressField" accessibleText="Number station address field" layoutX="189.0" layoutY="63.0" prefHeight="25.0" prefWidth="390.0" AnchorPane.leftAnchor="189.0" AnchorPane.rightAnchor="-1.0" />
                         <Label layoutX="182.0" layoutY="99.0" text="Username:" AnchorPane.rightAnchor="340.0" />
+                        <TextField fx:id="usernameField" accessibleText="Numbers station username field" layoutX="245.0" layoutY="95.0" prefHeight="25.0" prefWidth="124.0" AnchorPane.rightAnchor="209.0" />
                         <Label layoutX="380.0" layoutY="99.0" text="Password:" AnchorPane.rightAnchor="142.0" />
-                        <TextField fx:id="usernameField" layoutX="245.0" layoutY="95.0" prefHeight="25.0" prefWidth="124.0" AnchorPane.rightAnchor="209.0" />
-                        <PasswordField fx:id="passwordField" layoutX="444.0" layoutY="95.0" prefHeight="25.0" prefWidth="135.0" AnchorPane.rightAnchor="0.0" />
-                        <TextField fx:id="externalProgramCommandField" layoutX="-2.0" layoutY="131.0" prefHeight="25.0" prefWidth="580.0" AnchorPane.leftAnchor="-2.0" AnchorPane.rightAnchor="0.0" />
-                        <Button fx:id="testConnectionButton" layoutX="464.0" layoutY="7.0" mnemonicParsing="false" onAction="#handleTestConnectionButtonPress" text="Test connection" AnchorPane.rightAnchor="0.0" />
-                        <Label fx:id="connectionTestStatusLabel" alignment="CENTER_RIGHT" layoutX="136.0" layoutY="11.0" prefHeight="17.0" prefWidth="317.0" textAlignment="RIGHT" AnchorPane.leftAnchor="136.0" AnchorPane.rightAnchor="125.0" />
+                        <PasswordField fx:id="passwordField" accessibleText="Numbers station password field" layoutX="444.0" layoutY="95.0" prefHeight="25.0" prefWidth="135.0" AnchorPane.rightAnchor="0.0" />
+                        <TextField fx:id="externalProgramCommandField" accessibleText="Numbers station external program field" layoutX="-2.0" layoutY="131.0" prefHeight="25.0" prefWidth="580.0" AnchorPane.leftAnchor="-2.0" AnchorPane.rightAnchor="0.0" />
                      
                    </children>
                      <VBox.margin>
@@ -108,14 +108,14 @@
                               <Font size="14.0" />
                            </font>
                         </Label>
-                        <Button layoutX="520.5" layoutY="4.0" mnemonicParsing="false" onAction="#handleAddPrefixButtonPress" text="+" AnchorPane.rightAnchor="33.0" />
-                        <Button layoutX="552.0" layoutY="4.0" mnemonicParsing="false" onAction="#handleRemovePrefixButtonPress" prefHeight="25.0" prefWidth="25.0" text="-" AnchorPane.rightAnchor="1.0" />
+                        <Button accessibleText="Add prefix field" layoutX="520.5" layoutY="4.0" mnemonicParsing="false" onAction="#handleAddPrefixButtonPress" text="+" AnchorPane.rightAnchor="33.0" />
+                        <Button accessibleText="Remove selected prefix button" layoutX="552.0" layoutY="4.0" mnemonicParsing="false" onAction="#handleRemovePrefixButtonPress" prefHeight="25.0" prefWidth="25.0" text="-" AnchorPane.rightAnchor="1.0" />
                      </children>
                      <VBox.margin>
                         <Insets bottom="10.0" />
                      </VBox.margin>
                   </AnchorPane>
-                  <ListView fx:id="prefixListView" prefHeight="103.0" prefWidth="578.0" />
+                  <ListView fx:id="prefixListView" accessibleText="Prefix list" prefHeight="103.0" prefWidth="578.0" />
                   <Separator prefWidth="200.0">
                      <VBox.margin>
                         <Insets bottom="10.0" top="10.0" />
@@ -144,12 +144,9 @@
                                <fx:reference source="messagePeriodGroup" />
                             </toggleGroup>
                         </RadioButton>
-                        <DatePicker fx:id="scheduleStartDatePicker" disable="${manageScheduleExternallyCheckBox.selected}" layoutX="115.0" layoutY="34.0" />
                         <Label layoutX="115.0" layoutY="8.0" text="Starting from:" />
-                        <TextField fx:id="scheduleStartTimeField" disable="${manageScheduleExternallyCheckBox.selected}" layoutX="115.0" layoutY="64.0" text="23:24:49" />
-                        <Label layoutX="48.0" layoutY="102.0" text="TODO: Jitter" />
-                        <Button layoutX="377.0" layoutY="73.0" mnemonicParsing="false" onAction="#handleTestRegisterScheduleButtonPress" text="Test register scheduled task" />
-                        <Button layoutX="377.0" layoutY="107.0" mnemonicParsing="false" onAction="#handleTestRunScheduleButtonPress" text="Test running scheduled task" />
+                        <DatePicker fx:id="scheduleStartDatePicker" accessibleText="Message schedule start date" disable="${manageScheduleExternallyCheckBox.selected}" layoutX="115.0" layoutY="34.0" />
+                        <TextField fx:id="scheduleStartTimeField" accessibleText="Message schedule start time" disable="${manageScheduleExternallyCheckBox.selected}" layoutX="115.0" layoutY="64.0" text="23:24:49" />
                      
                    </children>
                   </AnchorPane>

src/main/resources/help-index.html

@@ -108,7 +108,6 @@
       <li>
         Set other options (See application reference for details)
       </li>
-      <li>TODO: test post</li>
     </ol>
 
     <h3>