Code cleanup

Author: tarlepp

tests/Integration/TestCase/EntityTestCase.php

@@ -179,7 +179,6 @@ public function testThatSetterReturnsInstanceOfEntity(
         string $type,
         FieldMapping|AssociationMapping $meta,
     ): void {
-        //if (!(isset($meta->fieldName) && $meta->columnDefinition === null)) {
         if (method_exists($meta, 'isManyToManyOwningSide')
             && (
                 $meta->isManyToManyOwningSide()
@@ -257,28 +256,6 @@ public function testThatGetterReturnsExpectedValue(
             );
         }
 
-        /*
-        //if (array_key_exists('columnName', $meta) || array_key_exists('joinColumns', $meta)) {
-        if (isset($meta->fieldName) && $meta->columnDefinition === null) {
-            $value = PhpUnitUtil::getValidValueForType($type, $meta);
-            $entity->{$setter}($value);
-            self::assertSame(
-                $value,
-                $callable(),
-                sprintf(
-                    'Getter method of %s:%s did not return expected value type (%s) and it returned (%s)',
-                    static::$entityName,
-                    $getter,
-                    gettype($value),
-                    gettype($callable()),
-                ),
-            );
-        } else {
-            $type = ArrayCollection::class;
-            self::assertInstanceOf($type, $callable());
-        }
-         */
-
         try {
             $method = 'assertIs' . ucfirst($type);
 

tests/Utils/PhpUnitUtil.php

@@ -270,7 +270,6 @@ private static function getValidValue(
 
         if (substr_count($type, '\\') > 1 && !str_contains($type, '|')) {
             /** @var class-string $class */
-            //$class = $meta !== [] && array_key_exists('targetEntity', $meta) ? $meta['targetEntity'] : $type;
             $class = $meta->targetEntity ?? $type;
 
             $type = self::TYPE_CUSTOM_CLASS;