[TASK] Cleanup AbstractEntityTest 82/55282/2
authorAlexander Schnitzler <git@alexanderschnitzler.de>
Fri, 5 Jan 2018 14:11:52 +0000 (15:11 +0100)
committerGeorg Ringer <georg.ringer@gmail.com>
Mon, 8 Jan 2018 13:20:42 +0000 (14:20 +0100)
Replace eval with anonymous classes

Releases: master
Resolves: #83486
Change-Id: Ic90e72f2cec212d44a32aa0ee7767e821c808ecf
Reviewed-on: https://review.typo3.org/55282
Reviewed-by: Mathias Brodala <mbrodala@pagemachine.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Reiner Teubner <reiner.teubner@me.com>
Tested-by: Reiner Teubner <reiner.teubner@me.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
typo3/sysext/extbase/Tests/Unit/DomainObject/AbstractEntityTest.php

index 592b7c2..704a747 100644 (file)
@@ -1,4 +1,5 @@
 <?php
+declare(strict_types=1);
 namespace TYPO3\CMS\Extbase\Tests\Unit\DomainObject;
 
 /*
@@ -24,16 +25,12 @@ class AbstractEntityTest extends \TYPO3\TestingFramework\Core\Unit\UnitTestCase
      */
     public function objectIsNotDirtyAfterCallingMemorizeCleanStateWithSimpleProperties()
     {
-        $domainObjectName = $this->getUniqueId('DomainObject_');
-        $domainObjectNameWithNS = __NAMESPACE__ . '\\' . $domainObjectName;
-        eval('namespace ' . __NAMESPACE__ . '; class ' . $domainObjectName . ' extends \\' . \TYPO3\CMS\Extbase\DomainObject\AbstractEntity::class . ' {
-                       public $foo;
-                       public $bar;
-               }');
-        $domainObject = new $domainObjectNameWithNS();
-        $domainObject->foo = 'Test';
-        $domainObject->bar = 'It is raining outside';
+        $domainObject = new class extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
+            public $foo = 'Test';
+            public $bar = 'It is raining outside';
+        };
         $domainObject->_memorizeCleanState();
+
         $this->assertFalse($domainObject->_isDirty());
     }
 
@@ -42,17 +39,13 @@ class AbstractEntityTest extends \TYPO3\TestingFramework\Core\Unit\UnitTestCase
      */
     public function objectIsDirtyAfterCallingMemorizeCleanStateWithSimplePropertiesAndModifyingThePropertiesAfterwards()
     {
-        $domainObjectName = $this->getUniqueId('DomainObject_');
-        $domainObjectNameWithNS = __NAMESPACE__ . '\\' . $domainObjectName;
-        eval('namespace ' . __NAMESPACE__ . '; class ' . $domainObjectName . ' extends \\' . \TYPO3\CMS\Extbase\DomainObject\AbstractEntity::class . ' {
-                       public $foo;
-                       public $bar;
-               }');
-        $domainObject = new $domainObjectNameWithNS();
-        $domainObject->foo = 'Test';
-        $domainObject->bar = 'It is raining outside';
+        $domainObject = new class extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
+            public $foo = 'Test';
+            public $bar = 'It is raining outside';
+        };
         $domainObject->_memorizeCleanState();
         $domainObject->bar = 'Now it is sunny.';
+
         $this->assertTrue($domainObject->_isDirty());
     }
 
@@ -61,16 +54,13 @@ class AbstractEntityTest extends \TYPO3\TestingFramework\Core\Unit\UnitTestCase
      */
     public function objectIsNotDirtyAfterCallingMemorizeCleanStateWithObjectProperties()
     {
-        $domainObjectName = $this->getUniqueId('DomainObject_');
-        $domainObjectNameWithNS = __NAMESPACE__ . '\\' . $domainObjectName;
-        eval('namespace ' . __NAMESPACE__ . '; class ' . $domainObjectName . ' extends \\' . \TYPO3\CMS\Extbase\DomainObject\AbstractEntity::class . ' {
-                       public $foo;
-                       public $bar;
-               }');
-        $domainObject = new $domainObjectNameWithNS();
+        $domainObject = new class extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
+            public $foo;
+            public $bar = 'It is raining outside';
+        };
         $domainObject->foo = new \DateTime();
-        $domainObject->bar = 'It is raining outside';
         $domainObject->_memorizeCleanState();
+
         $this->assertFalse($domainObject->_isDirty());
     }
 
@@ -79,24 +69,21 @@ class AbstractEntityTest extends \TYPO3\TestingFramework\Core\Unit\UnitTestCase
      */
     public function objectIsNotDirtyAfterCallingMemorizeCleanStateWithOtherDomainObjectsAsProperties()
     {
-        $domainObjectName = $this->getUniqueId('DomainObject_');
-        $domainObjectNameWithNS = __NAMESPACE__ . '\\' . $domainObjectName;
-        eval('namespace ' . __NAMESPACE__ . '; class ' . $domainObjectName . ' extends \\' . \TYPO3\CMS\Extbase\DomainObject\AbstractEntity::class . ' {
-                       public $foo;
-                       public $bar;
-               }');
-        $secondDomainObjectName = $this->getUniqueId('DomainObject_');
-        $secondDomainObjectNameWithNS = __NAMESPACE__ . '\\' . $secondDomainObjectName;
-        eval('namespace ' . __NAMESPACE__ . '; class ' . $secondDomainObjectName . ' extends \\' . \TYPO3\CMS\Extbase\DomainObject\AbstractEntity::class . ' {
-                       public $foo;
-                       public $bar;
-               }');
-        $secondDomainObject = new $secondDomainObjectNameWithNS();
+        $domainObject = new class extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
+            public $foo;
+            public $bar;
+        };
+
+        $secondDomainObject = new class extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
+            public $foo;
+            public $bar;
+        };
+
         $secondDomainObject->_memorizeCleanState();
-        $domainObject = new $domainObjectNameWithNS();
         $domainObject->foo = $secondDomainObject;
         $domainObject->bar = 'It is raining outside';
         $domainObject->_memorizeCleanState();
+
         $this->assertFalse($domainObject->_isDirty());
     }
 }