Add two more tests for 100% coverage

bulldogcreative · Aug 11, 2018 · a146948 · a146948
1 parent c4d6ac3
commit a146948
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 1 deletion.
diff --git a/tests/RouterTests.php b/tests/RouterTests.php
@@ -88,4 +88,12 @@ public function testDeleteMethod()
         $this->assertEquals($routes[0]['method'], 'DELETE');
         $this->assertEquals($routes[0]['callable'], 'callable');
     }
+
+    public function testRouteInfoMethod()
+    {
+        $router = new Router;
+        $router->delete('/', 'callable');
+        $routeInfo = $router->routeInfo();
+        $this->assertInternalType('array', $routeInfo);
+    }
 }
diff --git a/tests/RoutingTests.php b/tests/RoutingTests.php
@@ -32,7 +32,19 @@ public function testRouteWithVariables()
         $this->assertSame('callable', $routeInfo->handler());
         $this->assertInternalType('array', $routeInfo->vars());
         $this->assertSame(['id' => '1'], $routeInfo->vars());
-        var_dump($routeInfo->vars());
     }
 
+    public function testFilterUriMethod()
+    {
+        $router = new Router;
+        $router->get('/user/{id}', 'callable');
+
+        $request = new ServerRequest([], [], '/user/1?wee=true', 'GET');
+        $routeInfo = $router->run($request);
+
+        $this->assertInstanceOf(ServerRequestInterface::class, $request);
+        $this->assertSame('callable', $routeInfo->handler());
+        $this->assertInternalType('array', $routeInfo->vars());
+        $this->assertSame(['id' => '1'], $routeInfo->vars());
+    }
 }