From d6668292604bc9eb3080c12c39b94176842907da Mon Sep 17 00:00:00 2001 From: Matthew Dean Date: Fri, 25 Mar 2016 14:43:56 -0700 Subject: [PATCH] Fix for Jasmine tests - don't log messages in production --- test/browser/common.js | 37 ++++++++++++++++++----- test/browser/runner-production-options.js | 2 +- 2 files changed, 30 insertions(+), 9 deletions(-) diff --git a/test/browser/common.js b/test/browser/common.js index 6ab2da68..887e9cc6 100644 --- a/test/browser/common.js +++ b/test/browser/common.js @@ -6,19 +6,40 @@ jasmine.getEnv().addReporter(new jasmine.JSReporter2()); var logMessages = []; window.less = window.less || {}; + +var logLevel_debug = 4, + logLevel_info = 3, + logLevel_warn = 2, + logLevel_error = 1; + +// The amount of logging in the javascript console. +// 3 - Debug, information and errors +// 2 - Information and errors +// 1 - Errors +// 0 - None +// Defaults to 2 + less.loggers = [ { - info: function (msg) { - logMessages.push(msg); + debug: function(msg) { + if (less.options.logLevel >= logLevel_debug) { + logMessages.push(msg); + } }, - debug: function (msg) { - logMessages.push(msg); + info: function(msg) { + if (less.options.logLevel >= logLevel_info) { + logMessages.push(msg); + } }, - warn: function (msg) { - logMessages.push(msg); + warn: function(msg) { + if (less.options.logLevel >= logLevel_warn) { + logMessages.push(msg); + } }, - error: function (msg) { - logMessages.push(msg); + error: function(msg) { + if (less.options.logLevel >= logLevel_error) { + logMessages.push(msg); + } } } ]; diff --git a/test/browser/runner-production-options.js b/test/browser/runner-production-options.js index c93773f2..a17f8a95 100644 --- a/test/browser/runner-production-options.js +++ b/test/browser/runner-production-options.js @@ -1,3 +1,3 @@ -var less = {logLevel: 4, +var less = {logLevel: 1, errorReporting: "console"}; less.env = "production";