SmartWeb\ModuleTesting\Console\Command\GenerateTest

Complexity

Total Complexity

3

Size/Duplication

Total Lines	62
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	handle()	24	24	2
A	provideArguments()	8	8	1

<?php


namespace SmartWeb\ModuleTesting\Console\Command;

use Exception;
use SmartWeb\ModuleTesting\Codeception\SmartWeb\Setup;
use SmartWeb\ModuleTesting\Console\CommandArgument;
use SmartWeb\ModuleTesting\Console\CommandArguments;
use SmartWeb\ModuleTesting\Console\InputArgumentType;
use SmartWeb\ModuleTesting\Generator\Test\TestGenerator;

/**
 * Command for generating a Codeception test for a Module.
 *
 * @package SmartWeb\ModuleTesting\Console
 */
class GenerateTest extends BaseCommand

This class seems to be duplicated in your project.

{
    
    /**
     * The console command name.
     *
     * @var string
     */
    protected $name = 'make-test';
    
    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Generate a Codeception Test for the given module and suite';
    
    /**
     * Execute the console command.
     *
     * @return void
     *
     * @throws Exception
     */
    public function handle()
    {
        /**
         * @var string $moduleName
         * @var string $suite
         * @var string $name
         */
        $moduleName = $this->argument('moduleName');
        $suite = $this->argument('suite');
        $name = $this->argument('name');
        
        $this->info("Creating '{$name}' for module '{$moduleName}' in suite '{$suite}'.");
        
        $setup = new Setup($moduleName);
        
        try {
            $generator = new TestGenerator($setup, $suite, $name);

It seems like $suite defined by $this->argument('suite') on line 50 can also be of type array; however, SmartWeb\ModuleTesting\Generator\Test\TestGenerator::__construct() does only seem to accept string, maybe add an additional type check?

It seems like $name defined by $this->argument('name') on line 51 can also be of type array; however, SmartWeb\ModuleTesting\Generator\Test\TestGenerator::__construct() does only seem to accept string, maybe add an additional type check?

            $generator->generate();
            $this->info("Created test!");
        } catch (Exception $e) {
            $this->error("Failed to create test!");
            throw $e;
        }
    }
    
    /**
     * @return CommandArguments
     */
    protected function provideArguments() : CommandArguments
    {
        return new CommandArguments([
            new CommandArgument('moduleName', InputArgumentType::required(), 'The name of the module.'),
            new CommandArgument('suite', InputArgumentType::required(), 'The suite to generate test in.'),
            new CommandArgument('name', InputArgumentType::required(), 'The name of the test.'),
        ]);
    }
    
}