✅ Add tests for backbone.sync.js

Author: wwwredfish

app/scripts/backbone.noworker.sync.js

@@ -14,7 +14,7 @@ define([
 ], function(Q, _, Sync, DB) {
     'use strict';
 
-    var Adapter = _.extend(Sync, {
+    var Adapter = _.extend({}, Sync, {
         promises: [],
 
         /**

app/scripts/backbone.sync.js

@@ -9,61 +9,77 @@
 define([
     'q',
     'underscore'
-], function(Q) {
+], function(Q, _) {
     'use strict';
 
     var Adapter = {
-        promises: [],
+        promises: {},
 
         /**
          * Create a new worker and start listening to its events.
          *
          * @return function
          */
         sync: function() {
-            var self = this,
-                sync;
+            _.bindAll(this, 'listenToWorker', 'backboneSync');
 
             this.worker = new Worker('scripts/workers/localForage.js');
 
             // Promise which signifies whether the worker is ready
             this.workerPromise = Q.defer();
 
-            this.worker.onmessage = function(data) {
-                var msg = data.data;
+            // Start listening to WebWorker events
+            this.worker.onmessage = this.listenToWorker;
 
-                switch (msg.msg) {
+            // A function for Backbone sync
+            return this.backboneSync;
+        },
 
-                    // Database webworker is ready
-                    case 'ready':
-                        self.workerPromise.resolve();
-                        break;
+        /**
+         * Resolve promises after receiving WebWorker messages.
+         *
+         * @type object data
+         */
+        listenToWorker: function(data) {
+            var msg = data.data;
 
-                    // Request was fullfilled
-                    case 'done':
-                        self.promises[msg.promiseId].resolve(msg.data);
-                        delete self.promises[msg.promiseId];
-                        break;
+            switch (msg.msg) {
 
-                    // Request failed with errors
-                    case 'fail':
-                        self.promises[msg.promiseId].reject(msg.data);
-                        delete self.promises[msg.promiseId];
-                        break;
+                // Database webworker is ready
+                case 'ready':
+                    this.workerPromise.resolve();
+                    break;
 
-                    default:
-                }
-            };
+                // Request was fullfilled
+                case 'done':
+                    this.promises[msg.promiseId].resolve(msg.data);
+                    delete this.promises[msg.promiseId];
+                    break;
 
-            // A function for Backbone sync
-            sync = function(method, model, options) {
-                return self.workerPromise.promise
-                .then(function() {
-                    return self[method](model, options);
-                });
-            };
-
-            return sync;
+                // Request failed with errors
+                case 'fail':
+                    this.promises[msg.promiseId].reject(msg.data);
+                    delete this.promises[msg.promiseId];
+                    break;
+
+                default:
+            }
+        },
+
+        /**
+         * With this method Backbone.sync will be overriden.
+         *
+         * @return promise
+         */
+        backboneSync: function(method, model, options) {
+
+            // First, make sure WebWorker is ready
+            return this.workerPromise.promise
+            .then(_.bind(function() {
+
+                // Execute the method (read, create, update)
+                return this[method](model, options);
+            }, this));
         },
 
         /**