123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460 |
- <?php
- declare(strict_types=1);
- /**
- * SPDX-FileCopyrightText: 2016 ownCloud, Inc.
- * SPDX-License-Identifier: AGPL-3.0-only
- */
- namespace OCA\UpdateNotification\Tests\BackgroundJob;
- use OC\Installer;
- use OC\Updater\VersionCheck;
- use OCA\UpdateNotification\BackgroundJob\UpdateAvailableNotifications;
- use OCP\App\IAppManager;
- use OCP\AppFramework\Services\IAppConfig;
- use OCP\AppFramework\Utility\ITimeFactory;
- use OCP\IConfig;
- use OCP\IGroup;
- use OCP\IGroupManager;
- use OCP\IUser;
- use OCP\Notification\IManager;
- use OCP\Notification\INotification;
- use PHPUnit\Framework\MockObject\MockObject;
- use Test\TestCase;
- class UpdateAvailableNotificationsTest extends TestCase {
- private IConfig|MockObject $config;
- private IManager|MockObject $notificationManager;
- private IGroupManager|MockObject $groupManager;
- private IAppManager|MockObject $appManager;
- private IAppConfig|MockObject $appConfig;
- private ITimeFactory|MockObject $timeFactory;
- private Installer|MockObject $installer;
- private VersionCheck|MockObject $versionCheck;
- protected function setUp(): void {
- parent::setUp();
- $this->config = $this->createMock(IConfig::class);
- $this->appConfig = $this->createMock(IAppConfig::class);
- $this->notificationManager = $this->createMock(IManager::class);
- $this->groupManager = $this->createMock(IGroupManager::class);
- $this->appManager = $this->createMock(IAppManager::class);
- $this->timeFactory = $this->createMock(ITimeFactory::class);
- $this->installer = $this->createMock(Installer::class);
- $this->versionCheck = $this->createMock(VersionCheck::class);
- }
- /**
- * @param array $methods
- * @return UpdateAvailableNotifications|MockObject
- */
- protected function getJob(array $methods = []) {
- if (empty($methods)) {
- return new UpdateAvailableNotifications(
- $this->timeFactory,
- $this->config,
- $this->appConfig,
- $this->notificationManager,
- $this->groupManager,
- $this->appManager,
- $this->installer,
- $this->versionCheck,
- );
- }
- {
- return $this->getMockBuilder(UpdateAvailableNotifications::class)
- ->setConstructorArgs([
- $this->timeFactory,
- $this->config,
- $this->appConfig,
- $this->notificationManager,
- $this->groupManager,
- $this->appManager,
- $this->installer,
- $this->versionCheck,
- ])
- ->onlyMethods($methods)
- ->getMock();
- }
- }
- public function testRun() {
- $job = $this->getJob([
- 'checkCoreUpdate',
- 'checkAppUpdates',
- ]);
- $job->expects($this->once())
- ->method('checkCoreUpdate');
- $job->expects($this->once())
- ->method('checkAppUpdates');
- $this->config->expects($this->exactly(2))
- ->method('getSystemValueBool')
- ->withConsecutive(
- ['has_internet_connection', true],
- ['debug', false],
- )
- ->willReturnOnConsecutiveCalls(
- true,
- true,
- );
- self::invokePrivate($job, 'run', [null]);
- }
- public function testRunNoInternet() {
- $job = $this->getJob([
- 'checkCoreUpdate',
- 'checkAppUpdates',
- ]);
- $job->expects($this->never())
- ->method('checkCoreUpdate');
- $job->expects($this->never())
- ->method('checkAppUpdates');
- $this->config->method('getSystemValueBool')
- ->with('has_internet_connection', true)
- ->willReturn(false);
- self::invokePrivate($job, 'run', [null]);
- }
- public function dataCheckCoreUpdate(): array {
- return [
- ['daily', null, null, null, null],
- ['git', null, null, null, null],
- ['beta', [], null, null, null],
- ['beta', false, false, null, null],
- ['beta', false, false, null, 13],
- ['beta', [
- 'version' => '9.2.0',
- 'versionstring' => 'Nextcloud 11.0.0',
- ], '9.2.0', 'Nextcloud 11.0.0', null],
- ['stable', [], null, null, null],
- ['stable', false, false, null, null],
- ['stable', false, false, null, 6],
- ['stable', [
- 'version' => '9.2.0',
- 'versionstring' => 'Nextcloud 11.0.0',
- ], '9.2.0', 'Nextcloud 11.0.0', null],
- ['production', [], null, null, null],
- ['production', false, false, null, null],
- ['production', false, false, null, 2],
- ['production', [
- 'version' => '9.2.0',
- 'versionstring' => 'Nextcloud 11.0.0',
- ], '9.2.0', 'Nextcloud 11.0.0', null],
- ];
- }
- /**
- * @dataProvider dataCheckCoreUpdate
- *
- * @param string $channel
- * @param mixed $versionCheck
- * @param null|string $version
- * @param null|string $readableVersion
- * @param null|int $errorDays
- */
- public function testCheckCoreUpdate(string $channel, $versionCheck, $version, $readableVersion, $errorDays) {
- $job = $this->getJob([
- 'getChannel',
- 'createNotifications',
- 'clearErrorNotifications',
- 'sendErrorNotifications',
- ]);
- $job->expects($this->once())
- ->method('getChannel')
- ->willReturn($channel);
- if ($versionCheck === null) {
- $this->versionCheck->expects($this->never())
- ->method('check');
- } else {
- $this->versionCheck->expects($this->once())
- ->method('check')
- ->willReturn($versionCheck);
- }
- if ($version === null) {
- $job->expects($this->never())
- ->method('createNotifications');
- $job->expects($versionCheck === null ? $this->never() : $this->once())
- ->method('clearErrorNotifications');
- } elseif ($version === false) {
- $job->expects($this->never())
- ->method('createNotifications');
- $job->expects($this->never())
- ->method('clearErrorNotifications');
- $this->appConfig->expects($this->once())
- ->method('getAppValueInt')
- ->willReturn($errorDays ?? 0);
- $this->appConfig->expects($this->once())
- ->method('setAppValueInt')
- ->with('update_check_errors', $errorDays + 1);
- $job->expects($errorDays !== null ? $this->once() : $this->never())
- ->method('sendErrorNotifications')
- ->with($errorDays + 1);
- } else {
- $this->appConfig->expects($this->once())
- ->method('setAppValueInt')
- ->with('update_check_errors', 0);
- $job->expects($this->once())
- ->method('clearErrorNotifications');
- $job->expects($this->once())
- ->method('createNotifications')
- ->with('core', $version, $readableVersion);
- }
- self::invokePrivate($job, 'checkCoreUpdate');
- }
- public function dataCheckAppUpdates(): array {
- return [
- [
- ['app1', 'app2'],
- [
- ['app1', false],
- ['app2', '1.9.2'],
- ],
- [
- ['app2', '1.9.2'],
- ],
- ],
- ];
- }
- /**
- * @dataProvider dataCheckAppUpdates
- *
- * @param string[] $apps
- * @param array $isUpdateAvailable
- * @param array $notifications
- */
- public function testCheckAppUpdates(array $apps, array $isUpdateAvailable, array $notifications) {
- $job = $this->getJob([
- 'isUpdateAvailable',
- 'createNotifications',
- ]);
- $this->appManager->expects($this->once())
- ->method('getInstalledApps')
- ->willReturn($apps);
- $job->expects($this->exactly(\count($apps)))
- ->method('isUpdateAvailable')
- ->willReturnMap($isUpdateAvailable);
- $mockedMethod = $job->expects($this->exactly(\count($notifications)))
- ->method('createNotifications');
- \call_user_func_array([$mockedMethod, 'withConsecutive'], $notifications);
- self::invokePrivate($job, 'checkAppUpdates');
- }
- public function dataCreateNotifications(): array {
- return [
- ['app1', '1.0.0', '1.0.0', false, false, null, null],
- ['app2', '1.0.1', '1.0.0', '1.0.0', true, ['user1'], [['user1']]],
- ['app3', '1.0.1', false, false, true, ['user2', 'user3'], [['user2'], ['user3']]],
- ];
- }
- /**
- * @dataProvider dataCreateNotifications
- *
- * @param string $app
- * @param string $version
- * @param string|false $lastNotification
- * @param string|false $callDelete
- * @param bool $createNotification
- * @param string[]|null $users
- * @param array|null $userNotifications
- */
- public function testCreateNotifications(string $app, string $version, $lastNotification, $callDelete, $createNotification, $users, $userNotifications) {
- $job = $this->getJob([
- 'deleteOutdatedNotifications',
- 'getUsersToNotify',
- ]);
- $this->appConfig->expects($this->once())
- ->method('getAppValueString')
- ->with($app, '')
- ->willReturn($lastNotification ? $lastNotification : '');
- if ($lastNotification !== $version) {
- $this->appConfig->expects($this->once())
- ->method('setAppValueString')
- ->with($app, $version);
- }
- if ($callDelete === false) {
- $job->expects($this->never())
- ->method('deleteOutdatedNotifications');
- } else {
- $job->expects($this->once())
- ->method('deleteOutdatedNotifications')
- ->with($app, $callDelete);
- }
- if ($users === null) {
- $job->expects($this->never())
- ->method('getUsersToNotify');
- } else {
- $job->expects($this->once())
- ->method('getUsersToNotify')
- ->willReturn($users);
- }
- if ($createNotification) {
- $notification = $this->createMock(INotification::class);
- $notification->expects($this->once())
- ->method('setApp')
- ->with('updatenotification')
- ->willReturnSelf();
- $notification->expects($this->once())
- ->method('setDateTime')
- ->willReturnSelf();
- $notification->expects($this->once())
- ->method('setObject')
- ->with($app, $version)
- ->willReturnSelf();
- $notification->expects($this->once())
- ->method('setSubject')
- ->with('update_available')
- ->willReturnSelf();
- if ($userNotifications !== null) {
- $mockedMethod = $notification->expects($this->exactly(\count($userNotifications)))
- ->method('setUser')
- ->willReturnSelf();
- \call_user_func_array([$mockedMethod, 'withConsecutive'], $userNotifications);
- $this->notificationManager->expects($this->exactly(\count($userNotifications)))
- ->method('notify');
- }
- $this->notificationManager->expects($this->once())
- ->method('createNotification')
- ->willReturn($notification);
- } else {
- $this->notificationManager->expects($this->never())
- ->method('createNotification');
- }
- self::invokePrivate($job, 'createNotifications', [$app, $version]);
- }
- public function dataGetUsersToNotify(): array {
- return [
- [['g1', 'g2'], ['g1' => null, 'g2' => ['u1', 'u2']], ['u1', 'u2']],
- [['g3', 'g4'], ['g3' => ['u1', 'u2'], 'g4' => ['u2', 'u3']], ['u1', 'u2', 'u3']],
- ];
- }
- /**
- * @dataProvider dataGetUsersToNotify
- * @param string[] $groups
- * @param array $groupUsers
- * @param string[] $expected
- */
- public function testGetUsersToNotify(array $groups, array $groupUsers, array $expected) {
- $job = $this->getJob();
- $this->appConfig->expects($this->once())
- ->method('getAppValueArray')
- ->with('notify_groups', ['admin'])
- ->willReturn($groups);
- $groupMap = [];
- foreach ($groupUsers as $gid => $uids) {
- if ($uids === null) {
- $group = null;
- } else {
- $group = $this->getGroup($gid);
- $group->expects($this->any())
- ->method('getUsers')
- ->willReturn($this->getUsers($uids));
- }
- $groupMap[] = [$gid, $group];
- }
- $this->groupManager->expects($this->exactly(\count($groups)))
- ->method('get')
- ->willReturnMap($groupMap);
- $result = self::invokePrivate($job, 'getUsersToNotify');
- $this->assertEquals($expected, $result);
- // Test caching
- $result = self::invokePrivate($job, 'getUsersToNotify');
- $this->assertEquals($expected, $result);
- }
- public function dataDeleteOutdatedNotifications(): array {
- return [
- ['app1', '1.1.0'],
- ['app2', '1.2.0'],
- ];
- }
- /**
- * @dataProvider dataDeleteOutdatedNotifications
- * @param string $app
- * @param string $version
- */
- public function testDeleteOutdatedNotifications(string $app, string $version) {
- $notification = $this->createMock(INotification::class);
- $notification->expects($this->once())
- ->method('setApp')
- ->with('updatenotification')
- ->willReturnSelf();
- $notification->expects($this->once())
- ->method('setObject')
- ->with($app, $version)
- ->willReturnSelf();
- $this->notificationManager->expects($this->once())
- ->method('createNotification')
- ->willReturn($notification);
- $this->notificationManager->expects($this->once())
- ->method('markProcessed')
- ->with($notification);
- $job = $this->getJob();
- self::invokePrivate($job, 'deleteOutdatedNotifications', [$app, $version]);
- }
- /**
- * @param string[] $userIds
- * @return IUser[]|MockObject[]
- */
- protected function getUsers(array $userIds): array {
- $users = [];
- foreach ($userIds as $uid) {
- $user = $this->createMock(IUser::class);
- $user->expects($this->any())
- ->method('getUID')
- ->willReturn($uid);
- $users[] = $user;
- }
- return $users;
- }
- /**
- * @param string $gid
- * @return \OCP\IGroup|MockObject
- */
- protected function getGroup(string $gid) {
- $group = $this->createMock(IGroup::class);
- $group->expects($this->any())
- ->method('getGID')
- ->willReturn($gid);
- return $group;
- }
- }
|