-
-
Notifications
You must be signed in to change notification settings - Fork 315
Open
Description
Given:
src/index.spec.ts
describe('a test suite', () => {
it('should pass', () => {
expect(true).toBeTrue();
});
it('should fail', () => {
expect(true).toBeFalse();
});
describe('an inner test suite', () => {
it("should pass 2", () => {
expect(undefined).toBeUndefined();
});
it("should fail 2", () => {
expect(undefined).toBeTrue();
});
describe('another inner test suite', () => {
it("should pass 3", () => {
expect(null).toBeNull();
});
it("should fail 3", () => {
expect(null).toBeTrue();
});
});
});
});
Expected:
Actual:
Quick Analysis:
Seems to work by calling logSuite
instead of logResults
, in line 97
web/packages/test-runner/src/reporter/summaryReporter.ts
Lines 94 to 101 in d5ae228
reportTestFileResults({ logger, sessionsForTestFile }) { | |
cachedLogger = logger; | |
for (const session of sessionsForTestFile) { | |
logResults(logger, session.testResults, '', session.browser); | |
logger.log(''); | |
} | |
reportBrowserLogs(logger, sessionsForTestFile); | |
}, |
However, not sure if it's expected to have the browser name on all testSuite lines, or only the root one.
gian1200
Metadata
Metadata
Assignees
Labels
No labels