From c74c23b0ffada047507409d4cd39535b30d8f040 Mon Sep 17 00:00:00 2001
From: Sander Declerck <sander@sdsoftware.be>
Date: Thu, 23 Oct 2025 10:52:03 +0200
Subject: [PATCH] Fix unit tests

---
 packages/safe-chain/src/utils/safeSpawn.js      | 3 ++-
 packages/safe-chain/src/utils/safeSpawn.spec.js | 6 ++++++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/packages/safe-chain/src/utils/safeSpawn.js b/packages/safe-chain/src/utils/safeSpawn.js
index 32669b3..96c0603 100644
--- a/packages/safe-chain/src/utils/safeSpawn.js
+++ b/packages/safe-chain/src/utils/safeSpawn.js
@@ -1,4 +1,5 @@
 import { spawn, execSync } from "child_process";
+import os from "os";
 
 function escapeArg(arg) {
   // Shell metacharacters that need escaping
@@ -42,7 +43,7 @@ export async function safeSpawn(command, args, options = {}) {
     // array args (safer, no escaping needed).
     // See: https://nodejs.org/api/child_process.html#child_processspawncommand-args-options
     let child;
-    if (process.platform === "win32") {
+    if (os.platform() === "win32") {
       const fullCommand = buildCommand(command, args);
       child = spawn(fullCommand, { ...options, shell: true });
     } else {
diff --git a/packages/safe-chain/src/utils/safeSpawn.spec.js b/packages/safe-chain/src/utils/safeSpawn.spec.js
index 6d8dd26..4ad005e 100644
--- a/packages/safe-chain/src/utils/safeSpawn.spec.js
+++ b/packages/safe-chain/src/utils/safeSpawn.spec.js
@@ -33,6 +33,12 @@ describe("safeSpawn", () => {
       },
     });
 
+    mock.module("os", {
+      namedExports: {
+        platform: () => "win32",
+      },
+    });
+
     // Import after mocking
     const safeSpawnModule = await import("./safeSpawn.js");
     safeSpawn = safeSpawnModule.safeSpawn;