Zenstruck\ScheduleBundle\Schedule\Task\Runner\ProcessTaskRunner

Complexity

Total Complexity

3

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Test Coverage

Coverage

81.82%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	14	2
A	supports()	0	3	1

<?php

namespace Zenstruck\ScheduleBundle\Schedule\Task\Runner;

use Zenstruck\ScheduleBundle\Schedule\Task;
use Zenstruck\ScheduleBundle\Schedule\Task\ProcessTask;
use Zenstruck\ScheduleBundle\Schedule\Task\Result;
use Zenstruck\ScheduleBundle\Schedule\Task\TaskRunner;

/**
 * @author Kevin Bond <kevinbond@gmail.com>
 */
final class ProcessTaskRunner implements TaskRunner
{
    /**
     * @param ProcessTask $task
     */
    public function __invoke(Task $task): Result
    {
        $process = clone $task->getProcess();

The method getProcess() does not exist on Zenstruck\ScheduleBundle\Schedule\Task. It seems like you code against a sub-type of Zenstruck\ScheduleBundle\Schedule\Task such as Zenstruck\ScheduleBundle\Schedule\Task\ProcessTask.

        $process->run();

        if ($process->isSuccessful()) {
            return Result::successful($task, $process->getOutput());
        }

        return Result::failure(
            $task,
            "Exit {$process->getExitCode()}: {$process->getExitCodeText()}",
            $process->getOutput().$process->getErrorOutput()
        );
    }

    public function supports(Task $task): bool
    {
        return $task instanceof ProcessTask;
    }
}