feat: add/remove/edit helm repo (#673)

Signed-off-by: Andre Dietisheim <[email protected]>

Author: adietish

build.gradle

@@ -140,7 +140,7 @@ dependencies {
             'io.fabric8:openshift-client:6.12.0',
             'org.apache.commons:commons-compress:1.26.1',
             'org.apache.commons:commons-exec:1.4.0',
-            'com.redhat.devtools.intellij:intellij-common:1.9.5',
+            'com.redhat.devtools.intellij:intellij-common:1.9.6-SNAPSHOT',
             'io.jsonwebtoken:jjwt-impl:0.12.5',
             'io.jsonwebtoken:jjwt-jackson:0.12.5',
             'org.keycloak:keycloak-installed-adapter:24.0.2',
@@ -150,7 +150,7 @@ dependencies {
             'org.junit.platform:junit-platform-launcher:1.10.2',
             'org.mockito:mockito-core:5.11.0',
             'org.easytesting:fest-assert:1.4',
-            'com.redhat.devtools.intellij:intellij-common:1.9.5:test',
+            'com.redhat.devtools.intellij:intellij-common:1.9.6-SNAPSHOT:test',
             'org.awaitility:awaitility:4.2.1',
             'org.mock-server:mockserver-client-java:5.15.0',
             'org.mock-server:mockserver-netty:5.15.0',

src/it/java/org/jboss/tools/intellij/openshift/test/ui/annotations/UITest.java

@@ -10,12 +10,12 @@
  ******************************************************************************/
 package org.jboss.tools.intellij.openshift.test.ui.annotations;
 
-import java.lang.annotation.RetentionPolicy;
-import java.lang.annotation.Target;
+import org.junit.jupiter.api.Tag;
+
 import java.lang.annotation.ElementType;
 import java.lang.annotation.Retention;
-
-import org.junit.jupiter.api.Tag;
+import java.lang.annotation.RetentionPolicy;
+import java.lang.annotation.Target;
 
 /**
  * @author Ondrej Dockal

src/it/java/org/jboss/tools/intellij/openshift/test/ui/common/ImportProjectFromVersionControlFixture.java

@@ -18,7 +18,9 @@
 import com.intellij.remoterobot.fixtures.FixtureName;
 import com.intellij.remoterobot.fixtures.JTextFieldFixture;
 import org.jetbrains.annotations.NotNull;
+
 import java.time.Duration;
+
 import static com.intellij.remoterobot.search.locators.Locators.byXpath;
 
 /**

src/it/java/org/jboss/tools/intellij/openshift/test/ui/common/ProjectTreeFixture.java

@@ -17,7 +17,9 @@
 import com.intellij.remoterobot.fixtures.FixtureName;
 import com.intellij.remoterobot.fixtures.JTreeFixture;
 import org.jetbrains.annotations.NotNull;
+
 import java.time.Duration;
+
 import static com.intellij.remoterobot.search.locators.Locators.byXpath;
 import static com.intellij.remoterobot.utils.RepeatUtilsKt.waitFor;
 

src/it/java/org/jboss/tools/intellij/openshift/test/ui/common/WelcomeDialogFixture.java

@@ -14,10 +14,12 @@
 import com.intellij.remoterobot.data.RemoteComponent;
 import com.intellij.remoterobot.fixtures.ComponentFixture;
 import com.intellij.remoterobot.fixtures.ContainerFixture;
-import org.jetbrains.annotations.NotNull;
 import com.intellij.remoterobot.fixtures.DefaultXpath;
 import com.intellij.remoterobot.fixtures.FixtureName;
+import org.jetbrains.annotations.NotNull;
+
 import java.time.Duration;
+
 import static com.intellij.remoterobot.search.locators.Locators.byXpath;
 
 /**

src/it/java/org/jboss/tools/intellij/openshift/test/ui/views/GettingStartedView.java

@@ -12,7 +12,11 @@
 
 import com.intellij.remoterobot.RemoteRobot;
 import com.intellij.remoterobot.data.RemoteComponent;
-import com.intellij.remoterobot.fixtures.*;
+import com.intellij.remoterobot.fixtures.ComponentFixture;
+import com.intellij.remoterobot.fixtures.ContainerFixture;
+import com.intellij.remoterobot.fixtures.DefaultXpath;
+import com.intellij.remoterobot.fixtures.FixtureName;
+import com.intellij.remoterobot.fixtures.JTreeFixture;
 import com.intellij.remoterobot.search.locators.Locator;
 import com.intellij.remoterobot.utils.Keyboard;
 import com.intellij.remoterobot.utils.WaitForConditionTimeoutException;

src/it/java/org/jboss/tools/intellij/openshift/utils/helm/Charts.java

@@ -31,7 +31,7 @@ public static Chart get(String name, Helm helm) throws Exception {
   }
 
   public static void addRepository(Pair<String, String> pair, Helm helm) throws IOException {
-    helm.addRepo(pair.first, pair.second);
+    helm.addRepo(pair.first, pair.second, null);
   }
 
 }

src/it/java/org/jboss/tools/intellij/openshift/utils/helm/HelmCliRepoTest.java

@@ -20,7 +20,7 @@ public void testListRepos_should_list_repo_that_was_added() throws IOException {
         // given openshift repo was added to helm repos
         String name = "openshift";
         String url = "https://charts.openshift.io/";
-        helm.addRepo(name, url);
+        helm.addRepo(name, url, null);
         // when
         List<HelmRepository> repositories = helm.listRepos();
         // then

src/main/java/org/jboss/tools/intellij/openshift/actions/helm/AddHelmRepoAction.java

@@ -0,0 +1,78 @@
+/*******************************************************************************
+ * Copyright (c) 2023 Red Hat, Inc.
+ * Distributed under license by Red Hat, Inc. All rights reserved.
+ * This program is made available under the terms of the
+ * Eclipse Public License v2.0 which accompanies this distribution,
+ * and is available at http://www.eclipse.org/legal/epl-v20.html
+ *
+ * Contributors:
+ * Red Hat, Inc. - initial API and implementation
+ ******************************************************************************/
+package org.jboss.tools.intellij.openshift.actions.helm;
+
+import com.intellij.openapi.actionSystem.AnActionEvent;
+import com.intellij.openapi.project.Project;
+import com.redhat.devtools.intellij.common.utils.ApplicationUtils;
+import com.redhat.devtools.intellij.common.utils.SwingUtils;
+import java.awt.Point;
+import java.io.IOException;
+import java.util.Collection;
+import java.util.Collections;
+import java.util.concurrent.CompletableFuture;
+import org.jboss.tools.intellij.openshift.actions.HelmAction;
+import org.jboss.tools.intellij.openshift.telemetry.TelemetryService;
+import org.jboss.tools.intellij.openshift.tree.application.HelmRepositoriesNode;
+import org.jboss.tools.intellij.openshift.ui.helm.AddHelmRepoDialog;
+import org.jboss.tools.intellij.openshift.utils.helm.Helm;
+import org.jboss.tools.intellij.openshift.utils.helm.HelmRepository;
+import org.jetbrains.annotations.NotNull;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+
+public class AddHelmRepoAction extends HelmAction {
+
+  private static final Logger LOGGER = LoggerFactory.getLogger(AddHelmRepoAction.class);
+
+  @Override
+  public void actionPerformedOnSelectedObject(AnActionEvent anActionEvent, Object selected, @NotNull Helm helm) {
+    Project project = getEventProject(anActionEvent);
+    if (!(selected instanceof HelmRepositoriesNode repositoriesNode)) {
+      return;
+    }
+    openAddHelmRepoDialog(repositoriesNode, helm, project, SwingUtils.getMouseLocation(anActionEvent));
+  }
+
+  private void openAddHelmRepoDialog(HelmRepositoriesNode repositoriesNode, Helm helm, Project project, Point location) {
+    CompletableFuture.supplyAsync(
+      () -> listRepositories(helm),
+      ApplicationUtils.PLATFORM_EXECUTOR
+    ).thenAcceptAsync(
+      repositories -> {
+        AddHelmRepoDialog dialog = new AddHelmRepoDialog(repositories, repositoriesNode, helm, project, location);
+        sendTelemetryResults(TelemetryService.TelemetryResult.SUCCESS);
+        dialog.show();
+      },
+      ApplicationUtils.UI_EXECUTOR
+    );
+  }
+
+  private Collection<HelmRepository> listRepositories(Helm helm) {
+    try {
+      return helm.listRepos();
+    } catch (IOException e) {
+      LOGGER.warn("Could not list helm repositories", e);
+      return Collections.emptyList();
+    }
+  }
+
+  @Override
+  public String getTelemetryActionName() {
+    return "helm-add repo";
+  }
+
+  @Override
+  public boolean isVisible(Object selected) {
+    return selected instanceof HelmRepositoriesNode;
+  }
+}

src/main/java/org/jboss/tools/intellij/openshift/actions/helm/OpenHelmChartsAction.java

@@ -13,10 +13,10 @@
 import com.intellij.openapi.actionSystem.AnActionEvent;
 import com.intellij.openapi.project.Project;
 import org.jboss.tools.intellij.openshift.actions.HelmAction;
+import org.jboss.tools.intellij.openshift.actions.NodeUtils;
 import org.jboss.tools.intellij.openshift.telemetry.TelemetryService;
 import org.jboss.tools.intellij.openshift.tree.application.ApplicationsRootNode;
 import org.jboss.tools.intellij.openshift.tree.application.NamespaceNode;
-import org.jboss.tools.intellij.openshift.tree.application.ParentableNode;
 import org.jboss.tools.intellij.openshift.ui.helm.ChartsDialog;
 import org.jboss.tools.intellij.openshift.utils.helm.Helm;
 import org.jboss.tools.intellij.openshift.utils.odo.Odo;
@@ -27,11 +27,10 @@ public class OpenHelmChartsAction extends HelmAction {
     @Override
     public void actionPerformedOnSelectedObject(AnActionEvent anActionEvent, Object selected, @NotNull Helm helm) {
         Project project = getEventProject(anActionEvent);
-        ParentableNode<?> parentableNode = ((ParentableNode<?>) selected);
-        if (parentableNode == null) {
+        ApplicationsRootNode rootNode = NodeUtils.getRoot(selected);
+        if (rootNode == null) {
             return;
         }
-        ApplicationsRootNode rootNode = parentableNode.getRoot();
         Odo odo = rootNode.getOdo().getNow(null);
         if (odo == null) {
             return;