Skip to content
Snippets Groups Projects
Commit c0ea93b5 authored by Tim van Dijen's avatar Tim van Dijen
Browse files

fix redirect authproc

parent 71f916ee
No related branches found
No related tags found
No related merge requests found
...@@ -175,7 +175,7 @@ class ExampleAuth ...@@ -175,7 +175,7 @@ class ExampleAuth
* *
* @return \SimpleSAML\HTTP\RunnableResponse * @return \SimpleSAML\HTTP\RunnableResponse
*/ */
public function redirect(Request $request): RunnableResponse public function redirecttest(Request $request): RunnableResponse
{ {
/** /**
* Request handler for redirect filter test. * Request handler for redirect filter test.
......
exampleauth-authpage: exampleauth-authpage:
path: /authpage path: /authpage
defaults: { _controller: 'SimpleSAML\Module\exampleauth\Controller\ExampleAuth::authpage' } defaults: { _controller: 'SimpleSAML\Module\exampleauth\Controller\ExampleAuth::authpage' }
exampleauth-redirect: exampleauth-redirecttest:
path: /redirect path: /redirecttest
defaults: { _controller: 'SimpleSAML\Module\exampleauth\Controller\ExampleAuth::redirect' } defaults: { _controller: 'SimpleSAML\Module\exampleauth\Controller\ExampleAuth::redirecttest' }
exampleauth-resume: exampleauth-resume:
path: /resume path: /resume
defaults: { _controller: 'SimpleSAML\Module\exampleauth\Controller\ExampleAuth::resume' } defaults: { _controller: 'SimpleSAML\Module\exampleauth\Controller\ExampleAuth::resume' }
...@@ -197,14 +197,14 @@ class ExampleAuthTest extends TestCase ...@@ -197,14 +197,14 @@ class ExampleAuthTest extends TestCase
/** /**
* Test that accessing the redirect-endpoint leads to a redirect * Test that accessing the redirecttest-endpoint leads to a redirect
* *
* @return void * @return void
*/ */
public function testRedirect(): void public function testRedirect(): void
{ {
$request = Request::create( $request = Request::create(
'/redirect', '/redirecttest',
'GET', 'GET',
['StateId' => 'someState'] ['StateId' => 'someState']
); );
...@@ -217,21 +217,21 @@ class ExampleAuthTest extends TestCase ...@@ -217,21 +217,21 @@ class ExampleAuthTest extends TestCase
} }
}); });
$response = $c->redirect($request); $response = $c->redirecttest($request);
$this->assertTrue($response->isSuccessful()); $this->assertTrue($response->isSuccessful());
$this->assertInstanceOf(RunnableResponse::class, $response); $this->assertInstanceOf(RunnableResponse::class, $response);
} }
/** /**
* Test that accessing the redirect-endpoint without StateId leads to an exception * Test that accessing the redirecttest-endpoint without StateId leads to an exception
* *
* @return void * @return void
*/ */
public function testRedirectMissingStateId(): void public function testRedirectMissingStateId(): void
{ {
$request = Request::create( $request = Request::create(
'/redirect', '/redirecttest',
'GET', 'GET',
); );
...@@ -240,6 +240,6 @@ class ExampleAuthTest extends TestCase ...@@ -240,6 +240,6 @@ class ExampleAuthTest extends TestCase
$this->expectException(Error\BadRequest::class); $this->expectException(Error\BadRequest::class);
$this->expectExceptionMessage('Missing required StateId query parameter.'); $this->expectExceptionMessage('Missing required StateId query parameter.');
$c->redirect($request); $c->redirecttest($request);
} }
} }
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment