Skip to content

Commit ce2b12e

Browse files
committed
adding notes before making change
unecessary dif temp add logs for pending tests fix linting issues undo unecessary change
1 parent 7007f13 commit ce2b12e

File tree

3 files changed

+26
-0
lines changed

3 files changed

+26
-0
lines changed

lib/reporters/default.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -370,8 +370,11 @@ export default class Reporter {
370370
}
371371

372372
this.lineWriter.writeLine(`${testsInFile.size} tests were pending in ${this.relativeFile(file)}\n`);
373+
const testTitleToLogs = evt.pendingTestsLogReference.get(file) ?? new Map();
373374
for (const title of testsInFile) {
375+
const logs = testTitleToLogs.get(title) ?? [];
374376
this.lineWriter.writeLine(`${figures.circleDotted} ${this.prefixTitle(file, title)}`);
377+
this.writeLogs({logs});
375378
}
376379

377380
this.lineWriter.writeLine('');

lib/run-status.js

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ export default class RunStatus extends Emittery {
99
super();
1010

1111
this.pendingTests = new Map();
12+
this.pendingTestsLogReference = new Map();
1213

1314
this.emptyParallelRun = parallelRuns
1415
&& parallelRuns.currentFileCount === 0
@@ -60,6 +61,7 @@ export default class RunStatus extends Emittery {
6061
});
6162

6263
this.pendingTests.set(testFile, new Set());
64+
this.pendingTestsLogReference.set(testFile, new Map());
6365
worker.onStateChange(data => this.emitStateChange(data));
6466
}
6567

@@ -124,22 +126,31 @@ export default class RunStatus extends Emittery {
124126
fileStats.remainingTests--;
125127
this.removePendingTest(event);
126128
break;
129+
case 'test-register-log-reference':
130+
this.addLogReference(event);
131+
break;
127132
case 'timeout':
128133
stats.timeouts++;
129134
event.pendingTests = this.pendingTests;
135+
event.pendingTestsLogReference = this.pendingTestsLogReference;
130136
this.pendingTests = new Map();
137+
this.pendingTestsLogReference = new Map();
131138
for (const testsInFile of event.pendingTests.values()) {
132139
stats.timedOutTests += testsInFile.size;
133140
}
134141

135142
break;
136143
case 'interrupt':
137144
event.pendingTests = this.pendingTests;
145+
event.pendingTestsLogReference = this.pendingTestsLogReference;
138146
this.pendingTests = new Map();
147+
this.pendingTestsLogReference = new Map();
139148
break;
140149
case 'process-exit':
141150
event.pendingTests = this.pendingTests;
151+
event.pendingTestsLogReference = this.pendingTestsLogReference;
142152
this.pendingTests = new Map();
153+
this.pendingTestsLogReference = new Map();
143154
break;
144155
case 'uncaught-exception':
145156
stats.uncaughtExceptions++;
@@ -198,6 +209,12 @@ export default class RunStatus extends Emittery {
198209
return 0;
199210
}
200211

212+
addLogReference(event) {
213+
if (this.pendingTestsLogReference.has(event.testFile)) {
214+
this.pendingTestsLogReference.get(event.testFile).set(event.title, event.logs);
215+
}
216+
}
217+
201218
addPendingTest(event) {
202219
if (this.pendingTests.has(event.testFile)) {
203220
this.pendingTests.get(event.testFile).add(event.title);

lib/runner.js

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -358,6 +358,12 @@ export default class Runner extends Emittery {
358358
notifyTimeoutUpdate: this.notifyTimeoutUpdate,
359359
});
360360

361+
this.emit('stateChange', {
362+
type: 'test-register-log-reference',
363+
title: task.title,
364+
logs: test.logs,
365+
});
366+
361367
const result = await this.runSingle(test);
362368
testOk = result.passed;
363369

0 commit comments

Comments
 (0)