Added / route with redirects

4 files changed, 76 insertions, 20 deletions

diff --git a/tests/Unit/Http/Exceptions/HttpPermanentRedirectTest.php b/tests/Unit/Http/Exceptions/HttpPermanentRedirectTest.php
new file mode 100644
index 00000000..3f6a832c
--- /dev/null
+++ b/tests/Unit/Http/Exceptions/HttpPermanentRedirectTest.php
@@ -0,0 +1,20 @@
+<?php
+
+namespace Engelsystem\Test\Unit\Http;
+
+use Engelsystem\Http\Exceptions\HttpPermanentRedirect;
+use Engelsystem\Http\Exceptions\HttpRedirect;
+use PHPUnit\Framework\TestCase;
+
+class HttpPermanentRedirectTest extends TestCase
+{
+    /**
+     * @covers \Engelsystem\Http\Exceptions\HttpPermanentRedirect::__construct
+     */
+    public function testConstruct()
+    {
+        $exception = new HttpPermanentRedirect('https://lorem.ipsum/foo/bar');
+        $this->assertInstanceOf(HttpRedirect::class, $exception);
+        $this->assertEquals(301, $exception->getStatusCode());
+    }
+}
diff --git a/tests/Unit/Http/Exceptions/HttpRedirectTest.php b/tests/Unit/Http/Exceptions/HttpRedirectTest.php
new file mode 100644
index 00000000..04190f28
--- /dev/null
+++ b/tests/Unit/Http/Exceptions/HttpRedirectTest.php
@@ -0,0 +1,26 @@
+<?php
+
+namespace Engelsystem\Test\Unit\Http;
+
+use DMS\PHPUnitExtensions\ArraySubset\ArraySubsetAsserts;
+use Engelsystem\Http\Exceptions\HttpRedirect;
+use PHPUnit\Framework\TestCase;
+
+class HttpRedirectTest extends TestCase
+{
+    use ArraySubsetAsserts;
+
+    /**
+     * @covers \Engelsystem\Http\Exceptions\HttpRedirect::__construct
+     */
+    public function testConstruct()
+    {
+        $exception = new HttpRedirect('https://lorem.ipsum/foo/bar');
+        $this->assertEquals(302, $exception->getStatusCode());
+        $this->assertArraySubset(['Location' => 'https://lorem.ipsum/foo/bar'], $exception->getHeaders());
+
+        $exception = new HttpRedirect('/test', 301, ['lorem' => 'ipsum']);
+        $this->assertEquals(301, $exception->getStatusCode());
+        $this->assertArraySubset(['lorem' => 'ipsum'], $exception->getHeaders());
+    }
+}
diff --git a/tests/Unit/Http/Exceptions/HttpTemporaryRedirectTest.php b/tests/Unit/Http/Exceptions/HttpTemporaryRedirectTest.php
new file mode 100644
index 00000000..6096830f
--- /dev/null
+++ b/tests/Unit/Http/Exceptions/HttpTemporaryRedirectTest.php
@@ -0,0 +1,20 @@
+<?php
+
+namespace Engelsystem\Test\Unit\Http;
+
+use Engelsystem\Http\Exceptions\HttpRedirect;
+use Engelsystem\Http\Exceptions\HttpTemporaryRedirect;
+use PHPUnit\Framework\TestCase;
+
+class HttpTemporaryRedirectTest extends TestCase
+{
+    /**
+     * @covers \Engelsystem\Http\Exceptions\HttpTemporaryRedirect::__construct
+     */
+    public function testConstruct()
+    {
+        $exception = new HttpTemporaryRedirect('https://lorem.ipsum/foo/bar');
+        $this->assertInstanceOf(HttpRedirect::class, $exception);
+        $this->assertEquals(302, $exception->getStatusCode());
+    }
+}
diff --git a/tests/Unit/Middleware/LegacyMiddlewareTest.php b/tests/Unit/Middleware/LegacyMiddlewareTest.php
index 0589340a..f14a38ed 100644
--- a/tests/Unit/Middleware/LegacyMiddlewareTest.php
+++ b/tests/Unit/Middleware/LegacyMiddlewareTest.php
@@ -46,37 +46,29 @@ class LegacyMiddlewareTest extends TestCase
             ->disableOriginalConstructor()
             ->getMock();
 
-        $middleware->expects($this->exactly(2))
+        $middleware->expects($this->once())
             ->method('loadPage')
-            ->withConsecutive(['user_worklog'], ['login'])
-            ->willReturnOnConsecutiveCalls(
-                ['title', 'content'],
-                ['title2', 'content2']
-            );
+            ->with('user_worklog')
+            ->willReturn(['title', 'content']);
 
-        $middleware->expects($this->exactly(3))
+        $middleware->expects($this->exactly(2))
             ->method('renderPage')
             ->withConsecutive(
                 ['user_worklog', 'title', 'content'],
-                ['404', 'Page not found', 'It\'s not available!'],
-                ['login', 'title2', 'content2']
+                ['404', 'Page not found', 'It\'s not available!']
             )
             ->willReturn($response);
 
-        $container->expects($this->exactly(4))
+        $container->expects($this->exactly(3))
             ->method('get')
-            ->withConsecutive(['request'], ['request'], ['translator'], ['request'])
+            ->withConsecutive(['request'], ['request'], ['translator'])
             ->willReturnOnConsecutiveCalls(
                 $defaultRequest,
                 $defaultRequest,
-                $translator,
-                $defaultRequest
+                $translator
             );
 
         $auth->expects($this->atLeastOnce())
-            ->method('user')
-            ->willReturn(false);
-        $auth->expects($this->atLeastOnce())
             ->method('can')
             ->willReturn(false);
 
@@ -92,17 +84,15 @@ class LegacyMiddlewareTest extends TestCase
             ->method('path')
             ->willReturn('user-worklog');
 
-        $parameters->expects($this->exactly(3))
+        $parameters->expects($this->exactly(2))
             ->method('get')
             ->with('p')
             ->willReturnOnConsecutiveCalls(
                 null,
-                'foo',
-                '/'
+                'foo'
             );
 
         $middleware->process($request, $handler);
         $middleware->process($request, $handler);
-        $middleware->process($request, $handler);
     }
 }