Fix test imports after deps.ts removal

Update test files to import interfaces from their respective source
modules instead of the deleted deps.ts file. Remove export from
interfaces that are only used internally.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: vdiez

src/properties.ts

@@ -52,12 +52,12 @@ import {
   ScannerProperty,
 } from './types';
 
-export interface PropertiesFsDeps {
+interface PropertiesFsDeps {
   readFileSync: typeof fs.readFileSync;
   existsSync: typeof fs.existsSync;
 }
 
-export interface PropertiesProcessDeps {
+interface PropertiesProcessDeps {
   platform: NodeJS.Platform;
   arch: NodeJS.Architecture;
   env: NodeJS.ProcessEnv;

test/unit/java.test.ts

@@ -23,8 +23,12 @@ import axios from 'axios';
 import MockAdapter from 'axios-mock-adapter';
 import { LogLevel } from '../../src/logging';
 import { API_V2_JRE_ENDPOINT, SONARQUBE_JRE_PROVISIONING_MIN_VERSION } from '../../src/constants';
-import { FsDeps } from '../../src/deps';
-import { fetchJRE, fetchServerVersion, serverSupportsJREProvisioning } from '../../src/java';
+import {
+  fetchJRE,
+  fetchServerVersion,
+  serverSupportsJREProvisioning,
+  JavaFsDeps,
+} from '../../src/java';
 import * as request from '../../src/request';
 import {
   AnalysisJresResponseType,
@@ -204,13 +208,13 @@ describe('java', () => {
         const mockDownload = mock.fn(() => Promise.resolve());
         const mockValidateChecksum = mock.fn(() => Promise.reject(new Error('Checksum mismatch')));
         const mockRemove = mock.fn(() => Promise.resolve());
-        const mockFsDeps: Partial<FsDeps> = {
+        const mockFsDeps: Partial<JavaFsDeps> = {
           remove: mockRemove,
         };
 
         await assert.rejects(async () => {
           await fetchJRE(MOCKED_PROPERTIES, {
-            fsDeps: mockFsDeps as FsDeps,
+            fsDeps: mockFsDeps as JavaFsDeps,
             getCacheFileLocationFn: mockGetCacheFileLocation,
             getCacheDirectoriesFn: mockGetCacheDirectories,
             downloadFn: mockDownload,

test/unit/platform.test.ts

@@ -21,13 +21,13 @@
 import { describe, it, mock, beforeEach } from 'node:test';
 import assert from 'node:assert';
 import * as platform from '../../src/platform';
-import { FsDeps, ProcessDeps } from '../../src/deps';
+import { PlatformFsDeps, PlatformProcessDeps } from '../../src/platform';
 
 // Mock logging to suppress output
 const mockLog = mock.fn();
 mock.method(console, 'log', mockLog);
 
-function createMockProcessDeps(overrides: Partial<ProcessDeps> = {}): ProcessDeps {
+function createMockProcessDeps(overrides: Partial<PlatformProcessDeps> = {}): PlatformProcessDeps {
   return {
     platform: 'linux',
     arch: 'x64',
@@ -37,14 +37,14 @@ function createMockProcessDeps(overrides: Partial<ProcessDeps> = {}): ProcessDep
   };
 }
 
-function createMockFsDeps(readFileSyncResult?: string | Error): Partial<FsDeps> {
+function createMockFsDeps(readFileSyncResult?: string | Error): Partial<PlatformFsDeps> {
   return {
     readFileSync: mock.fn(() => {
       if (readFileSyncResult instanceof Error) {
         throw readFileSyncResult;
       }
       return Buffer.from(readFileSyncResult ?? '');
-    }) as unknown as FsDeps['readFileSync'],
+    }) as unknown as PlatformPlatformFsDeps['readFileSync'],
   };
 }
 
@@ -78,7 +78,7 @@ describe('getPlatformInfo', () => {
     const processDeps = createMockProcessDeps({ platform: 'linux', arch: 'x64' });
     const fsDeps = createMockFsDeps('NAME="Alpine Linux"\nID=alpine');
 
-    assert.strictEqual(platform.getSupportedOS(processDeps, fsDeps as FsDeps), 'alpine');
+    assert.strictEqual(platform.getSupportedOS(processDeps, fsDeps as PlatformFsDeps), 'alpine');
     assert.strictEqual(platform.getArch(processDeps), 'x64');
   });
 
@@ -87,29 +87,29 @@ describe('getPlatformInfo', () => {
 
     // First call throws, second returns alpine
     let callCount = 0;
-    const fsDeps: Partial<FsDeps> = {
+    const fsDeps: Partial<PlatformFsDeps> = {
       readFileSync: mock.fn((filePath: string) => {
         callCount++;
         if (callCount === 1) {
           throw new Error('File not found');
         }
         return Buffer.from('NAME="Alpine Linux"\nID=alpine');
-      }) as unknown as FsDeps['readFileSync'],
+      }) as unknown as PlatformPlatformFsDeps['readFileSync'],
     };
 
-    assert.strictEqual(platform.getSupportedOS(processDeps, fsDeps as FsDeps), 'alpine');
+    assert.strictEqual(platform.getSupportedOS(processDeps, fsDeps as PlatformFsDeps), 'alpine');
     assert.strictEqual(platform.getArch(processDeps), 'x64');
   });
 
   it('failed to detect alpine', () => {
     const processDeps = createMockProcessDeps({ platform: 'linux', arch: 'x64' });
-    const fsDeps: Partial<FsDeps> = {
+    const fsDeps: Partial<PlatformFsDeps> = {
       readFileSync: mock.fn(() => {
         throw new Error('File not found');
-      }) as unknown as FsDeps['readFileSync'],
+      }) as unknown as PlatformPlatformFsDeps['readFileSync'],
     };
 
-    assert.strictEqual(platform.getSupportedOS(processDeps, fsDeps as FsDeps), 'linux');
+    assert.strictEqual(platform.getSupportedOS(processDeps, fsDeps as PlatformFsDeps), 'linux');
     assert.strictEqual(platform.getArch(processDeps), 'x64');
 
     // Check that warning was logged - the message could be in arguments[0] or arguments[1]

test/unit/process.test.ts

@@ -20,18 +20,15 @@
 import { describe, it, mock } from 'node:test';
 import assert from 'node:assert';
 import { SCANNER_CLI_DEFAULT_BIN_NAME, WINDOWS_WHERE_EXE_PATH } from '../../src/constants';
-import { ProcessDeps } from '../../src/deps';
-import { locateExecutableFromPath } from '../../src/process';
+import { locateExecutableFromPath, ProcessProcessDeps } from '../../src/process';
 
 // Mock console.log to suppress output
 mock.method(console, 'log', () => {});
 
-function createMockProcessDeps(overrides: Partial<ProcessDeps> = {}): ProcessDeps {
+function createMockProcessDeps(overrides: Partial<ProcessProcessDeps> = {}): ProcessProcessDeps {
   return {
     platform: 'linux',
     arch: 'x64',
-    env: {},
-    cwd: () => '/test',
     ...overrides,
   };
 }