diff options
author | Roeland Jago Douma <rullzer@users.noreply.github.com> | 2020-04-15 12:28:40 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-04-15 12:28:40 +0300 |
commit | 95ad9ab4ac0398c1d06f51ab382fd7b5081cc344 (patch) | |
tree | 1aaac85f1d8642a374f7ce7084ce747ba518a4ae /tests/Core | |
parent | 19171d77e20827e4a587b86e143eb2926140edae (diff) | |
parent | 6c49dc2d1f602752831901a4202a87806f7e4732 (diff) |
Merge pull request #20401 from nextcloud/fix/login-sso-redirct
Fix absolute redirect
Diffstat (limited to 'tests/Core')
-rw-r--r-- | tests/Core/Controller/LoginControllerTest.php | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tests/Core/Controller/LoginControllerTest.php b/tests/Core/Controller/LoginControllerTest.php index 80be53ed26d..e9d4a89aa7d 100644 --- a/tests/Core/Controller/LoginControllerTest.php +++ b/tests/Core/Controller/LoginControllerTest.php @@ -509,7 +509,7 @@ class LoginControllerTest extends TestCase { ->method('getUID') ->willReturn('jane'); $password = 'secret'; - $originalUrl = 'another%20url'; + $originalUrl = 'another url'; $redirectUrl = 'http://localhost/another url'; $this->request @@ -551,7 +551,7 @@ class LoginControllerTest extends TestCase { $this->request, $user, $password, - '%2Fapps%2Fmail' + '/apps/mail' ); $loginResult = LoginResult::success($loginData); $this->chain->expects($this->once()) @@ -563,11 +563,11 @@ class LoginControllerTest extends TestCase { ->willReturn(true); $this->urlGenerator->expects($this->once()) ->method('getAbsoluteURL') - ->with(urldecode('/apps/mail')) + ->with('/apps/mail') ->willReturn($redirectUrl); $expected = new \OCP\AppFramework\Http\RedirectResponse($redirectUrl); - $response = $this->loginController->tryLogin($user, $password, '%2Fapps%2Fmail'); + $response = $this->loginController->tryLogin($user, $password, '/apps/mail'); $this->assertEquals($expected, $response); } |