test(cli): fix flaky path-resolve tests on Windows

Fix 4 Windows-specific test failures by normalizing paths for cross-platform
compatibility. Tests were comparing Unix-style paths against Windows paths.

Changes:
- Use normalizePath() in mock implementations to handle path separators
- Use normalizePath() when asserting expected results
- Ensures tests work on both Unix (forward slash) and Windows (backslash)

Fixes flaky tests in findNpmDirPathSync():
- "finds npm directory in lib/node_modules structure"
- "finds npm directory with node_modules in current path"
- "finds npm directory with node_modules in parent path"
- "handles nvm directory structure"

All 21 tests in path-resolve.test.mts now pass on all platforms.

Author: jdalton

packages/cli/src/utils/fs/path-resolve.test.mts

@@ -473,7 +473,7 @@ describe('Path Resolve', () => {
       const { isDirSync } = vi.mocked(await import('@socketsecurity/lib/fs'))
 
       isDirSync.mockImplementation(p => {
-        const pathStr = String(p)
+        const pathStr = normalizePath(String(p))
         if (pathStr.includes('lib/node_modules/npm')) {
           return true
         }
@@ -485,42 +485,42 @@ describe('Path Resolve', () => {
 
       const result = findNpmDirPathSync('/usr/local/bin/npm')
 
-      expect(result).toBe('/usr/local/bin/npm/lib/node_modules/npm')
+      expect(normalizePath(result)).toBe(normalizePath('/usr/local/bin/npm/lib/node_modules/npm'))
     })
 
     it('finds npm directory with node_modules in current path', async () => {
       const { isDirSync } = vi.mocked(await import('@socketsecurity/lib/fs'))
 
       isDirSync.mockImplementation(p => {
-        const pathStr = String(p)
-        if (pathStr === '/usr/local/npm/node_modules') {
+        const pathStr = normalizePath(String(p))
+        if (pathStr === normalizePath('/usr/local/npm/node_modules')) {
           return true
         }
         return false
       })
 
       const result = findNpmDirPathSync('/usr/local/npm')
 
-      expect(result).toBe('/usr/local/npm')
+      expect(normalizePath(result)).toBe(normalizePath('/usr/local/npm'))
     })
 
     it('finds npm directory with node_modules in parent path', async () => {
       const { isDirSync } = vi.mocked(await import('@socketsecurity/lib/fs'))
 
       isDirSync.mockImplementation(p => {
-        const pathStr = String(p)
-        if (pathStr === '/usr/local/npm/node_modules') {
+        const pathStr = normalizePath(String(p))
+        if (pathStr === normalizePath('/usr/local/npm/node_modules')) {
           return false
         }
-        if (pathStr === '/usr/local/node_modules') {
+        if (pathStr === normalizePath('/usr/local/node_modules')) {
           return true
         }
         return false
       })
 
       const result = findNpmDirPathSync('/usr/local/npm')
 
-      expect(result).toBe('/usr/local')
+      expect(normalizePath(result)).toBe(normalizePath('/usr/local'))
     })
 
     it('returns undefined when no npm directory found', async () => {
@@ -537,7 +537,7 @@ describe('Path Resolve', () => {
       const { isDirSync } = vi.mocked(await import('@socketsecurity/lib/fs'))
 
       isDirSync.mockImplementation(p => {
-        const pathStr = String(p)
+        const pathStr = normalizePath(String(p))
         if (pathStr.includes('.nvm') && pathStr.endsWith('/node_modules')) {
           return true
         }
@@ -548,7 +548,7 @@ describe('Path Resolve', () => {
         '/Users/user/.nvm/versions/node/v18.0.0/bin/npm',
       )
 
-      expect(result).toBe('/Users/user/.nvm/versions/node/v18.0.0/bin/npm')
+      expect(normalizePath(result)).toBe(normalizePath('/Users/user/.nvm/versions/node/v18.0.0/bin/npm'))
     })
   })
 })