@shopgate/pwa-common 7.32.0-beta.2 → 7.32.0-beta.21

package/helpers/mocks/mockRenderOptions.js

@@ -0,0 +1,21 @@
+import PropTypes from 'prop-types';
+const options = {
+  context: {
+    i18n: () => ({
+      // eslint-disable-next-line no-underscore-dangle
+      __(input) {
+        if (input) {
+          return input;
+        }
+        return '';
+      },
+      _p: () => 'p',
+      _d: () => 'd',
+      _n: () => 'n'
+    })
+  },
+  childContextTypes: {
+    i18n: PropTypes.func
+  }
+};
+export default options;

package/helpers/router/index.js

@@ -1,7 +1,9 @@
 import "core-js/modules/es.array.reduce.js";
+import "core-js/modules/es.string.replace.js";
 import "core-js/modules/web.url-search-params.js";
 import { createBrowserHistory } from 'history';
 import { router } from '@virtuous/conductor';
+import { logger } from '@shopgate/pwa-core/helpers';
 const match = /^(.*)index.html/.exec(window.location.pathname);
 const {
   getCurrentRoute
@@ -56,4 +58,87 @@ export const parseObjectToQueryString = (obj, includePrefix = true) => {
     return `?${urlParams.toString()}`;
   }
   return urlParams.toString();
+};
+
+// Angle brackets and their HTML entity representations.
+const ANGLE_BRACKET_ENTITIES = '&(lt|gt|#0*6[02]|#x0*3[ce]);?';
+const UNSAFE_CHARS_REGEX = new RegExp(`[<>]|${ANGLE_BRACKET_ENTITIES}`, 'i');
+const UNSAFE_PROTOCOL_REGEX = /^\s*(javascript|data|vbscript):/i;
+
+/**
+ * Removes HTML tags, angle brackets and their entity representations from a string.
+ * @param {string} value The (decoded) value.
+ * @returns {string}
+ */
+const stripHtml = value => value.replace(new RegExp(ANGLE_BRACKET_ENTITIES, 'gi'), '').replace(/<[^>]*>/g, '').replace(/[<>]/g, '');
+
+/**
+ * Removes HTML from a single URI component. The component is only modified when it contains
+ * suspicious characters, so regular components (e.g. encoded product ids) stay untouched.
+ * @param {string} component A single URI encoded component like a path segment.
+ * @returns {string}
+ */
+const sanitizeUriComponent = component => {
+  let decoded;
+  try {
+    decoded = decodeURIComponent(component);
+  } catch (e) {
+    // Malformed encoding - fall back to the raw component.
+    decoded = component;
+  }
+  if (!UNSAFE_CHARS_REGEX.test(decoded)) {
+    return component;
+  }
+  return encodeURIComponent(stripHtml(decoded));
+};
+
+/**
+ * Removes HTML from a decoded query parameter key or value.
+ * @param {string} value The decoded value.
+ * @returns {string}
+ */
+const sanitizeQueryValue = value => UNSAFE_CHARS_REGEX.test(value) ? stripHtml(value) : value;
+
+/**
+ * Sanitizes a link which was received from an external source (e.g. deep links or push messages)
+ * before it's passed to the router. HTML markup is removed from the path, query and hash to prevent
+ * that it's rendered somewhere within the app. Links with a script protocol are rejected.
+ * Links without suspicious content are returned unchanged.
+ * @param {string} link The link to sanitize.
+ * @returns {string} The sanitized link or an empty string when the link was rejected.
+ */
+export const sanitizeLink = link => {
+  if (typeof link !== 'string' || link === '') {
+    return '';
+  }
+  if (UNSAFE_PROTOCOL_REGEX.test(link)) {
+    logger.warn('sanitizeLink: Rejected link with unsafe protocol', link);
+    return '';
+  }
+  const hashIndex = link.indexOf('#');
+  const beforeHash = hashIndex === -1 ? link : link.slice(0, hashIndex);
+  const hash = hashIndex === -1 ? null : link.slice(hashIndex + 1);
+  const queryIndex = beforeHash.indexOf('?');
+  const base = queryIndex === -1 ? beforeHash : beforeHash.slice(0, queryIndex);
+  const query = queryIndex === -1 ? null : beforeHash.slice(queryIndex + 1);
+  let sanitized = base.split('/').map(sanitizeUriComponent).join('/');
+  if (query !== null) {
+    const params = new URLSearchParams(query);
+    const sanitizedParams = new URLSearchParams();
+    let modified = false;
+    params.forEach((value, key) => {
+      const sanitizedKey = sanitizeQueryValue(key);
+      const sanitizedValue = sanitizeQueryValue(value);
+      modified = modified || sanitizedKey !== key || sanitizedValue !== value;
+      sanitizedParams.append(sanitizedKey, sanitizedValue);
+    });
+    sanitized += `?${modified ? sanitizedParams.toString() : query}`;
+  }
+  if (hash !== null) {
+    sanitized += `#${sanitizeUriComponent(hash)}`;
+  }
+  if (sanitized !== link) {
+    logger.warn('sanitizeLink: Removed unsafe content from link', link);
+  }
+  return sanitized;
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@shopgate/pwa-common",
-  "version": "7.32.0-beta.2",
+  "version": "7.32.0-beta.21",
   "description": "Common library for the Shopgate Connect PWA.",
   "license": "Apache-2.0",
   "author": "Shopgate <support@shopgate.com>",
@@ -17,7 +17,7 @@
   "dependencies": {
     "@redux-devtools/extension": "^3.3.0",
     "@sentry/browser": "6.0.1",
-    "@shopgate/pwa-benchmark": "7.32.0-beta.2",
+    "@shopgate/pwa-benchmark": "7.32.0-beta.21",
     "@virtuous/conductor": "~2.5.0",
     "@virtuous/react-conductor": "~2.5.0",
     "@virtuous/redux-persister": "1.1.0-beta.7",
@@ -40,7 +40,7 @@
     "swiper": "12.2.0"
   },
   "devDependencies": {
-    "@shopgate/pwa-core": "7.32.0-beta.2",
+    "@shopgate/pwa-core": "7.32.0-beta.21",
     "@types/lodash": "^4.17.24",
     "@types/react-portal": "^3.0.9",
     "lodash": "^4.17.23",

package/providers/toast/index.js

@@ -32,47 +32,41 @@ let ToastProvider = /*#__PURE__*/function (_Component) {
       if (!toast.message) {
         return;
       }
-      const {
-        toasts
-      } = _this.state;
-
-      // Check if the toast id already is present.
-      const found = toasts.find(({
-        id
-      }) => toast.id === id);
+      const nextToast = {
+        id: toast.id,
+        action: toast.action,
+        actionLabel: toast.actionLabel,
+        onLongPress: toast.onLongPress,
+        message: toast.message,
+        messageParams: toast.messageParams,
+        duration: toast.duration || _this.props.theme.transitions.duration.toast
+      };
 
-      // If found, update the toast with the new data.
-      if (found) {
-        found.action = toast.action;
-        found.actionLabel = toast.actionLabel;
-        found.message = toast.message;
-        found.messageParams = toast.messageParams;
-        found.duration = toast.duration || _this.props.theme.transitions.duration.toast;
-      } else {
-        toasts.push({
-          id: toast.id,
-          action: toast.action,
-          actionLabel: toast.actionLabel,
-          message: toast.message,
-          messageParams: toast.messageParams,
-          duration: toast.duration || _this.props.theme.transitions.duration.toast
-        });
-      }
-      _this.setState({
+      // Update the queue immutably: consumers rely on the array reference changing to re-render
+      // (e.g. the SnackBar memoizes the currently shown toast on this reference). A toast whose id
+      // is already queued replaces that entry with the new data; otherwise it is appended.
+      _this.setState(({
         toasts
+      }) => {
+        const exists = toasts.some(({
+          id
+        }) => id === toast.id);
+        return {
+          toasts: exists ? toasts.map(item => item.id === toast.id ? nextToast : item) : [].concat(toasts, [nextToast])
+        };
       });
     };
     /**
      * Removes the first toast from the list.
      */
     _this.removeToast = () => {
-      const {
-        toasts
-      } = _this.state;
-      toasts.shift();
-      _this.setState({
+      // Drop the first toast immutably so the array reference changes and the SnackBar advances to
+      // the next queued toast (rather than re-rendering the same, now-stale, memoized toast).
+      _this.setState(({
         toasts
-      });
+      }) => ({
+        toasts: toasts.slice(1)
+      }));
     };
     _this.flushToasts = () => {
       if (_this.state.toasts.length) {
@@ -90,11 +84,21 @@ let ToastProvider = /*#__PURE__*/function (_Component) {
   }
 
   /**
-   * Returns the context value to be passed to consumers.
-   * @returns {Object}
+   * Removes the UIEvents listeners registered in the constructor.
    */
   _inheritsLoose(ToastProvider, _Component);
   var _proto = ToastProvider.prototype;
+  _proto.componentWillUnmount = function componentWillUnmount() {
+    UIEvents.removeListener(this.constructor.ADD, this.addToast);
+    UIEvents.removeListener(this.constructor.FLUSH, this.flushToasts);
+  }
+
+  /**
+   * Returns the context value to be passed to consumers. The object reference is cached and only
+   * rebuilt when `toasts` changes, so consumers (Toaster → SnackBarContainer → SnackBar) don't
+   * re-render on every unrelated ToastProvider render.
+   * @returns {Object}
+   */;
   /**
    * @returns {JSX}
    */
@@ -107,14 +111,24 @@ let ToastProvider = /*#__PURE__*/function (_Component) {
   return _createClass(ToastProvider, [{
     key: "provided",
     get: function () {
-      return {
-        addToast: this.addToast,
-        removeToast: this.removeToast,
-        toasts: this.state.toasts
-      };
+      if (!this.providedCache || this.providedCache.toasts !== this.state.toasts) {
+        this.providedCache = {
+          addToast: this.addToast,
+          removeToast: this.removeToast,
+          toasts: this.state.toasts
+        };
+      }
+      return this.providedCache;
     }
   }]);
 }(Component);
 ToastProvider.ADD = 'toast_add';
 ToastProvider.FLUSH = 'toast_flush';
-export default withTheme(ToastProvider);
+const ThemedToastProvider = withTheme(ToastProvider);
+
+// The HOC returns a new component which doesn't carry over the statics of the wrapped class. They
+// are part of the public API of this module (consumers emit UIEvents on ToastProvider.ADD), so they
+// are re-attached explicitly.
+ThemedToastProvider.ADD = ToastProvider.ADD;
+ThemedToastProvider.FLUSH = ToastProvider.FLUSH;
+export default ThemedToastProvider;

package/subscriptions/error.js

@@ -1,7 +1,4 @@
 import "core-js/modules/es.array.includes.js";
-import after from 'lodash/after';
-import before from 'lodash/before';
-import over from 'lodash/over';
 import { isAvailable } from '@shopgate/native-modules';
 import { init, addBreadcrumb, configureScope, captureException, captureMessage, captureEvent, withScope, Severity as SentrySeverity } from '@sentry/browser';
 import { EBIGAPI, emitter, errorManager, ETIMEOUT, ENETUNREACH, EUNKNOWN, EFAVORITE } from '@shopgate/pwa-core';
@@ -12,7 +9,7 @@ import {
 // eslint-disable-next-line import/no-named-default
 default as appConfig, themeName, pckVersion } from "../helpers/config";
 import { env } from "../helpers/environment";
-import { transformGeneralPipelineError } from "./helpers/pipeline";
+import { transformGeneralPipelineError, getDisplayErrorMessage } from "./helpers/pipeline";
 import { historyPop } from "../actions/router";
 import showModal from "../actions/modal/showModal";
 import { getUserData } from "../selectors/user";
@@ -24,6 +21,9 @@ import { getRouterStack } from "../selectors/router";
 import { MODAL_PIPELINE_ERROR } from "../constants/ModalTypes";
 import ToastProvider from "../providers/toast";
 
+// Generic, translated fallback shown when a backend error carries no code we can map to a message.
+const GENERIC_ERROR_MESSAGE = 'modal.body_error';
+
 /**
  * App errors subscriptions.
  * @param {Function} subscribe The subscribe function.
@@ -60,15 +60,35 @@ export default subscribe => {
       error
     } = action;
     const {
-      message,
       code,
       context,
       meta = {}
     } = error;
     const {
-      behavior
+      behavior,
+      message: originalMessage
     } = meta;
-    if (behavior) {
+
+    // Never surface a raw backend message to the user: show the extension's own translated
+    // message, a code-mapped translated message, or a generic fallback. The resolver is the single
+    // source of truth for both the text and whether it is ready-to-display (`displayTranslated`) or
+    // a locale key that must still go through I18n.Text.
+    const {
+      message: displayMessage,
+      translated: displayTranslated
+    } = getDisplayErrorMessage(error, GENERIC_ERROR_MESSAGE);
+
+    // Unknown/generic and connection-style backend errors are treated as general platform issues.
+    // EUNKNOWN is always treated as general — even when the backend supplied a translated message —
+    // so it is shown as a toast with its translated message or generic fallback, never a modal.
+    const isConnectionError = displayMessage === 'error.general' || [ETIMEOUT, ENETUNREACH].includes(code);
+    const isGeneralError = isConnectionError || displayMessage === GENERIC_ERROR_MESSAGE || code === EUNKNOWN;
+
+    // Some pipeline actions (e.g. fetchCategory) register an error behavior for *every* error via
+    // `setResponseBehavior`. Those behaviors are only meaningful for the specific errors they
+    // expect; when the platform fails with an unexpected/unmappable message we skip the behavior
+    // and fall through to the generic toast handling instead of, for example, showing a modal.
+    if (behavior && !isGeneralError) {
       behavior({
         dispatch,
         getState,
@@ -78,38 +98,41 @@ export default subscribe => {
       return;
     }
 
-    /** Show modal thunk */
-    const showModalError = () => {
+    /**
+     * Shows the pipeline error modal. When openWithDetails is set, the modal opens directly on the
+     * developer detail view (pipeline, code, raw message, params) — used for the toast long-press.
+     * @param {boolean} [openWithDetails=false] Whether to open the modal in developer detail mode.
+     */
+    const showModalError = (openWithDetails = false) => {
       dispatch(showModal({
         confirm: 'modal.ok',
         dismiss: null,
         title: null,
-        message,
+        message: displayMessage,
         type: MODAL_PIPELINE_ERROR,
         params: {
           pipeline: context,
           request: meta.input,
-          message: meta.message,
-          code
+          message: originalMessage,
+          code,
+          translated: displayTranslated,
+          messageParams: meta.additionalParams,
+          openWithDetails
         }
       }));
     };
-    let shouldShowToast = message === 'error.general';
-    if ([ETIMEOUT, ENETUNREACH].includes(code) && message === 'modal.body_error') {
-      shouldShowToast = true;
-    }
-    // It was transformed general error. let it popup after 10 toast clicks
-    if (shouldShowToast) {
-      const showToastAfter = after(9, showModalError);
-      // Recursively show same toast message until clicked 10 times
-      const showToast = before(10, () => {
-        events.emit(ToastProvider.ADD, {
-          id: 'pipeline.error',
-          message: 'error.general',
-          action: over([showToast, showToastAfter])
-        });
+
+    // General platform errors surface as a toast instead of a modal.
+    // Long-pressing the toast opens the error modal in developer detail mode.
+    if (isGeneralError) {
+      // Connection-style errors show the generic connection message, but never override a message
+      // an extension already translated for us.
+      const useGenericConnectionText = isConnectionError && !displayTranslated;
+      events.emit(ToastProvider.ADD, {
+        id: 'pipeline.error',
+        message: useGenericConnectionText ? 'error.general' : displayMessage,
+        onLongPress: () => showModalError(true)
       });
-      showToast();
       return;
     }
     showModalError();

package/subscriptions/helpers/pipeline.js

@@ -1,3 +1,5 @@
+import { i18n } from '@shopgate/engage/core/helpers';
+
 /**
  * @param {Object} error error
  * @returns {string|*}
@@ -12,4 +14,70 @@ export function transformGeneralPipelineError(error) {
     return 'error.general';
   }
   return message;
+}
+
+/**
+ * @typedef {Object} DisplayErrorMessage
+ * @property {string} message The text to display — an already-translated string or a locale key.
+ * @property {boolean} translated Whether `message` is ready-to-display text (`true`) or a locale
+ *   key that still needs to go through I18n.Text (`false`).
+ */
+
+/**
+ * Resolves the message to display for a pipeline error, without ever surfacing raw backend text.
+ * Also reports whether the result is ready-to-display text or a locale key
+ * Precedence:
+ * 1. An extension's own message when it is explicitly flagged as already translated.
+ * 2. A resolvable i18n key — safe to surface even when errorManager did not remap it, because a
+ *    translation key (e.g. `cart.error_out_of_stock`) is not raw backend text. `i18n.has` confirms
+ *    the key actually resolves in the loaded locales, rather than merely looking like a key.
+ * 3. The code-mapped message. When errorManager maps the error code it resolves a message that
+ *    differs from the raw backend text (`error.meta.message`).
+ * 4. A generic, translated fallback otherwise — including when nothing mapped and errorManager fell
+ *    back to the raw backend message (`error.message === error.meta.message`).
+ * @param {Object} error The queued pipeline error.
+ * @param {string} error.message The message resolved by errorManager (code-mapped or raw fallback).
+ * @param {Object} [error.meta] The error meta data.
+ * @param {boolean} [error.meta.translated] Whether the backend message is already translated.
+ * @param {string} [error.meta.message] The original, raw backend message.
+ * @param {string} genericMessage The generic fallback message key.
+ * @returns {DisplayErrorMessage}
+ */
+export function getDisplayErrorMessage(error, genericMessage) {
+  const {
+    message,
+    meta = {}
+  } = error;
+  const {
+    translated,
+    message: originalMessage
+  } = meta;
+  if (translated && originalMessage) {
+    return {
+      message: originalMessage,
+      translated: true
+    };
+  }
+
+  // A translation key is never raw backend text, so surface it even when errorManager left it
+  // untouched (`message === originalMessage`).
+  if (typeof message === 'string' && i18n.has(message)) {
+    return {
+      message,
+      translated: false
+    };
+  }
+
+  // Only trust `message` when we have the raw backend message to compare against and a mapping
+  // actually replaced it. Without the raw message we cannot rule out that `message` is raw text.
+  if (originalMessage !== undefined && message !== originalMessage) {
+    return {
+      message,
+      translated: false
+    };
+  }
+  return {
+    message: genericMessage,
+    translated: false
+  };
 }

package/subscriptions/router.js

@@ -9,7 +9,7 @@ import Route from '@virtuous/conductor/Route';
 import { Linking } from '@shopgate/native-modules';
 import { HISTORY_RESET_TO } from '@shopgate/pwa-common/constants/ActionTypes';
 import { logger } from '@shopgate/pwa-core';
-import { isPageAdminPreviewActive } from '@shopgate/engage/admin-preview/helpers';
+import { isNavigationBlocked } from '@shopgate/engage/admin-preview/helpers';
 import addCouponsToCart from '@shopgate/pwa-common-commerce/cart/actions/addCouponsToCart';
 import { getCurrentRoute, getRouterStackIndex } from "../selectors/router";
 import { LoadingProvider } from "../providers";
@@ -21,6 +21,7 @@ import { navigate$, userDidLogin$, appWillStart$, windowOpenOverride$ } from "..
 import { isUserLoggedIn } from "../selectors/user";
 import { getIsConnected } from "../selectors/client";
 import { INDEX_PATH } from "../constants/RoutePaths";
+import { sanitizeLink } from "../helpers/router";
 import appConfig from "../helpers/config";
 import authRoutes from "../collections/AuthRoutes";
 import ToastProvider from "../providers/toast";
@@ -31,7 +32,7 @@ import ToastProvider from "../providers/toast";
  */
 export default function routerSubscriptions(subscribe) {
   subscribe(navigate$, async params => {
-    if (isPageAdminPreviewActive()) {
+    if (isNavigationBlocked()) {
       // No navigation is allowed in page preview mode.
       return;
     }
@@ -108,6 +109,10 @@ export default function routerSubscriptions(subscribe) {
         }
       case HISTORY_RESET_TO:
         {
+          const sanitizedResetToPathname = sanitizeLink(String(resetToPathname || ''));
+          if (!sanitizedResetToPathname) {
+            return;
+          }
           await router.pop({
             steps: historyLength - 1,
             state: routeState,
@@ -115,7 +120,7 @@ export default function routerSubscriptions(subscribe) {
             emitAfter: false
           });
           await router.replace({
-            pathname: resetToPathname,
+            pathname: sanitizedResetToPathname,
             state: routeState
           });
           return;
@@ -124,6 +129,13 @@ export default function routerSubscriptions(subscribe) {
         break;
     }
 
+    // Remove HTML markup from the location (e.g. within query parameters of links from CMS content)
+    // to prevent that it's rendered within pages or sent within requests. Links with a script
+    // protocol are rejected.
+    if (location) {
+      location = sanitizeLink(String(location));
+    }
+
     // Remove trailing slashes from internal links, since they might break the routing mechanism.
     // External links are treated as valid, since we don't know about the requirements at the
     // 3rd party server (e.g. google maps links might require trailing slashes).
@@ -331,9 +343,10 @@ export default function routerSubscriptions(subscribe) {
      * so that the router can decide how to handle the URL.
      */
     Linking.addEventListener('windowOpenRequested', event => {
-      const {
-        targetUrl
-      } = event.detail;
+      const targetUrl = sanitizeLink(String(event.detail?.targetUrl || ''));
+      if (!targetUrl) {
+        return;
+      }
       dispatch(historyPush({
         pathname: targetUrl,
         state: {}
@@ -371,9 +384,10 @@ export default function routerSubscriptions(subscribe) {
     action,
     dispatch
   }) => {
-    if (action.pathname) {
+    const pathname = sanitizeLink(String(action.pathname || ''));
+    if (pathname) {
       dispatch(historyPush({
-        pathname: action.pathname
+        pathname
       }));
     }
   });

package/action-creators/app/spec.js

@@ -1,96 +0,0 @@
-import { APP_WILL_START, APP_DID_START, PWA_DID_APPEAR, PWA_DID_DISAPPEAR, WILL_REGISTER_LINK_EVENTS, DID_REGISTER_LINK_EVENTS, OPEN_DEEP_LINK, OPEN_PUSH_NOTIFICATION, OPEN_UNIVERSAL_LINK } from "../../constants/ActionTypes";
-import { appWillStart, appDidStart, pwaDidAppear, pwaDidDisappear, willRegisterLinkEvents, didRegisterLinkEvents, openDeepLink, openPushNotification, openUniversalLink } from "./index";
-const dataMock = {
-  some: 'data'
-};
-describe('Action Creators: app', () => {
-  describe('appWillStart()', () => {
-    it('should work as expected', () => {
-      const expected = {
-        type: APP_WILL_START,
-        location: dataMock
-      };
-      expect(appWillStart(dataMock)).toEqual(expected);
-    });
-  });
-  describe('appDidStart()', () => {
-    it('should work as expected', () => {
-      const expected = {
-        type: APP_DID_START
-      };
-      expect(appDidStart()).toEqual(expected);
-    });
-  });
-  describe('pwaDidAppear()', () => {
-    it('should work as expected', () => {
-      const expected = {
-        type: PWA_DID_APPEAR
-      };
-      expect(pwaDidAppear()).toEqual(expected);
-    });
-  });
-  describe('pwaDidDisappear()', () => {
-    it('should work as expected', () => {
-      const expected = {
-        type: PWA_DID_DISAPPEAR
-      };
-      expect(pwaDidDisappear()).toEqual(expected);
-    });
-  });
-  describe('willRegisterLinkEvents()', () => {
-    it('should work as expected', () => {
-      const expected = {
-        type: WILL_REGISTER_LINK_EVENTS
-      };
-      expect(willRegisterLinkEvents()).toEqual(expected);
-    });
-  });
-  describe('didRegisterLinkEvents()', () => {
-    it('should work as expected', () => {
-      const expected = {
-        type: DID_REGISTER_LINK_EVENTS
-      };
-      expect(didRegisterLinkEvents()).toEqual(expected);
-    });
-  });
-  describe('openDeepLink()', () => {
-    it('should work as expected', () => {
-      const expected = {
-        type: OPEN_DEEP_LINK,
-        payload: dataMock
-      };
-      expect(openDeepLink(dataMock)).toEqual(expected);
-    });
-  });
-  describe('openPushNotification()', () => {
-    const notificationId = 'abc123';
-    const link = '/link/to/somewhere';
-    it('should work as expected', () => {
-      const expected = {
-        type: OPEN_PUSH_NOTIFICATION,
-        notificationId,
-        link
-      };
-      expect(openPushNotification(notificationId, link)).toEqual(expected);
-    });
-    it('should work as expected when the link is empty', () => {
-      const expected = {
-        type: OPEN_PUSH_NOTIFICATION,
-        notificationId,
-        link: ''
-      };
-      expect(openPushNotification(notificationId)).toEqual(expected);
-    });
-  });
-  describe('openUniversalLink()', () => {
-    const expected = {
-      type: OPEN_UNIVERSAL_LINK,
-      payload: {
-        link: 'https://testshop.shopgate.com/item/313131313132',
-        wasOpenedFromSearchIndex: true,
-        linkSerial: 1003
-      }
-    };
-    expect(openUniversalLink(expected.payload)).toEqual(expected);
-  });
-});