rename redis-wrapper redis-adapter

Author: rlindner81

src/feature-toggles.js

@@ -17,7 +17,7 @@ const pathlib = require("path");
 const { readFile } = require("fs");
 const VError = require("verror");
 const yaml = require("yaml");
-const redis = require("./redis-wrapper");
+const redis = require("./redis-adapter");
 const { REDIS_INTEGRATION_MODE } = redis;
 const { CfEnv } = require("./shared/cf-env");
 const { Logger } = require("./shared/logger");

src/plugin.js

@@ -8,7 +8,7 @@ const pathlib = require("path");
 const cds = require("@sap/cds");
 const cdsPackage = require("@sap/cds/package.json");
 const toggles = require("./");
-const { closeMainClient, closeSubscriberClient } = require("./redis-wrapper");
+const { closeMainClient, closeSubscriberClient } = require("./redis-adapter");
 
 const FEATURE_KEY_REGEX = /\/fts\/([^\s/]+)$/;
 const FTS_AUTO_CONFIG = {

src/redis-wrapper.js src/redis-adapter.jssrc/redis-wrapper.js renamed to src/redis-adapter.js

@@ -1,8 +1,8 @@
 /**
- * RedisWrapper is a thin wrapper around the node redis module.
+ * redis-adapter is a thin wrapper around the node redis module.
  *
  * @see {@link https://www.npmjs.com/package/redis|redis on npm}
- * @module redis-wrapper
+ * @module redis-adapter
  */
 "use strict";
 
@@ -14,8 +14,8 @@ const { HandlerCollection } = require("./shared/handler-collection");
 const { Semaphore } = require("./shared/semaphore");
 const { tryJsonParse } = require("./shared/static");
 
-const COMPONENT_NAME = "/RedisWrapper";
-const VERROR_CLUSTER_NAME = "RedisWrapperError";
+const COMPONENT_NAME = "/RedisAdapter";
+const VERROR_CLUSTER_NAME = "RedisAdapterError";
 
 const INTEGRATION_MODE = Object.freeze({
   CF_REDIS_CLUSTER: "CF_REDIS_CLUSTER",

src/service/feature-service.js

@@ -7,7 +7,7 @@
 
 const VError = require("verror");
 const toggles = require("../");
-const redis = require("../redis-wrapper");
+const redis = require("../redis-adapter");
 const { Logger } = require("../shared/logger");
 
 const COMPONENT_NAME = "/FeatureService";

test/__mocks__/redis-wrapper.js test/__mocks__/redis-adapter.jstest/__mocks__/redis-wrapper.js renamed to test/__mocks__/redis-adapter.js

@@ -1,6 +1,6 @@
 "use strict";
 
-const { REDIS_INTEGRATION_MODE } = jest.requireActual("../../src/redis-wrapper");
+const { REDIS_INTEGRATION_MODE } = jest.requireActual("../../src/redis-adapter");
 
 const redisKey = "feature-key";
 let mockRedisState = {};

test/feature-toggles.test.js

@@ -15,7 +15,7 @@ const { stateFromInfo } = require("../__common__/from-info");
 const { FEATURE, mockConfig: config } = require("../__common__/mockdata");
 
 let featureTogglesLoggerSpy;
-let redisWrapperLoggerSpy;
+let redisAdapterLoggerSpy;
 let toggles;
 
 describe("local integration test", () => {
@@ -31,11 +31,11 @@ describe("local integration test", () => {
       error: jest.spyOn(featureTogglesModule._._getLogger(), "error"),
     };
 
-    const redisWrapper = require("../../src/redis-wrapper");
-    redisWrapperLoggerSpy = {
-      info: jest.spyOn(redisWrapper._._getLogger(), "info"),
-      warning: jest.spyOn(redisWrapper._._getLogger(), "warning"),
-      error: jest.spyOn(redisWrapper._._getLogger(), "error"),
+    const redisAdapter = require("../../src/redis-adapter");
+    redisAdapterLoggerSpy = {
+      info: jest.spyOn(redisAdapter._._getLogger(), "info"),
+      warning: jest.spyOn(redisAdapter._._getLogger(), "warning"),
+      error: jest.spyOn(redisAdapter._._getLogger(), "error"),
     };
   });
 
@@ -292,8 +292,8 @@ describe("local integration test", () => {
       `);
       expect(featureTogglesLoggerSpy.warning).toHaveBeenCalledTimes(0);
       expect(featureTogglesLoggerSpy.error).toHaveBeenCalledTimes(0);
-      expect(redisWrapperLoggerSpy.info).toHaveBeenCalledTimes(0);
-      expect(redisWrapperLoggerSpy.warning).toHaveBeenCalledTimes(0);
+      expect(redisAdapterLoggerSpy.info).toHaveBeenCalledTimes(0);
+      expect(redisAdapterLoggerSpy.warning).toHaveBeenCalledTimes(0);
     });
 
     test("getFeatureValue, changeFeatureValue without scopes", async () => {
@@ -322,9 +322,9 @@ describe("local integration test", () => {
       `);
       expect(featureTogglesLoggerSpy.warning).toHaveBeenCalledTimes(0);
       expect(featureTogglesLoggerSpy.error).toHaveBeenCalledTimes(0);
-      expect(redisWrapperLoggerSpy.info).toHaveBeenCalledTimes(0);
-      expect(redisWrapperLoggerSpy.warning.mock.calls).toMatchSnapshot();
-      expect(redisWrapperLoggerSpy.error).toHaveBeenCalledTimes(0);
+      expect(redisAdapterLoggerSpy.info).toHaveBeenCalledTimes(0);
+      expect(redisAdapterLoggerSpy.warning.mock.calls).toMatchSnapshot();
+      expect(redisAdapterLoggerSpy.error).toHaveBeenCalledTimes(0);
     });
 
     test("getFeatureValue with bad scopes", async () => {
@@ -459,9 +459,9 @@ describe("local integration test", () => {
       `);
       expect(featureTogglesLoggerSpy.warning).toHaveBeenCalledTimes(0);
       expect(featureTogglesLoggerSpy.error).toHaveBeenCalledTimes(0);
-      expect(redisWrapperLoggerSpy.info).toHaveBeenCalledTimes(0);
-      expect(redisWrapperLoggerSpy.warning.mock.calls).toMatchSnapshot();
-      expect(redisWrapperLoggerSpy.error).toHaveBeenCalledTimes(0);
+      expect(redisAdapterLoggerSpy.info).toHaveBeenCalledTimes(0);
+      expect(redisAdapterLoggerSpy.warning.mock.calls).toMatchSnapshot();
+      expect(redisAdapterLoggerSpy.error).toHaveBeenCalledTimes(0);
     });
 
     test("getFeatureValue, changeFeatureValue with scopes and clearSubScopes, resetFeatureValue", async () => {
@@ -556,9 +556,9 @@ describe("local integration test", () => {
       `);
       expect(featureTogglesLoggerSpy.warning).toHaveBeenCalledTimes(0);
       expect(featureTogglesLoggerSpy.error).toHaveBeenCalledTimes(0);
-      expect(redisWrapperLoggerSpy.info).toHaveBeenCalledTimes(0);
-      expect(redisWrapperLoggerSpy.warning.mock.calls).toMatchSnapshot();
-      expect(redisWrapperLoggerSpy.error).toHaveBeenCalledTimes(0);
+      expect(redisAdapterLoggerSpy.info).toHaveBeenCalledTimes(0);
+      expect(redisAdapterLoggerSpy.warning.mock.calls).toMatchSnapshot();
+      expect(redisAdapterLoggerSpy.error).toHaveBeenCalledTimes(0);
     });
 
     test("validateFeatureValue with invalid scopes", async () => {