mirror of
https://github.com/nextcloud/server.git
synced 2025-01-31 06:43:12 +00:00
04f313d1d6
Previously if a server status got set to failure, it stayed that way until an addressbook-sync found changes. Now the server status is set to OK after each successful sync check (if that's not the case already), regardless of addressbook changes. This change also includes two new logging statements, which could help next time someone debugs this. Signed-off-by: Pablo Zimdahl <pablo@nextcloud.com>
145 lines
4.9 KiB
PHP
145 lines
4.9 KiB
PHP
<?php
|
|
/**
|
|
* @copyright Copyright (c) 2016, ownCloud, Inc.
|
|
*
|
|
* @author Bjoern Schiessle <bjoern@schiessle.org>
|
|
* @author Björn Schießle <bjoern@schiessle.org>
|
|
* @author Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
* @author Joas Schilling <coding@schilljs.com>
|
|
* @author Lukas Reschke <lukas@statuscode.ch>
|
|
* @author Morris Jobke <hey@morrisjobke.de>
|
|
* @author Roeland Jago Douma <roeland@famdouma.nl>
|
|
* @author Thomas Müller <thomas.mueller@tmit.eu>
|
|
*
|
|
* @license AGPL-3.0
|
|
*
|
|
* This code is free software: you can redistribute it and/or modify
|
|
* it under the terms of the GNU Affero General Public License, version 3,
|
|
* as published by the Free Software Foundation.
|
|
*
|
|
* This program is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
* GNU Affero General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU Affero General Public License, version 3,
|
|
* along with this program. If not, see <http://www.gnu.org/licenses/>
|
|
*
|
|
*/
|
|
namespace OCA\Federation\Tests;
|
|
|
|
use OC\OCS\DiscoveryService;
|
|
use OCA\Federation\DbHandler;
|
|
use OCA\Federation\SyncFederationAddressBooks;
|
|
use PHPUnit\Framework\MockObject\MockObject;
|
|
use Psr\Log\LoggerInterface;
|
|
|
|
class SyncFederationAddressbooksTest extends \Test\TestCase {
|
|
|
|
/** @var array */
|
|
private $callBacks = [];
|
|
|
|
/** @var MockObject | DiscoveryService */
|
|
private $discoveryService;
|
|
|
|
/** @var MockObject|LoggerInterface */
|
|
private $logger;
|
|
|
|
protected function setUp(): void {
|
|
parent::setUp();
|
|
|
|
$this->discoveryService = $this->getMockBuilder(DiscoveryService::class)
|
|
->disableOriginalConstructor()->getMock();
|
|
$this->discoveryService->expects($this->any())->method('discover')->willReturn([]);
|
|
$this->logger = $this->createMock(LoggerInterface::class);
|
|
}
|
|
|
|
public function testSync() {
|
|
/** @var DbHandler | MockObject $dbHandler */
|
|
$dbHandler = $this->getMockBuilder('OCA\Federation\DbHandler')
|
|
->disableOriginalConstructor()
|
|
->getMock();
|
|
$dbHandler->method('getAllServer')
|
|
->willReturn([
|
|
[
|
|
'url' => 'https://cloud.drop.box',
|
|
'url_hash' => 'sha1',
|
|
'shared_secret' => 'iloveowncloud',
|
|
'sync_token' => '0'
|
|
]
|
|
]);
|
|
$dbHandler->expects($this->once())->method('setServerStatus')->
|
|
with('https://cloud.drop.box', 1, '1');
|
|
$syncService = $this->getMockBuilder('OCA\DAV\CardDAV\SyncService')
|
|
->disableOriginalConstructor()
|
|
->getMock();
|
|
$syncService->expects($this->once())->method('syncRemoteAddressBook')
|
|
->willReturn('1');
|
|
|
|
/** @var \OCA\DAV\CardDAV\SyncService $syncService */
|
|
$s = new SyncFederationAddressBooks($dbHandler, $syncService, $this->discoveryService, $this->logger);
|
|
$s->syncThemAll(function ($url, $ex) {
|
|
$this->callBacks[] = [$url, $ex];
|
|
});
|
|
$this->assertEquals('1', count($this->callBacks));
|
|
}
|
|
|
|
public function testException() {
|
|
/** @var DbHandler | MockObject $dbHandler */
|
|
$dbHandler = $this->getMockBuilder('OCA\Federation\DbHandler')->
|
|
disableOriginalConstructor()->
|
|
getMock();
|
|
$dbHandler->method('getAllServer')->
|
|
willReturn([
|
|
[
|
|
'url' => 'https://cloud.drop.box',
|
|
'url_hash' => 'sha1',
|
|
'shared_secret' => 'iloveowncloud',
|
|
'sync_token' => '0'
|
|
]
|
|
]);
|
|
$syncService = $this->getMockBuilder('OCA\DAV\CardDAV\SyncService')
|
|
->disableOriginalConstructor()
|
|
->getMock();
|
|
$syncService->expects($this->once())->method('syncRemoteAddressBook')
|
|
->willThrowException(new \Exception('something did not work out'));
|
|
|
|
/** @var \OCA\DAV\CardDAV\SyncService $syncService */
|
|
$s = new SyncFederationAddressBooks($dbHandler, $syncService, $this->discoveryService, $this->logger);
|
|
$s->syncThemAll(function ($url, $ex) {
|
|
$this->callBacks[] = [$url, $ex];
|
|
});
|
|
$this->assertEquals(2, count($this->callBacks));
|
|
}
|
|
|
|
public function testSuccessfulSyncWithoutChangesAfterFailure() {
|
|
/** @var DbHandler | MockObject $dbHandler */
|
|
$dbHandler = $this->getMockBuilder('OCA\Federation\DbHandler')
|
|
->disableOriginalConstructor()
|
|
->getMock();
|
|
$dbHandler->method('getAllServer')
|
|
->willReturn([
|
|
[
|
|
'url' => 'https://cloud.drop.box',
|
|
'url_hash' => 'sha1',
|
|
'shared_secret' => 'ilovenextcloud',
|
|
'sync_token' => '0'
|
|
]
|
|
]);
|
|
$dbHandler->method('getServerStatus')->willReturn(\OCA\Federation\TrustedServers::STATUS_FAILURE);
|
|
$dbHandler->expects($this->once())->method('setServerStatus')->
|
|
with('https://cloud.drop.box', 1);
|
|
$syncService = $this->getMockBuilder('OCA\DAV\CardDAV\SyncService')
|
|
->disableOriginalConstructor()
|
|
->getMock();
|
|
$syncService->expects($this->once())->method('syncRemoteAddressBook')
|
|
->willReturn('0');
|
|
|
|
/** @var \OCA\DAV\CardDAV\SyncService $syncService */
|
|
$s = new SyncFederationAddressBooks($dbHandler, $syncService, $this->discoveryService, $this->logger);
|
|
$s->syncThemAll(function ($url, $ex) {
|
|
$this->callBacks[] = [$url, $ex];
|
|
});
|
|
$this->assertEquals('1', count($this->callBacks));
|
|
}
|
|
}
|