#1802 GUI State management implementation

cli/pom.xml

@@ -138,8 +138,6 @@
               </archive>
             </configuration>
           </execution>
-
-          <!-- jar for test classes -->
           <execution>
             <id>test-jar</id>
             <goals>

cli/src/test/java/com/devonfw/tools/ide/context/IdeContextTest.java

@@ -5,13 +5,6 @@
 import java.util.Properties;
 
 import org.junit.jupiter.api.Test;
-import com.devonfw.tools.ide.security.ToolVersionChoice;
-import com.devonfw.tools.ide.tool.ToolEditionAndVersion;
-import com.devonfw.tools.ide.security.ToolVulnerabilities;
-import com.devonfw.tools.ide.version.VersionIdentifier;
-import com.devonfw.tools.ide.version.VersionRange;
-import com.devonfw.tools.ide.url.model.file.json.Cve;
-import com.devonfw.tools.ide.tool.ToolEdition;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -24,13 +17,20 @@
 import com.devonfw.tools.ide.log.IdeLogLevel;
 import com.devonfw.tools.ide.os.SystemInfo;
 import com.devonfw.tools.ide.os.SystemInfoMock;
+import com.devonfw.tools.ide.security.ToolVersionChoice;
+import com.devonfw.tools.ide.security.ToolVulnerabilities;
+import com.devonfw.tools.ide.tool.ToolEdition;
+import com.devonfw.tools.ide.tool.ToolEditionAndVersion;
+import com.devonfw.tools.ide.url.model.file.json.Cve;
 import com.devonfw.tools.ide.variable.IdeVariables;
 import com.devonfw.tools.ide.version.IdeVersion;
+import com.devonfw.tools.ide.version.VersionIdentifier;
+import com.devonfw.tools.ide.version.VersionRange;
 
 /**
  * Test of {@link IdeContext}.
  */
-class IdeContextTest extends AbstractIdeContextTest {
+public class IdeContextTest extends AbstractIdeContextTest {
 
   private static final Logger LOG = LoggerFactory.getLogger(IdeContextTest.class);
 
@@ -284,7 +284,7 @@ void testFindBashOnSystemPathOnWindowsWithInvalidBashPathSet() {
   @Test
   void testQuestionWithMultipleOptions() {
     IdeTestContext context = newContext(PROJECT_BASIC, null, false);
-    String[] options = {"option1", "option2"};
+    String[] options = { "option1", "option2" };
     context.setAnswers("option1");
     String result = context.question(options, "Which option?");
     assertThat(result).isEqualTo("option1");
@@ -293,7 +293,7 @@ void testQuestionWithMultipleOptions() {
   @Test
   void testQuestionWithSingleOptionAndEmptyAnswer() {
     IdeTestContext context = newContext(PROJECT_BASIC, null, false);
-    String[] options = {"onlyOption"};
+    String[] options = { "onlyOption" };
     context.setAnswers(""); // Empty answer (Enter)
     String result = context.question(options, "Which option?");
     assertThat(result).isEqualTo("onlyOption");
@@ -306,7 +306,7 @@ void testQuestionWithSingleToolVersionChoiceAndEmptyAnswer() {
     VersionIdentifier version = VersionIdentifier.of("17");
     Cve cve = new Cve("CVE-2023-XXXX", 9.8, List.of(VersionRange.of("[17,18)")));
     ToolVersionChoice choice = new ToolVersionChoice(new ToolEditionAndVersion(edition, version), "current", ToolVulnerabilities.of(List.of(cve)));
-    ToolVersionChoice[] options = {choice};
+    ToolVersionChoice[] options = { choice };
 
     context.setAnswers(""); // Empty answer (Enter)
     ToolVersionChoice result = context.question(options, "Which version?");

gui/pom.xml

@@ -30,7 +30,6 @@
     <dependency>
       <groupId>${project.groupId}</groupId>
       <artifactId>ide-cli</artifactId>
-      <classifier>tests</classifier>
       <version>${project.version}</version>
       <type>test-jar</type>
       <scope>test</scope>
@@ -68,6 +67,14 @@
       <artifactId>openjfx-monocle</artifactId>
       <scope>test</scope>
     </dependency>
+
+    <!-- required for using ide-cli testing classes -->
+    <dependency>
+      <groupId>org.wiremock</groupId>
+      <artifactId>wiremock</artifactId>
+      <version>3.13.2</version>
+      <scope>test</scope>
+    </dependency>
   </dependencies>
 
   <build>

gui/src/main/java/com/devonfw/ide/gui/App.java

@@ -2,6 +2,7 @@
 
 import java.io.IOException;
 import javafx.application.Application;
+import javafx.application.Platform;
 import javafx.fxml.FXMLLoader;
 import javafx.geometry.Rectangle2D;
 import javafx.scene.Parent;
@@ -10,6 +11,10 @@
 import javafx.stage.Screen;
 import javafx.stage.Stage;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+import com.devonfw.ide.gui.modal.IdeDialog;
 import com.devonfw.tools.ide.variable.IdeVariables;
 import com.devonfw.tools.ide.version.IdeVersion;
 
@@ -20,9 +25,17 @@ public class App extends Application {
 
   Parent root;
 
+  private static final Logger LOG = LoggerFactory.getLogger(App.class);
+
   @Override
   public void start(Stage primaryStage) throws IOException {
 
+    Thread.setDefaultUncaughtExceptionHandler((thread, throwable) -> {
+          LOG.error("Uncaught exception in thread {}: {}", thread.getName(), throwable.getMessage(), throwable);
+          Platform.runLater(() -> new IdeDialog(IdeDialog.AlertType.ERROR, throwable.getMessage()).showAndWait());
+        }
+    );
+
     FXMLLoader fxmlLoader = new FXMLLoader(App.class.getResource("main-view.fxml"));
     fxmlLoader.setController(
         new MainController(System.getenv(IdeVariables.IDE_ROOT.getName()))

gui/src/main/java/com/devonfw/ide/gui/AppLauncher.java

@@ -2,10 +2,11 @@
 
 /**
  * Launcher class for the App. Workaround for "Error: JavaFX runtime components are missing, and are required to run this application." Inspired by
- * <ahref=https://stackoverflow.com/questions/56894627/how-to-fix-error-javafx-runtime-components-are-missing-and-are-required-to-ru>StackOverflow</a>
+ * <a href=https://stackoverflow.com/questions/56894627/how-to-fix-error-javafx-runtime-components-are-missing-and-are-required-to-ru>StackOverflow</a>
  */
 public class AppLauncher {
 
+  @SuppressWarnings("MissingJavadoc")
   public static void main(final String[] args) {
 
     App.main(args);

gui/src/main/java/com/devonfw/ide/gui/MainController.java

@@ -1,20 +1,18 @@
 package com.devonfw.ide.gui;
 
-import java.io.IOException;
-import java.nio.file.Files;
+import java.io.FileNotFoundException;
 import java.nio.file.Path;
-import java.util.stream.Stream;
+import java.util.List;
 import javafx.fxml.FXML;
 import javafx.scene.control.Button;
 import javafx.scene.control.ComboBox;
 
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-import com.devonfw.tools.ide.context.IdeContext;
-import com.devonfw.tools.ide.context.IdeStartContextImpl;
-import com.devonfw.tools.ide.log.IdeLogLevel;
-import com.devonfw.tools.ide.log.IdeLogListenerBuffer;
+import com.devonfw.ide.gui.context.IdeGuiStateManager;
+import com.devonfw.ide.gui.context.ProjectManager;
+import com.devonfw.ide.gui.modal.IdeDialog;
 
 /**
  * Controller of the main screen of the dashboard GUI.
@@ -23,6 +21,9 @@ public class MainController {
 
   private static Logger LOG = LoggerFactory.getLogger(MainController.class);
 
+  private ProjectManager projectManager;
+
+
   @FXML
   private ComboBox<String> selectedProject;
 
@@ -49,8 +50,11 @@ public class MainController {
    * Constructor
    */
   public MainController(String directoryPath) {
+
     LOG.debug("IDE_ROOT path={}", directoryPath);
     this.directoryPath = directoryPath;
+
+    this.projectManager = IdeGuiStateManager.getInstance().getProjectManager();
   }
 
   @FXML
@@ -83,66 +87,56 @@ private void openVsCode() {
     openIDE("vscode");
   }
 
-
   private void setProjectsComboBox() {
 
     assert (directoryPath != null) : "directoryPath is null! Please check the setup of your environment variables (IDE_ROOT)";
 
+    List<String> projects = projectManager.getProjectNames();
+
     selectedProject.getItems().clear();
-    Path directory = Path.of(directoryPath);
-
-    if (Files.exists(directory) && Files.isDirectory(directory)) {
-      try (Stream<Path> subPaths = Files.list(directory)) {
-        subPaths
-            .filter(Files::isDirectory)
-            .map(Path::getFileName)
-            .map(Path::toString)
-            .filter(name -> !name.startsWith("_"))
-            .forEach(name -> selectedProject.getItems().add(name));
-      } catch (IOException e) {
-        throw new IllegalStateException("Failed to list projects!", e);
-      }
-    }
+    selectedProject.getItems().addAll(projects);
 
     selectedProject.setOnAction(actionEvent -> {
 
-      projectValue = Path.of(selectedProject.getValue()).resolve(IdeContext.FOLDER_WORKSPACES);
+      setWorkspaceComboBox();
+
       selectedWorkspace.setDisable(false);
+    });
+  }
+
+  private void setWorkspaceComboBox() {
+
+    List<String> workspaces = projectManager.getWorkspaceNames(selectedProject.getValue());
+
+    selectedWorkspace.getItems().clear();
+    selectedWorkspace.getItems().addAll(workspaces);
+
+    selectedWorkspace.setOnAction(actionEvent -> {
+      updateContext(selectedProject.getValue(), selectedWorkspace.getValue());
+
       androidStudioOpen.setDisable(false);
       eclipseOpen.setDisable(false);
       intellijOpen.setDisable(false);
       vsCodeOpen.setDisable(false);
-      selectedWorkspace.setValue("main");
-      this.workspaceValue = Path.of("main");
     });
   }
 
-  @FXML
-  private void setWorkspaceValue() {
+  private void openIDE(String inIde) {
 
-    selectedWorkspace.getItems().clear();
-    Path directory = Path.of(directoryPath).resolve(projectValue);
-    if (Files.exists(directory) && Files.isDirectory(directory)) {
-      try (Stream<Path> subPaths = Files.list(directory)) {
-        subPaths
-            .filter(Files::isDirectory)
-            .map(Path::getFileName)
-            .map(Path::toString)
-            .forEach(name -> selectedWorkspace.getItems().add(name));
-
-      } catch (IOException e) {
-        throw new RuntimeException("Error occurred while fetching workspace names.", e);
-      }
-    }
-    this.workspaceValue = Path.of(selectedWorkspace.getValue());
+    IdeGuiStateManager
+        .getInstance()
+        .getCurrentContext()
+        .getCommandletManager()
+        .getCommandlet(inIde)
+        .run();
   }
 
-  private void openIDE(String inIde) {
-
-    final IdeLogListenerBuffer buffer = new IdeLogListenerBuffer();
-    IdeLogLevel logLevel = IdeLogLevel.INFO;
-    IdeStartContextImpl startContext = new IdeStartContextImpl(logLevel, buffer);
-    IdeGuiContext context = new IdeGuiContext(startContext, Path.of(this.directoryPath).resolve(this.projectValue).resolve(this.workspaceValue));
-    context.getCommandletManager().getCommandlet(inIde).run();
+  private void updateContext(String selectedProjectName, String selectedWorkspaceName) {
+    try {
+      IdeGuiStateManager.getInstance().switchContext(selectedProjectName, selectedWorkspaceName);
+    } catch (FileNotFoundException e) {
+      IdeDialog errorDialog = new IdeDialog(IdeDialog.AlertType.ERROR, e.getMessage());
+      errorDialog.showAndWait();
+    }
   }
 }

gui/src/main/java/com/devonfw/ide/gui/context/GuiContextChangeListener.java

@@ -0,0 +1,15 @@
+package com.devonfw.ide.gui.context;
+
+/**
+ * Interface that notifies listeners of context changes.
+ */
+public interface GuiContextChangeListener {
+
+  /**
+   * This method is called when the context changes. It can be used to update the GUI based on the new context.
+   *
+   * @param newContext the new {@link IdeGuiContext}.
+   */
+  void onContextChange(IdeGuiContext newContext);
+
+}

gui/src/main/java/com/devonfw/ide/gui/IdeGuiContext.java → gui/src/main/java/com/devonfw/ide/gui/context/IdeGuiContext.java

@@ -1,4 +1,4 @@
-package com.devonfw.ide.gui;
+package com.devonfw.ide.gui.context;
 
 import java.nio.file.Path;
 
@@ -12,25 +12,26 @@
  */
 public class IdeGuiContext extends AbstractIdeContext {
 
-
   /**
    * The constructor.
    *
    * @param startContext the {@link IdeStartContextImpl}.
    * @param workingDirectory the optional {@link Path} to current working directory.
    */
   public IdeGuiContext(IdeStartContextImpl startContext, Path workingDirectory) {
+
     super(startContext, workingDirectory);
   }
 
   @Override
   protected String readLine() {
+
     return "";
   }
 
   @Override
   public IdeProgressBar newProgressBar(String title, long size, String unitName, long unitSize) {
+
     return new IdeProgressBarNone(title, 0, unitName, unitSize);
   }
-
 }