Code quality improvements

Author: wazabii8

FileIterator.php

@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace MaplePHP\Unitary;
@@ -13,7 +14,7 @@
 
 class FileIterator
 {
-    const PATTERN = 'unitary-*.php';
+    public const PATTERN = 'unitary-*.php';
 
     private array $args;
 
@@ -74,7 +75,7 @@ private function findFiles(string $dir): array
         $pattern = static::PATTERN;
         foreach ($iterator as $file) {
             if (($file instanceof SplFileInfo) && fnmatch($pattern, $file->getFilename()) &&
-                (isset($this->args['path']) || !str_contains($file->getPathname(), DIRECTORY_SEPARATOR . "vendor" . DIRECTORY_SEPARATOR ))) {
+                (isset($this->args['path']) || !str_contains($file->getPathname(), DIRECTORY_SEPARATOR . "vendor" . DIRECTORY_SEPARATOR))) {
                 if(!$this->findExcluded($this->exclude(), $dir, $file->getPathname())) {
                     $files[] = $file->getPathname();
                 }
@@ -87,9 +88,9 @@ private function findFiles(string $dir): array
      * Get exclude parameter
      * @return array
      */
-    function exclude(): array
+    public function exclude(): array
     {
-        $excl = array();
+        $excl = [];
         if(isset($this->args['exclude']) && is_string($this->args['exclude'])) {
             $exclude = explode(',', $this->args['exclude']);
             foreach ($exclude as $file) {
@@ -112,7 +113,7 @@ function exclude(): array
      * @param string $file
      * @return bool
      */
-    function findExcluded(array $exclArr, string $relativeDir, string $file): bool
+    public function findExcluded(array $exclArr, string $relativeDir, string $file): bool
     {
         $file = $this->getNaturalPath($file);
         foreach ($exclArr as $excl) {
@@ -129,7 +130,7 @@ function findExcluded(array $exclArr, string $relativeDir, string $file): bool
      * @param string $path
      * @return string
      */
-    function getNaturalPath(string $path): string
+    public function getNaturalPath(string $path): string
     {
         return str_replace("\\", "/", $path);
     }
@@ -142,8 +143,7 @@ function getNaturalPath(string $path): string
     private function requireUnitFile(string $file): ?Closure
     {
         $clone = clone $this;
-        $call = function() use ($file, $clone): void
-        {
+        $call = function () use ($file, $clone): void {
             $cli = new CliHandler();
             if(isset(self::$headers['args']['trace'])) {
                 $cli->enableTraceLines(true);
@@ -155,7 +155,7 @@ private function requireUnitFile(string $file): ?Closure
             if (!is_file($file)) {
                 throw new RuntimeException("File \"$file\" do not exists.");
             }
-            require_once ($file);
+            require_once($file);
 
             $clone->getUnit()->execute();
 
@@ -180,4 +180,4 @@ protected function getUnit(): Unit
         return $unit;
 
     }
-}
+}

Handlers/FileHandler.php

@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace MaplePHP\Unitary\Handlers;
@@ -45,4 +46,4 @@ public function execute(): void
         $upload = new UploadedFile($this->stream);
         $upload->moveTo($this->file);
     }
-}
+}

Handlers/HandlerInterface.php

@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace MaplePHP\Unitary\Handlers;
@@ -7,7 +8,6 @@
 
 interface HandlerInterface
 {
-
     /**
      * Access the command stream
      * @return Command
@@ -20,4 +20,4 @@ public function getCommand(): Command;
      * @return void
      */
     public function execute(): void;
-}
+}

Handlers/HtmlHandler.php

@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace MaplePHP\Unitary\Handlers;
@@ -44,4 +45,4 @@ public function execute(): void
         $out = str_replace(["[", "]"], ['<span style="background-color: #666; color: #FFF; padding: 4px 2px">', '</span>'], $out);
         echo '<pre style="' . $style . '">' . $out . '</pre>';
     }
-}
+}