@applicaster/zapp-react-native-utils 16.0.0-alpha.1593888345 → 16.0.0-alpha.1733304298

package/appUtils/contextKeysManager/__tests__/getKey/success/fullSearch.test.ts

@@ -1,4 +1,4 @@
-import { ContextKeysManager, NOTHING } from "../../..";
+import { ContextKeysManager } from "../../..";
 
 describe("Context Keys Manager - getKey", () => {
   it("returns value if found in session-storage", async () => {
@@ -35,10 +35,6 @@ describe("Context Keys Manager - getKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const getKeyByReference = jest
-      .spyOn(contextManager, "getKeyByReference")
-      .mockResolvedValueOnce(NOTHING);
-
     // run
     const result = await contextManager.getKey(key);
 
@@ -47,8 +43,6 @@ describe("Context Keys Manager - getKey", () => {
 
     expect(mockedLogger.warn).not.toHaveBeenCalled();
 
-    expect(getKeyByReference).toHaveBeenCalledWith("key", "namespace");
-
     expect(mockedSessionStorage.getItem).toHaveBeenCalledWith(
       "key",
       "namespace"
@@ -92,10 +86,6 @@ describe("Context Keys Manager - getKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const getKeyByReference = jest
-      .spyOn(contextManager, "getKeyByReference")
-      .mockResolvedValueOnce(NOTHING);
-
     // run
     const result = await contextManager.getKey(key);
 
@@ -104,8 +94,6 @@ describe("Context Keys Manager - getKey", () => {
 
     expect(mockedLogger.warn).not.toHaveBeenCalled();
 
-    expect(getKeyByReference).toHaveBeenCalledWith("key", "namespace");
-
     expect(mockedSessionStorage.getItem).toHaveBeenCalledWith(
       "key",
       "namespace"
@@ -149,10 +137,6 @@ describe("Context Keys Manager - getKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const getKeyByReference = jest
-      .spyOn(contextManager, "getKeyByReference")
-      .mockResolvedValueOnce(NOTHING);
-
     // run
     const result = await contextManager.getKey(key);
 
@@ -161,8 +145,6 @@ describe("Context Keys Manager - getKey", () => {
 
     expect(mockedLogger.warn).not.toHaveBeenCalled();
 
-    expect(getKeyByReference).toHaveBeenCalledWith("key", "namespace");
-
     expect(mockedSessionStorage.getItem).toHaveBeenCalledWith(
       "key",
       "namespace"
@@ -209,10 +191,6 @@ describe("Context Keys Manager - getKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const getKeyByReference = jest
-      .spyOn(contextManager, "getKeyByReference")
-      .mockResolvedValueOnce(NOTHING);
-
     // run
     const result = await contextManager.getKey(key);
 
@@ -221,8 +199,6 @@ describe("Context Keys Manager - getKey", () => {
 
     expect(mockedLogger.warn).not.toHaveBeenCalled();
 
-    expect(getKeyByReference).toHaveBeenCalledWith("key", "namespace");
-
     expect(mockedSessionStorage.getItem).toHaveBeenCalledWith(
       "key",
       "namespace"

package/appUtils/contextKeysManager/__tests__/getKey/success/searchInSecure.test.ts

@@ -1,4 +1,4 @@
-import { ContextKeysManager, NOTHING } from "../../..";
+import { ContextKeysManager } from "../../..";
 import { DEFAULT_NAMESPACE } from "../../../consts";
 
 describe("Context Keys Manager - getKey", () => {
@@ -36,10 +36,6 @@ describe("Context Keys Manager - getKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const getKeyByReference = jest
-      .spyOn(contextManager, "getKeyByReference")
-      .mockResolvedValueOnce(NOTHING);
-
     // run
     const result = await contextManager.getKey(key);
 
@@ -48,8 +44,6 @@ describe("Context Keys Manager - getKey", () => {
 
     expect(mockedLogger.warn).not.toHaveBeenCalled();
 
-    expect(getKeyByReference).toHaveBeenCalledWith("key", DEFAULT_NAMESPACE);
-
     expect(mockedSessionStorage.getItem).toHaveBeenCalledWith(
       "key",
       DEFAULT_NAMESPACE

package/appUtils/contextKeysManager/__tests__/removeKey/failure.test.ts

@@ -34,11 +34,6 @@ describe("Context Keys Manager - removeKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const removeReferenceForKey = jest.spyOn(
-      contextManager,
-      "removeReferenceForKey"
-    );
-
     // run
     const result = await contextManager.removeKey(key);
 
@@ -51,8 +46,6 @@ describe("Context Keys Manager - removeKey", () => {
       data: { key },
     });
 
-    expect(removeReferenceForKey).not.toHaveBeenCalled();
-
     expect(mockedSessionStorage.removeItem).not.toHaveBeenCalled();
     expect(mockedLocalStorage.removeItem).not.toHaveBeenCalled();
     expect(mockedSecureStorage.removeItem).not.toHaveBeenCalled();
@@ -92,10 +85,6 @@ describe("Context Keys Manager - removeKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const removeReferenceForKey = jest
-      .spyOn(contextManager, "removeReferenceForKey")
-      .mockResolvedValueOnce(true);
-
     // run
     const result = await contextManager.removeKey(key);
 
@@ -107,8 +96,6 @@ describe("Context Keys Manager - removeKey", () => {
       data: { key },
     });
 
-    expect(removeReferenceForKey).toHaveBeenCalledWith("key", "namespace");
-
     expect(mockedSessionStorage.removeItem).toHaveBeenCalledWith(
       "key",
       "namespace"
@@ -159,10 +146,6 @@ describe("Context Keys Manager - removeKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const removeReferenceForKey = jest
-      .spyOn(contextManager, "removeReferenceForKey")
-      .mockResolvedValueOnce(true);
-
     // run
     const result = await contextManager.removeKey(key);
 
@@ -174,8 +157,6 @@ describe("Context Keys Manager - removeKey", () => {
       data: { key },
     });
 
-    expect(removeReferenceForKey).toHaveBeenCalledWith("key", "namespace");
-
     expect(mockedSessionStorage.removeItem).toHaveBeenCalledWith(
       "key",
       "namespace"
@@ -226,10 +207,6 @@ describe("Context Keys Manager - removeKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const removeReferenceForKey = jest
-      .spyOn(contextManager, "removeReferenceForKey")
-      .mockResolvedValueOnce(true);
-
     // run
     const result = await contextManager.removeKey(key);
 
@@ -241,8 +218,6 @@ describe("Context Keys Manager - removeKey", () => {
       data: { key },
     });
 
-    expect(removeReferenceForKey).toHaveBeenCalledWith("key", "namespace");
-
     expect(mockedSessionStorage.removeItem).toHaveBeenCalledWith(
       "key",
       "namespace"

package/appUtils/contextKeysManager/__tests__/removeKey/success.test.ts

@@ -34,10 +34,6 @@ describe("Context Keys Manager - removeKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const removeReferenceForKey = jest
-      .spyOn(contextManager, "removeReferenceForKey")
-      .mockResolvedValueOnce(true);
-
     // run
     const result = await contextManager.removeKey(key);
 
@@ -46,8 +42,6 @@ describe("Context Keys Manager - removeKey", () => {
 
     expect(mockedLogger.warn).not.toHaveBeenCalled();
 
-    expect(removeReferenceForKey).toHaveBeenCalledWith("key", "namespace");
-
     expect(mockedSessionStorage.removeItem).toHaveBeenCalledWith(
       "key",
       "namespace"

package/appUtils/contextKeysManager/__tests__/setKey/failure/failedToSaveInStorage.test.ts

@@ -38,10 +38,6 @@ describe("Context Keys Manager - setKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const setReferenceForKey = jest
-      .spyOn(contextManager, "setReferenceForKey")
-      .mockResolvedValueOnce(true);
-
     // run
     const result = await contextManager.setKey({ key, value, storageLevel });
 
@@ -68,12 +64,6 @@ describe("Context Keys Manager - setKey", () => {
 
     expect(mockedSecureStorage.getItem).not.toHaveBeenCalled();
     expect(mockedSecureStorage.setItem).not.toHaveBeenCalled();
-
-    expect(setReferenceForKey).toHaveBeenCalledWith(
-      "key",
-      "namespace",
-      storageLevel
-    );
   });
 
   it("returns false if failed to saved with storage-level=persisted", async () => {
@@ -112,10 +102,6 @@ describe("Context Keys Manager - setKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const setReferenceForKey = jest
-      .spyOn(contextManager, "setReferenceForKey")
-      .mockResolvedValueOnce(true);
-
     // run
     const result = await contextManager.setKey({ key, value, storageLevel });
 
@@ -142,12 +128,6 @@ describe("Context Keys Manager - setKey", () => {
 
     expect(mockedSecureStorage.getItem).not.toHaveBeenCalled();
     expect(mockedSecureStorage.setItem).not.toHaveBeenCalled();
-
-    expect(setReferenceForKey).toHaveBeenCalledWith(
-      "key",
-      "namespace",
-      storageLevel
-    );
   });
 
   it("returns false if failed to saved with storage-level=secure", async () => {
@@ -186,10 +166,6 @@ describe("Context Keys Manager - setKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const setReferenceForKey = jest
-      .spyOn(contextManager, "setReferenceForKey")
-      .mockResolvedValueOnce(true);
-
     // run
     const result = await contextManager.setKey({ key, value, storageLevel });
 
@@ -216,12 +192,6 @@ describe("Context Keys Manager - setKey", () => {
       value,
       "namespace"
     );
-
-    expect(setReferenceForKey).toHaveBeenCalledWith(
-      "key",
-      "namespace",
-      storageLevel
-    );
   });
 
   it("returns false if failed to saved with storage-level=undefined", async () => {
@@ -259,10 +229,6 @@ describe("Context Keys Manager - setKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const setReferenceForKey = jest
-      .spyOn(contextManager, "setReferenceForKey")
-      .mockResolvedValueOnce(true);
-
     // run
     const result = await contextManager.setKey({ key, value });
 
@@ -289,11 +255,5 @@ describe("Context Keys Manager - setKey", () => {
 
     expect(mockedSecureStorage.getItem).not.toHaveBeenCalled();
     expect(mockedSecureStorage.setItem).not.toHaveBeenCalled();
-
-    expect(setReferenceForKey).toHaveBeenCalledWith(
-      "key",
-      "namespace",
-      StorageLevel.default
-    );
   });
 });

package/appUtils/contextKeysManager/__tests__/setKey/{success/saveItemAndReference.test.ts → success.test.ts}

@@ -1,5 +1,5 @@
-import { ContextKeysManager } from "../../..";
-import { StorageLevel } from "../../../consts";
+import { ContextKeysManager } from "../..";
+import { StorageLevel } from "../../consts";
 
 describe("Context Keys Manager - setKey", () => {
   it("returns true if saved with success with storage-level=undefined", async () => {
@@ -37,10 +37,6 @@ describe("Context Keys Manager - setKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const setReferenceForKey = jest
-      .spyOn(contextManager, "setReferenceForKey")
-      .mockResolvedValueOnce(true);
-
     // run
     const result = await contextManager.setKey({ key, value, storageLevel });
 
@@ -62,12 +58,6 @@ describe("Context Keys Manager - setKey", () => {
 
     expect(mockedSecureStorage.getItem).not.toHaveBeenCalled();
     expect(mockedSecureStorage.setItem).not.toHaveBeenCalled();
-
-    expect(setReferenceForKey).toHaveBeenCalledWith(
-      "key",
-      "namespace",
-      StorageLevel.default
-    );
   });
 
   it("returns true if saved with success with storage-level=default", async () => {
@@ -105,10 +95,6 @@ describe("Context Keys Manager - setKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const setReferenceForKey = jest
-      .spyOn(contextManager, "setReferenceForKey")
-      .mockResolvedValueOnce(true);
-
     // run
     const result = await contextManager.setKey({ key, value, storageLevel });
 
@@ -130,12 +116,6 @@ describe("Context Keys Manager - setKey", () => {
 
     expect(mockedSecureStorage.getItem).not.toHaveBeenCalled();
     expect(mockedSecureStorage.setItem).not.toHaveBeenCalled();
-
-    expect(setReferenceForKey).toHaveBeenCalledWith(
-      "key",
-      "namespace",
-      StorageLevel.default
-    );
   });
 
   it("returns true if saved with success with storage-level=persisted", async () => {
@@ -173,10 +153,6 @@ describe("Context Keys Manager - setKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const setReferenceForKey = jest
-      .spyOn(contextManager, "setReferenceForKey")
-      .mockResolvedValueOnce(true);
-
     // run
     const result = await contextManager.setKey({ key, value, storageLevel });
 
@@ -198,12 +174,6 @@ describe("Context Keys Manager - setKey", () => {
 
     expect(mockedSecureStorage.getItem).not.toHaveBeenCalled();
     expect(mockedSecureStorage.setItem).not.toHaveBeenCalled();
-
-    expect(setReferenceForKey).toHaveBeenCalledWith(
-      "key",
-      "namespace",
-      storageLevel
-    );
   });
 
   it("returns true if saved with success with storage-level=secure", async () => {
@@ -241,10 +211,6 @@ describe("Context Keys Manager - setKey", () => {
       secureStorage: mockedSecureStorage,
     });
 
-    const setReferenceForKey = jest
-      .spyOn(contextManager, "setReferenceForKey")
-      .mockResolvedValueOnce(true);
-
     // run
     const result = await contextManager.setKey({ key, value, storageLevel });
 
@@ -266,11 +232,5 @@ describe("Context Keys Manager - setKey", () => {
       value,
       "namespace"
     );
-
-    expect(setReferenceForKey).toHaveBeenCalledWith(
-      "key",
-      "namespace",
-      storageLevel
-    );
   });
 });

package/appUtils/contextKeysManager/index.ts

@@ -1,6 +1,6 @@
 import { isNil } from "@applicaster/zapp-react-native-utils/utils";
 
-import { StorageLevel, StorageType } from "./consts";
+import { StorageLevel } from "./consts";
 import {
   getNamespaceAndKey,
   keyIsValid,
@@ -59,18 +59,14 @@ type Setup = {
   sessionStorage?: Storage;
   localStorage?: Storage;
   secureStorage?: Storage;
-  referenceStorage?: Storage;
 };
 
-export const REFERENCE_NAMESPACE = "applicaster.v2.references";
-
 export class ContextKeysManager {
   private static _instance: ContextKeysManager;
   private _logger: Logger;
   private _sessionStorage: Storage;
   private _localStorage: Storage;
   private _secureStorage: Storage;
-  private _referenceStorage: Storage;
 
   public static get instance() {
     if (!this._instance) {
@@ -79,7 +75,6 @@ export class ContextKeysManager {
         sessionStorage: SessionStorage,
         localStorage: LocalStorage,
         secureStorage: SecureStorage,
-        referenceStorage: SessionStorage,
       });
 
       this._instance = instance;
@@ -94,13 +89,11 @@ export class ContextKeysManager {
     sessionStorage = SessionStorage,
     localStorage = LocalStorage,
     secureStorage = SecureStorage,
-    referenceStorage = SessionStorage,
-  }: Setup) {
+  }: Setup = {}) {
     this._logger = logger;
     this._sessionStorage = sessionStorage;
     this._localStorage = localStorage;
     this._secureStorage = secureStorage;
-    this._referenceStorage = referenceStorage;
   }
 
   private parseKey(key: KeyName): ParsedKeyAndNamespace {
@@ -119,41 +112,6 @@ export class ContextKeysManager {
     return { isValid: true, ...parsedKey };
   }
 
-  // GET
-  public async getKeyByReference(
-    key: string,
-    namespace: string
-  ): Promise<ValueOrNothing> {
-    // 1) look for storage
-    // 2) if storage found - look value in it
-    // 3) if storage not found return NOTHING
-
-    return this._referenceStorage
-      .getItem(buildNamespaceKey(key, namespace), REFERENCE_NAMESPACE)
-      .then((storage: StorageType | unknown) => {
-        switch (storage) {
-          case StorageType.session:
-            return this._sessionStorage.getItem(key, namespace);
-          case StorageType.local:
-            return this._localStorage.getItem(key, namespace);
-          case StorageType.secure:
-            return this._secureStorage.getItem(key, namespace);
-          default:
-            // unknown storage
-            return NOTHING;
-        }
-      })
-      .catch((error) => {
-        // something went wrong
-        this._logger.warn({
-          message: error.message,
-          data: { key, namespace },
-        });
-
-        return NOTHING;
-      });
-  }
-
   public async getKeys(
     keys: Array<KeyName>,
     contextObj?: { key: string; required?: boolean }[]
@@ -228,19 +186,7 @@ export class ContextKeysManager {
       return NOTHING;
     }
 
-    // at first - try to find by reference
-    // if not found, look in session > local > secure
-
-    const resultByReference = await this.getKeyByReference(
-      parsedKey,
-      parsedNamespace
-    );
-
-    if (!isNil(resultByReference)) {
-      return resultByReference;
-    }
-
-    // try to full-search
+    // look in session > local > secure
     try {
       const resultFromSessionStorage = await this._sessionStorage.getItem(
         parsedKey,
@@ -284,41 +230,6 @@ export class ContextKeysManager {
   // GET
 
   // SET
-  public async setReferenceForKey(
-    key: string,
-    namespace: string,
-    storageLevel: StorageLevel
-  ): Promise<boolean> {
-    return this._referenceStorage
-      .setItem(
-        buildNamespaceKey(key, namespace),
-        storageLevel,
-        REFERENCE_NAMESPACE
-      )
-      .then((result) => {
-        if (savingResultIsSuccess(result)) {
-          return true;
-        }
-
-        this._logger.warn({
-          message:
-            "While saving reference - reference storage return non expected value: true",
-          data: { key, namespace, storageLevel },
-        });
-
-        return false;
-      })
-      .catch((error) => {
-        // something went wrong
-        this._logger.warn({
-          message: error.message,
-          data: { key, namespace, storageLevel },
-        });
-
-        return false;
-      });
-  }
-
   // for all keys where saving were succeed return true
   // for others return false
   public async setKeys(
@@ -352,29 +263,36 @@ export class ContextKeysManager {
     }
 
     try {
+      let setItemResult: boolean;
+
       switch (storageLevel) {
         case StorageLevel.persisted:
-          return await Promise.all([
-            this._localStorage.setItem(parsedKey, value, parsedNamespace),
-            this.setReferenceForKey(parsedKey, parsedNamespace, storageLevel),
-          ]).then(([setItemResult]) => {
-            return savingResultIsSuccess(setItemResult);
-          });
+          setItemResult = await this._localStorage.setItem(
+            parsedKey,
+            value,
+            parsedNamespace
+          );
+
+          break;
         case StorageLevel.secure:
-          return await Promise.all([
-            this._secureStorage.setItem(parsedKey, value, parsedNamespace),
-            this.setReferenceForKey(parsedKey, parsedNamespace, storageLevel),
-          ]).then(([setItemResult]) => {
-            return savingResultIsSuccess(setItemResult);
-          });
+          setItemResult = await this._secureStorage.setItem(
+            parsedKey,
+            value,
+            parsedNamespace
+          );
+
+          break;
         default:
-          return await Promise.all([
-            this._sessionStorage.setItem(parsedKey, value, parsedNamespace),
-            this.setReferenceForKey(parsedKey, parsedNamespace, storageLevel),
-          ]).then(([setItemResult]) => {
-            return savingResultIsSuccess(setItemResult);
-          });
+          setItemResult = await this._sessionStorage.setItem(
+            parsedKey,
+            value,
+            parsedNamespace
+          );
+
+          break;
       }
+
+      return savingResultIsSuccess(setItemResult);
     } catch (error) {
       // something went wrong
       this._logger.warn({
@@ -388,24 +306,6 @@ export class ContextKeysManager {
   // SET
 
   // REMOVE
-  public async removeReferenceForKey(
-    key: string,
-    namespace: string
-  ): Promise<boolean> {
-    return this._referenceStorage
-      .removeItem(buildNamespaceKey(key, namespace), REFERENCE_NAMESPACE)
-      .then(() => true)
-      .catch((error) => {
-        // something went wrong
-        this._logger.warn({
-          message: error.message,
-          data: { key, namespace },
-        });
-
-        return false;
-      });
-  }
-
   public async removeKey(key: KeyName): Promise<boolean> {
     const {
       isValid: isValidKey,
@@ -422,7 +322,6 @@ export class ContextKeysManager {
       this._sessionStorage.removeItem(parsedKey, parsedNamespace),
       this._localStorage.removeItem(parsedKey, parsedNamespace),
       this._secureStorage.removeItem(parsedKey, parsedNamespace),
-      this.removeReferenceForKey(parsedKey, parsedNamespace),
     ])
       .then(() => true)
       .catch((error) => {

package/appUtils/playerManager/OverlayObserver/OverlaysObserver.ts

@@ -2,8 +2,7 @@ import { BehaviorSubject, Observable } from "rxjs";
 import { distinctUntilChanged } from "rxjs/operators";
 import { Player } from "../player";
 import { createLogger, utilsLogger } from "../../../logger";
-import { appStore } from "@applicaster/zapp-react-native-redux/AppStore";
-import { findPluginByIdentifier } from "../../../pluginUtils";
+import { getPluginByIdentifier } from "@applicaster/zapp-react-native-utils/pluginUtils";
 import {
   loadFeedEntry,
   loadFeedAndPrefetchThumbnailImage,
@@ -237,11 +236,8 @@ export class OverlaysObserver {
 
   preparePlayNext = async () => {
     try {
-      const plugins = appStore.get("plugins");
-
-      const playNextPlugin: any = findPluginByIdentifier(
-        "QuickBrickPlayNextOverlay",
-        plugins
+      const playNextPlugin: any = getPluginByIdentifier(
+        "QuickBrickPlayNextOverlay"
       );
 
       if (!playNextPlugin) {
@@ -252,10 +248,7 @@ export class OverlaysObserver {
         return;
       }
 
-      const playNextFeedUrl: string = await retrieveFeedUrl(
-        this.entry,
-        plugins
-      );
+      const playNextFeedUrl: string = await retrieveFeedUrl(this.entry);
 
       if (!playNextFeedUrl || typeof playNextFeedUrl !== "string") {
         log_debug(