From 0c7c358d134e4f12b18364f23e32b991176dabc1 Mon Sep 17 00:00:00 2001
From: Joas Schilling <coding@schilljs.com>
Date: Wed, 30 Sep 2020 16:17:18 +0200
Subject: [PATCH] Increase the timeout of statuses
Signed-off-by: Joas Schilling <coding@schilljs.com>
---
apps/user_status/lib/Service/StatusService.php | 2 +-
.../Unit/BackgroundJob/ClearOldStatusesBackgroundJobTest.php | 2 +-
apps/user_status/tests/Unit/Service/StatusServiceTest.php | 4 ++--
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/apps/user_status/lib/Service/StatusService.php b/apps/user_status/lib/Service/StatusService.php
index 85368342b25..39edad35850 100644
--- a/apps/user_status/lib/Service/StatusService.php
+++ b/apps/user_status/lib/Service/StatusService.php
@@ -77,7 +77,7 @@ class StatusService {
];
/** @var int */
- public const INVALIDATE_STATUS_THRESHOLD = 5 /* minutes */ * 60 /* seconds */;
+ public const INVALIDATE_STATUS_THRESHOLD = 15 /* minutes */ * 60 /* seconds */;
/** @var int */
public const MAXIMUM_MESSAGE_LENGTH = 80;
diff --git a/apps/user_status/tests/Unit/BackgroundJob/ClearOldStatusesBackgroundJobTest.php b/apps/user_status/tests/Unit/BackgroundJob/ClearOldStatusesBackgroundJobTest.php
index ccb649ecce3..682151a3f5d 100644
--- a/apps/user_status/tests/Unit/BackgroundJob/ClearOldStatusesBackgroundJobTest.php
+++ b/apps/user_status/tests/Unit/BackgroundJob/ClearOldStatusesBackgroundJobTest.php
@@ -56,7 +56,7 @@ class ClearOldStatusesBackgroundJobTest extends TestCase {
->with(1337);
$this->mapper->expects($this->once())
->method('clearStatusesOlderThan')
- ->with(1037, 1337);
+ ->with(437, 1337);
$this->time->method('getTime')
->willReturn(1337);
diff --git a/apps/user_status/tests/Unit/Service/StatusServiceTest.php b/apps/user_status/tests/Unit/Service/StatusServiceTest.php
index 4f47070e7c1..1e61f5b09ca 100644
--- a/apps/user_status/tests/Unit/Service/StatusServiceTest.php
+++ b/apps/user_status/tests/Unit/Service/StatusServiceTest.php
@@ -152,7 +152,7 @@ class StatusServiceTest extends TestCase {
$status->setIsUserDefined(true);
$this->timeFactory->method('getTime')
- ->willReturn(1400);
+ ->willReturn(2600);
$this->mapper->expects($this->once())
->method('findByUserId')
->with('john.doe')
@@ -160,7 +160,7 @@ class StatusServiceTest extends TestCase {
$this->assertEquals($status, $this->service->findByUserId('john.doe'));
$this->assertEquals('offline', $status->getStatus());
- $this->assertEquals(1400, $status->getStatusTimestamp());
+ $this->assertEquals(2600, $status->getStatusTimestamp());
$this->assertFalse($status->getIsUserDefined());
}
--
GitLab