fix: do not swallow failing test results in the default reporter

Open mokimo opened this issue 2 years ago • 5 comments

What I did

Fix an issue that leads to failing tests to be potentially swallowed from the defaultReporter import { defaultReporter } from '@web/test-runner';

It's a very tedious issue to debug and might fail tests to fail on CI systems as well as locally.

Reproduceable test

import { readFile } from '@web/test-runner-commands';
import { expect } from '@esm-bundle/chai';

describe('minimal reproducible example', () => {
  it('this test works', async () => {
    document.body.innerHTML = await readFile({ path: './mocks/just-return-empty-body.html' });
    expect(true).to.be.true;
  });

  describe('this describe fails', async () => {
    document.body.innerHTML = await readFile({ path: './mocks/just-return-empty-body-2.html' });

    it('failing test', () => {
      expect(true).to.be.true;
    });
  });
});