test: remove common.skipIfReportDisabled()

The report feature won't ever be disabled moving forward, so
checking for its existence in the tests is no longer needed.

PR-URL: nodejs#32242
Fixes: nodejs#26293
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: David Carlier <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>

Author: cjihrig

test/common/index.js

@@ -566,12 +566,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');
@@ -714,7 +708,6 @@ const common = {
   skipIf32Bits,
   skipIfEslintMissing,
   skipIfInspectorDisabled,
-  skipIfReportDisabled,
   skipIfWorker,
 
   get enoughTestCpu() {

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',

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') {

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');
 

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.');
 

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');

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);

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');

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');