Nip\Router\Tests\Router\Traits\HasMatcherTraitTest

Complexity

Total Complexity

2

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	testRouteLiteral()	0	24	1
A	testRouteDynamic()	0	22	1

<?php

namespace Nip\Router\Tests\Router\Traits;

use Nip\Request;
use Nip\Router\Route\Route;
use Nip\Router\RouteFactory;
use Nip\Router\Router;
use Nip\Router\Tests\AbstractTest;
use Nip\Router\Tests\Fixtures\Application\Library\Router\Route\StandardRoute;

/**
 * Class HasMatcherTraitTest
 * @package Nip\Router\Tests\Router\Traits
 */
class HasMatcherTraitTest extends AbstractTest
{

    public function testRouteLiteral()
    {
        $router = new Router();
        $collection = $router->getRoutes();

        RouteFactory::generateLiteralRoute($collection, "admin.index", Route::class, "/admin", "/index");

It seems like $collection defined by $router->getRoutes() on line 22 can also be of type array<integer,object<Nip\Router\Route\Route>>; however, Nip\Router\RouteFactory::generateLiteralRoute() does only seem to accept object<Nip\Router\RouteCollection>, maybe add an additional type check?

        RouteFactory::generateLiteralRoute($collection, "api.index", Route::class, "/api", "/index");

It seems like $collection defined by $router->getRoutes() on line 22 can also be of type array<integer,object<Nip\Router\Route\Route>>; however, Nip\Router\RouteFactory::generateLiteralRoute() does only seem to accept object<Nip\Router\RouteCollection>, maybe add an additional type check?

        $request = Request::create('/api/404');
        $router->route($request);
        self::assertNull($router->getCurrent());

        $request = Request::create('/api/index');
        $params = $router->route($request);
        self::assertSame([], $params);

        $currentRoute = $router->getCurrent();
        self::assertInstanceOf(Route::class, $currentRoute);
        self::assertSame('api.index', $currentRoute->getName());

        $request = Request::create('/admin/index');
        $router->route($request);
        self::assertSame('admin.index', $router->getCurrent()->getName());
    }

    public function testRouteDynamic()
    {
        $router = new Router();
        $collection = $router->getRoutes();

        RouteFactory::generateStandardRoute($collection, "admin.standard", StandardRoute::class, "/admin",

It seems like $collection defined by $router->getRoutes() on line 47 can also be of type array<integer,object<Nip\Router\Route\Route>>; however, Nip\Router\RouteFactory::generateStandardRoute() does only seem to accept object<Nip\Router\RouteCollection>, maybe add an additional type check?

            '/:controller/:action', ['module' => 'admin']);
        RouteFactory::generateStandardRoute($collection, "api.standard", StandardRoute::class, "/api",

It seems like $collection defined by $router->getRoutes() on line 47 can also be of type array<integer,object<Nip\Router\Route\Route>>; however, Nip\Router\RouteFactory::generateStandardRoute() does only seem to accept object<Nip\Router\RouteCollection>, maybe add an additional type check?

            '/:controller/:action', ['module' => 'api']);

        $request = Request::create('/api/pages/delete');
        self::assertSame(
            ['module' => 'api', 'controller' => 'pages', 'action' => 'delete'],
            $router->route($request)
        );

        $request = Request::create('/admin/pages/delete');
        self::assertSame(
            ['module' => 'admin', 'controller' => 'pages', 'action' => 'delete'],
            $router->route($request)
        );
    }
}