fix(test): make tsconfig.spec typecheck pass with zero errors

Widen tsconfig.spec rootDir so test/ files are covered, then repair
every type error that surfaced:

- Make query DTO page/limit optional to match their @IsOptional()
  decorators (services already default them with ?? 1)
- Add I18nService arg to AdminController construction in specs
- Add missing tier to AuthUserPayload literals in e2e specs
- Type execFile spy against Node's overloads; wire the
  CacheMonitoringService into CacheHeadersInterceptor spec
- Break readStream circular inference, fix implicit-any mock params
  and getResponse() string|object access
- Track auth.service's family count() query in the reuse spec
  (service stopped fetching rows for the log line)

🤖 Generated with Codebuff
Co-Authored-By: Codebuff <noreply@codebuff.com>

Author: lovesmilesmall-hue

src/admin/dto/admin.dto.ts

@@ -37,14 +37,14 @@ export class AdminUsersQueryDto {
   @Type(() => Number)
   @IsInt()
   @Min(1)
-  page: number = 1;
+  page?: number = 1;
 
   @IsOptional()
   @Type(() => Number)
   @IsInt()
   @Min(1)
   @Max(100)
-  limit: number = 20;
+  limit?: number = 20;
 
   @IsOptional()
   @IsString()
@@ -84,14 +84,14 @@ export class ModerationQueueQueryDto {
   @Type(() => Number)
   @IsInt()
   @Min(1)
-  page: number = 1;
+  page?: number = 1;
 
   @IsOptional()
   @Type(() => Number)
   @IsInt()
   @Min(1)
   @Max(100)
-  limit: number = 20;
+  limit?: number = 20;
 
   @IsOptional()
   @IsString()
@@ -141,14 +141,14 @@ export class TransactionMonitoringQueryDto {
   @Type(() => Number)
   @IsInt()
   @Min(1)
-  page: number = 1;
+  page?: number = 1;
 
   @IsOptional()
   @Type(() => Number)
   @IsInt()
   @Min(1)
   @Max(100)
-  limit: number = 20;
+  limit?: number = 20;
 
   @IsOptional()
   @IsString()

src/fraud/dto/fraud.dto.ts

@@ -43,14 +43,14 @@ export class FraudAlertsQueryDto {
   @Type(() => Number)
   @IsInt()
   @Min(1)
-  page: number = 1;
+  page?: number = 1;
 
   @IsOptional()
   @Type(() => Number)
   @IsInt()
   @Min(1)
   @Max(100)
-  limit: number = 20;
+  limit?: number = 20;
 }
 
 export class ReviewFraudAlertDto {

test/admin/admin.controller.spec.ts

@@ -5,6 +5,7 @@ import { AdminService } from '../../src/admin/admin.service';
 import { EmailService } from '../../src/email/email.service';
 import { ArchiveService } from '../../src/archive/archive.service';
 import { CleanupService } from '../../src/database/cleanup.service';
+import { I18nService } from '../../src/i18n/i18n.service';
 import { RolesGuard } from '../../src/auth/guards/roles.guard';
 import { ROLES_KEY } from '../../src/auth/decorators/roles.decorator';
 import { AuthUserPayload } from '../../src/auth/types/auth-user.type';
@@ -69,6 +70,7 @@ describe('AdminController - role enforcement', () => {
       {} as unknown as EmailService,
       {} as unknown as ArchiveService,
       {} as unknown as CleanupService,
+      {} as unknown as I18nService,
     );
 
     reflector = new Reflector();

test/backup/backup.service.e2e-spec.ts

@@ -298,16 +298,21 @@ describe('BackupService e2e — schedule, status, and restore flows', () => {
 
     it('handles restore failure correctly and updates database status', async () => {
       // Mock execFile to simulate failure
+      type ExecFileParams = Parameters<typeof childProcess.execFile>;
       const execFileSpy = jest
         .spyOn(childProcess, 'execFile')
         .mockImplementationOnce(
           (
-            cmd: string,
-            args: string[],
-            callback: (error: any, stdout: string, stderr: string) => void,
-          ) => {
-            callback(new Error('psql command failed'), '', 'psql: error: connection failed');
-            return {};
+            _file: ExecFileParams[0],
+            _args: ExecFileParams[1],
+            options: ExecFileParams[2],
+            callback: ExecFileParams[3],
+          ): ReturnType<typeof childProcess.execFile> => {
+            // promisify() invokes execFile(file, args, callback) — the callback can
+            // arrive in the options slot, so accept both call shapes.
+            const cb = typeof options === 'function' ? options : callback;
+            cb?.(new Error('psql command failed'), '', 'psql: error: connection failed');
+            return {} as ReturnType<typeof childProcess.execFile>;
           },
         );
 

test/cache/cache-stats.spec.ts

@@ -53,7 +53,7 @@ describe('CacheStatsController', () => {
 
 describe('CacheHeadersInterceptor', () => {
   it('sets cache headers on response', (done) => {
-    const interceptor = new CacheHeadersInterceptor();
+    const interceptor = new CacheHeadersInterceptor(new CacheMonitoringService());
     const mockRes = { setHeader: jest.fn(), getHeader: jest.fn() };
     const mockReq = { path: '/' };
     const ctx = {

test/e2e/auth-property.e2e-spec.ts

@@ -8,7 +8,6 @@ import { PrismaService } from '../../src/database/prisma.service';
 class FakePrismaService {
   users = new Map<string, any>();
   properties = new Map<string, any>();
-  blacklistedToken = new Map<string, any>();
 
   async $connect() {}
   async $disconnect() {}

test/e2e/duplicate-detection-merge.e2e.spec.ts

@@ -12,6 +12,7 @@ import { DuplicateDetectionService } from '../../src/duplicate-detection/duplica
 import { FraudService } from '../../src/fraud/fraud.service';
 import { JwtAuthGuard } from '../../src/auth/guards/jwt-auth.guard';
 import { AuthUserPayload } from '../../src/auth/types/auth-user.type';
+import { UserTier } from '../../src/types/prisma.types';
 import { PrismaService } from '../../src/database/prisma.service';
 
 const OWNER_ID = '11111111-1111-4111-a111-111111111111';
@@ -29,6 +30,7 @@ class MockJwtAuthGuard implements CanActivate {
       sub: OWNER_ID,
       email: 'owner@example.com',
       role: 'USER' as any,
+      tier: UserTier.FREE,
       type: 'access',
     };
     requestContext.user = user;

test/e2e/property-crud-search-favorite.e2e.spec.ts

@@ -30,6 +30,7 @@ import { AuthService } from '../../src/auth/auth.service';
 import { JwtAuthGuard } from '../../src/auth/guards/jwt-auth.guard';
 import { RolesGuard } from '../../src/auth/guards/roles.guard';
 import { AuthUserPayload } from '../../src/auth/types/auth-user.type';
+import { UserTier } from '../../src/types/prisma.types';
 
 const OWNER_ID = '11111111-1111-4111-a111-111111111111';
 
@@ -45,6 +46,7 @@ class MockJwtAuthGuard implements CanActivate {
       sub: OWNER_ID,
       email: 'owner@example.com',
       role: role as any,
+      tier: UserTier.FREE,
       type: 'access',
     };
     req.user = user;

test/e2e/sessions-management.e2e.spec.ts

@@ -26,6 +26,7 @@ import { SessionsService } from '../../src/sessions/sessions.service';
 import { ConfigService } from '@nestjs/config';
 import { JwtAuthGuard } from '../../src/auth/guards/jwt-auth.guard';
 import { AuthUserPayload } from '../../src/auth/types/auth-user.type';
+import { UserTier } from '../../src/types/prisma.types';
 
 const TEST_USER_ID = 'sessions-user-001';
 
@@ -38,6 +39,7 @@ class MockJwtAuthGuard implements CanActivate {
       sub: TEST_USER_ID,
       email: 'sessions@example.com',
       role: 'USER',
+      tier: UserTier.FREE,
       type: 'access',
     };
     req.user = user;

test/e2e/user-registration-login.e2e.spec.ts

@@ -27,6 +27,7 @@ import { SessionsService } from '../../src/sessions/sessions.service';
 import { AuthService } from '../../src/auth/auth.service';
 import { JwtAuthGuard } from '../../src/auth/guards/jwt-auth.guard';
 import { AuthUserPayload } from '../../src/auth/types/auth-user.type';
+import { UserTier } from '../../src/types/prisma.types';
 
 const TEST_USER_ID = 'test-user-profile-001';
 
@@ -41,6 +42,7 @@ class MockJwtAuthGuard implements CanActivate {
       sub: TEST_USER_ID,
       email: 'user@example.com',
       role: 'USER',
+      tier: UserTier.FREE,
       type: 'access',
     };
     req.user = user;