Epfremme\ProcessQueue\Process\ProcessFactory

Complexity

Total Complexity

4

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes	3
Bugs	0	Features	2

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	make()	0	8	3

<?php
/**
 * File ProcessFactory.php
 *
 * @author Edward Pfremmer <epfremme@nerdery.com>
 */
namespace Epfremme\ProcessQueue\Process;

use Symfony\Component\Process\Process;

/**
 * Class ProcessFactory
 *
 * @package Epfremme\ProcessQueue\Process
 */
class ProcessFactory
{
    /**
     * @var string
     */
    private $cmd;

    /**
     * ProcessFactory constructor
     *
     * @param string $cmd
     */
    public function __construct($cmd)
    {
        $this->cmd = $cmd;
    }

    /**
     * @param \SplFileInfo|string $cwd
     * @return Process
     */
    public function make($cwd = null)
    {
        if (!is_null($cwd) && !is_dir($cwd)) {
            throw new Exception\InvalidWorkingDirectoryException($cwd);
        }

        return new Process($this->cmd, $cwd);

It seems like $cwd defined by parameter $cwd on line 37 can also be of type object<SplFileInfo>; however, Symfony\Component\Process\Process::__construct() does only seem to accept string|null, maybe add an additional type check?

    }
}