wip interface listener

Author: calvinalkan

src/Mappers/WordpressEventMapper.php

@@ -89,7 +89,13 @@ private function buildResolvableForMappedEvent(string $event) : Closure
 
             return function (...$args_from_wp) use ($event) {
 
-                $payload = $this->buildNamedConstructorArgs($event, $args_from_wp);
+                $args = collect($args_from_wp)->reject(function ( $arg )  {
+
+                    return empty($arg);
+
+                });
+
+                $payload = $this->buildNamedConstructorArgs($event, $args->all());
 
                 $event_object = $this->container->make($event, $payload);
 

tests/TestListeners/ListenerInterface.php

@@ -0,0 +1,11 @@
+<?php
+
+
+    namespace Tests\TestListeners;
+
+    interface ListenerInterface
+    {
+
+        public function handleEvent($event);
+
+    }

tests/Unit/BetterWpHooksTest.php

@@ -15,6 +15,7 @@
     use Tests\TestDependencies\SimpleClass;
     use Tests\TestListeners\ActionListener;
     use Tests\TestEvents\FilterableEvent;
+    use Tests\TestListeners\ListenerInterface;
     use Tests\TestStubs\DifferentContainer;
     use Tests\TestStubs\Plugin1;
     use Tests\TestStubs\Plugin2;
@@ -409,6 +410,9 @@ public function if_specified_mapped_events_can_be_resolved_from_the_container()
 
         }
 
+
+
+
         /**
          *
          *

tests/Unit/InterfaceListenerTest.php

@@ -0,0 +1,110 @@
+<?php
+
+
+    namespace Tests\Unit;
+
+    use BetterWpHooks\Dispatchers\WordpressDispatcher;
+    use BetterWpHooks\ListenerFactory;
+    use BetterWpHooks\WordpressApi;
+    use Codeception\AssertThrows;
+    use PHPUnit\Framework\TestCase;
+    use SniccoAdapter\BaseContainerAdapter;
+    use Tests\CustomAssertions;
+    use Tests\TestListeners\ListenerInterface;
+
+    class InterfaceListenerTest extends TestCase
+    {
+
+
+        use AssertThrows;
+        use CustomAssertions;
+
+        private $dispatcher;
+
+        private $wp;
+
+        /**
+         * @var BaseContainerAdapter
+         */
+        private $container;
+
+        /** @test */
+        public function a_listener_can_be_an_interface_if_bound_in_the_container () {
+
+            $this->container->singleton(ListenerInterface::class, function () {
+
+                return new InterfaceImplementation();
+
+            });
+
+            $this->dispatcher->listen('event1', ListenerInterface::class . '@handle');
+
+            $value = $this->dispatcher->dispatch('event1', 'foo');
+
+            $this->assertSame('foo', $value);
+
+        }
+
+        protected function setUp(): void {
+
+
+            parent::setUp();
+
+            $plugin_php = dirname( __DIR__, 2 ) . '/vendor/calvinalkan/wordpress-hook-api-clone/plugin.php';
+
+            require_once $plugin_php;
+
+            $this->assertEmpty( $GLOBALS['wp_filter'] );
+            $this->assertEmpty( $GLOBALS['wp_actions'] );
+            $this->assertEmpty( $GLOBALS['wp_current_filter'] );
+
+            $this->wp         = new WordpressApi();
+            $this->dispatcher = new WordpressDispatcher(
+
+                new ListenerFactory( $container = new BaseContainerAdapter() ),
+                $this->wp
+
+            );
+
+            $this->container = $container;
+
+        }
+
+        protected function tearDown(): void {
+
+            parent::tearDown();
+
+            $this->reset();
+
+        }
+
+        private function reset(): void {
+
+
+            $GLOBALS['wp_filter']         = [];
+            $GLOBALS['wp_actions']        = [];
+            $GLOBALS['wp_current_filter'] = [];
+
+            $this->dispatcher = new WordpressDispatcher(
+
+                new ListenerFactory(),
+                $this->wp
+
+            );
+
+            $this->assertEmpty( $this->dispatcher->getListeners() );
+
+
+        }
+
+
+    }
+
+    class InterfaceImplementation implements ListenerInterface {
+
+        public function handleEvent($event)
+        {
+            return 'foo';
+        }
+
+    }

tests/Unit/WordpressDispatcherTest.php

@@ -20,7 +20,6 @@
     use Tests\TestEvents\EventWithDefaultLogic;
     use Tests\TestEvents\EventWithDefaultNoTypeHit;
     use Tests\TestEvents\EventWithDefaults;
-    use Tests\TestEvents\EventWithoutParameters;
     use Tests\TestEvents\FilterableWithoutDefault;
 	use Tests\TestListeners\ActionListener;
 	use Exception;