Update tests

Author: jdalton

src/commands/manifest/cmd-manifest-cdxgen.test.mts

@@ -1,6 +1,5 @@
 import path from 'node:path'
 
-import semver from 'semver'
 import { describe, expect, it } from 'vitest'
 
 import { LOG_SYMBOLS } from '@socketsecurity/registry/lib/logger'
@@ -63,16 +62,10 @@ describe('socket manifest cdxgen', async () => {
           .replace(/(?<=CycloneDX\s+Generator\s+)[\d.]+/, '<redacted>')
           .replace(/(?<=Node\.js,\s+Version:\s+)[\d.]+/, '<redacted>')
 
-        // Node 24 on Windows currently fails this test with assertion failure:
-        // Assertion failed: !(handle->flags & UV_HANDLE_CLOSING), file src\win\async.c, line 76
-        const skipStdoutOnWin32Node24 =
-          constants.WIN32 && semver.parse(constants.NODE_VERSION)!.major >= 24
-        if (!skipStdoutOnWin32Node24) {
-          expect(redactedStdout).toMatchInlineSnapshot(`
-            "CycloneDX Generator <redacted>
-            Runtime: Node.js, Version: <redacted>"
-          `)
-        }
+        expect(redactedStdout).toMatchInlineSnapshot(`
+          "CycloneDX Generator <redacted>
+          Runtime: Node.js, Version: <redacted>"
+        `)
         expect(`\n   ${stderr}`).toMatchInlineSnapshot(`
           "
              _____         _       _        /---------------

src/commands/optimize/cmd-optimize.test.mts

@@ -375,7 +375,7 @@ describe('socket optimize', async () => {
         const output = stdout + stderr
         expect(output).toMatch(/Optimizing|Adding overrides|Finished/i)
       },
-      { timeout: 60_000 },
+      { timeout: 120_000 },
     )
 
     cmdit(
@@ -410,7 +410,7 @@ describe('socket optimize', async () => {
         const output = stdout + stderr
         expect(output).toMatch(/Optimizing|Adding overrides/i)
       },
-      { timeout: 60_000 },
+      { timeout: 120_000 },
     )
 
     cmdit(

src/commands/pnpm/cmd-pnpm.test.mts

@@ -3,7 +3,7 @@ import { describe, expect } from 'vitest'
 import constants from '../../../src/constants.mts'
 import { cmdit, spawnSocketCli } from '../../../test/utils.mts'
 
-describe('socket pnpm', async () => {
+describe.skipIf(constants.WIN32)('socket pnpm', async () => {
   const { binCliPath } = constants
 
   cmdit(

src/commands/scan/cmd-scan-reach.test.mts

@@ -1,5 +1,4 @@
 import { describe, expect } from 'vitest'
-import semver from 'semver'
 
 import constants from '../../../src/constants.mts'
 import { cmdit, spawnSocketCli } from '../../../test/utils.mts'
@@ -493,13 +492,7 @@ describe('socket scan reach', async () => {
       const { code, stdout } = await spawnSocketCli(binCliPath, cmd, {
         cwd: '/tmp',
       })
-      // Node 24 on Windows currently fails this test with assertion failure:
-      // Assertion failed: !(handle->flags & UV_HANDLE_CLOSING), file src\win\async.c, line 76
-      const skipStdoutOnWin32Node24 =
-        constants.WIN32 && semver.parse(constants.NODE_VERSION)!.major >= 24
-      if (!skipStdoutOnWin32Node24) {
-        expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
-      }
+      expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
       expect(code, 'should exit with code 0').toBe(0)
     },
   )

src/commands/yarn/cmd-yarn.test.mts

@@ -3,7 +3,7 @@ import { describe, expect } from 'vitest'
 import constants from '../../../src/constants.mts'
 import { cmdit, spawnSocketCli } from '../../../test/utils.mts'
 
-describe('socket yarn', async () => {
+describe.skipIf(constants.WIN32)('socket yarn', async () => {
   const { binCliPath } = constants
 
   cmdit(