fix: fix

Signed-off-by: Anton Misskii <[email protected]>

Author: amisskii

tests/playwright/src/ai-lab-extension.spec.ts

@@ -437,15 +437,15 @@ test.describe.serial(`AI Lab extension installation and verification`, () => {
     });
   });
 
-  ['Audio to Text', 'ChatBot', 'Summarizer', 'Code Generation', 'RAG Chatbot', 'Function calling'].forEach(appName => {
+  ['ChatBot', 'Function calling'].forEach(appName => {
     test.describe.serial(`AI Recipe installation`, () => {
       test.skip(
         !process.env.EXT_TEST_RAG_CHATBOT && appName === 'RAG Chatbot',
         'EXT_TEST_RAG_CHATBOT variable not set, skipping test',
       );
       let recipesCatalogPage: AILabRecipesCatalogPage;
 
-      test(`Navigate to Recipes Catalog for ${appName}`, async ({ runner, page, navigationBar }) => {
+      test.beforeAll(`Open Recipes Catalog`, async ({ runner, page, navigationBar }) => {
         [page, webview] = await handleWebview(runner, page, navigationBar);
         aiLabPage = new AILabPage(page, webview);
         await aiLabPage.navigationBar.waitForLoad();
@@ -461,46 +461,52 @@ test.describe.serial(`AI Lab extension installation and verification`, () => {
         await demoApp.startNewDeployment();
       });
 
-      /* eslint-disable sonarjs/no-nested-functions */
-      if (appName === 'Function calling') {
-        test(`Verify that model service for the ${appName} is working`, async ({ request }) => {
-          test.setTimeout(600_000);
-          const modelServicePage = await aiLabPage.navigationBar.openServices();
-          const serviceDetailsPage = await modelServicePage.openSercviceDetails(
-            'ibm-granite/granite-3.3-8b-instruct-GGUF',
-          );
-          await playExpect.poll(async () => await serviceDetailsPage.getServiceState()).toBe('RUNNING');
-          const port = await serviceDetailsPage.getInferenceServerPort();
-          const url = `http://localhost:${port}/v1/chat/completions`;
+      test(`Verify that model service for the ${appName} is working`, async ({ request }) => {
+        test.skip(appName !== 'Function calling');
+        test.setTimeout(600_000);
 
-          const response = await request.post(url, {
-            data: {
-              messages: [
-                {
-                  content: 'You are a helpful assistant.',
-                  role: 'system',
-                },
-                {
-                  content: 'What is the capital of Czech Republic?',
-                  role: 'user',
-                },
-              ],
-            },
-            timeout: 600_000,
-          });
+        const modelServicePage = await aiLabPage.navigationBar.openServices();
+        const serviceDetailsPage = await modelServicePage.openServiceDetails(
+          'ibm-granite/granite-3.3-8b-instruct-GGUF',
+        );
+        // eslint-disable-next-line sonarjs/no-nested-functions
+        await playExpect.poll(async () => await serviceDetailsPage.getServiceState()).toBe('RUNNING');
+        const port = await serviceDetailsPage.getInferenceServerPort();
+        const url = `http://localhost:${port}/v1/chat/completions`;
 
-          playExpect(response.ok()).toBeTruthy();
-          const body = await response.body();
-          const text = body.toString();
-          playExpect(text).toContain('Prague');
+        const response = await request.post(url, {
+          data: {
+            messages: [
+              {
+                content: 'You are a helpful assistant.',
+                role: 'system',
+              },
+              {
+                content: 'What is the capital of Czech Republic?',
+                role: 'user',
+              },
+            ],
+          },
+          timeout: 600_000,
         });
-      }
-      /* eslint-enable sonarjs/no-nested-functions */
 
-      test(`Stop ${appName} app`, async ({ navigationBar }) => {
+        playExpect(response.ok()).toBeTruthy();
+        const body = await response.body();
+        const text = body.toString();
+        playExpect(text).toContain('Prague');
+      });
+
+      test(`Stop ${appName} app`, async () => {
         test.setTimeout(150_000);
         await stopAndDeleteApp(appName);
         await cleanupServiceModels();
+      });
+
+      test.afterAll(`Ensure cleanup of "${appName}" app, related service, and images`, async ({ navigationBar }) => {
+        test.setTimeout(150_000);
+
+        await stopAndDeleteApp(appName);
+        await cleanupServiceModels();
         await deleteUnusedImages(navigationBar);
       });
     });
@@ -521,6 +527,9 @@ async function cleanupServiceModels(): Promise<void> {
 async function stopAndDeleteApp(appName: string): Promise<void> {
   const aiRunningAppsPage = await aiLabPage.navigationBar.openRunningApps();
   await aiRunningAppsPage.waitForLoad();
+  if (!(await aiRunningAppsPage.appExists(appName))) {
+    console.log(`"${appName}" is not present in the running apps list. Skipping stop and delete operations.`);
+  }
   await playExpect.poll(async () => await aiRunningAppsPage.appExists(appName), { timeout: 10_000 }).toBeTruthy();
   await playExpect
     .poll(async () => await aiRunningAppsPage.getCurrentStatusForApp(appName), { timeout: 60_000 })

tests/playwright/src/model/ai-lab-model-service-page.ts

@@ -67,7 +67,7 @@ export class AiModelServicePage extends AILabBasePage {
     return (await this.getAllTableRows()).length;
   }
 
-  async openSercviceDetails(modelName: string): Promise<AILabServiceDetailsPage> {
+  async openServiceDetails(modelName: string): Promise<AILabServiceDetailsPage> {
     const serviceRow = await this.getServiceByModel(modelName);
     if (serviceRow === undefined) {
       throw new Error(`Model [${modelName}] service doesn't exist`);
@@ -80,6 +80,7 @@ export class AiModelServicePage extends AILabBasePage {
   async getServiceByModel(modelName: string): Promise<Locator | undefined> {
     const rows = await this.getAllTableRows();
     for (let rowNum = 1; rowNum < rows.length; rowNum++) {
+      //skip header
       const serviceModel = rows[rowNum].getByRole('cell').nth(4);
       if ((await serviceModel.textContent()) === modelName) {
         return rows[rowNum];

tests/playwright/src/model/ai-lab-service-details-page.ts

@@ -59,9 +59,6 @@ export class AILabServiceDetailsPage extends AILabBasePage {
 
   async getServiceState(): Promise<string> {
     const serviceState = await this.webview.getByRole('status').getAttribute('title');
-    if (serviceState === 'RUNNING') {
-      return 'RUNNING';
-    }
     return serviceState ?? 'UNKNOWN';
   }
 }