#33 Add Class_ parameter into interface of parsing property

Author: Pavel Alexeev

src/Parser/AvroClassPropertyParser.php

@@ -8,17 +8,20 @@
 use PhpKafka\PhpAvroSchemaGenerator\PhpClass\PhpClassPropertyInterface;
 use PhpKafka\PhpAvroSchemaGenerator\PhpClass\PhpClassPropertyType;
 use PhpKafka\PhpAvroSchemaGenerator\PhpClass\PhpClassPropertyTypeItem;
+use PhpParser\Node\Stmt\Class_;
+use PhpParser\Node\Stmt\Property;
 
 /**
  * We will skip transient and private properties like starting at 'o_', '_', 'omitMandatoryCheck', 'allLazyKeysMarkedAsLoaded'
  */
 class AvroClassPropertyParser extends ClassPropertyParser {
 
     /**
-     * @throws SkipPropertyException
+     * @inheritdoc
+     * @override
      */
-    public function parseProperty($property): PhpClassPropertyInterface {
-        $prop = parent::parseProperty($property);
+    public function parseProperty(Class_ $class, Property $property): PhpClassPropertyInterface {
+        $prop = parent::parseProperty($class, $property);
         if (str_starts_with($prop->getPropertyName(), 'o_') or str_starts_with($prop->getPropertyName(), '_')
             or in_array($prop->getPropertyName(), ['omitMandatoryCheck', 'allLazyKeysMarkedAsLoaded'])) {
             throw new SkipPropertyException();

src/Parser/ClassParser.php

@@ -185,7 +185,7 @@ private function getAllClassProperties(Class_ $class, array $properties): array
         foreach ($class->stmts as $pStatement) {
             if ($pStatement instanceof Property) {
                 try {
-                    $properties[] = $this->propertyParser->parseProperty($pStatement);
+                    $properties[] = $this->propertyParser->parseProperty($class, $pStatement);
                 }
                 catch(SkipPropertyException $skip){ }
             }

src/Parser/ClassPropertyParser.php

@@ -4,7 +4,6 @@
 
 namespace PhpKafka\PhpAvroSchemaGenerator\Parser;
 
-use JetBrains\PhpStorm\Pure;
 use PhpKafka\PhpAvroSchemaGenerator\Avro\Avro;
 use PhpKafka\PhpAvroSchemaGenerator\PhpClass\PhpClassProperty;
 use PhpKafka\PhpAvroSchemaGenerator\PhpClass\PhpClassPropertyInterface;
@@ -14,9 +13,9 @@
 use PhpParser\Comment\Doc;
 use PhpParser\Node\Identifier;
 use PhpParser\Node\NullableType;
+use PhpParser\Node\Stmt\Class_;
 use PhpParser\Node\Stmt\Property;
 use PhpParser\Node\UnionType;
-use RuntimeException;
 
 class ClassPropertyParser implements ClassPropertyParserInterface
 {
@@ -31,15 +30,10 @@ public function __construct(DocCommentParserInterface $docParser)
     }
 
     /**
-     * @param Property|mixed $property
-     * @return PhpClassPropertyInterface
+     * @inheritdoc
      */
-    public function parseProperty($property): PhpClassPropertyInterface
+    public function parseProperty(Class_ $class, Property $property): PhpClassPropertyInterface
     {
-        if (false === $property instanceof Property) {
-            throw new RuntimeException(sprintf('Property must be of type: %s', Property::class));
-        }
-
         $propertyAttributes = $this->getPropertyAttributes($property);
 
         return new PhpClassProperty(

src/Parser/ClassPropertyParserInterface.php

@@ -6,14 +6,16 @@
 
 use PhpKafka\PhpAvroSchemaGenerator\Exception\SkipPropertyException;
 use PhpKafka\PhpAvroSchemaGenerator\PhpClass\PhpClassPropertyInterface;
+use PhpParser\Node\Stmt\Class_;
 use PhpParser\Node\Stmt\Property;
 
 interface ClassPropertyParserInterface
 {
     /**
-     * @param Property|mixed $property
+     * @param Class_ $class
+     * @param Property $property
      * @return PhpClassPropertyInterface
      * @throws SkipPropertyException Such property will then just skipped
      */
-    public function parseProperty($property): PhpClassPropertyInterface;
+    public function parseProperty(Class_ $class, Property $property): PhpClassPropertyInterface;
 }

tests/Unit/Parser/ClassPropertyParserTest.php

@@ -10,6 +10,7 @@
 use PhpParser\Comment\Doc;
 use PhpParser\Node\Identifier;
 use PhpParser\Node\NullableType;
+use PhpParser\Node\Stmt\Class_;
 use PhpParser\Node\Stmt\Property;
 use PhpParser\Node\Stmt\PropertyProperty;
 use PhpParser\Node\UnionType;
@@ -20,6 +21,7 @@ class ClassPropertyParserTest extends TestCase
 {
     public function testParseProperty(): void
     {
+        $class = $this->getMockBuilder(Class_::class)->disableOriginalConstructor()->getMock();
         $doc = $this->getMockBuilder(Doc::class)->disableOriginalConstructor()->getMock();
         $varId = $this->getMockBuilder(VarLikeIdentifier::class)->disableOriginalConstructor()->getMock();
         $varId->name = 'bla';
@@ -48,19 +50,9 @@ public function testParseProperty(): void
         $property4->props = [$propertyProperty];
         $cpp = new ClassPropertyParser($docParser);
 
-        self::assertInstanceOf(PhpClassPropertyInterface::class, $cpp->parseProperty($property1));
-        self::assertInstanceOf(PhpClassPropertyInterface::class, $cpp->parseProperty($property2));
-        self::assertInstanceOf(PhpClassPropertyInterface::class, $cpp->parseProperty($property3));
-        self::assertInstanceOf(PhpClassPropertyInterface::class, $cpp->parseProperty($property4));
+        self::assertInstanceOf(PhpClassPropertyInterface::class, $cpp->parseProperty($class, $property1));
+        self::assertInstanceOf(PhpClassPropertyInterface::class, $cpp->parseProperty($class, $property2));
+        self::assertInstanceOf(PhpClassPropertyInterface::class, $cpp->parseProperty($class, $property3));
+        self::assertInstanceOf(PhpClassPropertyInterface::class, $cpp->parseProperty($class, $property4));
     }
-
-    public function testParsePropertyExceptionOnNonProperty(): void
-    {
-        self::expectException(\RuntimeException::class);
-        self::expectExceptionMessage('Property must be of type: PhpParser\Node\Stmt\Property');
-        $docParser = $this->getMockForAbstractClass(DocCommentParserInterface::class);
-        $cpp = new ClassPropertyParser($docParser);
-
-        $cpp->parseProperty(1);
-    }
-}
+}