feat: implement docker-compose and adjust tests workflow to use docker

.github/workflows/tests.yml

@@ -14,6 +14,12 @@ jobs:
       matrix:
         node-version: [10, 12, 14]
     steps:
+      - name: Initiate docker containers
+        run: docker-compose -f docker-compose.yml up -d
+      - name: Check running containers
+        run: docker ps -a
+      - name: Check docker logs
+        run: docker logs redis
       - uses: actions/checkout@v2
       - name: Setup NodeJS v${{ matrix.node-version }}
         uses: actions/setup-node@v1

docker-compose.yml

@@ -0,0 +1,7 @@
+version: '3'
+services:
+  redis:
+    image: redis
+    container_name: redis
+    ports:
+      - '6379:6379'

package.json

@@ -38,6 +38,7 @@
   "devDependencies": {
     "@commitlint/cli": "^12.0.0",
     "@commitlint/config-conventional": "^12.0.0",
+    "@golevelup/ts-jest": "^0.3.1",
     "@nestjs/cli": "^7.5.5",
     "@nestjs/common": "^7.6.13",
     "@nestjs/core": "^7.6.13",

src/throttler-storage-redis.service.ts

@@ -6,8 +6,14 @@ import { ThrottlerStorageRedis } from './throttler-storage-redis.interface';
 export class ThrottlerStorageRedisService implements ThrottlerStorageRedis {
   redis: Redis.Redis;
 
-  constructor(options?: Redis.RedisOptions) {
-    this.redis = new Redis(options);
+  constructor(redis?: Redis.Redis);
+  constructor(options?: Redis.RedisOptions);
+  constructor(redisOrOptions?: Redis.Redis | Redis.RedisOptions) {
+    if (redisOrOptions instanceof Redis) {
+      this.redis = redisOrOptions;
+    } else {
+      this.redis = new Redis(redisOrOptions);
+    }
   }
 
   async getRecord(key: string): Promise<number[]> {
@@ -16,6 +22,6 @@ export class ThrottlerStorageRedisService implements ThrottlerStorageRedis {
   }
 
   async addRecord(key: string, ttl: number): Promise<void> {
-    this.redis.set(`${key}:${Date.now() + ttl * 1000}`, ttl, 'EX', ttl);
+    await this.redis.set(`${key}:${Date.now() + ttl * 1000}`, ttl, 'EX', ttl);
   }
 }

test/app/controllers/controller.module.ts

@@ -1,5 +1,5 @@
 import { Module } from '@nestjs/common';
-import { ThrottlerModule } from '@nestjs/throttler';
+import { ThrottlerModule, ThrottlerStorageService } from '@nestjs/throttler';
 import { AppService } from '../app.service';
 import { AppController } from './app.controller';
 import { DefaultController } from './default.controller';
@@ -11,6 +11,7 @@ import { LimitController } from './limit.controller';
       limit: 5,
       ttl: 60,
       ignoreUserAgents: [/throttler-test/g],
+      storage: new ThrottlerStorageService(),
     }),
   ],
   controllers: [AppController, DefaultController, LimitController],