Removes the path() method from the MiddlewareFactory

This is better accomplished using `Zend\Stratigility\path()` + the
factory.

Author: weierophinney

src/MiddlewareFactory.php

@@ -13,7 +13,6 @@
 use Psr\Http\Server\MiddlewareInterface;
 use Zend\Stratigility\MiddlewarePipe;
 use Zend\Stratigility\Middleware\CallableMiddlewareDecorator;
-use Zend\Stratigility\Middleware\PathMiddlewareDecorator;
 
 /**
  * Marshal middleware for use in the application.
@@ -121,20 +120,4 @@ public function pipeline(...$middleware) : MiddlewarePipe
         }
         return $pipeline;
     }
-
-    /**
-     * Segregate one or more middleware by path.
-     *
-     * Creates and returns a PathMiddlewareDecorator instance after first
-     * passing the $middleware argument to prepare().
-     *
-     * @param string|array|MiddlewareInterface $middleware
-     */
-    public function path(string $path, $middleware) : PathMiddlewareDecorator
-    {
-        return new PathMiddlewareDecorator(
-            $path,
-            $this->prepare($middleware)
-        );
-    }
 }

test/MiddlewareFactoryTest.php

@@ -19,8 +19,6 @@
 use Zend\Expressive\Middleware\LazyLoadingMiddleware;
 use Zend\Stratigility\MiddlewarePipe;
 use Zend\Stratigility\Middleware\CallableMiddlewareDecorator;
-use Zend\Stratigility\Middleware\DoublePassMiddlewareDecorator;
-use Zend\Stratigility\Middleware\PathMiddlewareDecorator;
 
 class MiddlewareFactoryTest extends TestCase
 {
@@ -196,50 +194,4 @@ public function testPipelineAllowsPipingArraysOfMiddlewareAndCastsThemToInternal
         $this->assertCallableMiddleware($callable, $received[0]);
         $this->assertSame($middleware, $received[1]);
     }
-
-    /**
-     * @dataProvider validPrepareTypes
-     * @param string|callable|MiddlewareInterface $middleware
-     * @param mixed $expected Expected type or value for use with assertion
-     */
-    public function testPathCreatesPathMiddlewareDecoratorUsingAnyMiddlewareTypeSupportedByPrepare(
-        $middleware,
-        string $assertion,
-        $expected
-    ) {
-        $decorator = $this->factory->path('/foo', $middleware);
-        $this->assertInstanceOf(PathMiddlewareDecorator::class, $decorator);
-
-        $this->assertAttributeSame('/foo', 'prefix', $decorator);
-
-        $r = new ReflectionProperty($decorator, 'middleware');
-        $r->setAccessible(true);
-        $received = $r->getValue($decorator);
-
-        $this->{$assertion}($expected, $received);
-    }
-
-    public function testPathCanAcceptAnArrayOfMiddleware()
-    {
-        $callable = function ($request, $handler) {
-        };
-        $middleware = new DispatchMiddleware();
-
-        $internalPipeline = [$callable, $middleware];
-
-        $decorator = $this->factory->path('/foo', $internalPipeline);
-
-        $this->assertInstanceOf(PathMiddlewareDecorator::class, $decorator);
-
-        $this->assertAttributeSame('/foo', 'prefix', $decorator);
-
-        $r = new ReflectionProperty($decorator, 'middleware');
-        $r->setAccessible(true);
-        $received = $r->getValue($decorator);
-
-        $received = $this->reflectPipeline($received);
-        $this->assertCount(2, $received);
-        $this->assertCallableMiddleware($callable, $received[0]);
-        $this->assertSame($middleware, $received[1]);
-    }
 }