Log all tests by default except console, which limits to 5

Author: Lloople

README.md

@@ -54,7 +54,7 @@ Write the tests in a CSV file ready for import.
 
 Default options are:
 
-- rows: `5`
+- rows: `null` (all the tests)
 - file: `phpunit_results.csv`
 
 ## Json
@@ -67,7 +67,7 @@ Write the tests in a JSON file ready for import.
 
 Default options are:
 
-- rows: `5`
+- rows: `null` (all the tests)
 - file: `phpunit_results.json`
 
 ### MySQL
@@ -80,7 +80,7 @@ Store the test name and the time into a MySQL database. It will override existin
 
 Default credentials are (as array):
 
-- rows: `5`
+- rows: `null` (all the tests)
 - database: `phpunit_results`
 - table: `default`
 - username: `root`
@@ -97,10 +97,52 @@ Store the test name and the time into a SQLite database. It will override existi
 
 Default credentials are (as array):
 
-- rows: `5`
+- rows: `null` (all the tests)
 - database: `phpunit_results.db`
 - table: `default`
 
+## Arguments
+
+To override the default configuration per extension, you need to use `<arguments>`in your `phpunit.xml` file
+
+```xml
+<extension class="Lloople\PHPUnitExtensions\Runners\SlowestTests\Json">
+  <arguments>
+    <integer>10</integer>
+    <string>phpunit_results_as_json.json</string>
+  </arguments>
+</extension>
+```
+
+In the case of the MySQL and SQLite, which needs a database connection, configuration goes as array
+
+<extension class="Lloople\PHPUnitExtensions\Runners\SlowestTests\MySQL">
+  <arguments>
+    <null/> <!-- This allows you to log all the tests -->
+    <array>
+      <element key="database">
+        <string>my_phpunit_results</string>
+      </element>
+      <element key="table">
+        <string>project1_test_results</string>
+      </element>
+      <element key="username">
+        <string>homestead</string>
+      </element>
+      <element key="password">
+        <string>secret</string>
+      </element>
+      <element key="host">
+        <string>192.168.12.14</string>
+      </element>
+    </array>
+  </arguments>
+</extension>
+```
+
+You don't need to override those credentials that already fit to your 
+usecase, since the class will merge your configuration with the default one
+
 ### Changelog
 
 Please see [CHANGELOG](CHANGELOG.md) for more information what has changed recently.

src/Runners/SlowestTests/Channel.php

@@ -11,7 +11,7 @@ abstract class Channel implements AfterTestHook, AfterLastTestHook
 
     protected $rows;
 
-    public function __construct(int $rows = 5)
+    public function __construct(?int $rows = null)
     {
         $this->rows = $rows;
     }
@@ -42,6 +42,10 @@ abstract protected function printResults(): void;
 
     protected function testsToPrint(): array
     {
+        if ($this->rows === null) {
+            return $this->tests;
+        }
+
         return array_slice($this->tests, 0, $this->rows, true);
     }
 

src/Runners/SlowestTests/Console.php

@@ -4,6 +4,12 @@
 
 class Console extends Channel
 {
+
+    public function __construct(?int $rows = 5)
+    {
+        parent::__construct($rows);
+    }
+    
     protected function printResults(): void
     {
         echo PHP_EOL . "Showing the top {$this->rows} slowest tests:" . PHP_EOL;

src/Runners/SlowestTests/Csv.php

@@ -6,9 +6,10 @@ class Csv extends Channel
 {
     protected $file;
 
-    public function __construct(int $rows = 5, string $file = 'phpunit_results.csv')
+    public function __construct(?int $rows = null, string $file = 'phpunit_results.csv')
     {
-        $this->rows = $rows;
+        parent::__construct($rows);
+
         $this->file = $file;
     }
 

src/Runners/SlowestTests/Json.php

@@ -6,9 +6,10 @@ class Json extends Channel
 {
     protected $file;
 
-    public function __construct(int $rows = 5, string $file = 'phpunit_results.json')
+    public function __construct(?int $rows = null, string $file = 'phpunit_results.json')
     {
-        $this->rows = $rows;
+        parent::__construct($rows);
+
         $this->file = $file;
     }
 

src/Runners/SlowestTests/MySQL.php

@@ -17,7 +17,7 @@ class MySQL extends Channel
         'host' => '127.0.0.1'
     ];
 
-    public function __construct(int $rows = 5, array $credentials = [])
+    public function __construct(?int $rows = null, array $credentials = [])
     {
         parent::__construct($rows);
 
@@ -45,7 +45,7 @@ protected function connect(): void
     protected function createTableIfNotExists(): void
     {
         $this->connection->prepare(
-            "CREATE TABLE IF NOT EXISTS {$this->credentials['table']} (
+            "CREATE TABLE IF NOT EXISTS `{$this->credentials['table']}` (
                 `time` float DEFAULT NULL,
                 `name` varchar(255) COLLATE utf8_unicode_ci NOT NULL,
                 `method` varchar(255) COLLATE utf8_unicode_ci DEFAULT NULL,