diff --git a/test/common/index.js b/test/common/index.js index 28ce841c48cc3f..e77e7b95059947 100644 --- a/test/common/index.js +++ b/test/common/index.js @@ -553,12 +553,6 @@ function skipIfInspectorDisabled() { } } -function skipIfReportDisabled() { - if (!process.config.variables.node_report) { - skip('Diagnostic reporting is disabled'); - } -} - function skipIf32Bits() { if (bits < 64) { skip('The tested feature is not available in 32bit builds'); @@ -700,7 +694,6 @@ const common = { skipIf32Bits, skipIfEslintMissing, skipIfInspectorDisabled, - skipIfReportDisabled, skipIfWorker, get enoughTestCpu() { diff --git a/test/report/test-report-config.js b/test/report/test-report-config.js index 4f6fdf2a80e111..49781ac9582adc 100644 --- a/test/report/test-report-config.js +++ b/test/report/test-report-config.js @@ -1,7 +1,6 @@ // Flags: --experimental-report --report-on-fatalerror --report-on-signal --report-uncaught-exception 'use strict'; const common = require('../common'); -common.skipIfReportDisabled(); const assert = require('assert'); common.expectWarning('ExperimentalWarning', diff --git a/test/report/test-report-fatal-error.js b/test/report/test-report-fatal-error.js index 8ecd058cf8a4de..60eeef6c454ba7 100644 --- a/test/report/test-report-fatal-error.js +++ b/test/report/test-report-fatal-error.js @@ -1,7 +1,6 @@ 'use strict'; const common = require('../common'); -common.skipIfReportDisabled(); const assert = require('assert'); // Testcase to produce report on fatal error (javascript heap OOM) if (process.argv[2] === 'child') { diff --git a/test/report/test-report-getreport.js b/test/report/test-report-getreport.js index ba645df69fdd7b..efadaa73f10cb9 100644 --- a/test/report/test-report-getreport.js +++ b/test/report/test-report-getreport.js @@ -1,7 +1,6 @@ // Flags: --experimental-report 'use strict'; const common = require('../common'); -common.skipIfReportDisabled(); const assert = require('assert'); const helper = require('../common/report'); diff --git a/test/report/test-report-signal.js b/test/report/test-report-signal.js index 51244fcade2196..c1c98d2588aba5 100644 --- a/test/report/test-report-signal.js +++ b/test/report/test-report-signal.js @@ -2,7 +2,7 @@ 'use strict'; // Test producing a report via signal. const common = require('../common'); -common.skipIfReportDisabled(); + if (common.isWindows) return common.skip('Unsupported on Windows.'); diff --git a/test/report/test-report-uncaught-exception.js b/test/report/test-report-uncaught-exception.js index 4a2627c13c88ff..d35b5f276f9f4e 100644 --- a/test/report/test-report-uncaught-exception.js +++ b/test/report/test-report-uncaught-exception.js @@ -2,7 +2,6 @@ 'use strict'; // Test producing a report on uncaught exception. const common = require('../common'); -common.skipIfReportDisabled(); const assert = require('assert'); const helper = require('../common/report'); const tmpdir = require('../common/tmpdir'); diff --git a/test/report/test-report-uv-handles.js b/test/report/test-report-uv-handles.js index ea0c189b859c1a..bf5e53d154b32d 100644 --- a/test/report/test-report-uv-handles.js +++ b/test/report/test-report-uv-handles.js @@ -5,7 +5,6 @@ const common = require('../common'); if (common.isIBMi) common.skip('IBMi does not support fs.watch()'); -common.skipIfReportDisabled(); if (process.argv[2] === 'child') { // Exit on loss of parent process const exit = () => process.exit(2); diff --git a/test/report/test-report-worker.js b/test/report/test-report-worker.js index 39a2c87f65f044..6e77deecf57137 100644 --- a/test/report/test-report-worker.js +++ b/test/report/test-report-worker.js @@ -1,7 +1,6 @@ // Flags: --experimental-report 'use strict'; const common = require('../common'); -common.skipIfReportDisabled(); const assert = require('assert'); const { Worker } = require('worker_threads'); const { once } = require('events'); diff --git a/test/report/test-report-writereport.js b/test/report/test-report-writereport.js index a72744fcd4fcb7..f04a19f3e01bc0 100644 --- a/test/report/test-report-writereport.js +++ b/test/report/test-report-writereport.js @@ -3,7 +3,6 @@ // Test producing a report via API call, using the no-hooks/no-signal interface. const common = require('../common'); -common.skipIfReportDisabled(); const assert = require('assert'); const { spawnSync } = require('child_process'); const fs = require('fs');