parsing enum file (#276)

Author: AlessandroMinoccheri

src/Analyzer/FileVisitor.php

@@ -126,6 +126,13 @@ public function enterNode(Node $node): void
                 ->addDependency(new ClassDependency($node->class->toString(), $node->getLine()));
         }
 
+        if ($node instanceof Node\Stmt\Enum_ && null !== $node->namespacedName) {
+            /** @psalm-suppress UndefinedPropertyFetch */
+            $this->classDescriptionBuilder = ClassDescriptionBuilder::create(
+                $node->namespacedName->toCodeString()
+            );
+        }
+
         /**
          * matches parameters dependency in functions and method definitions like
          * public function __construct(Symfony\Component\HttpFoundation\Request $request).
@@ -154,6 +161,12 @@ public function leaveNode(Node $node): void
 
             $this->classDescriptions[] = $classDescription;
         }
+
+        if ($node instanceof Node\Stmt\Enum_ && null !== $this->classDescriptionBuilder) {
+            $classDescription = $this->classDescriptionBuilder->get();
+
+            $this->classDescriptions[] = $classDescription;
+        }
     }
 
     private function isSelfOrStaticOrParent(string $dependencyClass): bool

src/CLI/Runner.php

@@ -4,6 +4,7 @@
 
 namespace Arkitect\CLI;
 
+use Arkitect\Analyzer\ClassDescription;
 use Arkitect\Analyzer\FileParser;
 use Arkitect\Analyzer\FileParserFactory;
 use Arkitect\Analyzer\Parser;
@@ -60,6 +61,7 @@ public function check(
                 $parsingErrors->add($parsedError);
             }
 
+            /** @var ClassDescription $classDescription */
             foreach ($fileParser->getClassDescriptions() as $classDescription) {
                 foreach ($classSetRule->getRules() as $rule) {
                     $rule->check($classDescription, $violations);

src/Rules/ArchRule.php

@@ -37,7 +37,6 @@ public function check(ClassDescription $classDescription, Violations $violations
             return;
         }
 
-        //TODO
         $this->shoulds->checkAll($classDescription, $violations, $this->because);
     }
 }

src/Rules/Constraints.php

@@ -16,7 +16,6 @@ public function add(Expression $expression): void
         $this->expressions[] = $expression;
     }
 
-    //TODO
     public function checkAll(ClassDescription $classDescription, Violations $violations, string $because): void
     {
         /** @var Expression $expression */

tests/Unit/Analyzer/FileVisitorTest.php

@@ -10,6 +10,7 @@
 use Arkitect\Analyzer\FullyQualifiedClassName;
 use Arkitect\CLI\TargetPhpVersion;
 use Arkitect\Expression\ForClasses\DependsOnlyOnTheseNamespaces;
+use Arkitect\Expression\ForClasses\Implement;
 use Arkitect\Expression\ForClasses\NotHaveDependencyOutsideNamespace;
 use Arkitect\Rules\ParsingError;
 use Arkitect\Rules\Violations;
@@ -419,4 +420,32 @@ public function getStatic(): self
 
         $this->assertCount(1, $violations);
     }
+
+    public function test_it_can_parse_enum(): void
+    {
+        $code = <<< 'EOF'
+<?php
+namespace Root\Cars;
+enum Enum
+{
+    case Hearts;
+    case Diamonds;
+    case Clubs;
+    case Spades;
+}
+EOF;
+
+        /** @var FileParser $fp */
+        $fp = FileParserFactory::createFileParser(TargetPhpVersion::create('8.1'));
+        $fp->parse($code, 'relativePathName');
+
+        $cd = $fp->getClassDescriptions();
+
+        $violations = new Violations();
+
+        $notHaveDependencyOutsideNamespace = new Implement('MyInterface');
+        $notHaveDependencyOutsideNamespace->evaluate($cd[0], $violations, 'we want to add this rule for our software');
+
+        $this->assertCount(1, $violations);
+    }
 }