Added "test.dp.args" config option.

Author: Bilge

README.md

@@ -55,11 +55,12 @@ Pip makes no attempt to modify the test summary; only runtime output is changed.
 
 Pip's behaviour can be customized by adding `<parameter>` nodes as children of the `<bootstrap>` node in `phpunit.xml`, with `name` and `value` attributes corresponding to the table below.
 
-| Parameter name  | Default value | Description                                                |
-|-----------------|---------------|------------------------------------------------------------|
-| perf.slow       | 200 (ms)      | Sets the performance threshold for _slow_ (yellow) tests   |
-| perf.vslow      | 1000 (ms)     | Sets the performance threshold for _very slow_ (red) tests |
-| test.name.strip | ''            | Strips the specified matching portion of the test name     |
+| Parameter name  | Default value | Description                                                            |
+|-----------------|---------------|------------------------------------------------------------------------|
+| perf.slow       | 200 (ms)      | Sets the performance threshold for _slow_ (yellow) tests               |
+| perf.vslow      | 1000 (ms)     | Sets the performance threshold for _very slow_ (red) tests             |
+| test.dp.args    | true          | True to show the arguments passed by the data provider, false to hide. |
+| test.name.strip | ''            | Strips the specified matching portion of the test name                 |
 
 ## Requirements
 

src/PipConfig.php

@@ -7,5 +7,6 @@ final class PipConfig
 {
     public int $perfSlow = 200;
     public int $perfVslow = 1_000;
+    public bool $testDpArgs = true;
     public string $testNameStrip = '';
 }

src/PipExtension.php

@@ -15,6 +15,7 @@ public function bootstrap(Configuration $configuration, Facade $facade, Paramete
         $config = new PipConfig();
         $parameters->has('perf.slow') && $config->perfSlow = +$parameters->get('perf.slow');
         $parameters->has('perf.vslow') && $config->perfVslow = +$parameters->get('perf.vslow');
+        $parameters->has('test.dp.args') && $config->testDpArgs = $parameters->get('test.dp.args') === 'true';
         $parameters->has('test.name.strip') && $config->testNameStrip = $parameters->get('test.name.strip');
 
         $facade->registerTracer(new Printer($config));

src/Printer.php

@@ -102,7 +102,15 @@ public function trace(Event $event): void
             // Data provider case.
             if ($event->test()->isTestMethod() && $event->test()->testData()->hasDataFromDataProvider()) {
                 $id = substr($id, 0, strrpos($id, '#'));
-                $id .= $event->test()->testData()->dataFromDataProvider()->dataAsStringForResultOutput();
+
+                $data = $event->test()->testData()->dataFromDataProvider()->dataAsStringForResultOutput();
+                if (!$this->config->testDpArgs) {
+                    $data = substr($data, 0, 17 + strlen(
+                        $event->test()->testData()->dataFromDataProvider()->dataSetName())
+                    );
+                }
+
+                $id .= $data;
             }
 
             $ms = round($event->telemetryInfo()->time()->duration($this->start)->asFloat() * 1_000);

test/config/TestDpArgs.xml

@@ -0,0 +1,7 @@
+<phpunit>
+  <extensions>
+    <bootstrap class="ScriptFUSION\Pip\PipExtension">
+      <parameter name="test.dp.args" value="false"/>
+    </bootstrap>
+  </extensions>
+</phpunit>

test/config/test dp args.phpt

@@ -0,0 +1,22 @@
+--TEST--
+Tests that when "test dp args" is disabled, the arguments passed from the data provider are hidden in the output.
+
+--ARGS--
+-c test/config/TestDpArgs.xml --colors=always test/CapabilitiesTest.php --filter ::testDataProvider\h
+
+--FILE_EXTERNAL--
+../PHPUnit runner.php
+
+--EXPECTF--
+PHPUnit %s
+
+Runtime: %s
+Configuration: %s
+
+ 50% . ScriptFUSIONTest\Pip\CapabilitiesTest::testDataProvider with data set "foo" (%d ms)
+100% . ScriptFUSIONTest\Pip\CapabilitiesTest::testDataProvider with data set "baz" (%d ms)
+
+
+Time: %s
+
+OK (2 tests, 2 assertions)