diff options
Diffstat (limited to 'apps/updatenotification/tests/BackgroundJob/UpdateAvailableNotificationsTest.php')
-rw-r--r-- | apps/updatenotification/tests/BackgroundJob/UpdateAvailableNotificationsTest.php | 40 |
1 files changed, 20 insertions, 20 deletions
diff --git a/apps/updatenotification/tests/BackgroundJob/UpdateAvailableNotificationsTest.php b/apps/updatenotification/tests/BackgroundJob/UpdateAvailableNotificationsTest.php index fd9affc87b3..a271d6988fe 100644 --- a/apps/updatenotification/tests/BackgroundJob/UpdateAvailableNotificationsTest.php +++ b/apps/updatenotification/tests/BackgroundJob/UpdateAvailableNotificationsTest.php @@ -211,19 +211,19 @@ class UpdateAvailableNotificationsTest extends TestCase { $job->expects($this->never()) ->method('clearErrorNotifications'); - $this->config->expects($this->once()) - ->method('getAppValue') - ->willReturn($errorDays); - $this->config->expects($this->once()) - ->method('setAppValue') - ->with('updatenotification', 'update_check_errors', $errorDays + 1); + $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->config->expects($this->once()) - ->method('setAppValue') - ->with('updatenotification', 'update_check_errors', 0); + $this->appConfig->expects($this->once()) + ->method('setAppValueInt') + ->with('update_check_errors', 0); $job->expects($this->once()) ->method('clearErrorNotifications'); $job->expects($this->once()) @@ -302,15 +302,15 @@ class UpdateAvailableNotificationsTest extends TestCase { 'getUsersToNotify', ]); - $this->config->expects($this->once()) - ->method('getAppValue') - ->with('updatenotification', $app, false) - ->willReturn($lastNotification); + $this->appConfig->expects($this->once()) + ->method('getAppValueString') + ->with($app, '') + ->willReturn($lastNotification ? $lastNotification : ''); if ($lastNotification !== $version) { - $this->config->expects($this->once()) - ->method('setAppValue') - ->with('updatenotification', $app, $version); + $this->appConfig->expects($this->once()) + ->method('setAppValueString') + ->with($app, $version); } if ($callDelete === false) { @@ -386,10 +386,10 @@ class UpdateAvailableNotificationsTest extends TestCase { public function testGetUsersToNotify(array $groups, array $groupUsers, array $expected) { $job = $this->getJob(); - $this->config->expects($this->once()) - ->method('getAppValue') - ->with('updatenotification', 'notify_groups', '["admin"]') - ->willReturn(json_encode($groups)); + $this->appConfig->expects($this->once()) + ->method('getAppValueArray') + ->with('notify_groups', ['admin']) + ->willReturn($groups); $groupMap = []; foreach ($groupUsers as $gid => $uids) { |