fix: show 'no url/binding' node, use async leafstate for component (#687)

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

Author: adietish

src/main/java/org/jboss/tools/intellij/openshift/tree/application/ApplicationsTreeStructure.java

@@ -95,9 +95,7 @@ public Object getApplicationsRoot() {
 
     @Override
     public @NotNull LeafState getLeafState(@NotNull Object element) {
-        if (element instanceof ComponentNode) {
-            return LeafState.ALWAYS;
-        } else if (element instanceof ChartReleaseNode) {
+        if (element instanceof ChartReleaseNode) {
             return LeafState.ALWAYS;
         } else if (element instanceof MessageNode<?>) {
             return LeafState.ALWAYS;
@@ -112,8 +110,8 @@ private Object[] createComponentChildren(ComponentNode componentNode) {
         if (odo == null) {
             return new Object[] { new MessageNode<>(root, componentNode, "Could not get components") };
         }
-        List<URLNode> urls = getURLs(componentNode);
-        List<BindingNode> bindings = getBindings(componentNode, odo);
+        List<BaseNode<ComponentNode>> urls = getURLs(componentNode, odo);
+        List<BaseNode<ComponentNode>> bindings = getBindings(componentNode, odo);
         return Stream.of(urls, bindings)
           .filter(item -> !item.isEmpty())
           .flatMap(Collection::stream)
@@ -226,34 +224,47 @@ private List<BaseNode<?>> load(Callable<List<BaseNode<?>>> callable, NamespaceNo
             return Collections.singletonList(new MessageNode<>(namespace.getRoot(), namespace, errorMessage));
         }
     }
-    private List<URLNode> getURLs(ComponentNode element) {
-        List<URLNode> results = new ArrayList<>();
-        Odo odo = element.getRoot().getOdo().getNow(null);
-        if (odo == null) {
-            return Collections.emptyList();
-        }
+
+    private List<BaseNode<ComponentNode>> getURLs(ComponentNode componentNode, @NotNull Odo odo) {
+        List<BaseNode<ComponentNode>> nodes = new ArrayList<>();
         try {
-            odo.listURLs(element.getParent().getName(),
-                element.getComponent().getPath(), element.getName()).forEach(url ->
-                    results.add(new URLNode(element, url))
-                );
+            List<URLNode> urls = odo.listURLs(
+                componentNode.getParent().getName(),
+                componentNode.getComponent().getPath(),
+                componentNode.getName())
+              .stream()
+              .map(url -> new URLNode(componentNode, url))
+              .collect(Collectors.toList());
+            if (!urls.isEmpty()) {
+                nodes.addAll(urls);
+            } else {
+                nodes.add(new MessageNode<>(root, componentNode, "<No URLS>"));
+            }
         } catch (IOException e) {
             LOGGER.warn(e.getLocalizedMessage(), e);
         }
-        return results;
+        return nodes;
     }
 
-    private List<BindingNode> getBindings(ComponentNode element, @NotNull Odo odo) {
-        List<BindingNode> results = new ArrayList<>();
+    private List<BaseNode<ComponentNode>> getBindings(ComponentNode componentNode, @NotNull Odo odo) {
+        List<BaseNode<ComponentNode>> nodes = new ArrayList<>();
         try {
-            odo.listBindings(element.getParent().getName(),
-                element.getComponent().getPath(), element.getName()).forEach(binding ->
-                    results.add(new BindingNode(element, binding))
-                );
+            List<BindingNode> bindings = odo.listBindings(
+                componentNode.getParent().getName(),
+                componentNode.getComponent().getPath(),
+                componentNode.getName())
+              .stream()
+              .map(binding -> new BindingNode(componentNode, binding))
+              .collect(Collectors.toList());
+            if (!bindings.isEmpty()) {
+                nodes.addAll(bindings);
+            } else {
+                nodes.add(new MessageNode<>(root, componentNode, "<No Bindings>"));
+            }
         } catch (IOException e) {
             LOGGER.warn(e.getLocalizedMessage(), e);
         }
-        return results;
+        return nodes;
     }
 
     private Object[] getRegistries(ApplicationsRootNode root) {
@@ -323,6 +334,11 @@ public Object getParentElement(@NotNull Object element) {
     public void commit() {
     }
 
+    @Override
+    public boolean isToBuildChildrenInBackground(@NotNull Object element) {
+        return true;
+    }
+
     @Override
     public boolean hasSomethingToCommit() {
         return false;