test: change jenkins reporter

Ceres6 · targos · commit abca97f7e249 · 2025-02-05T10:35:23.000+01:00
PR-URL: #56808 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
diff --git a/test/es-module/test-esm-no-addons.mjs b/test/es-module/test-esm-no-addons.mjs
@@ -15,8 +15,7 @@ if (isMainThread) {
     loadFixture('pkgexports/no-addons').then(
       mustCall((module) => {
         const message = module.default;
-
-        if (process.execArgv.length === 0) {
+        if (!process.execArgv.includes('--no-addons')) {
           assert.strictEqual(message, 'using native addons');
         } else {
           assert.strictEqual(message, 'not using native addons');
diff --git a/tools/test.py b/tools/test.py
@@ -316,9 +316,7 @@ def HasRun(self, output):
 
 class ActionsAnnotationProgressIndicator(DotsProgressIndicator):
   def AboutToRun(self, case):
-    case.additional_flags = case.additional_flags.copy() if hasattr(case, 'additional_flags') else []
-    case.additional_flags.append('--test-reporter=./test/common/test-error-reporter.js')
-    case.additional_flags.append('--test-reporter-destination=stdout')
+    pass
 
   def GetAnnotationInfo(self, test, output):
     traceback = output.stdout + output.stderr
@@ -1448,6 +1446,9 @@ def BuildOptions():
   result.add_option("--type",
       help="Type of build (simple, fips, coverage)",
       default=None)
+  result.add_option("--error-reporter",
+      help="use error reporter",
+      default=True, action="store_true")
   return result
 
 
@@ -1663,6 +1664,10 @@ def Main():
     options.node_args.append("--always-turbofan")
     options.progress = "deopts"
 
+  if options.error_reporter:
+    options.node_args.append('--test-reporter=./test/common/test-error-reporter.js')
+    options.node_args.append('--test-reporter-destination=stdout')
+
   if options.worker:
     run_worker = join(workspace, "tools", "run-worker.js")
     options.node_args.append(run_worker)
