MOB-44726: Add possibility to. add customer reporters (#1962)

* MOB-44726: Fix playwright custom reporter

* MOB-44726: Add possibility to. add customer reporters

Author: henrychv

bzt/modules/javascript.py

@@ -149,14 +149,24 @@ def startup(self):
 
         reporter = "@taurus/playwright-custom-reporter"
 
+        # Add custom reporters if specified in scenario config
+        custom_reporters = self.get_scenario().get("reporters", [])
+        has_additional_reporter = False
+        if custom_reporters and isinstance(custom_reporters, list):
+            # Sanitize reporter names for command line usage
+            safe_reporters = [r.translate(str.maketrans("", "", " \t\r\n\v\f'\""))
+                              for r in custom_reporters if r and isinstance(r, str)]
+            if safe_reporters:
+                reporter = reporter + "," + ",".join(safe_reporters)
+                has_additional_reporter = True
+
         # self.env.set({"TAURUS_PWREPORT_VERBOSE": "true"})
-        # TODO: set to false if we will add support for customer reporter
-        #  - keep stdout to customer reporter (or default)
-        self.env.set({"TAURUS_PWREPORT_STDOUT": "true"})
+        # Keep stdout for custom reporters if any
+        self.env.set({"TAURUS_PWREPORT_STDOUT": "true" if not has_additional_reporter else "false"})
         self.env.set({"TAURUS_PWREPORT_DIR": self.engine.artifacts_dir})
         if max_duration:
             self.env.set({"TAURUS_PWREPORT_DURATION": str(int(max_duration * 1000))})
-        options = ["--reporter " + reporter,
+        options = ["--reporter \"" + reporter + "\"",
                    "--output " + self.engine.artifacts_dir + "/test-output",
                    "--workers " + str(concurrency),
                    "--repeat-each " + str(repeat_each)]

bzt/resources/playwright-custom-reporter/playwright-custom-reporter.js

@@ -40,7 +40,7 @@ class TaurusReporter {
     }
 
     if (fs.existsSync(this.options.outputFile)) {
-      fs.rmSync(p, {
+      fs.rmSync(this.options.outputFile, {
         force: true,
         maxRetries: 2
       });

tests/unit/modules/_selenium/test_javascript.py

@@ -263,6 +263,35 @@ def test_command_line(self):
         self.assertIn("--repeat-each 30", self.CMD_LINE)
         self.assertIn("--workers 10", self.CMD_LINE)
         self.assertIn("-project=firefox", self.CMD_LINE)
+        self.assertIn("-reporter \"@taurus/playwright-custom-reporter\"", self.CMD_LINE)
+        self.assertIn("-g 'has title'", self.CMD_LINE)
+        self.assertEqual('60000', self.ENV.get("TAURUS_PWREPORT_DURATION", "undefined"))
+
+    def test_command_line_additional_reporter(self):
+        self.simple_run({
+            'execution': {
+                'iterations': 3,
+                'concurrency': 10,
+                'hold-for': '1m',
+                'settings': {
+                    'env': {
+                        'BASE_URL': 'https://blazedemo.com/'
+                    }
+                },
+                'scenario': {
+                    "script": RESOURCES_DIR + "playwright",
+                    'browser': 'firefox',
+                    'test': 'has title',
+                    'reporters': ['"json" ']
+                },
+                'executor': 'playwright',
+            },
+        })
+        self.assertIn("npx playwright test", self.CMD_LINE)
+        self.assertIn("--repeat-each 30", self.CMD_LINE)
+        self.assertIn("--workers 10", self.CMD_LINE)
+        self.assertIn("-project=firefox", self.CMD_LINE)
+        self.assertIn("-reporter \"@taurus/playwright-custom-reporter,json\"", self.CMD_LINE)
         self.assertIn("-g 'has title'", self.CMD_LINE)
         self.assertEqual('60000', self.ENV.get("TAURUS_PWREPORT_DURATION", "undefined"))