Merge pull request #15615 from apalumbo/next

Trailing comma handling for "-s" command line paramenter

lib/core-server/src/cli/__tests__/utils.test.ts

@@ -0,0 +1,26 @@
+import { parseList } from '../utils';
+
+describe('parseList', () => {
+  test.each`
+    source                     | expected
+    ${'item1,item2, item3'}    | ${['item1', 'item2', 'item3']}
+    ${'item1 , item2, item3 '} | ${['item1', 'item2', 'item3']}
+  `(`Items will be trimmed, source "$source" should return $expected`, ({ source, expected }) => {
+    const result = parseList(source);
+    expect(result).toEqual(expected);
+  });
+
+  test.each`
+    source              | expected
+    ${'item1,item2, '}  | ${['item1', 'item2']}
+    ${'item1, ,item3 '} | ${['item1', 'item3']}
+    ${'item1,,item3 '}  | ${['item1', 'item3']}
+    ${'item1,, '}       | ${['item1']}
+  `(
+    `Empty items will be stripped, source "$source" should return $expected`,
+    ({ source, expected }) => {
+      const result = parseList(source);
+      expect(result).toEqual(expected);
+    }
+  );
+});

lib/core-server/src/cli/utils.ts

@@ -2,7 +2,10 @@ import deprecate from 'util-deprecate';
 import dedent from 'ts-dedent';
 
 export function parseList(str: string): string[] {
-  return str.split(',');
+  return str
+    .split(',')
+    .map((item) => item.trim())
+    .filter((item) => item.length > 0);
 }
 
 export function getEnvConfig(program: Record<string, any>, configEnv: Record<string, any>): void {