fix: deprecate na reporter (#1625)

* fix: deprecate na reporter * properly restore console
dequelabs · Jun 10, 2019 · 3e807f0 · 3e807f0
1 parent 96a1ea8
commit 3e807f0
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/lib/core/reporters/na.js b/lib/core/reporters/na.js
@@ -3,6 +3,10 @@
 axe.addReporter('na', function(results, options, callback) {
 	'use strict';
 
+	console.warn(
+		'"na" reporter will be deprecated in axe v4.0. Use the "v2" reporter instead.'
+	);
+
 	if (typeof options === 'function') {
 		callback = options;
 		options = {};

diff --git a/test/core/log.js b/test/core/log.js
@@ -5,10 +5,10 @@ describe('axe.log', function() {
 		assert.isFunction(axe.log);
 	});
 	it('should invoke console.log', function() {
+		var orig = window.console;
 		if (!window.console || window.console.log) {
 			window.console = { log: function() {} };
 		}
-		var orig = window.console.log;
 		var expected = ['hi', 'hello'];
 		var success = false;
 
@@ -21,6 +21,6 @@ describe('axe.log', function() {
 		axe.log.apply(axe.log, expected);
 		assert.isTrue(success);
 
-		window.console.log = orig;
+		window.console = orig;
 	});
 });