Merge pull request #9 from Swaagie/clone

[fix] Clone by default.

lib/memory.js

@@ -1,6 +1,7 @@
 'use strict';
 
 var fuse = require('fusing')
+  , clone = require('clone')
   , has = Object.prototype.hasOwnProperty;
 
 /**
@@ -45,7 +46,7 @@ Memory.readable('api', function api(dynamis) {
    * @api private
    */
   function get(key, fn) {
-    fn(undefined, memory.database[key]);
+    fn(undefined, clone(memory.database[key]));
   }
 
   /**

package.json

@@ -33,6 +33,7 @@
   "homepage": "https://github.com/swaagie/dynamis",
   "dependencies": {
     "async": "~1.4.0",
+    "clone": "~1.0.2",
     "debug": "~2.2.0",
     "eventemitter3": "~1.1.1",
     "fusing": "~1.0.0"

test/lib/memory.test.js

@@ -108,6 +108,22 @@ describe('Memory persistence layer', function () {
         done();
       });
     });
+
+    it('returns a clone of the set object', function (done) {
+      dynamis.set('foo', { example: 'lol' }, function () {
+        dynamis.get('foo', function (error, result) {
+          expect(result).to.be.a('object');
+          expect(result.lol).to.equal(undefined);
+
+          result.lol = 1;
+
+          dynamis.get('foo', function (error, res) {
+            expect(res.lol).to.equal(undefined);
+            done();
+          });
+        });
+      });
+    });
   });
 
   describe('#del', function () {