Stop extra semicolons in exclude paths from preventing results

pzavolinsky · Aug 31, 2023 · 8c8ccee · 8c8ccee
1 parent 28d4f1b
commit 8c8ccee
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/features/exclude-paths-from-report.feature b/features/exclude-paths-from-report.feature
@@ -16,3 +16,8 @@ Scenario: Ignore none
   Given file "exports.ts" is export const a = 1;
   When analyzing "tsconfig.json" with files ["--excludePathsFromReport=other-1;other-2"]
   Then the result is { "exports.ts": ["a"] }
+
+Scenario: Extra semi-colons doesn't filter all results
+  Given file "exports.ts" is export const a = 1;
+  When analyzing "tsconfig.json" with files ["--excludePathsFromReport=other-1;;other-2;"]
+  Then the result is { "exports.ts": ["a"] }
diff --git a/src/argsParser.ts b/src/argsParser.ts
@@ -43,7 +43,9 @@ function processOptions(
         {
           const paths = optionValue.split(';');
           paths.forEach((path) => {
-            pathsToExcludeFromReport.push(path);
+            if (path) {
+              pathsToExcludeFromReport.push(path);
+            }
           });
         }
         break;