From d7d805ef7904063b3fad77d42e400b42d9ae1400 Mon Sep 17 00:00:00 2001 From: Joas Schilling Date: Wed, 30 Sep 2020 16:17:18 +0200 Subject: [PATCH] Increase the timeout of statuses Signed-off-by: Joas Schilling --- 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 85368342b2..39edad3585 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 ccb649ecce..682151a3f5 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 4f47070e7c..1e61f5b09c 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()); }