Automatic registration of CollectTestExecutionMemoryFootprints

This patch allows for `CollectTestExecutionMemoryFootprints` to be automatically
registered when running `vendor/bin/roave-no-leaks`: this means that the
`phpunit.xml` changes are no longer required for end users

README.md

@@ -11,20 +11,6 @@ composer require --dev roave/no-leaks
 
 ## Usage
 
-In your `phpunit.xml` configuration, add following section:
-
-```xml
-    <listeners>
-        <listener class="Roave\NoLeaks\PHPUnit\CollectTestExecutionMemoryFootprints"/>
-    </listeners>
-
-    <extensions>
-        <extension class="Roave\NoLeaks\PHPUnit\CollectTestExecutionMemoryFootprints"/>
-    </extensions>
-```
-
-Then run:
-
 ```sh
 vendor/bin/roave-no-leaks
 ```

bin/roave-no-leaks.php

@@ -4,7 +4,9 @@
 
 namespace Roave\NoLeaks\CLI;
 
-use Roave\NoLeaks\PHPUnit\PHPUnitCommand;
+use PHPUnit\TextUI\Command;
+use PHPUnit\TextUI\TestRunner;
+use Roave\NoLeaks\PHPUnit\CollectTestExecutionMemoryFootprints;
 
 (function () {
     if (file_exists(__DIR__ . '/../vendor/autoload.php')) {
@@ -13,7 +15,33 @@
         require_once __DIR__ . '/../../../autoload.php';
     }
 
-    $_SERVER['argv'][] = '--repeat=3';
+    (new class extends Command
+    {
+        /** @var CollectTestExecutionMemoryFootprints */
+        private $collector;
 
-    PHPUnitCommand::main();
+        public function __construct()
+        {
+            $this->collector = new CollectTestExecutionMemoryFootprints();
+        }
+
+        protected function handleArguments(array $argv) : void
+        {
+            parent::handleArguments($argv);
+
+            $this->arguments['listeners'] = array_merge(
+                $this->arguments['listeners'] ?? [],
+                [$this->collector]
+            );
+        }
+
+        protected function createRunner() : TestRunner
+        {
+            $runner = parent::createRunner();
+
+            $runner->addExtension($this->collector);
+
+            return $runner;
+        }
+    })->run(array_merge($_SERVER['argv'], ['--repeat=3']), true);
 })();

phpunit.xml.dist

@@ -12,11 +12,6 @@
     verbose="true"
     executionOrder="random"
 >
-    <php>
-        <!-- to autoconfigure roave/no-leaks -->
-        <env name="REGISTER_NO_LEAKS" value="false" />
-    </php>
-
     <testsuites>
         <testsuite name="unit">
             <directory>./test/unit</directory>

test/e2e/phpunit-mock-suite.xml

@@ -11,14 +11,6 @@
     beStrictAboutTodoAnnotatedTests="true"
     verbose="true"
 >
-    <listeners>
-        <listener class="Roave\NoLeaks\PHPUnit\CollectTestExecutionMemoryFootprints"/>
-    </listeners>
-
-    <extensions>
-        <extension class="Roave\NoLeaks\PHPUnit\CollectTestExecutionMemoryFootprints"/>
-    </extensions>
-
     <testsuites>
         <testsuite name="Memleak detection e2e tests">
             <file>./LeakyIntegrationTest.php</file>