From 65e55a0c49fea460aba177439878c4538dfaf1c4 Mon Sep 17 00:00:00 2001 From: Marco Pivetta Date: Thu, 15 May 2014 17:37:41 +0200 Subject: [PATCH] DDC-3123 - using `assertAttributeEmpty` (with message) instead of explicit reflection --- .../Tests/ORM/Functional/Ticket/DDC3123Test.php | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/tests/Doctrine/Tests/ORM/Functional/Ticket/DDC3123Test.php b/tests/Doctrine/Tests/ORM/Functional/Ticket/DDC3123Test.php index 25c38426c..bb4f57f15 100644 --- a/tests/Doctrine/Tests/ORM/Functional/Ticket/DDC3123Test.php +++ b/tests/Doctrine/Tests/ORM/Functional/Ticket/DDC3123Test.php @@ -28,17 +28,13 @@ class DDC3123Test extends \Doctrine\Tests\OrmFunctionalTestCase $this->_em->persist($user); $uow->scheduleExtraUpdate($user, array('name' => 'changed name')); - $listener = $this->getMock('stdClass', array('postFlush')); + $listener = $this->getMock('stdClass', array(Events::postFlush)); $listener ->expects($this->once()) - ->method('postFlush') + ->method(Events::postFlush) ->will($this->returnCallback(function () use ($uow, $test) { - $extraUpdatesReflection = new \ReflectionProperty($uow, 'extraUpdates'); - - $extraUpdatesReflection->setAccessible(true); - - $test->assertEmpty($extraUpdatesReflection->getValue($uow)); + $test->assertAttributeEmpty('extraUpdates', $uow, 'ExtraUpdates are reset before postFlush'); })); $this->_em->getEventManager()->addEventListener(Events::postFlush, $listener);