Merge pull request #422 from helyakin/feature/addIsReadonlyCheck

feature: Add IsReadonly Expression

Author: fain182

README.md

@@ -251,6 +251,15 @@ $rules[] = Rule::allClasses()
     ->because('we want to be sure that aggregates are final classes');
 ```
 
+### Is readonly
+
+```php
+$rules[] = Rule::allClasses()
+    ->that(new ResideInOneOfTheseNamespaces('App\Domain\ValueObjects'))
+    ->should(new IsReadonly())
+    ->because('we want to be sure that value objects are readonly classes');
+```
+
 ### Is interface
 
 ```php
@@ -296,6 +305,15 @@ $rules[] = Rule::allClasses()
     ->because('we want to be sure that our adapters are not final classes');
 ```
 
+### Is not readonly
+
+```php
+$rules[] = Rule::allClasses()
+    ->that(new ResideInOneOfTheseNamespaces('App\Domain\Entity'))
+    ->should(new IsNotReadonly())
+    ->because('we want to be sure that there are no readonly entities');
+```
+
 ### Is not interface
 
 ```php

src/Analyzer/ClassDescription.php

@@ -20,6 +20,9 @@ class ClassDescription
     /** @var bool */
     private $final;
 
+    /** @var bool */
+    private $readonly;
+
     /** @var bool */
     private $abstract;
 
@@ -50,6 +53,7 @@ public function __construct(
         array $interfaces,
         ?FullyQualifiedClassName $extends,
         bool $final,
+        bool $readonly,
         bool $abstract,
         bool $interface,
         bool $trait,
@@ -62,6 +66,7 @@ public function __construct(
         $this->interfaces = $interfaces;
         $this->extends = $extends;
         $this->final = $final;
+        $this->readonly = $readonly;
         $this->abstract = $abstract;
         $this->docBlock = $docBlock;
         $this->attributes = $attributes;
@@ -130,6 +135,11 @@ public function isFinal(): bool
         return $this->final;
     }
 
+    public function isReadonly(): bool
+    {
+        return $this->readonly;
+    }
+
     public function isAbstract(): bool
     {
         return $this->abstract;

src/Analyzer/ClassDescriptionBuilder.php

@@ -22,6 +22,9 @@ class ClassDescriptionBuilder
     /** @var bool */
     private $final = false;
 
+    /** @var bool */
+    private $readonly = false;
+
     /** @var bool */
     private $abstract = false;
 
@@ -47,6 +50,7 @@ public function clear(): void
         $this->interfaces = [];
         $this->extend = null;
         $this->final = false;
+        $this->readonly = false;
         $this->abstract = false;
         $this->docBlock = [];
         $this->attributes = [];
@@ -92,6 +96,13 @@ public function setFinal(bool $final): self
         return $this;
     }
 
+    public function setReadonly(bool $readonly): self
+    {
+        $this->readonly = $readonly;
+
+        return $this;
+    }
+
     public function setAbstract(bool $abstract): self
     {
         $this->abstract = $abstract;
@@ -145,6 +156,7 @@ public function build(): ClassDescription
             $this->interfaces,
             $this->extend,
             $this->final,
+            $this->readonly,
             $this->abstract,
             $this->interface,
             $this->trait,

src/Analyzer/FileVisitor.php

@@ -42,6 +42,10 @@ public function enterNode(Node $node): void
                 $this->classDescriptionBuilder->setFinal(true);
             }
 
+            if ($node->isReadonly()) {
+                $this->classDescriptionBuilder->setReadonly(true);
+            }
+
             if ($node->isAbstract()) {
                 $this->classDescriptionBuilder->setAbstract(true);
             }

src/Expression/ForClasses/IsNotReadonly.php

@@ -0,0 +1,34 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Arkitect\Expression\ForClasses;
+
+use Arkitect\Analyzer\ClassDescription;
+use Arkitect\Expression\Description;
+use Arkitect\Expression\Expression;
+use Arkitect\Rules\Violation;
+use Arkitect\Rules\ViolationMessage;
+use Arkitect\Rules\Violations;
+
+class IsNotReadonly implements Expression
+{
+    public function describe(ClassDescription $theClass, string $because): Description
+    {
+        return new Description("{$theClass->getName()} should not be readonly", $because);
+    }
+
+    public function evaluate(ClassDescription $theClass, Violations $violations, string $because): void
+    {
+        if (!$theClass->isReadonly()) {
+            return;
+        }
+
+        $violation = Violation::create(
+            $theClass->getFQCN(),
+            ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+        );
+
+        $violations->add($violation);
+    }
+}

src/Expression/ForClasses/IsReadonly.php

@@ -0,0 +1,34 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Arkitect\Expression\ForClasses;
+
+use Arkitect\Analyzer\ClassDescription;
+use Arkitect\Expression\Description;
+use Arkitect\Expression\Expression;
+use Arkitect\Rules\Violation;
+use Arkitect\Rules\ViolationMessage;
+use Arkitect\Rules\Violations;
+
+class IsReadonly implements Expression
+{
+    public function describe(ClassDescription $theClass, string $because): Description
+    {
+        return new Description("{$theClass->getName()} should be readonly", $because);
+    }
+
+    public function evaluate(ClassDescription $theClass, Violations $violations, string $because): void
+    {
+        if ($theClass->isReadonly()) {
+            return;
+        }
+
+        $violation = Violation::create(
+            $theClass->getFQCN(),
+            ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+        );
+
+        $violations->add($violation);
+    }
+}

tests/Unit/Expressions/ForClasses/ContainDocBlockLikeTest.php

@@ -26,6 +26,7 @@ public function test_it_should_return_true_if_contains_doc_block(): void
             false,
             false,
             false,
+            false,
             ['/**  */myDocBlock with other information']
         );
         $because = 'we want to add this rule for our software';
@@ -53,6 +54,7 @@ public function test_it_should_return_true_if_contains_doc_block_without_because
             false,
             false,
             false,
+            false,
             ['/**  */myDocBlock with other information']
         );
         $violations = new Violations();
@@ -79,6 +81,7 @@ public function test_it_should_return_false_if_not_contains_doc_block(): void
             false,
             false,
             false,
+            false,
             ['/**  */myDocBlock with other information']
         );
         $because = 'we want to add this rule for our software';

tests/Unit/Expressions/ForClasses/ExtendTest.php

@@ -86,6 +86,7 @@ public function test_it_should_return_violation_error_if_extend_is_null(): void
             false,
             false,
             false,
+            false,
             false
         );
 

tests/Unit/Expressions/ForClasses/HaveAttributeTest.php

@@ -26,6 +26,7 @@ public function test_it_should_return_true_if_contains_doc_block(): void
             false,
             false,
             false,
+            false,
             [],
             [FullyQualifiedClassName::fromString('myAttribute')]
         );
@@ -54,6 +55,7 @@ public function test_it_should_return_true_if_contains_doc_block_without_because
             false,
             false,
             false,
+            false,
             [],
             [FullyQualifiedClassName::fromString('myAttribute')]
         );
@@ -81,6 +83,7 @@ public function test_it_should_return_false_if_not_contains_doc_block(): void
             false,
             false,
             false,
+            false,
             [],
             [FullyQualifiedClassName::fromString('myAttribute')]
         );

tests/Unit/Expressions/ForClasses/ImplementTest.php

@@ -26,6 +26,7 @@ public function test_it_should_return_violation_error(): void
             false,
             false,
             false,
+            false,
             false
         );
 
@@ -53,6 +54,7 @@ public function test_it_should_return_true_if_not_depends_on_namespace(): void
             false,
             false,
             false,
+            false,
             false
         );
         $because = 'we want to add this rule for our software';
@@ -75,6 +77,7 @@ public function test_it_should_return_false_if_depends_on_namespace(): void
             false,
             false,
             false,
+            false,
             false
         );
         $because = 'we want to add this rule for our software';
@@ -98,6 +101,7 @@ public function test_it_should_check_the_complete_fqcn(): void
             false,
             false,
             false,
+            false,
             false
         );
         $violations = new Violations();
@@ -118,6 +122,7 @@ public function test_it_should_return_if_is_an_interface(): void
             false,
             false,
             false,
+            false,
             true,
             false
         );