@onlineapps/conn-base-cache 1.0.0 → 1.0.2

package/package.json

@@ -1,12 +1,12 @@
 {
   "name": "@onlineapps/conn-base-cache",
-  "version": "1.0.0",
+  "version": "1.0.2",
   "description": "Redis cache connector with TTL, invalidation, and namespace support for OA Drive microservices",
   "main": "src/index.js",
   "scripts": {
     "test": "jest",
-    "test:unit": "jest test/unit",
-    "test:integration": "jest test/integration",
+    "test:unit": "jest tests/unit",
+    "test:integration": "jest tests/integration",
     "test:watch": "jest --watch",
     "docs": "jsdoc2md --files src/**/*.js > API.md"
   },

package/src/index.js

@@ -68,10 +68,21 @@ class CacheConnector {
    * });
    */
   constructor(config = {}) {
+    // Host and port are REQUIRED - no fallbacks (fail-fast principle)
+    const host = config.host || process.env.REDIS_HOST;
+    const port = config.port || process.env.REDIS_PORT;
+    
+    if (!host) {
+      throw new Error('[CacheConnector] Missing required config: host (set REDIS_HOST env or pass config.host)');
+    }
+    if (!port) {
+      throw new Error('[CacheConnector] Missing required config: port (set REDIS_PORT env or pass config.port)');
+    }
+
     // Configuration
     this.config = {
-      host: config.host || process.env.REDIS_HOST || 'localhost',
-      port: config.port || process.env.REDIS_PORT || 6379,
+      host,
+      port: parseInt(port, 10),
       password: config.password || process.env.REDIS_PASSWORD,
       db: config.db || process.env.REDIS_DB || 0,
       keyPrefix: 'cache:', // ALWAYS use cache: prefix
@@ -135,10 +146,32 @@ class CacheConnector {
    * await cache.connect();
    */
   async connect() {
-    if (this.connected || this.connecting) {
+    // If already connected, return success
+    if (this.connected) {
       return true;
     }
 
+    // If connection is in progress, wait for it to complete
+    if (this.connecting) {
+      return new Promise((resolve, reject) => {
+        const checkInterval = setInterval(() => {
+          if (this.connected) {
+            clearInterval(checkInterval);
+            resolve(true);
+          } else if (!this.connecting) {
+            clearInterval(checkInterval);
+            reject(new Error('Connection attempt failed'));
+          }
+        }, 100);
+
+        // Timeout after 5 seconds
+        setTimeout(() => {
+          clearInterval(checkInterval);
+          reject(new Error('Connection timeout'));
+        }, 5000);
+      });
+    }
+
     this.connecting = true;
 
     try {

package/{test → tests}/component/cache.component.test.js

@@ -109,7 +109,7 @@ class RedisSimulator {
   }
 }
 
-describe('CacheConnector - Component Tests', () => {
+describe('CacheConnector - Component Tests @component', () => {
   let cacheConnector;
   let redisSimulator;
 

package/{test → tests}/integration/cache.integration.test.js

@@ -11,7 +11,7 @@ const SKIP_INTEGRATION = process.env.SKIP_INTEGRATION === 'true';
 const REDIS_HOST = process.env.REDIS_HOST || 'localhost';
 const REDIS_PORT = process.env.REDIS_PORT || 6379;
 
-describe('CacheConnector - Integration Tests', () => {
+describe('CacheConnector - Integration Tests @integration', () => {
   if (SKIP_INTEGRATION) {
     it.skip('Skipping integration tests (SKIP_INTEGRATION=true)', () => {});
     return;

package/{test → tests}/unit/CacheConnector.edge.test.js

@@ -9,7 +9,7 @@ jest.mock('ioredis');
 const Redis = require('ioredis');
 const CacheConnector = require('../../src/index');
 
-describe('CacheConnector - Edge Cases & Full Coverage', () => {
+describe('CacheConnector - Edge Cases & Full Coverage @unit', () => {
   let cacheConnector;
   let mockRedisClient;
 

package/{test → tests}/unit/CacheConnector.test.js

@@ -9,7 +9,7 @@ jest.mock('ioredis');
 const Redis = require('ioredis');
 const CacheConnector = require('../../src/index');
 
-describe('CacheConnector - Unit Tests', () => {
+describe('CacheConnector - Unit Tests @unit', () => {
   let cacheConnector;
   let mockRedisClient;