From e8995d1b80e68244ff6e56fa24eb201374d484d2 Mon Sep 17 00:00:00 2001 From: cjihrig Date: Sat, 23 Feb 2019 09:52:03 -0500 Subject: [PATCH] test: increase getReport() coverage PR-URL: https://github.com/nodejs/node/pull/26276 Reviewed-By: Richard Lau Reviewed-By: James M Snell --- test/node-report/test-api-getreport.js | 21 +++++++++++++++++++-- 1 file changed, 19 insertions(+), 2 deletions(-) diff --git a/test/node-report/test-api-getreport.js b/test/node-report/test-api-getreport.js index 780a5147b3f100..9f40e61c2e7efe 100644 --- a/test/node-report/test-api-getreport.js +++ b/test/node-report/test-api-getreport.js @@ -8,5 +8,22 @@ const helper = require('../common/report'); common.expectWarning('ExperimentalWarning', 'report is an experimental feature. This feature could ' + 'change at any time'); -helper.validateContent(process.report.getReport()); -assert.deepStrictEqual(helper.findReports(process.pid, process.cwd()), []); + +{ + // Test with no arguments. + helper.validateContent(process.report.getReport()); + assert.deepStrictEqual(helper.findReports(process.pid, process.cwd()), []); +} + +{ + // Test with an error argument. + helper.validateContent(process.report.getReport(new Error('test error'))); + assert.deepStrictEqual(helper.findReports(process.pid, process.cwd()), []); +} + +// Test with an invalid error argument. +[null, 1, Symbol(), function() {}, 'foo'].forEach((error) => { + common.expectsError(() => { + process.report.getReport(error); + }, { code: 'ERR_INVALID_ARG_TYPE' }); +});