remove unneeded log, fix CR

Author: Mai-Saad

src/support/steps/general.ts

@@ -18,7 +18,7 @@ import {WP_BASE_URL} from '../../../config/wp.config';
 import scenarioUrls from "./../../../config/scenarioUrls.json";
 import { compareReference, isTagPresent, getScenarioTag, batchUpdateVRTestUrl, openMobileMenu} from "../../../utils/helpers";
 import type { Section } from "../../../utils/types";
-import { Page } from '@playwright/test';
+import { Page, ConsoleMessage } from '@playwright/test';
 import {
     deactivatePlugin, installRemotePlugin
 } from "../../../utils/commands";
@@ -479,7 +479,7 @@ Then('no error in the console different than nowprocket page {string}', async fu
 const getConsoleMsg = async (page: Page, url: string): Promise<Array<string>> => {
     const consoleMsg: string[] = [];
 
-    const consoleHandler = (msg): void => {
+    const consoleHandler = (msg: ConsoleMessage): void => {
         consoleMsg.push(msg.text());
     };
 
@@ -548,7 +548,7 @@ const getConsoleMsg = async (page: Page, url: string): Promise<Array<string>> =>
 const getConsoleMsgWithMenuExpansion = async (page: Page, url: string): Promise<Array<string>> => {
     const consoleMsg: string[] = [];
 
-    const consoleHandler = (msg): void => {
+    const consoleHandler = (msg: ConsoleMessage): void => {
         consoleMsg.push(msg.text());
     };
 

utils/commands.ts

@@ -493,13 +493,10 @@ export async function switchTheme(theme: string): Promise<void> {
     // Check if theme is already active
     let isActive = await isThemeActivated(theme);
     if (isActive) {
-        console.log(`[DEBUG] Theme ${theme} is already active, skipping activation`);
         return; // Theme is already active, no need to activate
     }
 
-    // Sanitize theme name to prevent shell injection and quote issues
-    // const sanitizedTheme = theme.replace(/[^\w-]/g, '');
-    // await wp(`theme activate "${sanitizedTheme}"`);
+    // Activate the theme
     await wp(`theme activate ${theme}`)
 
     // Verify theme is actually activated