Add 'Reveal in Explorer' command for pushed/not pushed components (#2064)

* Add 'Reveal in Explorer' command for pushed/not pushed components

This PR fixes #2011.

Signed-off-by: Denis Golovin dgolovin@redhat.com

Author: dgolovin

package.json

@@ -782,6 +782,10 @@
         "title": "Open Add CRC Cluster Wizard",
         "category": "OpenShift"
       },
+      {
+        "command": "openshift.component.revealContextInExplorer",
+        "title": "Reveal in Explorer"
+      },
       {
         "command": "openshift.component.test",
         "title": "Test",
@@ -1054,6 +1058,10 @@
         {
           "command": "openshift.component.watch.showLog",
           "when": "false"
+        },
+        {
+          "command": "openshift.component.revealContextInExplorer",
+          "when": "false"
         }
       ],
       "view/title": [
@@ -1320,19 +1328,24 @@
           "group": "c4@6"
         },
         {
-          "command": "openshift.component.delete",
-          "when": "view == openshiftProjectExplorer && viewItem == componentNoContext",
+          "command": "openshift.component.revealContextInExplorer",
+          "when": "view == openshiftProjectExplorer && viewItem == component || viewItem == componentNotPushed",
           "group": "c5@1"
         },
+        {
+          "command": "openshift.component.delete",
+          "when": "view == openshiftProjectExplorer && viewItem == componentNoContext || ",
+          "group": "c6@1"
+        },
         {
           "command": "openshift.component.delete",
           "when": "view == openshiftProjectExplorer && viewItem == component",
-          "group": "c5@1"
+          "group": "c6@1"
         },
         {
           "command": "openshift.component.delete",
           "when": "view == openshiftProjectExplorer && viewItem == componentNotPushed",
-          "group": "c5@1"
+          "group": "c6@1"
         },
         {
           "command": "openshift.service.create",

src/componentsView.ts

@@ -14,7 +14,6 @@ import { Command } from './odo/command';
 import { EnvInfo } from './odo/env';
 import { vsCommand } from './vscommand';
 
-
 export interface WorkspaceEntry {
     uri: vsc.Uri;
     type: vsc.FileType;
@@ -133,6 +132,7 @@ export class ComponentsTreeDataProvider extends BaseTreeDataProvider<Entry> {
             await vsc.commands.executeCommand('revealInExplorer', context.contextUri);
         }
     }
+
     createTreeView(id: string): vsc.TreeView<Entry> {
         if (!this.treeView) {
             this.treeView = vsc.window.createTreeView(id, {

src/openshift/component.ts

@@ -976,6 +976,12 @@ export class Component extends OpenShiftItem {
         await Component.odo.executeInTerminal(Command.testComponent(), component.contextPath.fsPath, `OpenShift: Test '${component.getName()}' Component`);
     }
 
+    @vsCommand('openshift.component.revealContextInExplorer')
+    public static async revealContextInExplorer(context: OpenShiftComponent): Promise<void> {
+        await commands.executeCommand('workbench.view.explorer');
+        await commands.executeCommand('revealInExplorer', context.contextPath);
+    }
+
     @vsCommand('openshift.component.import')
     // @clusterRequired() - not required because available only from context menu in application explorer
     static async import(component: OpenShiftObject): Promise<string | null> {

test/unit/openshift/component.test.ts

@@ -70,7 +70,7 @@ suite('OpenShift/Component', () => {
         getApps = sandbox.stub(OpenShiftItem, 'getApplicationNames').resolves([appItem]);
         sandbox.stub(OpenShiftItem, 'getComponentNames').resolves([componentItem]);
         sandbox.stub(OpenShiftItem, 'getServiceNames').resolves([serviceItem]);
-        commandStub = sandbox.stub(vscode.commands, 'executeCommand');
+        commandStub = sandbox.stub(vscode.commands, 'executeCommand').resolves();
         sandbox.stub()
     });
 
@@ -92,6 +92,15 @@ suite('OpenShift/Component', () => {
         });
     });
 
+    suite('reveal in explorer', () => {
+
+        test('called revealInExplorer with component\'s context', async () => {
+            await Component.revealContextInExplorer(componentItem);
+            expect(commandStub).calledWith('revealInExplorer', componentItem.contextPath);
+        });
+
+    });
+
     suite('create', () => {
         const componentType = new ComponentTypeAdapter(ComponentKind.S2I, 'nodejs', 'latest', 'builder,nodejs');
         const version = 'latest';