From de8290ebadf3add4c22c2ae5c968917431a63b3b Mon Sep 17 00:00:00 2001 From: Ansh Chaturvedi Date: Fri, 17 May 2024 11:09:10 -0400 Subject: [PATCH] refactor: assert error message when file cannot be parsed --- packages/openapi-generator/src/sourceFile.ts | 2 +- packages/openapi-generator/test/project.test.ts | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/packages/openapi-generator/src/sourceFile.ts b/packages/openapi-generator/src/sourceFile.ts index bd12baba..3b375c16 100644 --- a/packages/openapi-generator/src/sourceFile.ts +++ b/packages/openapi-generator/src/sourceFile.ts @@ -38,7 +38,7 @@ export async function parseSource( span: module.span, }; } catch (e: unknown) { - console.error('Error parsing source file: ', path, e); + console.error(`Error parsing source file: ${path}`, e); return undefined; } } diff --git a/packages/openapi-generator/test/project.test.ts b/packages/openapi-generator/test/project.test.ts index 02ef8cdd..60bebb2a 100644 --- a/packages/openapi-generator/test/project.test.ts +++ b/packages/openapi-generator/test/project.test.ts @@ -107,7 +107,9 @@ test('non-strict files are ignored and logged to stderr', async () => { console.error = (...args) => { errorCalled = true; - originalConsoleError(...args); + console.error = originalConsoleError; + const errorRegex = /Error parsing source file: \/index.ts/; + assert(errorRegex.test(args[0])); }; const project = new TestProject({ '/index.ts': NON_STRICT_MODE_SRC }, {}); @@ -116,6 +118,4 @@ test('non-strict files are ignored and logged to stderr', async () => { assert.strictEqual(sourceFile, undefined); assert.strictEqual(errorCalled, true, new Error('console.error was not called')); - - console.error = originalConsoleError; });