Split TestableAuthorize into seperate file

pradtke · pradtke · commit c6ac4befcf54 · 2019-03-27T14:35:49.000-06:00
diff --git a/tests/modules/authorize/lib/Auth/Process/AuthorizeTest.php b/tests/modules/authorize/lib/Auth/Process/AuthorizeTest.php
@@ -6,7 +6,6 @@
 namespace SimpleSAML\Test\Module\consent\Auth\Process;
 
 use PHPUnit\Framework\TestCase;
-use SimpleSAML\Module\authorize\Auth\Process\Authorize;
 use SimpleSAML\Utils\Attributes;
 
 class AuthorizeTest extends TestCase
@@ -161,15 +160,3 @@ public function noregexScenarioProvider()
         ];
     }
 }
-// phpcs:ignore
-class TestableAuthorize extends Authorize
-{
-    /**
-     * Override the redirect behavior since its difficult to test
-     * @param array $request the state
-     */
-    protected function unauthorized(&$request)
-    {
-        $request['NOT_AUTHORIZED'] = true;
-    }
-}
diff --git a/tests/modules/authorize/lib/Auth/Process/TestableAuthorize.php b/tests/modules/authorize/lib/Auth/Process/TestableAuthorize.php
@@ -0,0 +1,20 @@
+<?php
+/**
+ * Subclass authorize filter to make it unit testable.
+ */
+
+namespace SimpleSAML\Test\Module\consent\Auth\Process;
+
+use SimpleSAML\Module\authorize\Auth\Process\Authorize;
+
+class TestableAuthorize extends Authorize
+{
+    /**
+     * Override the redirect behavior since its difficult to test
+     * @param array $request the state
+     */
+    protected function unauthorized(&$request)
+    {
+        $request['NOT_AUTHORIZED'] = true;
+    }
+}
