refactor doAction to take an object

Author: khalillechelt

.eslintrc

@@ -1,6 +1,7 @@
 {
   "extends": "eslint-config-cycle",
   "env": {
-    "browser": true
+    "browser": true,
+    "node": true
   }
 }

index.js

@@ -3,9 +3,7 @@ import responseCollection from './lib/responseCollection'
 
 export default function storageDriver(request$) {
   // Execute writing actions.
-  request$.subscribe((req) => {
-    doAction(req.target, req.action, req.key, req.value)
-  })
+  request$.subscribe((request) => doAction(request))
 
   // Return reading functions.
   return responseCollection

lib/doAction.js

@@ -2,17 +2,21 @@
  * @function doAction
  * @description
  * A universal write function for localStorage and sessionStorage.
- * @param {string} [target=local] - a string determines which storage to use
- * @param {string} [action=setItem] - a string determines the write action
- * @param {string} key - the key of a storage item
- * @param {string} value - the value of a storage item
+ * @param {object} request - the storage request object
+ * @param {string} request.target - a string determines which storage to use
+ * @param {string} request.action - a string determines the write action
+ * @param {string} request.key - the key of a storage item
+ * @param {string} request.value - the value of a storage item
  */
-export default function doAction(target = 'local', action = 'setItem', key = undefined, value = undefined) { // eslint-disable-line
+function doAction(request) {
+  const args = [request.target, request.action, request.key, request.value]
   // Store the third and the fourth parameter in a new array, if defined.
-  const storageArgs = Array.prototype.slice.call(arguments, 2)
+  const storageArgs = args.slice(2)
   // Determine the storage target.
-  const storage = target === `local` ? localStorage : sessionStorage
+  const storage = request.target === `local` ? localStorage : sessionStorage
 
   // Execute the storage action and pass arguments if they were defined.
-  storage[action](...storageArgs)
+  storage[request.action](...storageArgs)
 }
+
+export default doAction