Update test files for path handling

Author: jdalton

test/registry/cacache.test.mts

@@ -1,12 +1,54 @@
-import { afterEach, describe, expect, it } from 'vitest'
+import { mkdtempSync } from 'node:fs'
+import os from 'node:os'
+import path from 'node:path'
+
+import {
+  afterAll,
+  afterEach,
+  beforeAll,
+  beforeEach,
+  describe,
+  expect,
+  it,
+} from 'vitest'
 
 import * as cacache from '../../registry/dist/lib/cacache.js'
+import { trash } from '../../scripts/utils/fs.mjs'
 
 // Test key for cache operations.
 const TEST_KEY = 'test-cacache-key'
 const TEST_DATA = 'test data content'
 
 describe('cacache module', () => {
+  let testCacheDir: string
+  let originalEnv: string | undefined
+
+  beforeAll(() => {
+    // Create a temporary directory for test-specific cacache.
+    testCacheDir = mkdtempSync(path.join(os.tmpdir(), 'cacache-test-'))
+    // Override cacache directory for tests.
+    originalEnv = process.env['SOCKET_CACACHE_DIR']
+    process.env['SOCKET_CACACHE_DIR'] = testCacheDir
+  })
+
+  afterAll(async () => {
+    // Restore original environment.
+    if (originalEnv === undefined) {
+      delete process.env['SOCKET_CACACHE_DIR']
+    } else {
+      process.env['SOCKET_CACACHE_DIR'] = originalEnv
+    }
+    // Clean up test cache directory.
+    await trash(testCacheDir)
+  })
+
+  beforeEach(async () => {
+    // Remove test key before each test for defensive cleanup.
+    await cacache.remove(TEST_KEY).catch(() => {
+      // Ignore cleanup errors.
+    })
+  })
+
   afterEach(async () => {
     // Clean up test entries after each test.
     await cacache.remove(TEST_KEY).catch(() => {

test/registry/debug.test.mts

@@ -428,8 +428,8 @@ describe('debug module', () => {
   })
 
   describe('isDebugNs', () => {
-    it('should return false when SOCKET_CLI_DEBUG is not set', () => {
-      delete process.env['SOCKET_CLI_DEBUG']
+    it('should return false when SOCKET_DEBUG is not set', () => {
+      delete process.env['SOCKET_DEBUG']
       expect(isDebugNs('test')).toBe(false)
     })
   })

test/registry/sea-build.test.mts

@@ -1,18 +1,18 @@
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
 
 // Mock the httpRequest module before importing sea-build module.
-vi.mock('../../registry/dist/lib/http-request.js', () => {
+vi.mock('../../registry/src/lib/http-request', () => {
   return {
     httpRequest: vi.fn(),
   }
 })
 
-import { httpRequest as httpRequestActual } from '../../registry/dist/lib/http-request.js'
+import { httpRequest as httpRequestActual } from '../../registry/src/lib/http-request'
 import {
   getBuildTargets,
   getDefaultNodeVersion,
   getLatestCurrentRelease,
-} from '../../registry/dist/lib/sea-build.js'
+} from '../../registry/src/lib/sea-build'
 
 const httpRequest = httpRequestActual as unknown as ReturnType<typeof vi.fn>