temporarily disable Windows sandbox tests fixed in a separate PR

ehedlund · ehedlund · commit 58f8646cfaee · 2026-04-03T10:33:43.000-04:00
diff --git a/packages/core/src/sandbox/windows/WindowsSandboxManager.test.ts b/packages/core/src/sandbox/windows/WindowsSandboxManager.test.ts
@@ -25,7 +25,8 @@ vi.mock('../../utils/shell-utils.js', async (importOriginal) => {
   };
 });
 
-describe('WindowsSandboxManager', () => {
+// TODO: reenable once test is fixed
+describe.skipIf(os.platform() === 'win32')('WindowsSandboxManager', () => {
   let manager: WindowsSandboxManager;
   let testCwd: string;
 
diff --git a/packages/core/src/services/sandboxManager.integration.test.ts b/packages/core/src/services/sandboxManager.integration.test.ts
@@ -103,7 +103,8 @@ function ensureSandboxAvailable(): boolean {
   if (platform === 'win32') {
     // Windows sandboxing relies on icacls, which is a core system utility and
     // always available.
-    return true;
+    // TODO: reenable once test is fixed
+    return false;
   }
 
   if (platform === 'darwin') {

Original file line number	Diff line number	Diff line change
`@@ -103,7 +103,8 @@ function ensureSandboxAvailable(): boolean {`
`103`	`103`	`if (platform === 'win32') {`
`104`	`104`	`// Windows sandboxing relies on icacls, which is a core system utility and`
`105`	`105`	`// always available.`
`106`		`- return true;`
	`106`	`+ // TODO: reenable once test is fixed`
	`107`	`+ return false;`
`107`	`108`	`}`
`108`	`109`
`109`	`110`	`if (platform === 'darwin') {`