From a1c7a658a2104457b74deaf1860b3cc719754958 Mon Sep 17 00:00:00 2001 From: Floris Kraak Date: Mon, 5 Dec 2016 08:55:08 +0100 Subject: [PATCH] Initialize var suites early As https://github.com/karma-runner/karma-junit-reporter/issues/61#issuecomment-264468229 Minor cleanup. Doing it this way should be faster though (no constructor call overhead, less code to begin with). --- index.js | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/index.js b/index.js index 19b6c25..8452be5 100644 --- a/index.js +++ b/index.js @@ -20,7 +20,7 @@ var JUnitReporter = function (baseReporterDecorator, config, logger, helper, for var classNameFormatter = reporterConfig.classNameFormatter var properties = reporterConfig.properties - var suites + var suites = [] var pendingFileWritings = 0 var fileWritingFinished = function () {} var allMessages = [] @@ -107,8 +107,6 @@ var JUnitReporter = function (baseReporterDecorator, config, logger, helper, for // "run_start" - a test run is beginning for all browsers this.onRunStart = function (browsers) { - suites = Object.create(null) - // TODO(vojta): remove once we don't care about Karma 0.10 browsers.forEach(initializeXmlForBrowser) }