Rename namespace Factory to Builder - Close #12

Author: sandrokeil

README.md

@@ -20,7 +20,7 @@ $printer = new PhpParser\PrettyPrinter\Standard(['shortArraySyntax' => true]);
 
 $ast = $parser->parse('');
 
-$classFactory = OpenCodeModeling\CodeAst\Factory\ClassFactory::fromScratch('TestClass', 'My\\Awesome\\Service');
+$classFactory = OpenCodeModeling\CodeAst\Builder\ClassBuilder::fromScratch('TestClass', 'My\\Awesome\\Service');
 $classFactory
     ->setFinal(true)
     ->setExtends('BaseClass')
@@ -112,7 +112,7 @@ EOF;
 
 $ast = $parser->parse($expected);
 
-$classFactory = OpenCodeModeling\CodeAst\Factory\ClassFactory::fromNodes(...$ast);
+$classFactory = OpenCodeModeling\CodeAst\Builder\ClassBuilder::fromNodes(...$ast);
 
 $classFactory->getName(); // TestClass
 $classFactory->getExtends(); // BaseClass

src/Factory/ClassFactory.php renamed to src/Builder/ClassBuilder.php

@@ -8,7 +8,7 @@
 
 declare(strict_types=1);
 
-namespace OpenCodeModeling\CodeAst\Factory;
+namespace OpenCodeModeling\CodeAst\Builder;
 
 use OpenCodeModeling\CodeAst\Code\ClassGenerator;
 use OpenCodeModeling\CodeAst\NodeVisitor\ClassExtends;
@@ -21,7 +21,7 @@
 use PhpParser\NodeTraverser;
 use PhpParser\NodeVisitor;
 
-final class ClassFactory
+final class ClassBuilder
 {
     /** @var string|null */
     private $namespace;
@@ -50,7 +50,7 @@ final class ClassFactory
     /** @var string[] */
     private $namespaceUse = [];
 
-    /** @var ClassConstFactory[] */
+    /** @var ClassConstBuilder[] */
     private $constants = [];
 
     private function __construct()
@@ -125,7 +125,7 @@ public function setNamespaceUse(string ...$namespaces): self
         return $this;
     }
 
-    public function setConstants(ClassConstFactory ...$constants): self
+    public function setConstants(ClassConstBuilder ...$constants): self
     {
         $this->constants = $constants;
 
@@ -184,7 +184,7 @@ public function getNamespaceUse(): array
     }
 
     /**
-     * @return ClassConstFactory[]
+     * @return ClassConstBuilder[]
      */
     public function getConstants(): array
     {
@@ -222,7 +222,7 @@ public function generate(): array
             \array_push(
                 $visitors,
                 ...\array_map(
-                    static function (ClassConstFactory $const) {
+                    static function (ClassConstBuilder $const) {
                         return $const->generate();
                     },
                     $this->constants
@@ -274,7 +274,7 @@ static function (Node\Name $name) {
                 );
                 break;
             case $node instanceof Node\Stmt\ClassConst:
-                $this->constants[] = ClassConstFactory::fromNode($node);
+                $this->constants[] = ClassConstBuilder::fromNode($node);
                 break;
             default:
                 break;

src/Factory/ClassConstFactory.php renamed to src/Builder/ClassConstBuilder.php

@@ -8,14 +8,14 @@
 
 declare(strict_types=1);
 
-namespace OpenCodeModeling\CodeAst\Factory;
+namespace OpenCodeModeling\CodeAst\Builder;
 
 use OpenCodeModeling\CodeAst\Code\ClassConstGenerator;
 use OpenCodeModeling\CodeAst\NodeVisitor\ClassConstant;
 use PhpParser\Node;
 use PhpParser\NodeVisitor;
 
-final class ClassConstFactory
+final class ClassConstBuilder
 {
     /** @var string */
     private $name;

tests/Factory/ClassFactoryTest.php renamed to tests/Builder/ClassBuilderTest.php

@@ -2,17 +2,17 @@
 
 declare(strict_types=1);
 
-namespace OpenCodeModelingTest\CodeAst\Factory;
+namespace OpenCodeModelingTest\CodeAst\Builder;
 
-use OpenCodeModeling\CodeAst\Factory\ClassConstFactory;
-use OpenCodeModeling\CodeAst\Factory\ClassFactory;
+use OpenCodeModeling\CodeAst\Builder\ClassConstBuilder;
+use OpenCodeModeling\CodeAst\Builder\ClassBuilder;
 use PhpParser\NodeTraverser;
 use PhpParser\Parser;
 use PhpParser\ParserFactory;
 use PhpParser\PrettyPrinter\Standard;
 use PHPUnit\Framework\TestCase;
 
-final class ClassFactoryTest extends TestCase
+final class ClassBuilderTest extends TestCase
 {
     /**
      * @var Parser
@@ -37,13 +37,13 @@ public function it_generates_class_for_empty_file(): void
     {
         $ast = $this->parser->parse('');
 
-        $classFactory = ClassFactory::fromScratch('TestClass', 'My\\Awesome\\Service');
+        $classFactory = ClassBuilder::fromScratch('TestClass', 'My\\Awesome\\Service');
         $classFactory
             ->setFinal(true)
             ->setExtends('BaseClass')
             ->setNamespaceUse('Foo\\Bar')
             ->setImplements('\\Iterator', 'Bar')
-            ->setConstants(ClassConstFactory::fromScratch('PRIV', 'private')->setPrivate());
+            ->setConstants(ClassConstBuilder::fromScratch('PRIV', 'private')->setPrivate());
 
         $nodeTraverser = new NodeTraverser();
         $classFactory->injectVisitors($nodeTraverser);
@@ -87,7 +87,7 @@ final class TestClass extends BaseClass implements \Iterator, Bar
 
         $ast = $this->parser->parse($expected);
 
-        $classFactory = ClassFactory::fromNodes(...$ast);
+        $classFactory = ClassBuilder::fromNodes(...$ast);
 
         $this->assertSame('TestClass', $classFactory->getName());
         $this->assertSame('BaseClass', $classFactory->getExtends());