fix: Electron dependency addition related fix for lightspeed [ACA-4624] (#2377)

* Code changes for fixing the issue and unmarking skip in ui tests

* lint fix

---------

Co-authored-by: tanwigeetika1618 <84617407+tanwigeetika1618@users.noreply.github.com>
Co-authored-by: Shatakshi Mishra <shatakshimishra01@gmail.com>

Author: 3 people

src/features/lightspeed/api.ts

@@ -42,10 +42,15 @@ const UNKNOWN_ERROR: string = "An unknown error occurred.";
 
 export function getFetch() {
   try {
-    return require("electron")?.net?.fetch;
+    const electronFetch = require("electron")?.net?.fetch;
+    if (electronFetch) {
+      return electronFetch;
+    }
   } catch {
-    return globalThis.fetch;
+    // electron not available (e.g., in tests or web environment)
   }
+  // Fallback to global fetch
+  return globalThis.fetch;
 }
 
 export class LightSpeedAPI {

test/ui/lightspeedUiTestPlaybookGenWebviewPart1Test.ts

@@ -22,10 +22,9 @@ describe("playbook generation features work", function () {
   let adtView: ViewSection;
 
   beforeEach(function () {
-    this.skip();
-    // if (!process.env.TEST_LIGHTSPEED_URL) {
-    //   this.skip();
-    // }
+    if (!process.env.TEST_LIGHTSPEED_URL) {
+      this.skip();
+    }
   });
 
   before(async function () {

test/ui/lightspeedUiTestPlaybookGenWebviewPart2Test.ts

@@ -21,8 +21,9 @@ import { PlaybookGenerationActionEvent } from "../../src/interfaces/lightspeed";
 config.truncateThreshold = 0;
 
 before(function () {
-  // Apparently broken on linux too
-  this.skip();
+  if (process.platform === "darwin") {
+    this.skip();
+  }
 });
 
 describe("playbook generation features work", function () {