Add more test coverage

Author: terremoth

demo-file.txt

@@ -1 +1 @@
-Hello users! At: 2026-01-15T09:46:38+00:00
+Hello users! At: 2026-01-15T12:35:35+00:00

src/Terremoth/Async/PhpFile.php

@@ -6,12 +6,12 @@
 use InvalidArgumentException;
 use Symfony\Component\Process\Process as SymfonyProcess;
 
-readonly class PhpFile
+class PhpFile
 {
     /**
      * @param string $file
-     * @throws Exception
      * @param list<string> $args
+     * @throws Exception
      */
     public function __construct(private string $file, private array $args = [])
     {
@@ -20,9 +20,9 @@ public function __construct(private string $file, private array $args = [])
                 . ' does not exists or is not readable!');
         }
 
-        $finfo = finfo_open(FILEINFO_MIME_TYPE);
-        $mimeType = finfo_file($finfo, $this->file);
-        finfo_close($finfo);
+        $fileInfo = finfo_open(FILEINFO_MIME_TYPE);
+        $mimeType = finfo_file($fileInfo, $this->file);
+        finfo_close($fileInfo);
 
         if (!in_array($mimeType, ['text/x-php', 'application/x-php', 'application/php', 'application/x-httpd-php'])) {
             throw new Exception('Error: file ' . $this->file . ' is not a PHP file!');
@@ -31,14 +31,32 @@ public function __construct(private string $file, private array $args = [])
 
     public function run(): void
     {
-        if (PHP_OS_FAMILY === 'Windows') {
+        if ($this->getOsFamily() === 'Windows') {
             $template = ['start', "", '/B', PHP_BINARY, $this->file, ...$this->args];
             $process = new SymfonyProcess($template);
-            $process->start();
+
+            $this->startProcess($process);
             return;
         }
 
-        $args = implode(' ', $this->args);
-        exec(PHP_BINARY . ' ' . $this->file . ' ' . $args . '  > /dev/null 2>&1 &');
+        $arguments = implode(' ', $this->args);
+        $command = PHP_BINARY . ' ' . $this->file . ' ' . $arguments . ' > /dev/null 2>&1 &';
+
+        $this->executeCommand($command);
+    }
+
+    protected function getOsFamily(): string
+    {
+        return PHP_OS_FAMILY;
+    }
+
+    protected function startProcess(SymfonyProcess $symfonyProcess): void
+    {
+        $symfonyProcess->start();
+    }
+
+    protected function executeCommand(string $command): void
+    {
+        exec($command);
     }
 }

tests/Terremoth/AsyncTest/PhpFileTest.php

@@ -3,6 +3,7 @@
 namespace Terremoth\AsyncTest;
 
 use PHPUnit\Framework\TestCase;
+use Symfony\Component\Process\Process as SymfonyProcess;
 use Terremoth\Async\PhpFile;
 use InvalidArgumentException;
 use Exception;
@@ -97,4 +98,76 @@ public function testRunDoesNotThrow(): void
 
         $this->expectNotToPerformAssertions();
     }
+
+
+    public function testRunExecutesWindowsFlowWhenOsIsWindows(): void
+    {
+        $tempFile = tempnam(sys_get_temp_dir(), 'test_win') . '.php';
+        file_put_contents($tempFile, '<?php echo "hello";');
+
+        try {
+            $phpFile = $this->getMockBuilder(PhpFile::class)
+                ->setConstructorArgs([$tempFile, ['arg1', 'arg2']])
+                ->onlyMethods(['getOsFamily', 'startProcess', 'executeCommand'])
+                ->getMock();
+
+            $phpFile->expects($this->once())
+                ->method('getOsFamily')
+                ->willReturn('Windows');
+
+            $phpFile->expects($this->once())
+                ->method('startProcess')
+                ->willReturnCallback(function (SymfonyProcess $process) use ($tempFile) {
+                    $commandLine = $process->getCommandLine();
+
+                    $this->assertStringContainsString('start', $commandLine);
+                    $this->assertStringContainsString('/B', $commandLine);
+                    $this->assertStringContainsString($tempFile, $commandLine);
+                    $this->assertStringContainsString('arg1', $commandLine);
+                });
+
+            $phpFile->expects($this->never())
+                ->method('executeCommand');
+
+            $phpFile->run();
+        } finally {
+            if (file_exists($tempFile)) {
+                unlink($tempFile);
+            }
+        }
+    }
+
+    public function testRunExecutesUnixFlowWhenOsIsLinux(): void
+    {
+        $tempFile = tempnam(sys_get_temp_dir(), 'test_unix') . '.php';
+        file_put_contents($tempFile, '<?php echo "hello";');
+
+        try {
+            $phpFile = $this->getMockBuilder(PhpFile::class)
+                ->setConstructorArgs([$tempFile, ['argA']])
+                ->onlyMethods(['getOsFamily', 'startProcess', 'executeCommand'])
+                ->getMock();
+
+            $phpFile->expects($this->once())
+                ->method('getOsFamily')
+                ->willReturn('Linux');
+
+            $phpFile->expects($this->never())
+                ->method('startProcess');
+
+            $phpFile->expects($this->once())
+                ->method('executeCommand')
+                ->willReturnCallback(function (string $command) use ($tempFile) {
+                    $this->assertStringContainsString('> /dev/null 2>&1 &', $command);
+                    $this->assertStringContainsString($tempFile, $command);
+                    $this->assertStringContainsString('argA', $command);
+                });
+
+            $phpFile->run();
+        } finally {
+            if (file_exists($tempFile)) {
+                unlink($tempFile);
+            }
+        }
+    }
 }