refactor(electron): simplify script handling in execute function

- Updated the `execute` function to pass scripts as-is instead of using `JSON.stringify`, improving handling of various script formats.
- Adjusted related test cases to reflect this change, ensuring accurate expectations for string arguments passed to the plugin.

Author: goosewobbler

packages/electron-service/src/commands/execute.ts

@@ -14,7 +14,7 @@ export async function execute<ReturnValue, InnerArguments extends unknown[]>(
     throw new Error('WDIO browser is not yet initialised');
   }
 
-  const scriptString = typeof script === 'function' ? script.toString() : JSON.stringify(script);
+  const scriptString = typeof script === 'function' ? script.toString() : script;
 
   const returnValue = await browser.execute(
     function executeWithinElectron(script: string, ...args) {

packages/electron-service/test/commands/execute.spec.ts

@@ -44,40 +44,37 @@ describe('execute Command', () => {
 
   it('should execute a stringified function', async () => {
     await execute(globalThis.browser, '() => 1 + 2 + 3');
-    expect(globalThis.browser.execute).toHaveBeenCalledWith(expect.any(Function), JSON.stringify('() => 1 + 2 + 3'));
-    expect(globalThis.wdioElectron.execute).toHaveBeenCalledWith(JSON.stringify('() => 1 + 2 + 3'), []);
+    expect(globalThis.browser.execute).toHaveBeenCalledWith(expect.any(Function), '() => 1 + 2 + 3');
+    expect(globalThis.wdioElectron.execute).toHaveBeenCalledWith('() => 1 + 2 + 3', []);
   });
 
   it('should handle scripts with quotes', async () => {
     const scriptWithQuotes = '() => "He said \\"hello\\""';
     await execute(globalThis.browser, scriptWithQuotes);
-    expect(globalThis.browser.execute).toHaveBeenCalledWith(expect.any(Function), JSON.stringify(scriptWithQuotes));
+    expect(globalThis.browser.execute).toHaveBeenCalledWith(expect.any(Function), scriptWithQuotes);
   });
 
   it('should handle scripts with newlines', async () => {
     const scriptWithNewlines = '() => "line1\\nline2"';
     await execute(globalThis.browser, scriptWithNewlines);
-    expect(globalThis.browser.execute).toHaveBeenCalledWith(expect.any(Function), JSON.stringify(scriptWithNewlines));
+    expect(globalThis.browser.execute).toHaveBeenCalledWith(expect.any(Function), scriptWithNewlines);
   });
 
   it('should handle scripts with unicode', async () => {
     const scriptWithUnicode = '() => "Hello 世界"';
     await execute(globalThis.browser, scriptWithUnicode);
-    expect(globalThis.browser.execute).toHaveBeenCalledWith(expect.any(Function), JSON.stringify(scriptWithUnicode));
+    expect(globalThis.browser.execute).toHaveBeenCalledWith(expect.any(Function), scriptWithUnicode);
   });
 
   it('should handle scripts with backslashes', async () => {
     const scriptWithBackslashes = '() => "C:\\\\path\\\\file"';
     await execute(globalThis.browser, scriptWithBackslashes);
-    expect(globalThis.browser.execute).toHaveBeenCalledWith(
-      expect.any(Function),
-      JSON.stringify(scriptWithBackslashes),
-    );
+    expect(globalThis.browser.execute).toHaveBeenCalledWith(expect.any(Function), scriptWithBackslashes);
   });
 
   it('should handle mixed special characters', async () => {
     const script = '() => "Test \\n \\t \\u001b and \\\\ backslash"';
     await execute(globalThis.browser, script);
-    expect(globalThis.browser.execute).toHaveBeenCalledWith(expect.any(Function), JSON.stringify(script));
+    expect(globalThis.browser.execute).toHaveBeenCalledWith(expect.any(Function), script);
   });
 });