fix path handling in `action_url()`, fixes #1105

Closes #1105

Merge request studip/studip!661

1 files changed, 26 insertions, 0 deletions

diff --git a/tests/unit/lib/classes/StudipControllerTest.php b/tests/unit/lib/classes/StudipControllerTest.php
index 3675f58..a7009fe 100644
--- a/tests/unit/lib/classes/StudipControllerTest.php
+++ b/tests/unit/lib/classes/StudipControllerTest.php
@@ -66,6 +66,19 @@ final class StudipControllerTest extends Codeception\Test\Unit
     }
 
     /**
+     * @dataProvider actionUrlProvider
+     * @covers StudipController::action_url
+     */
+    public function testActionUrl(string $expected, ...$args): void
+    {
+        $url = $this->getController()->action_url(...$args);
+        $this->assertEquals(
+            $expected,
+            $this->getRelativeURL($url)
+        );
+    }
+
+    /**
      * @dataProvider RedirectProvider
      * @covers StudipController::redirect
      */
@@ -192,6 +205,19 @@ final class StudipControllerTest extends Codeception\Test\Unit
         ];
     }
 
+    public function actionUrlProvider(): array
+    {
+        return [
+            'action'                   => ['dispatch.php/studip_controller_test/foo', 'foo'],
+            'action-and-parameter'     => ['dispatch.php/studip_controller_test/foo/23', 'foo/23'],
+            'action-and-parameters'    => ['dispatch.php/studip_controller_test/foo/23?bar=42', 'foo/23', ['bar' => 42]],
+
+            'fragment'                 => ['dispatch.php/studip_controller_test/foo/42/23#jump', 'foo/42/23#jump'],
+            'fragment-and-parameters'  => ['dispatch.php/studip_controller_test/foo/42/23#jump', 'foo#jump', 42, 23],
+            'url-encoding-parameters'  => ['dispatch.php/studip_controller_test/foo/%3Fabc/%2F', 'foo', '?abc', '/'],
+        ];
+    }
+
     public function RedirectProvider(): array
     {
         $result = $this->UrlForProvider();