feat: add unit testing

Author: sourcecode71

eslint.config.mjs

@@ -6,7 +6,7 @@ import tseslint from 'typescript-eslint';
 
 export default tseslint.config(
   {
-    ignores: ['eslint.config.mjs'],
+    ignores: ['eslint.config.mjs', 'jest.config.js', 'dist/'],
   },
   eslint.configs.recommended,
   ...tseslint.configs.recommendedTypeChecked,
@@ -28,7 +28,7 @@ export default tseslint.config(
     rules: {
       '@typescript-eslint/no-explicit-any': 'off',
       '@typescript-eslint/no-floating-promises': 'warn',
-      '@typescript-eslint/no-unsafe-argument': 'warn'
+      '@typescript-eslint/no-unsafe-argument': 'warn',
     },
   },
 );

package.json

@@ -16,9 +16,7 @@
     "lint": "eslint \"{src,apps,libs,test}/**/*.ts\" --fix",
     "test": "jest",
     "test:watch": "jest --watch",
-    "test:cov": "jest --coverage",
-    "test:debug": "node --inspect-brk -r tsconfig-paths/register -r ts-node/register node_modules/.bin/jest --runInBand",
-    "test:e2e": "jest --config ./test/jest-e2e.json"
+    "test:cov": "jest --coverage"
   },
   "dependencies": {
     "@nestjs/common": "^11.0.1",
@@ -68,22 +66,5 @@
     "tsconfig-paths": "^4.2.0",
     "typescript": "^5.7.3",
     "typescript-eslint": "^8.20.0"
-  },
-  "jest": {
-    "moduleFileExtensions": [
-      "js",
-      "json",
-      "ts"
-    ],
-    "rootDir": "src",
-    "testRegex": ".*\\.spec\\.ts$",
-    "transform": {
-      "^.+\\.(t|j)s$": "ts-jest"
-    },
-    "collectCoverageFrom": [
-      "**/*.(t|j)s"
-    ],
-    "coverageDirectory": "../coverage",
-    "testEnvironment": "node"
   }
 }

test/app.e2e-spec.ts

@@ -0,0 +1,127 @@
+import { Test, TestingModule } from '@nestjs/testing';
+import { NotificationController } from '../../src/notification/presentation/controllers/notification.controller';
+import { NotificationFactory } from '../../src/notification/application/factories/notification.factory';
+import { INotificationRepository } from '../../src/notification/domain/interfaces/notification-repository.interface';
+import { LoggerServiceFile } from '../../src/logger/services/logger.service.file';
+import { LoggerServiceDb } from '../../src/logger/services/logger.service.db';
+import { SendNotificationDto } from '../../src/notification/presentation/dtos/send-notification.dto';
+import { Notification } from '../../src/notification/domain/entities/notification.entity';
+import { NotificationChannel, NotificationType } from '../../src/config/notification.config';
+
+describe('NotificationController', () => {
+  let controller: NotificationController;
+  let factory: NotificationFactory;
+  let repository: INotificationRepository;
+
+  const mockNotification = new Notification(
+    'test@example.com',
+    'Test Subject',
+    'Test Body',
+    NotificationChannel.EMAIL,
+    NotificationType.ADMISSION_ID,
+    new Date(),
+  );
+
+  const mockSendDto: SendNotificationDto = {
+    recipient: 'test@example.com',
+    subject: 'Test Subject',
+    body: 'Test Body',
+    mediaType: NotificationChannel.EMAIL,
+    notificationType: NotificationType.ADMISSION_ID,
+  };
+
+  const mockStrategy = {
+    send: jest.fn().mockResolvedValue(undefined),
+  };
+
+  beforeEach(async () => {
+    const module: TestingModule = await Test.createTestingModule({
+      controllers: [NotificationController],
+      providers: [
+        {
+          provide: NotificationFactory,
+          useValue: {
+            createStrategy: jest.fn().mockReturnValue(mockStrategy),
+          },
+        },
+        {
+          provide: 'INotificationRepository',
+          useValue: {
+            save: jest.fn().mockResolvedValue(mockNotification),
+            findAll: jest.fn().mockResolvedValue([mockNotification]),
+            count: jest.fn().mockResolvedValue(1),
+          },
+        },
+        {
+          provide: LoggerServiceFile,
+          useValue: {
+            log: jest.fn(),
+            error: jest.fn(),
+          },
+        },
+        {
+          provide: LoggerServiceDb,
+          useValue: {
+            log: jest.fn(),
+            error: jest.fn().mockResolvedValue(undefined),
+          },
+        },
+      ],
+    }).compile();
+
+    controller = module.get<NotificationController>(NotificationController);
+    factory = module.get<NotificationFactory>(NotificationFactory);
+    repository = module.get<INotificationRepository>('INotificationRepository');
+
+    // Use spies to avoid unbound method errors
+    jest.spyOn(factory, 'createStrategy');
+    jest.spyOn(repository, 'save');
+    jest.spyOn(mockStrategy, 'send');
+  });
+
+  describe('send', () => {
+    it('should send notification and log successfully', async () => {
+      await controller.send(mockSendDto);
+
+      expect(factory.createStrategy).toHaveBeenCalledWith(mockSendDto.mediaType);
+      expect(repository.save).toHaveBeenCalledWith(expect.any(Notification));
+      expect(mockStrategy.send).toHaveBeenCalledWith(expect.any(Notification));
+    });
+
+    it('should throw error if strategy send fails', async () => {
+      jest.spyOn(factory, 'createStrategy').mockReturnValue({
+        send: jest.fn().mockRejectedValue(new Error('Send failed')),
+      });
+
+      await expect(controller.send(mockSendDto)).rejects.toThrow('Send failed');
+    });
+  });
+
+  describe('getNotificationHistory', () => {
+    it('should return notification history with default pagination', async () => {
+      jest.spyOn(repository, 'findAll');
+      const result = await controller.getNotificationHistory(1, 10);
+
+      expect(repository.findAll).toHaveBeenCalledWith(0, 10);
+      expect(result).toEqual({
+        data: [mockNotification],
+        total: 1,
+        page: 1,
+        totalPages: 1,
+      });
+    });
+
+    it('should handle custom pagination', async () => {
+      jest.spyOn(repository, 'findAll').mockResolvedValue([mockNotification, mockNotification]);
+      const result = await controller.getNotificationHistory(2, 5);
+
+      expect(repository.findAll).toHaveBeenCalledWith(5, 5);
+      expect(result).toEqual({
+        data: [mockNotification, mockNotification],
+        total: 2,
+        page: 2,
+        totalPages: 1,
+      });
+    });
+  });
+});

test/jest-e2e.json

@@ -17,5 +17,7 @@
     "noImplicitAny": true,
     "strictBindCallApply": true,
     "noFallthroughCasesInSwitch": true
-  }
-}
+  },
+  "include": ["src/**/*", "test/**/*"],
+  "exclude": ["node_modules", "dist"]
+}