aboutsummaryrefslogtreecommitdiffstats
path: root/apps/updatenotification/tests/BackgroundJob/UpdateAvailableNotificationsTest.php
diff options
context:
space:
mode:
Diffstat (limited to 'apps/updatenotification/tests/BackgroundJob/UpdateAvailableNotificationsTest.php')
-rw-r--r--apps/updatenotification/tests/BackgroundJob/UpdateAvailableNotificationsTest.php40
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) {