CS/QA: Tests: move TestWriter to separate file

php-parallel-lint · Mar 4, 2022 · 2b27e63 · 2b27e63
1 parent 9e024d4
commit 2b27e63
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 23 deletions.
diff --git a/phpcs.xml.dist b/phpcs.xml.dist
@@ -84,9 +84,6 @@
     </rule>
 
     <!-- To be addressed in test refactor. -->
-    <rule ref="PSR1.Classes.ClassDeclaration.MultipleClasses">
-        <exclude-pattern>/tests/OutputTest\.php$</exclude-pattern>
-    </rule>
     <rule ref="PSR1.Files.SideEffects.FoundWithSymbols">
         <exclude-pattern>/tests/*\.php$</exclude-pattern>
     </rule>

diff --git a/tests/Helpers/TestWriter.php b/tests/Helpers/TestWriter.php
@@ -0,0 +1,24 @@
+<?php
+
+namespace PHP_Parallel_Lint\PhpParallelLint\Tests\Helpers;
+
+use PHP_Parallel_Lint\PhpParallelLint\Writers\WriterInterface;
+
+class TestWriter implements WriterInterface
+{
+    /** @var string */
+    protected $logs = "";
+
+    /**
+     * @param string $string
+     */
+    public function write($string)
+    {
+        $this->logs .= $string;
+    }
+
+    public function getLogs()
+    {
+        return $this->logs;
+    }
+}
diff --git a/tests/OutputTest.php b/tests/OutputTest.php
@@ -15,7 +15,7 @@
 use PHP_Parallel_Lint\PhpParallelLint\Outputs\CheckstyleOutput;
 use PHP_Parallel_Lint\PhpParallelLint\Outputs\GitLabOutput;
 use PHP_Parallel_Lint\PhpParallelLint\Result;
-use PHP_Parallel_Lint\PhpParallelLint\Writers\WriterInterface;
+use PHP_Parallel_Lint\PhpParallelLint\Tests\Helpers\TestWriter;
 use Tester\Assert;
 use Tester\TestCase;
 
@@ -100,24 +100,5 @@ public function getGitLabOutputData()
     }
 }
 
-class TestWriter implements WriterInterface
-{
-    /** @var string */
-    protected $logs = "";
-
-    /**
-     * @param string $string
-     */
-    public function write($string)
-    {
-        $this->logs .= $string;
-    }
-
-    public function getLogs()
-    {
-        return $this->logs;
-    }
-}
-
 $testCase = new OutputTest();
 $testCase->run();