chore: Update test scripts in package.json and CI workflow to include unit and integration tests

Author: mt-ks

.github/workflows/ci.yaml

@@ -41,7 +41,7 @@ jobs:
       - name: Run library tests
         run: |
           npm ci
-          npm run test:lib
+          npm run test:unit
   test-integration:
     name: Test (Integration)
     needs: [test-library]
@@ -67,7 +67,7 @@ jobs:
       - name: Run integration tests
         run: |
           npm ci
-          npm run test:lib
+          npm run test:integration
         env:
           REDIS_PORT: 6379
           REDIS_CLUSTER_PORT: 7000

package.json

@@ -46,8 +46,10 @@
     "format:code": "xo --fix",
     "format:rest": "prettier --write .",
     "format": "run-s format:*",
+    "test:unit": "jest test/store-test.ts",
+    "test:integration": "jest test/redis-integration-test.ts",
     "test:lib": "jest",
-    "test": "run-s lint test:*",
+    "test": "run-s lint test:lib",
     "pre-commit": "lint-staged",
     "prepare": "run-s compile && husky install config/husky"
   },

test/redis-integration-test.ts

@@ -148,6 +148,20 @@ describe('Redis Integration Tests', () => {
 			store = new RedisStore({
 				async sendCommandCluster(details: SendCommandClusterDetails) {
 					const { command } = details
+
+					// If SCRIPT LOAD, send to all master nodes
+					if (command[0] === 'SCRIPT' && command[1] === 'LOAD') {
+						const nodes = client.nodes('master')
+						await Promise.all(
+							nodes.map(async (node) =>
+								node.call(command[0], ...command.slice(1)),
+							),
+						)
+						// Return the result from one of them (they should be identical)
+						const result = await client.call(command[0], ...command.slice(1))
+						return result as RedisReply
+					}
+
 					const result = await client.call(command[0], ...command.slice(1))
 					return result as RedisReply
 				},
@@ -203,6 +217,19 @@ describe('Redis Integration Tests', () => {
 			const shortStore = new RedisStore({
 				async sendCommandCluster(details: SendCommandClusterDetails) {
 					const { command } = details
+
+					// If SCRIPT LOAD, send to all master nodes
+					if (command[0] === 'SCRIPT' && command[1] === 'LOAD') {
+						const nodes = client.nodes('master')
+						await Promise.all(
+							nodes.map(async (node) =>
+								node.call(command[0], ...command.slice(1)),
+							),
+						)
+						const result = await client.call(command[0], ...command.slice(1))
+						return result as RedisReply
+					}
+
 					const result = await client.call(command[0], ...command.slice(1))
 					return result as RedisReply
 				},