Skip to content
This repository was archived by the owner on Jan 29, 2020. It is now read-only.

Commit c367037

Browse files
committed
qa: CS fixes per phpcs
whitespace and long lines
1 parent df2b093 commit c367037

4 files changed

+27
-11
lines changed

src/EventManager.php

+5-2
Original file line numberDiff line numberDiff line change
@@ -118,8 +118,11 @@ public static function createUsingListenerProvider(
118118
* createUsingListenerProvider to ensure that no provider is created during
119119
* instantiation.
120120
*/
121-
public function __construct(SharedEventManagerInterface $sharedEventManager = null, array $identifiers = [], $skipProviderCreation = false)
122-
{
121+
public function __construct(
122+
SharedEventManagerInterface $sharedEventManager = null,
123+
array $identifiers = [],
124+
$skipProviderCreation = false
125+
) {
123126
$this->eventPrototype = new Event();
124127

125128
if ($skipProviderCreation) {

test/EventManagerWithProviderTest.php

+5-2
Original file line numberDiff line numberDiff line change
@@ -63,8 +63,11 @@ public function attachableProviderMethods()
6363
* @param array $arguments Arguments to pass to $method
6464
* @param EventManager $manager Event manager on which to call $method
6565
*/
66-
public function testAttachmentMethodsRaiseExceptionForNonAttachableProvider($method, array $arguments, EventManager $manager)
67-
{
66+
public function testAttachmentMethodsRaiseExceptionForNonAttachableProvider(
67+
$method,
68+
array $arguments,
69+
EventManager $manager
70+
) {
6871
$this->expectException(RuntimeException::class);
6972
$this->expectExceptionMessage('instance is not of type ' . PrioritizedListenerAttachmentInterface::class);
7073
$manager->{$method}(...$arguments);

test/ListenerProvider/PrioritizedIdentifierListenerProviderTest.php

+12-2
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,18 @@ public function setUp()
2222
$this->provider = new PrioritizedIdentifierListenerProvider();
2323
}
2424

25-
public function getListeners(PrioritizedIdentifierListenerProvider $provider, array $identifiers, $event, $priority = 1)
26-
{
25+
/**
26+
* @param string[] $identifiers
27+
* @param string|object $event
28+
* @param int $priority
29+
* @return iterable
30+
*/
31+
public function getListeners(
32+
PrioritizedIdentifierListenerProvider $provider,
33+
array $identifiers,
34+
$event,
35+
$priority = 1
36+
) {
2737
$priority = (int) $priority;
2838
$listeners = $provider->getListenersForEventByPriority($event, $identifiers);
2939
if (! isset($listeners[$priority])) {

test/ListenerProvider/PrioritizedListenerProviderTest.php

+5-5
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ public function testCanDetachPreviouslyAttachedListenerFromEvent()
163163
$listener = function ($event) {
164164
};
165165
$this->provider->attach('test', $listener);
166-
166+
167167
$event = $this->createEvent();
168168
$listeners = iterator_to_array($this->provider->getListenersForEvent($event));
169169
$this->assertSame([$listener], $listeners, 'Expected one listener for event; none found?');
@@ -179,7 +179,7 @@ public function testCanDetachListenerFromAllEventsUsingNullEventToDetach()
179179
};
180180
$this->provider->attach('test', $listener);
181181
$this->provider->attach(Event::class, $listener);
182-
182+
183183
$event = $this->createEvent();
184184
$listeners = $this->flattenListeners($this->provider->getListenersForEvent($event));
185185
$this->assertSame([$listener, $listener], $listeners);
@@ -195,7 +195,7 @@ public function testCanDetachListenerFromAllEventsViaDetachWildcardListener()
195195
};
196196
$this->provider->attach('test', $listener);
197197
$this->provider->attach(Event::class, $listener);
198-
198+
199199
$event = $this->createEvent();
200200
$listeners = $this->flattenListeners($this->provider->getListenersForEvent($event));
201201
$this->assertSame([$listener, $listener], $listeners);
@@ -212,7 +212,7 @@ public function testCanDetachWildcardListenerFromAllEvents()
212212
$this->provider->attachWildcardListener($listener);
213213
$this->provider->attach('test', $listener);
214214
$this->provider->attach(Event::class, $listener);
215-
215+
216216
$event = $this->createEvent();
217217
$listeners = $this->flattenListeners($this->provider->getListenersForEvent($event));
218218
$this->assertSame([$listener, $listener, $listener], $listeners);
@@ -229,7 +229,7 @@ public function testCanClearListenersForASingleEventName()
229229
$this->provider->attachWildcardListener($listener);
230230
$this->provider->attach('test', $listener);
231231
$this->provider->attach(Event::class, $listener);
232-
232+
233233
$event = $this->createEvent();
234234
$listeners = $this->flattenListeners($this->provider->getListenersForEvent($event));
235235
$this->assertSame([$listener, $listener, $listener], $listeners);

0 commit comments

Comments
 (0)