add to tests - formatting

Author: taylorotwell

src/Illuminate/Support/Testing/Fakes/EventFake.php

@@ -48,6 +48,38 @@ public function __construct(Dispatcher $dispatcher, $eventsToFake = [])
         $this->eventsToFake = Arr::wrap($eventsToFake);
     }
 
+    /**
+     * Assert if an event has a listener attached to it.
+     *
+     * @param  string  $expectedEvent
+     * @param  string  $expectedListener
+     * @return void
+     */
+    public function assertListening($expectedEvent, $expectedListener)
+    {
+        foreach ($this->dispatcher->getListeners($expectedEvent) as $listenerClosure) {
+            $actualListener = (new ReflectionFunction($listenerClosure))
+                        ->getStaticVariables()['listener'];
+
+            if ($actualListener === $expectedListener ||
+                ($actualListener instanceof Closure &&
+                $expectedListener === Closure::class)) {
+                PHPUnit::assertTrue(true);
+
+                return;
+            }
+        }
+
+        PHPUnit::assertTrue(
+            false,
+            sprintf(
+                'Event [%s] does not have the [%s] listener attached to it',
+                $expectedEvent,
+                print_r($expectedListener, true)
+            )
+        );
+    }
+
     /**
      * Assert if an event was dispatched based on a truth-test callback.
      *
@@ -285,36 +317,4 @@ public function until($event, $payload = [])
     {
         return $this->dispatch($event, $payload, true);
     }
-
-    /**
-     * Assert if an event has a listener attached to it.
-     *
-     * @param string $expectedEvent
-     * @param string $expectedListener
-     * @return void
-     */
-    public function assertAttached($expectedEvent, $expectedListener)
-    {
-        foreach ($this->dispatcher->getListeners($expectedEvent) as $listenerClosure) {
-            $reflection = new ReflectionFunction($listenerClosure);
-            $actualListener = $reflection->getStaticVariables()['listener'];
-
-            if ($actualListener === $expectedListener) {
-                PHPUnit::assertTrue(true);
-
-                return;
-            }
-
-            if ($actualListener instanceof Closure && $expectedListener === Closure::class) {
-                PHPUnit::assertTrue(true);
-
-                return;
-            }
-        }
-
-        PHPUnit::assertTrue(
-            false,
-            sprintf('Event %s does not have the %s listener attached to it', $expectedEvent, print_r($expectedListener, true))
-        );
-    }
 }

tests/Integration/Events/EventFakeTest.php

@@ -128,18 +128,22 @@ public function testNonFakedHaltedEventGetsProperlyDispatchedAndReturnsResponse(
         Event::assertNotDispatched(NonImportantEvent::class);
     }
 
-    public function testAssertAttached()
+    public function testAssertListening()
     {
         Event::fake();
         Event::listen('event', 'listener');
+        Event::listen('event', PostEventSubscriber::class);
+        Event::listen('event', [PostEventSubscriber::class, 'foo']);
         Event::subscribe(PostEventSubscriber::class);
         Event::listen(function (NonImportantEvent $event) {
             // do something
         });
 
-        Event::assertAttached('event', 'listener');
-        Event::assertAttached('post-created', [PostEventSubscriber::class, 'handlePostCreated']);
-        Event::assertAttached(NonImportantEvent::class, Closure::class);
+        Event::assertListening('event', 'listener');
+        Event::assertListening('event', PostEventSubscriber::class);
+        Event::assertListening('event', [PostEventSubscriber::class, 'foo']);
+        Event::assertListening('post-created', [PostEventSubscriber::class, 'handlePostCreated']);
+        Event::assertListening(NonImportantEvent::class, Closure::class);
     }
 }