Fix unit test errrors

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

Author: dgolovin

src/openshift/component.ts

@@ -104,6 +104,10 @@ export class Component extends OpenShiftItem {
     }
 
     @vsCommand('openshift.component.create')
+    @clusterRequired()
+    @selectTargetApplication(
+        'In which Application you want to create a Component'
+    )
     static async create(application: OpenShiftApplication): Promise<string> {
         if (!application) return null;
 
@@ -115,11 +119,11 @@ export class Component extends OpenShiftItem {
 
         let command: Promise<string>;
         if (componentSource.label === SourceTypeChoice.GIT.label) {
-            command = Promise.resolve(commands.executeCommand('openshift.component.createFromGit', application));
+            command = Component.createFromGit(application);
         } else if (componentSource.label === SourceTypeChoice.BINARY.label) {
-            command = Promise.resolve(commands.executeCommand('openshift.component.createFromBinary', application));
+            command = Component.createFromBinary(application);
         } else if (componentSource.label === SourceTypeChoice.LOCAL.label) {
-            command = Promise.resolve(commands.executeCommand('openshift.component.createFromLocal', application));
+            command = Component.createFromLocal(application);
         }
         return command.catch((err) => Promise.reject(new VsCommandError(`Failed to create Component with error '${err}'`, 'Failed to create Component with error')));
     }

test/unit/openshift/catalog.test.ts

@@ -6,10 +6,11 @@
 import * as chai from 'chai';
 import * as sinonChai from 'sinon-chai';
 import * as sinon from 'sinon';
-import { OdoImpl } from '../../../src/odo';
+import { ContextType, OdoImpl } from '../../../src/odo';
 import { Command } from '../../../src/odo/command';
 import { Catalog } from '../../../src/openshift/catalog';
 import { Platform } from '../../../src/util/platform';
+import { TestItem } from './testOSItem';
 
 const {expect} = chai;
 chai.use(sinonChai);
@@ -20,21 +21,22 @@ suite('OpenShift/Catalog', () => {
 
     setup(() => {
         sandbox = sinon.createSandbox();
+        sandbox.stub(OdoImpl.prototype, 'getClusters').resolves([new TestItem(undefined, 'cluster', ContextType.CLUSTER)]);
         execStub = sandbox.stub(OdoImpl.prototype, 'executeInTerminal').resolves();
     });
 
     teardown(() => {
         sandbox.restore();
     });
 
-    test('listComponents calls odo catalog list components', () => {
-        Catalog.listComponents();
+    test('listComponents calls odo catalog list components', async () => {
+        await Promise.resolve(Catalog.listComponents());
 
         expect(execStub).calledOnceWith(Command.listCatalogComponents());
     });
 
-    test('listServices calls odo catalog list services', () => {
-        Catalog.listServices();
+    test('listServices calls odo catalog list services', async () => {
+        await Promise.resolve(Catalog.listServices());
 
         expect(execStub).calledOnceWith(Command.listCatalogServices(), Platform.getUserHomePath());
     });