Fix test for empty reporter option

MisanthropicBit · Feb 6, 2024 · 4a3162a · 4a3162a
1 parent 3be4ff7
commit 4a3162a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/mocha-multi-reporter.ts b/src/mocha-multi-reporter.ts
@@ -35,6 +35,7 @@ function parseReporterOptions(options: mocha.RunnerOptions): Record<string, Reco
 
     if (option.trim().length === 0) {
       console.error('Empty option for reporter')
+      continue
     }
 
     const [reporterName, configKey] = option.split(':')

diff --git a/tests/mocha-multi-reporter.test.ts b/tests/mocha-multi-reporter.test.ts
@@ -44,7 +44,7 @@ describe('mocha-multi-reporter', () => {
     expect(stderr, 'to equal', '')
   })
 
-  it.skip('reports empty option for reporter', async () => {
+  it('reports empty option for reporter', async () => {
     const command = [
       'mocha',
       '--reporter',
@@ -60,7 +60,7 @@ describe('mocha-multi-reporter', () => {
 
     expect(stdout, 'to contain', '"fullTitle"')
     expect(stdout, 'to contain', '1 passing')
-    expect(stderr, 'to equal', 'Empty option for reporter')
+    expect(stderr, 'to equal', 'Empty option for reporter\n')
   })
 
   it('reports unparsed reporter option', async () => {