Add database and support-tickets tests, remove @ts-nocheck from email and property-views

src/database/cleanup.service.spec.ts

@@ -0,0 +1,40 @@
+import { CleanupService } from './cleanup.service';
+import { PrismaService } from './prisma.service';
+
+describe('CleanupService', () => {
+  let service: CleanupService;
+  let prisma: jest.Mocked<Partial<PrismaService>>;
+
+  beforeEach(() => {
+    prisma = {
+      blacklistedToken: {
+        findMany: jest.fn().mockResolvedValue([]),
+        deleteMany: jest.fn().mockResolvedValue({ count: 0 }),
+      } as any,
+      passwordResetToken: {
+        findMany: jest.fn().mockResolvedValue([]),
+        deleteMany: jest.fn().mockResolvedValue({ count: 0 }),
+      } as any,
+      session: {
+        findMany: jest.fn().mockResolvedValue([]),
+        deleteMany: jest.fn().mockResolvedValue({ count: 0 }),
+      } as any,
+      loginHistory: {
+        findMany: jest.fn().mockResolvedValue([]),
+        deleteMany: jest.fn().mockResolvedValue({ count: 0 }),
+      } as any,
+    };
+    service = new CleanupService(prisma as unknown as PrismaService);
+  });
+
+  it('performCleanup returns summary with totalDeleted of 0 when no records exist', async () => {
+    const summary = await service.performCleanup();
+    expect(summary.totalDeleted).toBe(0);
+    expect(summary.results).toHaveLength(4);
+  });
+
+  it('getLastSummary returns null before any cleanup run', () => {
+    const result = service.getLastSummary();
+    expect(result).toBeNull();
+  });
+});

src/email/email-webhook.controller.ts

@@ -1,4 +1,3 @@
-// @ts-nocheck
 
 import { Controller, Post, Body, Get, HttpCode, UseGuards } from '@nestjs/common';
 import { EmailService } from './email.service';

src/email/email.module.ts

@@ -1,4 +1,3 @@
-// @ts-nocheck
 
 import { Module } from '@nestjs/common';
 import { EmailService } from './email.service';

src/email/email.processor.ts

@@ -1,4 +1,3 @@
-// @ts-nocheck
 
 import { Processor, WorkerHost } from '@nestjs/bullmq';
 import { Job } from 'bullmq';
@@ -33,7 +32,8 @@ export class EmailProcessor extends WorkerHost {
         context,
       });
       this.logger.log(`Email sent successfully to ${to}`);
-    } catch (error) {
+    } catch (err: unknown) {
+      const error = err instanceof Error ? err : new Error(String(err));
       this.logger.error(`Failed to send email to ${to}: ${error.message}`, error.stack);
       throw error; // BullMQ will handle retries if configured
     }

src/email/email.service.ts

@@ -1,4 +1,3 @@
-// @ts-nocheck
 
 import { Injectable, Logger } from '@nestjs/common';
 import { ConfigService } from '@nestjs/config';
@@ -264,7 +263,7 @@ export class EmailService {
     if (options.language && options.template) {
       const lang = options.language;
       const i18nKey = `email.${options.template}`;
-      const translated = this.i18nService.translate(i18nKey, lang, options.context);
+      const translated = this.i18nService.translate(i18nKey, { userPreference: lang }, options.context);
       if (translated !== i18nKey) {
         options.subject = options.subject || translated;
       }
@@ -329,7 +328,8 @@ export class EmailService {
       );
 
       this.logger.log(`📧 Email to ${options.to} queued for subject: ${options.subject}`);
-    } catch (error) {
+    } catch (err: unknown) {
+      const error = err instanceof Error ? err : new Error(String(err));
       this.logger.error(`❌ Failed to queue email to ${options.to}: ${error.message}`);
       throw error;
     }
@@ -347,7 +347,7 @@ export class EmailService {
     });
 
     const language = user?.languagePreference || 'en';
-    const translated = this.i18nService.translate(templateKey, language, params);
+    const translated = this.i18nService.translate(templateKey, { userPreference: language }, params);
 
     await this.sendEmail({
       to,

src/property-views/property-views.controller.ts

@@ -1,4 +1,3 @@
-// @ts-nocheck
 
 import {
   BadRequestException,
@@ -13,6 +12,10 @@ import {
   UseGuards,
 } from '@nestjs/common';
 import { Request } from 'express';
+
+interface RequestWithAuth extends Request {
+  authUser?: { sub: string };
+}
 import { ApiBearerAuth, ApiTags } from '@nestjs/swagger';
 import { PropertyViewsService } from './property-views.service';
 import {

src/property-views/property-views.module.ts

@@ -1,4 +1,3 @@
-// @ts-nocheck
 
 import { Module } from '@nestjs/common';
 import { PropertyViewsController } from './property-views.controller';

src/support-tickets/support-tickets.service.spec.ts

@@ -0,0 +1,34 @@
+import { SupportTicketsService } from './support-tickets.service';
+import { PrismaService } from '../database/prisma.service';
+import { NotificationsService } from '../notifications/notifications.service';
+
+describe('SupportTicketsService', () => {
+  let service: SupportTicketsService;
+  let prisma: jest.Mocked<Partial<PrismaService>>;
+  let notifications: jest.Mocked<Partial<NotificationsService>>;
+
+  beforeEach(() => {
+    prisma = {
+      supportTicket: {
+        create: jest.fn().mockResolvedValue({ id: 'ticket-1', priority: 'HIGH', slaDeadline: new Date() }),
+        findUnique: jest.fn().mockResolvedValue(null),
+        findMany: jest.fn().mockResolvedValue([]),
+        update: jest.fn().mockResolvedValue({}),
+      } as any,
+    };
+    notifications = { sendNotification: jest.fn().mockResolvedValue(undefined) };
+    service = new SupportTicketsService(
+      prisma as unknown as PrismaService,
+      notifications as unknown as NotificationsService,
+    );
+  });
+
+  it('creates a ticket with correct SLA deadline for HIGH priority', async () => {
+    const result = await service.createTicket('user-1', {
+      subject: 'Test', description: 'Desc', priority: 'HIGH',
+    } as any);
+    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+    expect(prisma.supportTicket!.create).toHaveBeenCalled();
+    expect(result.id).toBe('ticket-1');
+  });
+});