don't compare resolver data to as it is not url-decoded.

azjezz · azjezz · commit f457cee7f953 · 2021-06-05T02:11:52.000+01:00
Signed-off-by: azjezz &lt;azjezz@protonmail.com&gt;
diff --git a/tests/RouterTest.php b/tests/RouterTest.php
@@ -148,7 +148,7 @@ public function testMethodNotAllowedResponses(string $_name, callable $factory):
         [$responder, $_data] = $router->routeMethodAndPath(HttpMethod::HEAD, '/get');
 
         self::assertSame('get', $responder);
-        
+
         // GET -> HEAD
         try {
             $router->routeMethodAndPath(HttpMethod::GET, '/head');
@@ -204,10 +204,10 @@ public function testAllResolvers(string $_resolver_name, IResolver $resolver, st
         self::assertSame($expected_responder, $responder);
         self::assertSame($expected_data, $data);
 
-        [$responder, $data] = $resolver->resolve(HttpMethod::GET, $in);
+        // don't compare $_data to $expected_data as it is not url-decoded, that is done by the router.
+        [$responder, $_data] = $resolver->resolve(HttpMethod::GET, $in);
 
         self::assertSame($expected_responder, $responder);
-        self::assertSame($expected_data, $data);
 
         [$responder, $data] = $this->getRouter()->setResolver($resolver)->routeMethodAndPath(HttpMethod::HEAD, $in);
 
@@ -281,7 +281,7 @@ private function typeCheckCovariantTResponder(BaseRouter $_1, BaseRouter $_2): v
 
     /**
      * @param null|array<non-empty-string, array<string, string>> $routes
-     * 
+     *
      * @return TestRouter<string>
      */
     private function getRouter(?array $routes = null): TestRouter
