Normalize expected paths in tests for Windows compatibility

Path functions now return normalized paths with forward slashes on all platforms. Updated test expectations to use normalizePath() for consistent assertions across Windows and POSIX systems.

Author: jdalton

test/registry/edge-cases.test.mts

@@ -32,7 +32,7 @@ describe('edge case tests', () => {
       await fs.writeFile(testFile, '{}')
 
       const result = await fsUtils.findUp('package.json', { cwd: tmpDir })
-      expect(result).toBe(testFile)
+      expect(result).toBe(normalizePath(testFile))
     })
 
     it('should handle findUpSync edge cases', () => {
@@ -84,7 +84,7 @@ describe('edge case tests', () => {
     it('should handle uniqueSync with directories', () => {
       const dirPath = path.join(tmpDir, 'mydir')
       const unique1 = fsUtils.uniqueSync(dirPath)
-      expect(unique1).toBe(dirPath)
+      expect(unique1).toBe(normalizePath(dirPath))
 
       // Create the directory
       require('node:fs').mkdirSync(dirPath)

test/registry/fs.test.mts

@@ -387,7 +387,7 @@ describe('fs module', () => {
     it('should return same path if not exists', () => {
       const uniquePath = path.join(tmpDir, 'unique.txt')
       const result = uniqueSync(uniquePath)
-      expect(result).toBe(uniquePath)
+      expect(result).toBe(normalizePath(uniquePath))
     })
 
     it('should add number suffix for existing files', () => {

test/registry/paths.test.mts

@@ -3,6 +3,7 @@ import path from 'node:path'
 
 import { describe, expect, it } from 'vitest'
 
+import { normalizePath } from '../../registry/dist/lib/path.js'
 import {
   getSocketAppCacheDir,
   getSocketAppCacheTtlDir,
@@ -19,7 +20,7 @@ describe('paths module', () => {
   describe('getSocketUserDir', () => {
     it('should return ~/.socket path', () => {
       const result = getSocketUserDir()
-      expect(result).toBe(path.join(os.homedir(), '.socket'))
+      expect(result).toBe(normalizePath(path.join(os.homedir(), '.socket')))
     })
 
     it('should return consistent path on multiple calls', () => {
@@ -32,45 +33,57 @@ describe('paths module', () => {
   describe('getSocketAppDir', () => {
     it('should return app directory with underscore prefix', () => {
       const result = getSocketAppDir('test')
-      expect(result).toBe(path.join(os.homedir(), '.socket', '_test'))
+      expect(result).toBe(
+        normalizePath(path.join(os.homedir(), '.socket', '_test')),
+      )
     })
 
     it('should handle different app names', () => {
       const socket = getSocketAppDir('socket')
       const registry = getSocketAppDir('registry')
-      expect(socket).toBe(path.join(os.homedir(), '.socket', '_socket'))
-      expect(registry).toBe(path.join(os.homedir(), '.socket', '_registry'))
+      expect(socket).toBe(
+        normalizePath(path.join(os.homedir(), '.socket', '_socket')),
+      )
+      expect(registry).toBe(
+        normalizePath(path.join(os.homedir(), '.socket', '_registry')),
+      )
     })
   })
 
   describe('getSocketCacacheDir', () => {
     it('should return cacache directory', () => {
       const result = getSocketCacacheDir()
-      expect(result).toBe(path.join(os.homedir(), '.socket', '_cacache'))
+      expect(result).toBe(
+        normalizePath(path.join(os.homedir(), '.socket', '_cacache')),
+      )
     })
   })
 
   describe('getSocketDlxDir', () => {
     it('should return dlx directory', () => {
       const result = getSocketDlxDir()
-      expect(result).toBe(path.join(os.homedir(), '.socket', '_dlx'))
+      expect(result).toBe(
+        normalizePath(path.join(os.homedir(), '.socket', '_dlx')),
+      )
     })
   })
 
   describe('getSocketAppCacheDir', () => {
     it('should return app cache directory', () => {
       const result = getSocketAppCacheDir('test')
-      expect(result).toBe(path.join(os.homedir(), '.socket', '_test', 'cache'))
+      expect(result).toBe(
+        normalizePath(path.join(os.homedir(), '.socket', '_test', 'cache')),
+      )
     })
 
     it('should handle different app names', () => {
       const socketCache = getSocketAppCacheDir('socket')
       const registryCache = getSocketAppCacheDir('registry')
       expect(socketCache).toBe(
-        path.join(os.homedir(), '.socket', '_socket', 'cache'),
+        normalizePath(path.join(os.homedir(), '.socket', '_socket', 'cache')),
       )
       expect(registryCache).toBe(
-        path.join(os.homedir(), '.socket', '_registry', 'cache'),
+        normalizePath(path.join(os.homedir(), '.socket', '_registry', 'cache')),
       )
     })
   })
@@ -79,47 +92,59 @@ describe('paths module', () => {
     it('should return app TTL cache directory', () => {
       const result = getSocketAppCacheTtlDir('test')
       expect(result).toBe(
-        path.join(os.homedir(), '.socket', '_test', 'cache', 'ttl'),
+        normalizePath(
+          path.join(os.homedir(), '.socket', '_test', 'cache', 'ttl'),
+        ),
       )
     })
 
     it('should handle different app names', () => {
       const socketTtl = getSocketAppCacheTtlDir('socket')
       const registryTtl = getSocketAppCacheTtlDir('registry')
       expect(socketTtl).toBe(
-        path.join(os.homedir(), '.socket', '_socket', 'cache', 'ttl'),
+        normalizePath(
+          path.join(os.homedir(), '.socket', '_socket', 'cache', 'ttl'),
+        ),
       )
       expect(registryTtl).toBe(
-        path.join(os.homedir(), '.socket', '_registry', 'cache', 'ttl'),
+        normalizePath(
+          path.join(os.homedir(), '.socket', '_registry', 'cache', 'ttl'),
+        ),
       )
     })
   })
 
   describe('getSocketCliDir', () => {
     it('should return CLI directory', () => {
       const result = getSocketCliDir()
-      expect(result).toBe(path.join(os.homedir(), '.socket', '_socket'))
+      expect(result).toBe(
+        normalizePath(path.join(os.homedir(), '.socket', '_socket')),
+      )
     })
   })
 
   describe('getSocketRegistryDir', () => {
     it('should return Registry directory', () => {
       const result = getSocketRegistryDir()
-      expect(result).toBe(path.join(os.homedir(), '.socket', '_registry'))
+      expect(result).toBe(
+        normalizePath(path.join(os.homedir(), '.socket', '_registry')),
+      )
     })
   })
 
   describe('getSocketRegistryGithubCacheDir', () => {
     it('should return Registry GitHub cache directory', () => {
       const result = getSocketRegistryGithubCacheDir()
       expect(result).toBe(
-        path.join(
-          os.homedir(),
-          '.socket',
-          '_registry',
-          'cache',
-          'ttl',
-          'github',
+        normalizePath(
+          path.join(
+            os.homedir(),
+            '.socket',
+            '_registry',
+            'cache',
+            'ttl',
+            'github',
+          ),
         ),
       )
     })
@@ -158,14 +183,14 @@ describe('paths module', () => {
   })
 
   describe('platform compatibility', () => {
-    it('should use correct path separators', () => {
+    it('should use normalized forward slashes', () => {
       const result = getSocketAppDir('test')
-      expect(result.includes(path.sep)).toBe(true)
+      expect(result.includes('/')).toBe(true)
     })
 
-    it('should not contain hardcoded slashes', () => {
+    it('should normalize paths correctly', () => {
       const result = getSocketUserDir()
-      const parts = result.split(path.sep)
+      const parts = result.split('/')
       expect(parts[parts.length - 1]).toBe('.socket')
     })
   })