Hyde\Console\Commands\ValidateCommand

Complexity

Total Complexity

6

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Importance

Changes	5
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	runCheck()	0	8	1
A	handle()	0	15	2
A	timeTotal()	0	4	1
A	getCheckTime()	0	3	1

<?php

declare(strict_types=1);

namespace Hyde\Console\Commands;

use Hyde\Framework\Concerns\TracksExecutionTime;
use Hyde\Framework\Services\ValidationService;
use LaravelZero\Framework\Commands\Command;

/**
 * @see \Hyde\Framework\Testing\Feature\Commands\ValidateCommandTest
 */
class ValidateCommand extends Command
{
    use TracksExecutionTime;

    /** @var string */
    protected $signature = 'validate';

    /** @var string */
    protected $description = 'Run a series of tests to validate your setup and help you optimize your site.';

    protected ValidationService $service;

    public function __construct()
    {
        parent::__construct();

        $this->service = new ValidationService();
    }

    public function handle(): int
    {
        $this->startClock();

        $this->info('Running validation tests!');

        $this->newLine();

        foreach (ValidationService::checks() as $check) {
            $this->runCheck($check);
        }

        $this->info("All done! {$this->timeTotal()}");

        return Command::SUCCESS;
    }

    protected function runCheck(string $check): void
    {
        $timeStart = microtime(true);
        $result = $this->service->run($check);

        $this->line($result->formattedMessage($this->getCheckTime($timeStart)));

It seems like $timeStart can also be of type string; however, parameter $timeStart of Hyde\Console\Commands\ValidateCommand::getCheckTime() does only seem to accept double, maybe add an additional type check?

        $this->newline();
    }

    protected function getCheckTime(float $timeStart): string
    {
        return number_format((microtime(true) - $timeStart) * 1000, 2);
    }

    protected function timeTotal(): string
    {
        return sprintf("<fg=gray>Ran %s checks in {$this->getExecutionTimeString()}</>",
            sizeof(ValidationService::checks())
        );
    }
}