From 5e25365997abdd5460cb6d7d98365dc247b734f0 Mon Sep 17 00:00:00 2001 From: Juan Escalada Date: Thu, 21 Nov 2024 11:56:05 +0900 Subject: [PATCH] Add unit test for checking invalid comma parsing Signed-off-by: Juan Escalada --- .../src/commands/api-reports/api-reports.test.ts | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/packages/repo-tools/src/commands/api-reports/api-reports.test.ts b/packages/repo-tools/src/commands/api-reports/api-reports.test.ts index 955720c52e..6401f64439 100644 --- a/packages/repo-tools/src/commands/api-reports/api-reports.test.ts +++ b/packages/repo-tools/src/commands/api-reports/api-reports.test.ts @@ -473,6 +473,17 @@ describe('buildApiReports', () => { outputDir: mockDir.resolve('node_modules/.cache/api-extractor'), }); }); + + it('should throw an error if omitMessages ends with a comma', async () => { + const opts = { + omitMessages: 'ae-missing-release-tag,', + }; + const paths = ['packages/*']; + + await expect(buildApiReports(paths, opts)).rejects.toThrow( + `Invalid value for --omit-messages: ${opts.omitMessages}\nMust be a comma-separated list of strings without spaces or wrapped in quotations.`, + ); + }); }); describe('isCI', () => { it('should set localBuild to false if CI option is passed', async () => {