test: increase triggerReport() coverage

PR-URL: #26268 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · Feb 26, 2019 · 72cf8a3 · 72cf8a3
1 parent cae7232
commit 72cf8a3
Showing 1 changed file with 14 additions and 0 deletions.
diff --git a/test/node-report/test-api-nohooks.js b/test/node-report/test-api-nohooks.js
@@ -67,3 +67,17 @@ function validate() {
   helper.validate(filename);
   fs.unlinkSync(filename);
 }
+
+// Test with an invalid file argument.
+[null, 1, Symbol(), function() {}].forEach((file) => {
+  common.expectsError(() => {
+    process.report.triggerReport(file);
+  }, { code: 'ERR_INVALID_ARG_TYPE' });
+});
+
+// Test with an invalid error argument.
+[null, 1, Symbol(), function() {}, 'foo'].forEach((error) => {
+  common.expectsError(() => {
+    process.report.triggerReport('file', error);
+  }, { code: 'ERR_INVALID_ARG_TYPE' });
+});