Revert "feat(health-indicator): add Redis client dependency and integration tests"

This reverts commit f2db41c.

Author: CSenshi

packages/health-indicator/package.json

@@ -59,10 +59,8 @@
     "redis": "^5.0.0"
   },
   "devDependencies": {
-    "@nestjs-redis/client": "0.10.1",
     "@nestjs/testing": "^11.0.0",
-    "redis": "^5.0.0",
-    "supertest": "^7.1.4"
+    "redis": "^5.0.0"
   },
   "engines": {
     "node": ">=18.0.0",

packages/health-indicator/src/lib/health.indicator.int.spec.ts

@@ -1,15 +1,8 @@
-import { Controller, Get, Module } from '@nestjs/common';
-import {
-  HealthCheck,
-  HealthCheckResult,
-  HealthCheckService,
-  TerminusModule,
-} from '@nestjs/terminus';
+import { HealthIndicatorService } from '@nestjs/terminus';
 import { Test, TestingModule } from '@nestjs/testing';
-import { InjectRedis, Redis, RedisModule } from '@nestjs-redis/client';
 import { createClient } from 'redis';
-import request from 'supertest';
 import { RedisHealthIndicator } from './health.indicator';
+import { Redis } from './interfaces';
 
 // These tests require a running Redis instance
 const REDIS_URL = process.env.REDIS_URL || 'redis://localhost:6379';
@@ -20,8 +13,22 @@ describe('RedisHealthIndicator Integration Tests', () => {
 
   beforeAll(async () => {
     const module: TestingModule = await Test.createTestingModule({
-      imports: [TerminusModule],
-      providers: [RedisHealthIndicator],
+      providers: [
+        RedisHealthIndicator,
+        {
+          provide: HealthIndicatorService,
+          useValue: {
+            check: jest.fn().mockReturnValue({
+              up: jest.fn().mockImplementation((data) => ({
+                redis: { status: 'up', ...data },
+              })),
+              down: jest.fn().mockImplementation((data) => ({
+                redis: { status: 'down', ...data },
+              })),
+            }),
+          },
+        },
+      ],
     }).compile();
 
     healthIndicator = module.get<RedisHealthIndicator>(RedisHealthIndicator);
@@ -78,66 +85,4 @@ describe('RedisHealthIndicator Integration Tests', () => {
       }
     }, 15000);
   });
-
-  describe('Full RedisModule Integration', () => {
-    it('should integrate with the full RedisModule', async () => {
-      @Controller('health')
-      class HealthController {
-        constructor(
-          private health: HealthCheckService,
-          private redis: RedisHealthIndicator,
-          @InjectRedis() private readonly redisClient: Redis,
-        ) {}
-
-        @Get()
-        @HealthCheck()
-        check(): Promise<HealthCheckResult> {
-          return this.health.check([
-            () => this.redis.isHealthy('redis', { client: this.redisClient }),
-          ]);
-        }
-      }
-
-      @Module({
-        imports: [
-          RedisModule.forRoot({
-            type: 'client',
-            options: { url: process.env.REDIS_URL },
-          }),
-          TerminusModule,
-        ],
-        controllers: [HealthController],
-        providers: [RedisHealthIndicator],
-      })
-      class HealthModule {}
-
-      @Module({
-        imports: [HealthModule],
-        controllers: [],
-        providers: [],
-      })
-      class AppModule {}
-
-      const module: TestingModule = await Test.createTestingModule({
-        imports: [AppModule],
-      }).compile();
-
-      const app = module.createNestApplication();
-      await app.init();
-
-      request(app.getHttpServer())
-        .get('/health')
-        .expect(200)
-        .expect({
-          data: {
-            status: 'ok',
-            info: { redis: { status: 'up' } },
-            error: {},
-            details: { redis: { status: 'up' } },
-          },
-        });
-
-      await app.close();
-    });
-  });
 });