Skip to content

Commit 8950500

Browse files
committed
minor symfony#18711 [EventDispatcher] fix tests (xabbuh)
This PR was merged into the 3.0 branch. Discussion ---------- [EventDispatcher] fix tests | Q | A | ------------- | --- | Branch? | 3.0 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | License | MIT | Doc PR | The `getListenerPriority()` method was added to the event dispatcher interface in Symfony 3.0. Thus, it no longer makes sense to have a test that expects the method not to exist (the corresponding code in the `TraceableEventDispatcher` was already removed in 576a55c. Commits ------- 636401b [EventDispatcher] fix tests
2 parents 576a55c + 636401b commit 8950500

File tree

1 file changed

+0
-10
lines changed

1 file changed

+0
-10
lines changed

src/Symfony/Component/EventDispatcher/Tests/Debug/TraceableEventDispatcherTest.php

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -73,16 +73,6 @@ public function testGetListenerPriority()
7373
$this->assertSame(123, $tdispatcher->getListenerPriority('foo', $listeners[0]));
7474
}
7575

76-
public function testGetListenerPriorityReturnsZeroWhenWrappedMethodDoesNotExist()
77-
{
78-
$dispatcher = $this->getMock('Symfony\Component\EventDispatcher\EventDispatcherInterface');
79-
$traceableEventDispatcher = new TraceableEventDispatcher($dispatcher, new Stopwatch());
80-
$traceableEventDispatcher->addListener('foo', function () {}, 123);
81-
$listeners = $traceableEventDispatcher->getListeners('foo');
82-
83-
$this->assertSame(0, $traceableEventDispatcher->getListenerPriority('foo', $listeners[0]));
84-
}
85-
8676
public function testAddRemoveSubscriber()
8777
{
8878
$dispatcher = new EventDispatcher();

0 commit comments

Comments
 (0)