improve setItem helper to setStorageItem

src/helpers/index.js

@@ -27,5 +27,5 @@ export * from './random-response';
 export * from './get-descriptor-addon';
 export * from './parse-flags';
 export * from './match-request-props';
-export * from './local-storage-utils';
+export * from './storage-utils';
 export * from './parse-keyword-value';

src/helpers/storage-utils.js

@@ -0,0 +1,19 @@
+/**
+ * Set item to a specified storage
+ * @param {Object} storage Storage instance to set item into
+ * @param {string} key
+ * @param {string} value
+ * @param {boolean} shouldLog
+ */
+export const setStorageItem = (storage, key, value, shouldLog) => {
+    // eslint-disable-next-line no-console
+    const log = console.log.bind(console);
+    // setItem() may throw an exception if the storage is full.
+    try {
+        storage.setItem(key, value);
+    } catch (e) {
+        if (shouldLog) {
+            log(`Unable to set sessionStorage item due to: ${e.message}`);
+        }
+    }
+};

src/scriptlets/set-local-storage-item.js

@@ -1,7 +1,7 @@
 import {
     hit,
     nativeIsNaN,
-    setItem,
+    setStorageItem,
 } from '../helpers/index';
 
 /* eslint-disable max-len */
@@ -85,7 +85,8 @@ export function setLocalStorageItem(source, key, value) {
         return;
     }
 
-    setItem(key, keyValue);
+    const { localStorage } = window;
+    setStorageItem(localStorage, key, keyValue, source.verbose);
     hit(source);
 }
 
@@ -96,5 +97,5 @@ setLocalStorageItem.names = [
 setLocalStorageItem.injections = [
     hit,
     nativeIsNaN,
-    setItem,
+    setStorageItem,
 ];

src/scriptlets/set-session-storage-item.js

@@ -1,6 +1,7 @@
 import {
     hit,
     nativeIsNaN,
+    setStorageItem,
 } from '../helpers/index';
 
 /* eslint-disable max-len */
@@ -84,18 +85,8 @@ export function setSessionStorageItem(source, key, value) {
         return;
     }
 
-    const setItem = (key, value) => {
-        const { sessionStorage } = window;
-        // setItem() may throw an exception if the storage is full.
-        try {
-            sessionStorage.setItem(key, value);
-            hit(source);
-        } catch (e) {
-            log(`Was unable to set sessionStorage item due to: ${e.message}`);
-        }
-    };
-
-    setItem(key, keyValue);
+    const { sessionStorage } = window;
+    setStorageItem(sessionStorage, key, keyValue, source.verbose);
 }
 
 setSessionStorageItem.names = [
@@ -105,4 +96,5 @@ setSessionStorageItem.names = [
 setSessionStorageItem.injections = [
     hit,
     nativeIsNaN,
+    setStorageItem,
 ];

src/scriptlets/trusted-set-local-storage-item.js

@@ -1,7 +1,7 @@
 import {
     hit,
     nativeIsNaN,
-    setItem,
+    setStorageItem,
     parseKeywordValue,
 } from '../helpers/index';
 
@@ -69,7 +69,8 @@ export function trustedSetLocalStorageItem(source, key, value) {
 
     const parsedValue = parseKeywordValue(value);
 
-    setItem(key, parsedValue);
+    const { localStorage } = window;
+    setStorageItem(localStorage, key, parsedValue, source.verbose);
     hit(source);
 }
 
@@ -81,6 +82,6 @@ trustedSetLocalStorageItem.names = [
 trustedSetLocalStorageItem.injections = [
     hit,
     nativeIsNaN,
-    setItem,
+    setStorageItem,
     parseKeywordValue,
 ];