diff --git a/tests/Doctrine/Tests/ORM/Functional/EntityListenersOnMergeTest.php b/tests/Doctrine/Tests/ORM/Functional/EntityListenersOnMergeTest.php index e6f3d5fe8..6c38d9326 100644 --- a/tests/Doctrine/Tests/ORM/Functional/EntityListenersOnMergeTest.php +++ b/tests/Doctrine/Tests/ORM/Functional/EntityListenersOnMergeTest.php @@ -2,22 +2,23 @@ namespace Doctrine\Tests\ORM\Functional; +use Doctrine\ORM\Event\LifecycleEventArgs; use Doctrine\Tests\Models\Company\CompanyContractListener; use Doctrine\Tests\Models\Company\CompanyFixContract; use Doctrine\Tests\Models\DDC3597\DDC3597Image; use Doctrine\Tests\Models\DDC3597\DDC3597Media; use Doctrine\Tests\Models\DDC3597\DDC3597Root; +use Doctrine\Tests\OrmFunctionalTestCase; /** * @group DDC-1955 * @group 5570 * @group 6174 */ -class EntityListenersOnMergeTest extends \Doctrine\Tests\OrmFunctionalTestCase +class EntityListenersOnMergeTest extends OrmFunctionalTestCase { - /** - * @var \Doctrine\Tests\Models\Company\CompanyContractListener + * @var CompanyContractListener */ private $listener; @@ -26,34 +27,32 @@ class EntityListenersOnMergeTest extends \Doctrine\Tests\OrmFunctionalTestCase $this->useModelSet('company'); parent::setUp(); - $this->_schemaTool->createSchema( - [ - $this->_em->getClassMetadata(DDC3597Root::class), - $this->_em->getClassMetadata(DDC3597Media::class), - $this->_em->getClassMetadata(DDC3597Image::class), - ] - ); + $this->_schemaTool->createSchema([ + $this->_em->getClassMetadata(DDC3597Root::class), + $this->_em->getClassMetadata(DDC3597Media::class), + $this->_em->getClassMetadata(DDC3597Image::class), + ]); $this->listener = $this->_em->getConfiguration() ->getEntityListenerResolver() - ->resolve('Doctrine\Tests\Models\Company\CompanyContractListener'); + ->resolve(CompanyContractListener::class); } protected function tearDown() { parent::tearDown(); - $this->_schemaTool->dropSchema( - [ - $this->_em->getClassMetadata(DDC3597Root::class), - $this->_em->getClassMetadata(DDC3597Media::class), - $this->_em->getClassMetadata(DDC3597Image::class), - ] - ); + + $this->_schemaTool->dropSchema([ + $this->_em->getClassMetadata(DDC3597Root::class), + $this->_em->getClassMetadata(DDC3597Media::class), + $this->_em->getClassMetadata(DDC3597Image::class), + ]); } public function testMergeNewEntityLifecyleEventsModificationsShouldBeKept() { $imageEntity = new DDC3597Image('foobar'); + $imageEntity->setFormat('JPG'); $imageEntity->setSize(123); $imageEntity->getDimension()->setWidth(300); @@ -68,6 +67,7 @@ class EntityListenersOnMergeTest extends \Doctrine\Tests\OrmFunctionalTestCase public function testPrePersistListenersShouldBeFiredWithCorrectEntityData() { $fix = new CompanyFixContract(); + $fix->setFixPrice(2000); $this->listener->prePersistCalls = []; @@ -79,15 +79,8 @@ class EntityListenersOnMergeTest extends \Doctrine\Tests\OrmFunctionalTestCase $this->assertSame($fix, $this->listener->prePersistCalls[0][0]); - $this->assertInstanceOf( - 'Doctrine\Tests\Models\Company\CompanyFixContract', - $this->listener->prePersistCalls[0][0] - ); - - $this->assertInstanceOf( - 'Doctrine\ORM\Event\LifecycleEventArgs', - $this->listener->prePersistCalls[0][1] - ); + $this->assertInstanceOf(CompanyFixContract::class, $this->listener->prePersistCalls[0][0]); + $this->assertInstanceOf(LifecycleEventArgs::class, $this->listener->prePersistCalls[0][1]); $this->assertArrayHasKey('fixPrice', $this->listener->snapshots[CompanyContractListener::PRE_PERSIST][0]); $this->assertEquals( @@ -95,4 +88,4 @@ class EntityListenersOnMergeTest extends \Doctrine\Tests\OrmFunctionalTestCase $this->listener->snapshots[CompanyContractListener::PRE_PERSIST][0]['fixPrice'] ); } -} \ No newline at end of file +}