- refactored PreferencesManager tests in a single class

- implemented Hyperlink's actions to open their

Author: stefanofornari

pom.xml

@@ -245,11 +245,11 @@
                         -Dmonocle.platform.headless.height=768
                         -Dprism.order=sw
                         -Dtestfx.robot=glass
-                        -Dtestfx.headless=false       <!-- false to show UI -->
-                        <!-- comment out to show UI
+                        -Dtestfx.headless=true       <!-- false to show UI -->
+                        <!-- comment out to show UI -->
                         -Dglass.platform=Monocle
                         -Dmonocle.platform=Headless
-                        -->
+                        <!-- -->
                     </argLine>
                     <systemPropertyVariables>
                         <java.util.logging.config.file>src/test/resources/logging.properties</java.util.logging.config.file>

src/main/java/io/github/jeddict/ai/settings/JeddictPreferences.java

@@ -45,6 +45,7 @@
 import java.awt.event.KeyEvent;
 import java.io.IOException;
 import java.io.StringReader;
+import java.net.URI;
 import java.nio.file.Path;
 import java.nio.file.Paths;
 import java.util.HashMap;
@@ -77,6 +78,8 @@
 import org.apache.commons.lang3.StringUtils;
 import org.controlsfx.control.MasterDetailPane;
 import org.controlsfx.control.HyperlinkLabel;
+import org.openide.awt.HtmlBrowser.URLDisplayer;
+import org.openide.util.Exceptions;
 import static ste.lloop.Loop.on;
 import ste.netbeans.javafx.JFXPanel;
 
@@ -286,13 +289,16 @@ private void initComponents() {
         );
         modelsLink = new Hyperlink(); modelsLink.setId("modelsUrl");
         modelsLink.setOnAction(event -> {
+            browse(modelsLink.getText());
             modelsLink.setVisited(false); // Forces the link back to its unclicked default color
         });
         modelsLink.textProperty().bind(settings.string("modelsUrl"));
 
         apiKeyLink = new Hyperlink(); apiKeyLink.setId("apiKeyUrl");
         apiKeyLink.setOnAction(event -> {
+            browse(apiKeyLink.getText());
             apiKeyLink.setVisited(false); // Forces the link back to its unclicked default color
+
         });
         apiKeyLink.textProperty().bind(settings.string("apiKeyUrl"));
 
@@ -985,4 +991,12 @@ public ClassContextLegend() {
             setId("class-context-legend");
         }
     }
+
+    private void browse(final String url) {
+        try {
+            URLDisplayer.getDefault().showURL(URI.create(url).toURL());
+        } catch (Exception x) {
+            Exceptions.printStackTrace(x);
+        }
+    }
 }

src/test/java/io/github/jeddict/ai/settings/JeddictPreferencesTest.java

@@ -121,9 +121,6 @@ public void save_persists_ui_settings_to_preferences_manager() {
         then(pm.getMaxRetries()).isEqualTo(3);
         then(pm.getOrganizationId()).isEqualTo("org-1");
 
-        then(pm.isAllowCodeExecution()).isTrue();
-        then(pm.isIncludeCodeExecutionOutput()).isTrue();
-
         then(pm.getFileExtensionListToInclude()).containsExactly("java","kt");
         then(pm.getExcludeDirs()).containsExactly("a","b","c");