Merge pull request #107 from Kanaye/observable-to-json

Added toJSON to observable and tests for it.
astoilkov · Oct 26, 2015 · 743bea5 · 743bea5
2 parents c552b7d + d34d60d
commit 743bea5
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/src/query/observable.js b/src/query/observable.js
@@ -312,6 +312,9 @@ define([
             return value.toString();
           }
           return String(value);
+        },
+        toJSON: function() {
+          return this();
         }
       },
 

diff --git a/test/spec/query/observables.js b/test/spec/query/observables.js
@@ -562,4 +562,21 @@
       expect(observable.update).toHaveBeenCalled();
     });
   });
+
+  it('can be JSON.stringified', function () {
+
+    var values = [42,  'some string', [1,2,3], {
+        a:'some string',
+        b: 42,
+        c: [1,2,34],
+        d: {a: 'hello'},
+        e: blocks.observable('that works ?')
+      }];
+
+    for (var i = 0; i < values.length; i++) {
+      var value = values[i];
+      expect(JSON.stringify(value)).toBe(JSON.stringify(blocks.observable(value)));
+    }
+
+  });
 })();