Revert "Fix tests"

This reverts commit 997e4cf3fe.

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
This commit is contained in:
Morris Jobke 2016-09-23 13:59:55 +02:00
parent 7a135d8d81
commit 8c133451ac
No known key found for this signature in database
GPG Key ID: 9CE5ED29E7FCD38A
2 changed files with 7 additions and 16 deletions

View File

@ -46,14 +46,13 @@ class UpdateCheckerTest extends TestCase {
->method('check')
->willReturn([
'version' => 123,
'versionstring' => 'Nextcloud 123',
'versionstring' => 'ownCloud 123',
'web'=> 'javascript:alert(1)',
'url'=> 'javascript:alert(2)',
]);
$expected = [
'updateAvailable' => true,
'updateVersion' => 'Nextcloud 123',
'updateVersion' => 'ownCloud 123',
];
$this->assertSame($expected, $this->updateChecker->getUpdateState());
}
@ -64,16 +63,14 @@ class UpdateCheckerTest extends TestCase {
->method('check')
->willReturn([
'version' => 123,
'versionstring' => 'Nextcloud 123',
'web'=> 'https://docs.nextcloud.com/myUrl',
'url'=> 'https://downloads.nextcloud.org/server',
'versionstring' => 'ownCloud 123',
'web'=> 'https://owncloud.org/myUrl',
]);
$expected = [
'updateAvailable' => true,
'updateVersion' => 'Nextcloud 123',
'updateLink' => 'https://docs.nextcloud.com/myUrl',
'downloadLink' => 'https://downloads.nextcloud.org/server',
'updateVersion' => 'ownCloud 123',
'updateLink' => 'https://owncloud.org/myUrl',
];
$this->assertSame($expected, $this->updateChecker->getUpdateState());
}

View File

@ -127,7 +127,6 @@ class AdminControllerTest extends TestCase {
'channels' => $channels,
'newVersionString' => '8.1.2',
'updaterRequirementsFulfilled' => true,
'downloadLink' => '',
];
$expected = new TemplateResponse('updatenotification', 'admin', $params, '');
@ -161,10 +160,7 @@ class AdminControllerTest extends TestCase {
$this->updateChecker
->expects($this->once())
->method('getUpdateState')
->willReturn([
'updateVersion' => '8.1.2',
'downloadLink' => 'https://downloads.nextcloud.org/server',
]);
->willReturn(['updateVersion' => '8.1.2']);
$this->adminController
->expects($this->once())
->method('isCompatibleWithUpdater')
@ -177,7 +173,6 @@ class AdminControllerTest extends TestCase {
'channels' => $channels,
'newVersionString' => '8.1.2',
'updaterRequirementsFulfilled' => false,
'downloadLink' => 'https://downloads.nextcloud.org/server',
];
$expected = new TemplateResponse('updatenotification', 'admin', $params, '');
@ -224,7 +219,6 @@ class AdminControllerTest extends TestCase {
'channels' => $channels,
'newVersionString' => '',
'updaterRequirementsFulfilled' => true,
'downloadLink' => '',
];
$expected = new TemplateResponse('updatenotification', 'admin', $params, '');