ProtoneMedia\ApiHealth\Console\Check

Complexity

Total Complexity

2

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handle()	0	12	2

<?php

namespace ProtoneMedia\ApiHealth\Console;

use Illuminate\Console\Command;
use ProtoneMedia\ApiHealth\Checkers\Executor;

class Check extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'api-health:check {checkerClass}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Run the given checker';

    /**
     * Displays the result of the given checker.
     *
     * @return null
     */
    public function handle()
    {
        $executor = Executor::make(
            $checkerClass = $this->argument('checkerClass')

It seems like $checkerClass = $this->argument('checkerClass') can also be of type array and null; however, parameter $checkerClass of ProtoneMedia\ApiHealth\Checkers\Executor::make() does only seem to accept string, maybe add an additional type check?

        );

        $this->info('Running checker: ' . $checkerClass);

Are you sure $checkerClass of type array|null|string can be used in concatenation?

        if ($executor->passes()) {
            $this->info('Passes!');
        } else {
            $this->error('Fails! ' . $executor->getException()->getMessage());
        }
    }
}