admin/jasmine
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixed global variable leak when using ParallelReportDispatcher

Browse Source
This commit is contained in:
Steve Gravrock
2023-08-22 19:34:22 -07:00
parent 58bee05c36
commit 4ba42f3746
3 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
lib/jasmine-core/jasmine.js
View File

@@ -7161,6 +7161,8 @@ getJasmineRequireObj().NeverSkipPolicy = function(j$) {
}; };
getJasmineRequireObj().ParallelReportDispatcher = function(j$) { getJasmineRequireObj().ParallelReportDispatcher = function(j$) {
'use strict';
/** /**
* @class ParallelReportDispatcher * @class ParallelReportDispatcher
* @implements Reporter * @implements Reporter
@@ -7179,7 +7181,7 @@ getJasmineRequireObj().ParallelReportDispatcher = function(j$) {
const ReportDispatcher = deps.ReportDispatcher || j$.ReportDispatcher; const ReportDispatcher = deps.ReportDispatcher || j$.ReportDispatcher;
const QueueRunner = deps.QueueRunner || j$.QueueRunner; const QueueRunner = deps.QueueRunner || j$.QueueRunner;
const globalErrors = deps.globalErrors || new j$.GlobalErrors(); const globalErrors = deps.globalErrors || new j$.GlobalErrors();
const dispatcher = ReportDispatcher( const dispatcher = new ReportDispatcher(
j$.reporterEvents, j$.reporterEvents,
function(queueRunnerOptions) { function(queueRunnerOptions) {
queueRunnerOptions = { queueRunnerOptions = {
@@ -7905,6 +7907,8 @@ getJasmineRequireObj().QueueRunner = function(j$) {
}; };
getJasmineRequireObj().ReportDispatcher = function(j$) { getJasmineRequireObj().ReportDispatcher = function(j$) {
'use strict';
function ReportDispatcher(methods, queueRunnerFactory, onLateError) { function ReportDispatcher(methods, queueRunnerFactory, onLateError) {
const dispatchedMethods = methods || []; const dispatchedMethods = methods || [];

4
src/core/ParallelReportDispatcher.js
View File

@@ -1,4 +1,6 @@
getJasmineRequireObj().ParallelReportDispatcher = function(j$) { getJasmineRequireObj().ParallelReportDispatcher = function(j$) {
'use strict';
/** /**
* @class ParallelReportDispatcher * @class ParallelReportDispatcher
* @implements Reporter * @implements Reporter
@@ -17,7 +19,7 @@ getJasmineRequireObj().ParallelReportDispatcher = function(j$) {
const ReportDispatcher = deps.ReportDispatcher || j$.ReportDispatcher; const ReportDispatcher = deps.ReportDispatcher || j$.ReportDispatcher;
const QueueRunner = deps.QueueRunner || j$.QueueRunner; const QueueRunner = deps.QueueRunner || j$.QueueRunner;
const globalErrors = deps.globalErrors || new j$.GlobalErrors(); const globalErrors = deps.globalErrors || new j$.GlobalErrors();
const dispatcher = ReportDispatcher( const dispatcher = new ReportDispatcher(
j$.reporterEvents, j$.reporterEvents,
function(queueRunnerOptions) { function(queueRunnerOptions) {
queueRunnerOptions = { queueRunnerOptions = {

2
src/core/ReportDispatcher.js
View File

@@ -1,4 +1,6 @@
getJasmineRequireObj().ReportDispatcher = function(j$) { getJasmineRequireObj().ReportDispatcher = function(j$) {
'use strict';
function ReportDispatcher(methods, queueRunnerFactory, onLateError) { function ReportDispatcher(methods, queueRunnerFactory, onLateError) {
const dispatchedMethods = methods || []; const dispatchedMethods = methods || [];