Merge pull request #19 from gacela-project/feature/generate-command-w…

…ith-one-arg Pass only one argument to the "maker command"
gacela-project · Apr 18, 2021 · 9838f75 · 9838f75
2 parents 146b942 + e47a777
commit 9838f75
Show file tree

Hide file tree

Showing 20 changed files with 347 additions and 74 deletions.
diff --git a/.gitignore b/.gitignore
@@ -2,6 +2,5 @@
 /.idea/
 /.vscode/
 /vendor/
-/TestModule/
 .phpunit.*
 .php_cs.cache
diff --git a/composer.json b/composer.json
@@ -15,6 +15,7 @@
         }
     ],
     "require": {
+        "ext-json": "*",
         "php": ">=7.4"
     },
     "require-dev": {
@@ -39,14 +40,16 @@
     "scripts": {
         "test-all": [
             "@test-quality",
+            "@test-unit",
             "@test-integration"
         ],
         "test-quality": [
             "@csrun",
             "@psalm"
         ],
+        "test-unit": "./vendor/bin/phpunit --testsuite=unit",
         "test-integration": "./vendor/bin/phpunit --testsuite=integration",
-        "test-coverage": "XDEBUG_MODE=coverage ./vendor/bin/phpunit --testsuite=integration --coverage-html=coverage",
+        "test-coverage": "XDEBUG_MODE=coverage ./vendor/bin/phpunit --testsuite=unit,integration --coverage-html=coverage",
         "psalm": "./vendor/bin/psalm",
         "csfix": "./vendor/bin/php-cs-fixer fix --allow-risky=yes",
         "csrun": "./vendor/bin/php-cs-fixer fix --allow-risky=yes --dry-run"

diff --git a/phpunit.xml b/phpunit.xml
@@ -18,6 +18,10 @@
     </php>
 
     <testsuites>
+        <testsuite name="unit">
+            <directory suffix="Test.php">tests/Unit</directory>
+        </testsuite>
+
         <testsuite name="integration">
             <directory suffix="Test.php">tests/Integration</directory>
         </testsuite>

diff --git a/src/CodeGenerator/CodeGeneratorConfig.php b/src/CodeGenerator/CodeGeneratorConfig.php
@@ -5,6 +5,8 @@
 namespace Gacela\CodeGenerator;
 
 use Gacela\Framework\AbstractConfig;
+use Gacela\Framework\Config;
+use LogicException;
 
 final class CodeGeneratorConfig extends AbstractConfig
 {
@@ -32,4 +34,14 @@ private function getCommandTemplateContent(string $filename): string
     {
         return file_get_contents(__DIR__ . '/Infrastructure/Template/Command/' . $filename);
     }
+
+    public function getComposerJsonContentAsArray(): array
+    {
+        $filename = Config::getApplicationRootDir() . '/composer.json';
+        if (!file_exists($filename)) {
+            throw new LogicException('composer.json file not found but it is required');
+        }
+
+        return json_decode(file_get_contents($filename), true);
+    }
 }
diff --git a/src/CodeGenerator/CodeGeneratorFactory.php b/src/CodeGenerator/CodeGeneratorFactory.php
@@ -9,7 +9,7 @@
 use Gacela\CodeGenerator\Domain\Command\FacadeMaker;
 use Gacela\CodeGenerator\Domain\Command\FactoryMaker;
 use Gacela\CodeGenerator\Domain\Command\ModuleMaker;
-use Gacela\CodeGenerator\Domain\Io\CommandArgumentsParser;
+use Gacela\CodeGenerator\Domain\Io\CommandArguments\CommandArgumentsParser;
 use Gacela\CodeGenerator\Domain\Io\MakerIoInterface;
 use Gacela\CodeGenerator\Infrastructure\Io\SystemMakerIo;
 use Gacela\Framework\AbstractFactory;
@@ -71,6 +71,8 @@ private function createGeneratorIo(): MakerIoInterface
 
     public function createCommandArgumentsParser(): CommandArgumentsParser
     {
-        return new CommandArgumentsParser();
+        return new CommandArgumentsParser(
+            $this->getConfig()->getComposerJsonContentAsArray()
+        );
     }
 }
diff --git a/src/CodeGenerator/Domain/Command/AbstractMaker.php b/src/CodeGenerator/Domain/Command/AbstractMaker.php
@@ -20,13 +20,10 @@ public function __construct(MakerIoInterface $io, string $template)
 
     public function make(CommandArguments $commandArguments): void
     {
-        $pieces = explode('/', $commandArguments->targetDirectory());
-        $moduleName = end($pieces);
+        $this->io->createDirectory($commandArguments->directory());
 
-        $this->io->createDirectory($commandArguments->targetDirectory());
-
-        $path = sprintf('%s/%s.php', $commandArguments->targetDirectory(), $this->className());
-        $this->io->filePutContents($path, $this->generateFileContent("{$commandArguments->rootNamespace()}\\$moduleName"));
+        $path = sprintf('%s/%s.php', $commandArguments->directory(), $this->className());
+        $this->io->filePutContents($path, $this->generateFileContent($commandArguments->namespace()));
 
         $this->io->writeln("> Path '$path' created successfully");
     }

diff --git a/src/CodeGenerator/Domain/Command/ModuleMaker.php b/src/CodeGenerator/Domain/Command/ModuleMaker.php
@@ -29,7 +29,7 @@ public function make(CommandArguments $commandArguments): void
             $generator->make($commandArguments);
         }
 
-        $pieces = explode('/', $commandArguments->targetDirectory());
+        $pieces = explode('/', $commandArguments->directory());
         $moduleName = end($pieces);
         $this->io->writeln("Module $moduleName created successfully");
     }

diff --git a/src/CodeGenerator/Domain/Io/CommandArguments/CommandArgumentsParser.php b/src/CodeGenerator/Domain/Io/CommandArguments/CommandArgumentsParser.php
@@ -0,0 +1,90 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Gacela\CodeGenerator\Domain\Io\CommandArguments;
+
+use Gacela\CodeGenerator\Domain\Io\CommandArguments\Exception\CommandArgumentsException;
+use Gacela\CodeGenerator\Domain\ReadModel\CommandArguments;
+use InvalidArgumentException;
+
+final class CommandArgumentsParser
+{
+    private array $composerJson;
+
+    public function __construct(array $composerJson)
+    {
+        $this->composerJson = $composerJson;
+    }
+
+    /**
+     * @throws InvalidArgumentException
+     */
+    public function parse(array $arguments): CommandArguments
+    {
+        if (empty($arguments)) {
+            throw new InvalidArgumentException('Expected argument to be the location of the new module. For example: App/TestModule');
+        }
+
+        return $this->createCommandArguments($arguments[0]);
+    }
+
+    private function createCommandArguments(string $desiredNamespace): CommandArguments
+    {
+        if (!isset($this->composerJson['autoload'])) {
+            throw CommandArgumentsException::noAutoloadFound();
+        }
+
+        if (!isset($this->composerJson['autoload']['psr-4'])) {
+            throw CommandArgumentsException::noAutoloadPsr4Found();
+        }
+
+        $psr4 = $this->composerJson['autoload']['psr-4'];
+        $allPsr4Combinations = $this->allPossiblePsr4Combinations($desiredNamespace);
+
+        foreach ($allPsr4Combinations as $psr4Combination) {
+            $psr4Key = $psr4Combination . '\\';
+            if (isset($psr4[$psr4Key])) {
+                return $this->foundPsr4($psr4Key, $psr4[$psr4Key], $desiredNamespace);
+            }
+        }
+
+        throw CommandArgumentsException::noAutoloadPsr4MatchFound($desiredNamespace);
+    }
+
+    /**
+     * Combine all possible psr-4 combinations and return them ordered by longer to shorter.
+     * This way we'll be able to find the longer match first.
+     * For example: App/TestModule/TestSubModule will produce an array such as:
+     * [
+     *   'App/TestModule/TestSubModule',
+     *   'App/TestModule',
+     *   'App',
+     * ].
+     */
+    private function allPossiblePsr4Combinations(string $desiredNamespace): array
+    {
+        $result = [];
+
+        foreach (explode('/', $desiredNamespace) as $explodedArg) {
+            if (empty($result)) {
+                $result[] = $explodedArg;
+            } else {
+                $prevValue = $result[count($result) - 1];
+                $result[] = $prevValue . '\\' . $explodedArg;
+            }
+        }
+
+        return array_reverse($result);
+    }
+
+    private function foundPsr4(string $psr4Key, string $psr4Value, string $desiredNamespace): CommandArguments
+    {
+        $rootDir = substr($psr4Value, 0, -1);
+        $rootNamespace = substr($psr4Key, 0, -1);
+        $targetDirectory = str_replace(['/', $rootNamespace, '\\'], ['\\', $rootDir, '/'], $desiredNamespace);
+        $namespace = str_replace([$rootDir, '/'], [$rootNamespace, '\\'], $targetDirectory);
+
+        return new CommandArguments($namespace, $targetDirectory);
+    }
+}
diff --git a/src/CodeGenerator/Domain/Io/CommandArguments/Exception/CommandArgumentsException.php b/src/CodeGenerator/Domain/Io/CommandArguments/Exception/CommandArgumentsException.php
@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Gacela\CodeGenerator\Domain\Io\CommandArguments\Exception;
+
+use RuntimeException;
+
+final class CommandArgumentsException extends RuntimeException
+{
+    public static function noAutoloadFound(): self
+    {
+        return new self('No autoload found in your composer.json');
+    }
+
+    public static function noAutoloadPsr4Found(): self
+    {
+        return new self('No autoload psr-4 match found in your composer.json');
+    }
+
+    public static function noAutoloadPsr4MatchFound(string $desiredNamespace): self
+    {
+        return new self('No autoload psr-4 match found for ' . $desiredNamespace);
+    }
+}
diff --git a/src/CodeGenerator/Domain/Io/CommandArgumentsParser.php b/src/CodeGenerator/Domain/Io/CommandArgumentsParser.php
diff --git a/src/CodeGenerator/Domain/ReadModel/CommandArguments.php b/src/CodeGenerator/Domain/ReadModel/CommandArguments.php
@@ -6,22 +6,22 @@
 
 final class CommandArguments
 {
-    private string $rootNamespace;
-    private string $targetDirectory;
+    private string $namespace;
+    private string $directory;
 
-    public function __construct(string $rootNamespace, string $targetDirectory)
+    public function __construct(string $namespace, string $directory)
     {
-        $this->rootNamespace = $rootNamespace;
-        $this->targetDirectory = $targetDirectory;
+        $this->namespace = $namespace;
+        $this->directory = $directory;
     }
 
-    public function rootNamespace(): string
+    public function namespace(): string
     {
-        return $this->rootNamespace;
+        return $this->namespace;
     }
 
-    public function targetDirectory(): string
+    public function directory(): string
     {
-        return $this->targetDirectory;
+        return $this->directory;
     }
 }
diff --git a/tests/Integration/CodeGenerator/UsingIncorrectConfigurationTest.php b/tests/Integration/CodeGenerator/UsingIncorrectConfigurationTest.php
@@ -5,32 +5,37 @@
 namespace GacelaTest\Integration\CodeGenerator;
 
 use Gacela\CodeGenerator\CodeGeneratorFacade;
+use Gacela\CodeGenerator\Domain\Io\CommandArguments\Exception\CommandArgumentsException;
+use Gacela\Framework\Config;
 use InvalidArgumentException;
 use PHPUnit\Framework\TestCase;
 
 final class UsingIncorrectConfigurationTest extends TestCase
 {
-    public function test_make_unknown_command(): void
+    public function setUp(): void
     {
-        $this->expectException(InvalidArgumentException::class);
-
-        $codeGeneratorConfig = new CodeGeneratorFacade();
-        $codeGeneratorConfig->runCommand('make:unknown', [__NAMESPACE__, __DIR__ . '/Generated']);
+        Config::setApplicationRootDir(__DIR__);
     }
 
-    public function test_missing_target_directory(): void
+    public function test_make_unknown_command(): void
     {
         $this->expectException(InvalidArgumentException::class);
 
-        $codeGeneratorConfig = new CodeGeneratorFacade();
-        $codeGeneratorConfig->runCommand('', [__NAMESPACE__]);
+        $facade = new CodeGeneratorFacade();
+        $facade->runCommand('make:unknown', ['GacelaTest/Integration/CodeGenerator/Generated']);
     }
 
-    public function test_missing_root_namespace_and_target_directory(): void
+    public function test_missing_target(): void
     {
         $this->expectException(InvalidArgumentException::class);
+        $facade = new CodeGeneratorFacade();
+        $facade->runCommand('make:module', []);
+    }
 
-        $codeGeneratorConfig = new CodeGeneratorFacade();
-        $codeGeneratorConfig->runCommand('', []);
+    public function test_unknown_target(): void
+    {
+        $this->expectExceptionObject(CommandArgumentsException::noAutoloadPsr4MatchFound('UnknownNamespace'));
+        $facade = new CodeGeneratorFacade();
+        $facade->runCommand('make:module', ['UnknownNamespace']);
     }
 }
diff --git a/tests/Integration/CodeGenerator/UsingMakeConfigTest.php b/tests/Integration/CodeGenerator/UsingMakeConfigTest.php
@@ -5,17 +5,23 @@
 namespace GacelaTest\Integration\CodeGenerator;
 
 use Gacela\CodeGenerator\CodeGeneratorFacade;
+use Gacela\Framework\Config;
 use GacelaTest\Integration\CodeGenerator\Util\DirectoryUtil;
 use PHPUnit\Framework\TestCase;
 
 final class UsingMakeConfigTest extends TestCase
 {
+    public function setUp(): void
+    {
+        Config::setApplicationRootDir(__DIR__);
+    }
+
     public function test_make_config(): void
     {
         self::assertFileDoesNotExist(__DIR__ . '/Generated/Config.php');
 
-        $codeGeneratorConfig = new CodeGeneratorFacade();
-        $codeGeneratorConfig->runCommand('make:config', [__NAMESPACE__, __DIR__ . '/Generated']);
+        $facade = new CodeGeneratorFacade();
+        $facade->runCommand('make:config', ['GacelaTest/Integration/CodeGenerator/Generated']);
 
         $this->expectOutputRegex("~/Generated/Config.php' created successfully~");
         self::assertFileExists(__DIR__ . '/Generated/Config.php');

diff --git a/tests/Integration/CodeGenerator/UsingMakeDependencyProviderTest.php b/tests/Integration/CodeGenerator/UsingMakeDependencyProviderTest.php
@@ -5,17 +5,23 @@
 namespace GacelaTest\Integration\CodeGenerator;
 
 use Gacela\CodeGenerator\CodeGeneratorFacade;
+use Gacela\Framework\Config;
 use GacelaTest\Integration\CodeGenerator\Util\DirectoryUtil;
 use PHPUnit\Framework\TestCase;
 
 final class UsingMakeDependencyProviderTest extends TestCase
 {
+    public function setUp(): void
+    {
+        Config::setApplicationRootDir(__DIR__);
+    }
+
     public function test_make_dependency_provider(): void
     {
         self::assertFileDoesNotExist(__DIR__ . '/Generated/DependencyProvider.php');
 
-        $codeGeneratorDependencyProvider = new CodeGeneratorFacade();
-        $codeGeneratorDependencyProvider->runCommand('make:dependency-provider', [__NAMESPACE__, __DIR__ . '/Generated']);
+        $facade = new CodeGeneratorFacade();
+        $facade->runCommand('make:dependency-provider', ['GacelaTest/Integration/CodeGenerator/Generated']);
 
         $this->expectOutputRegex("~/Generated/DependencyProvider.php' created successfully~");
         self::assertFileExists(__DIR__ . '/Generated/DependencyProvider.php');