Restore the native JSON.parse (#482)

macbre · May 30, 2015 · 84e8af9 · 84e8af9
1 parent 53a3d6d
commit 84e8af9
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 0 deletions.
diff --git a/core/phantomas.js b/core/phantomas.js
@@ -514,6 +514,11 @@ phantomas.prototype = {
 
 	// called when all HTTP requests are completed
 	report: function() {
+		// restote the native JSON.parse (just in case, see #482)
+		this.page.evaluate(function() {
+			JSON.parse = window.__phantomas && window.__phantomas.JSON.parse;
+		});
+
 		this.emitInternal('report'); // @desc the report is about to be generated
 
 		var time = Date.now() - this.start;

diff --git a/core/scope.js b/core/scope.js
@@ -9,6 +9,12 @@
 	// create a scope
 	var phantomas = (scope.__phantomas = scope.__phantomas || {});
 
+	// keep the original JSON functions (#482)
+	phantomas.JSON = {
+		parse: JSON.parse,
+		stringify: JSON.stringify
+	};
+
 	// NodeRunner
 	var nodeRunner = function() {
 		// "Beep, Beep"

diff --git a/test/integration-spec.yaml b/test/integration-spec.yaml
@@ -207,3 +207,4 @@
   metrics:
     globalVariables: 3
     globalVariablesFalsy: 1
+    jsErrors: 0