SprykerTest\Shared\Testify\Fixtures\FixturesCommand::execute()

Complexity

Conditions	14
Paths	1152

Size

Total Lines	60
Code Lines	34

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function execute(InputInterface $input, OutputInterface $output): int
    {
        $this->options = $input->getOptions();

The property options does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->output = $output;

The property output does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        // load config
        $config = $this->getGlobalConfig();

        if (!$this->options['silent']) {
            $output->writeln(
                Codecept::versionString() . "\nPowered by " . Version::getVersionString(),
            );
            $output->writeln(
                'Running with seed: ' . $this->options['seed'] . "\n",
            );
        }
        if ($this->options['group']) {
            $output->writeln(sprintf('[Groups] <info>%s</info> ', implode(', ', $this->options['group'])));
        }
        if ($this->options['debug']) {
            $output->setVerbosity(OutputInterface::VERBOSITY_VERY_VERBOSE);
        }

        $userOptions = array_intersect_key($this->options, array_flip($this->passedOptionKeys($input)));
        $userOptions['verbosity'] = $output->getVerbosity();
        $userOptions['seed'] = (int)$this->options['seed'] ?: mt_rand();
        $userOptions['colors'] = $this->options['no-colors'] || ($input->hasOption('no-ansi') && $input->getOption('no-ansi')) ? false : $config['settings']['colors'];

        if ($this->options['group'] !== []) {
            $userOptions['groups'] = $this->options['group'];
        }

        if ($this->options['skip-group'] !== []) {
            $userOptions['excludeGroups'] = $this->options['skip-group'];
        }

        $this->codecept = new FixturesRunner($userOptions);

The property codecept does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $suites = Configuration::suites();
        $this->executed = $this->runSuites($suites);

The property executed does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        if (!empty($config['include'])) {
            $current_dir = Configuration::projectDir();
            $suites += $config['include'];
            $this->runIncludedSuites($config['include'], $current_dir);
        }

        if ($this->executed === 0) {
            throw new TestRuntimeException(
                sprintf("Suite '%s' could not be found", implode(', ', $suites)),
            );
        }

        $this->codecept->printResult();

        if (!$input->getOption('no-exit') && !$this->codecept->getResultAggregator()->wasSuccessful()) {
            exit(1);

In this branch, the function will implicitly return null which is incompatible with the type-hinted return integer. Consider adding a return statement or allowing null as return value.

        }

        return static::CODE_SUCCESS;
    }