Fix unit tests
Signed-off-by: Joas Schilling <coding@schilljs.com>
This commit is contained in:
parent
922f51e901
commit
b2d9c20aac
|
@ -24,6 +24,7 @@ namespace OCA\TwoFactorBackupCodes\Tests\Unit\Controller;
|
|||
|
||||
use OCA\TwoFactorBackupCodes\Controller\SettingsController;
|
||||
use OCA\TwoFactorBackupCodes\Service\BackupCodeStorage;
|
||||
use OCP\AppFramework\Http\JSONResponse;
|
||||
use OCP\IRequest;
|
||||
use OCP\IUser;
|
||||
use OCP\IUserSession;
|
||||
|
@ -89,7 +90,9 @@ class SettingsControllerTest extends TestCase {
|
|||
'codes' => $codes,
|
||||
'state' => 'state',
|
||||
];
|
||||
$this->assertEquals($expected, $this->controller->createCodes());
|
||||
$response = $this->controller->createCodes();
|
||||
$this->assertInstanceOf(JSONResponse::class, $response);
|
||||
$this->assertEquals($expected, $response->getData());
|
||||
}
|
||||
|
||||
}
|
||||
|
|
|
@ -153,7 +153,9 @@ class AuthSettingsControllerTest extends TestCase {
|
|||
'deviceToken' => ['dummy' => 'dummy', 'canDelete' => true],
|
||||
'loginName' => 'User13',
|
||||
];
|
||||
$this->assertEquals($expected, $this->controller->create($name));
|
||||
$response = $this->controller->create($name);
|
||||
$this->assertInstanceOf(JSONResponse::class, $response);
|
||||
$this->assertEquals($expected, $response->getData());
|
||||
}
|
||||
|
||||
public function testCreateSessionNotAvailable() {
|
||||
|
|
|
@ -49,6 +49,7 @@ use OCP\IConfig;
|
|||
use OCP\ILogger;
|
||||
use OCP\INavigationManager;
|
||||
use OCP\IRequest;
|
||||
use OCP\ISession;
|
||||
use OCP\IURLGenerator;
|
||||
use OCP\Security\ISecureRandom;
|
||||
|
||||
|
@ -63,6 +64,8 @@ class SecurityMiddlewareTest extends \Test\TestCase {
|
|||
private $secException;
|
||||
/** @var SecurityException */
|
||||
private $secAjaxException;
|
||||
/** @var ISession|\PHPUnit_Framework_MockObject_MockObject */
|
||||
private $session;
|
||||
/** @var IRequest|\PHPUnit_Framework_MockObject_MockObject */
|
||||
private $request;
|
||||
/** @var ControllerMethodReflector */
|
||||
|
@ -88,6 +91,7 @@ class SecurityMiddlewareTest extends \Test\TestCase {
|
|||
$this->logger = $this->createMock(ILogger::class);
|
||||
$this->navigationManager = $this->createMock(INavigationManager::class);
|
||||
$this->urlGenerator = $this->createMock(IURLGenerator::class);
|
||||
$this->session = $this->createMock(ISession::class);
|
||||
$this->request = $this->createMock(IRequest::class);
|
||||
$this->contentSecurityPolicyManager = $this->createMock(ContentSecurityPolicyManager::class);
|
||||
$this->csrfTokenManager = $this->createMock(CsrfTokenManager::class);
|
||||
|
@ -109,6 +113,7 @@ class SecurityMiddlewareTest extends \Test\TestCase {
|
|||
$this->navigationManager,
|
||||
$this->urlGenerator,
|
||||
$this->logger,
|
||||
$this->session,
|
||||
'files',
|
||||
$isLoggedIn,
|
||||
$isAdminUser,
|
||||
|
|
Loading…
Reference in New Issue