From 8814d03d4d397a655956f8a57ba914cf2146fa79 Mon Sep 17 00:00:00 2001 From: cjihrig Date: Thu, 28 Feb 2019 21:03:31 -0500 Subject: [PATCH] doc,lib,test: rename node-report to report This commit completes the renaming of node-report to report in order to better differentiate core's reporting from the node-report npm module. PR-URL: https://github.com/nodejs/node/pull/26371 Reviewed-By: Richard Lau Reviewed-By: Rich Trott Reviewed-By: Minwoo Jung Reviewed-By: James M Snell --- doc/api/report.md | 6 +++--- lib/internal/bootstrap/pre_execution.js | 2 +- lib/internal/process/execution.js | 4 ++-- test/common/index.js | 2 +- 4 files changed, 7 insertions(+), 7 deletions(-) diff --git a/doc/api/report.md b/doc/api/report.md index 419fe33417345d..90173a1cfaa154 100644 --- a/doc/api/report.md +++ b/doc/api/report.md @@ -32,7 +32,7 @@ is provided below for reference. "/home/nodeuser/project/node/out/Release/node", "--experimental-report", "--diagnostic-report-uncaught-exception", - "/home/nodeuser/project/node/test/node-report/test-exception.js", + "/home/nodeuser/project/node/test/report/test-exception.js", "child" ], "nodejsVersion": "v12.0.0-pre", @@ -66,8 +66,8 @@ is provided below for reference. "javascriptStack": { "message": "Error: *** test-exception.js: throwing uncaught Error", "stack": [ - "at myException (/home/nodeuser/project/node/test/node-report/test-exception.js:9:11)", - "at Object. (/home/nodeuser/project/node/test/node-report/test-exception.js:12:3)", + "at myException (/home/nodeuser/project/node/test/report/test-exception.js:9:11)", + "at Object. (/home/nodeuser/project/node/test/report/test-exception.js:12:3)", "at Module._compile (internal/modules/cjs/loader.js:718:30)", "at Object.Module._extensions..js (internal/modules/cjs/loader.js:729:10)", "at Module.load (internal/modules/cjs/loader.js:617:32)", diff --git a/lib/internal/bootstrap/pre_execution.js b/lib/internal/bootstrap/pre_execution.js index 072112e68ba07f..25becacf28cf5e 100644 --- a/lib/internal/bootstrap/pre_execution.js +++ b/lib/internal/bootstrap/pre_execution.js @@ -11,7 +11,7 @@ function prepareMainThreadExecution() { // Only main thread receives signals. setupSignalHandlers(); - // Process initial configurations of node-report, if any. + // Process initial diagnostic reporting configuration, if present. initializeReport(); initializeReportSignalHandlers(); // Main-thread-only. diff --git a/lib/internal/process/execution.js b/lib/internal/process/execution.js index a35feaacce28ff..575484d509a491 100644 --- a/lib/internal/process/execution.js +++ b/lib/internal/process/execution.js @@ -102,7 +102,7 @@ function createFatalException() { // call that threw and was never cleared. So clear it now. clearDefaultTriggerAsyncId(); - // If node-report is enabled, call into its handler to see + // If diagnostic reporting is enabled, call into its handler to see // whether it is interested in handling the situation. // Ignore if the error is scoped inside a domain. // use == in the checks as we want to allow for null and undefined @@ -119,7 +119,7 @@ function createFatalException() { report.onUnCaughtException(er ? er.stack : undefined); } } - } catch {} // NOOP, node_report unavailable. + } catch {} // Ignore the exception. Diagnostic reporting is unavailable. } if (exceptionHandlerState.captureFn !== null) { diff --git a/test/common/index.js b/test/common/index.js index 26a8fb4befa6c2..efb13c3f675e1a 100644 --- a/test/common/index.js +++ b/test/common/index.js @@ -635,7 +635,7 @@ function skipIfInspectorDisabled() { function skipIfReportDisabled() { if (!process.config.variables.node_report) { - skip('Node Report is disabled'); + skip('Diagnostic reporting is disabled'); } }