SeparateProcessExecutor::execute() - Code Metrics - Inspection of "Added process-executor which uses a process to exe..." - php-task/TaskBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#40)

by Wachter

created 2017-08-14 05:53 UTC

SeparateProcessExecutor::execute() A

↳ Parent: SeparateProcessExecutor

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	2

Importance

Changes	1
Bugs	1	Features	0

Metric	Value
c	1
b	1
f	0
dl	0
loc	14
ccs	8
cts	8
cp	1
rs	9.4285
cc	2
eloc	8
nc	2
nop	1
crap	2

<?php

/*
 * This file is part of php-task library.
 *
 * (c) php-task
 *
 * This source file is subject to the MIT license that is bundled
 * with this source code in the file LICENSE.
 */

namespace Task\TaskBundle\Executor;

use Symfony\Component\Process\ProcessBuilder;
use Task\Execution\TaskExecutionInterface;
use Task\Runner\ExecutorInterface;

/**
 * Uses a separate process to start the executions via console-command.
 */
class SeparateProcessExecutor implements ExecutorInterface
{
    /**
     * @var string
     */
    private $consolePath;

    /**
     * @var string
     */
    private $environment;

    /**
     * @param string $consolePath
     * @param string $environment
     */
    public function __construct($consolePath, $environment)
    {
        $this->consolePath = $consolePath;
        $this->environment = $environment;
    }

    /**
     * {@inheritdoc}
     */
    public function execute(TaskExecutionInterface $execution)
    {
        $process = ProcessBuilder::create(
            [$this->consolePath, 'task:execute', $execution->getUuid(), '-e ' . $this->environment]
        )->getProcess();

        $process->run();

        if (!$process->isSuccessful()) {
            throw new SeparateProcessException($process->getErrorOutput());
        }

        return $process->getOutput();
    }
}


1		<?php
2
3		/*
4		* This file is part of php-task library.
5		*
6		* (c) php-task
7		*
8		* This source file is subject to the MIT license that is bundled
9		* with this source code in the file LICENSE.
10		*/
11
12		namespace Task\TaskBundle\Executor;
13
14		use Symfony\Component\Process\ProcessBuilder;
15		use Task\Execution\TaskExecutionInterface;
16		use Task\Runner\ExecutorInterface;
17
18		/**
19		* Uses a separate process to start the executions via console-command.
20		*/
21		class SeparateProcessExecutor implements ExecutorInterface
22		{
23		/**
24		* @var string
25		*/
26		private $consolePath;
27
28		/**
29		* @var string
30		*/
31		private $environment;
32
33		/**
34		* @param string $consolePath
35		* @param string $environment
36		*/
37	13	public function __construct($consolePath, $environment)
38		{
39	13	$this->consolePath = $consolePath;
40	13	$this->environment = $environment;
41	13	}
42
43		/**
44		* {@inheritdoc}
45		*/
46	2	public function execute(TaskExecutionInterface $execution)
47		{
48	2	$process = ProcessBuilder::create(
49	2	[$this->consolePath, 'task:execute', $execution->getUuid(), '-e ' . $this->environment]
50	2	)->getProcess();
51
52	2	$process->run();
53
54	2	if (!$process->isSuccessful()) {
55	1	throw new SeparateProcessException($process->getErrorOutput());
56		}
57
58	1	return $process->getOutput();
59		}
60		}
61

php-task / TaskBundle

Pull Request — master (#40)

SeparateProcessExecutor::execute() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like