[tests] Fix Run Command test

- Fix broken imports (src/... → relative paths)
- Add suite timeout and existence assertions
- Use waitForItemStable for reliable component detection
- Improved componentCommands Public UI test

Signed-off-by: Victor Rubezhny <vrubezhny@redhat.com>
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: vrubezhny

src/devfile/parallelCompositeCommand.ts

@@ -3,8 +3,8 @@
  *  Licensed under the MIT License. See LICENSE file in the project root for license information.
  *-----------------------------------------------------------------------------------------------*/
 
-import { ComponentWorkspaceFolder } from 'src/odo/workspace';
 import { Command } from '../odo/componentTypeDescription';
+import { ComponentWorkspaceFolder } from '../odo/workspace';
 import { DevfileCommandRunner } from './devfileCommandRunner';
 
 export class ParallelCompositeCommand {

test/integration/command.test.ts

@@ -15,14 +15,14 @@ import { EventEmitter, Terminal, window, workspace } from 'vscode';
 import { parse, stringify } from 'yaml';
 import { CommandText } from '../../src/base/command';
 import { CliChannel } from '../../src/cli';
+import { DevfileCommandRunner } from '../../src/devfile/devfileCommandRunner';
 import { Oc } from '../../src/oc/ocWrapper';
 import { Command } from '../../src/odo/command';
 import { OdoPreference } from '../../src/odo/odoPreference';
 import { Odo } from '../../src/odo/odoWrapper';
+import { ComponentWorkspaceFolder } from '../../src/odo/workspace';
 import { LoginUtil } from '../../src/util/loginUtil';
 import { YAML_STRINGIFY_OPTIONS } from '../../src/util/utils';
-import { ComponentWorkspaceFolder } from 'src/odo/workspace';
-import { DevfileCommandRunner } from '../../src/devfile/devfileCommandRunner';
 
 const ODO = Odo.Instance;
 

test/ui/suite/componentCommands.ts

@@ -17,15 +17,16 @@ import {
     Workbench,
 } from 'vscode-extension-tester';
 import { parse } from 'yaml';
-import { itemExists } from '../common/conditions';
+import { waitForItemStable, warn } from '../common/conditions';
 import { VIEWS } from '../common/constants';
 import { reloadWindow } from '../common/overdrives';
 import { OpenshiftTerminalWebviewView } from '../common/ui/webviewView/openshiftTerminalWebviewView';
 
 export function testComponentCommands(path: string) {
     describe('Component Commands', function () {
+        this.timeout(30_000);
+
         let view: SideBarView;
-        let section: ViewSection;
         let commands: TreeItem[];
 
         const componentName = 'nodejs-starter';
@@ -46,10 +47,15 @@ export function testComponentCommands(path: string) {
             }
 
             // expect component is running
-            section = await view.getContent().getSection(VIEWS.components);
             try {
-                await itemExists(`${componentName} (dev running)`, section);
-            } catch {
+                const componentInDevName = `${componentName} (dev running)`;
+                const item = await waitForItemStable(getSection, componentInDevName, true, 40_000);
+                if (!item) {
+                    warn(`Component "${componentName}" not found or isn't running in Dev, skipping tests`);
+                    this.skip();
+                }
+            } catch (err) {
+                warn('Error in before hook: "Component Commands":', err);
                 this.skip();
             }
         });
@@ -70,7 +76,13 @@ export function testComponentCommands(path: string) {
             });
 
             //get component
+            const section = await getSection();
+            expect(section).to.exist;
+
             const components = await section.getVisibleItems();
+            expect(components).to.exist;
+            expect(components).to.have.length.greaterThan(0);
+
             const component = components[0] as TreeItem;
             await component.expand();
 
@@ -85,6 +97,11 @@ export function testComponentCommands(path: string) {
                 actualCommands.push(await command.getLabel());
             }
             expect(actualCommands).to.include.members(expectedCommands);
+
+            // The sets of devfile component commands (expectedCommands) and actual component commands (actualCommands)
+            // are to be the same
+            expect(actualCommands).to.have.members(expectedCommands);
+
         });
 
         it('Command can be ran', async function () {
@@ -117,5 +134,9 @@ export function testComponentCommands(path: string) {
 
             expect(terminalText).to.contain('Press any key to close this terminal');
         });
+
+        async function getSection(): Promise<ViewSection> {
+            return await view.getContent().getSection(VIEWS.components);
+        }
     });
 }