Add ProcessFactory::createRemoveFileProcess()

Author: steevanb

changelog.md

@@ -1,5 +1,7 @@
 ### master
 
+- Add `ProcessFactory::createRemoveFileProcess()`
+
 ### [0.4.0](../../compare/0.3.0...0.4.0) - 2022-02-14
 
 - Add `ParallelProcessesApplication::hasProcess()`

src/Process/ProcessFactory.php

@@ -5,26 +5,53 @@
 namespace Steevanb\ParallelProcess\Process;
 
 use Steevanb\ParallelProcess\Console\Application\ParallelProcessesApplication;
+use steevanb\PhpTypedArray\ScalarArray\StringArray;
 
 class ProcessFactory
 {
     public static function createRemoveDirectoryProcess(
-        string $directory,
+        string $path,
         ParallelProcessesApplication $application,
         string $name = null,
         string $cwd = null
     ): ?Process {
-        if (is_dir($directory)) {
-            $return = new Process(['rm', '-rf', $directory], $cwd);
-            if (is_string($name)) {
-                $return->setName($name);
-            }
+        if (is_dir($path)) {
+            $return = static::createProcess(new StringArray(['rm', '-rf', $path]), $application, $name, $cwd);
+        } else {
+            $return = null;
+        }
+
+        return $return;
+    }
 
-            $application->addProcess($return);
+    public static function createRemoveFileProcess(
+        string $filePathname,
+        ParallelProcessesApplication $application,
+        string $name = null,
+        string $cwd = null
+    ): ?Process {
+        if (is_file($filePathname)) {
+            $return = static::createProcess(new StringArray(['rm', $filePathname]), $application, $name, $cwd);
         } else {
             $return = null;
         }
 
         return $return;
     }
+
+    protected static function createProcess(
+        StringArray $command,
+        ParallelProcessesApplication $application,
+        string $name = null,
+        string $cwd = null
+    ): Process {
+        $return = new Process($command->toArray(), $cwd);
+        if (is_string($name)) {
+            $return->setName($name);
+        }
+
+        $application->addProcess($return);
+
+        return $return;
+    }
 }

tests/Process/ProcessFactory/CreateRemoveFileProcessTest.php

@@ -0,0 +1,58 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Steevanb\ParallelProcess\Tests\Process\ProcessFactory;
+
+use PHPUnit\Framework\TestCase;
+use Steevanb\ParallelProcess\{
+    Console\Application\ParallelProcessesApplication,
+    Process\Process,
+    Process\ProcessFactory
+};
+
+/** @covers \Steevanb\ParallelProcess\Process\ProcessFactory::createRemoveFileProcess */
+class CreateRemoveFileProcessTest extends TestCase
+{
+    public function testFileExists(): void
+    {
+        $application = new ParallelProcessesApplication();
+
+        /** @var Process $process */
+        $process = ProcessFactory::createRemoveFileProcess(__FILE__, $application);
+
+        static::assertInstanceOf(Process::class, $process);
+        static::assertTrue($application->hasProcess($process));
+    }
+
+    public function testFileNotFound(): void
+    {
+        $application = new ParallelProcessesApplication();
+
+        $process = ProcessFactory::createRemoveFileProcess(__FILE__ . '.foo.bar', $application);
+
+        static::assertNull($process);
+    }
+
+    public function testName(): void
+    {
+        $application = new ParallelProcessesApplication();
+
+        /** @var Process $process */
+        $process = ProcessFactory::createRemoveFileProcess(__FILE__, $application, 'foo');
+
+        static::assertInstanceOf(Process::class, $process);
+        static::assertSame('foo', $process->getName());
+    }
+
+    public function testCwd(): void
+    {
+        $application = new ParallelProcessesApplication();
+
+        /** @var Process $process */
+        $process = ProcessFactory::createRemoveFileProcess(__FILE__, $application, 'foo', '/foo/bar');
+
+        static::assertInstanceOf(Process::class, $process);
+        static::assertSame('/foo/bar', $process->getWorkingDirectory());
+    }
+}