add tests and make sure exposed JSON doesnt override the real window.…

…JSON object

Makefile

@@ -8,4 +8,7 @@ components:
 clean:
 	rm -fr build components template.js
 
-.PHONY: clean
+test:
+	@component test phantom
+
+.PHONY: clean test

component.json

@@ -5,8 +5,8 @@
   "version": "0.0.1",
   "keywords": ["json", "fallback"],
   "dependencies": {},
-  "development": {},
-  "scripts": [
-    "index.js"
-  ]
-}
+  "scripts": ["index.js"],
+  "development": {
+    "component/assert": "*"
+  }
+}

index.js

@@ -159,13 +159,11 @@
 // Create a JSON object only if one does not already exist. We create the
 // methods in a closure to avoid creating global variables.
 
-if (typeof JSON !== 'object') {
-    JSON = {};
-}
-
 (function () {
     'use strict';
 
+    var JSON = module.exports = {};
+
     function f(n) {
         // Format integers to have at least two digits.
         return n < 10 ? '0' + n : n;
@@ -487,5 +485,3 @@ if (typeof JSON !== 'object') {
         };
     }
 }());
-
-module.exports = JSON;

test/test.js

@@ -0,0 +1,17 @@
+
+describe('JSON', function(){
+  var json = require('json-fallback');
+  var assert = require('assert');
+
+  it('should not leak to the global scope', function(){
+    assert(json != window.JSON);
+  })
+
+  it('should parse json', function(){
+    assert(0 == json.parse('[0]')[0]);
+  })
+
+  it('should stringify json', function(){
+    assert('[0]' == json.stringify([0]));
+  })
+})