From 4621a4202040b2a9dfe88d2e806ccaf167f06308 Mon Sep 17 00:00:00 2001 From: Robin Appelman Date: Wed, 6 Nov 2019 15:10:15 +0100 Subject: [PATCH] fix tests for background file scan job Signed-off-by: Robin Appelman --- .../tests/BackgroundJob/ScanFilesTest.php | 21 ++++++++++++++++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/apps/files/tests/BackgroundJob/ScanFilesTest.php b/apps/files/tests/BackgroundJob/ScanFilesTest.php index 5b5557310d..e558800b9d 100644 --- a/apps/files/tests/BackgroundJob/ScanFilesTest.php +++ b/apps/files/tests/BackgroundJob/ScanFilesTest.php @@ -59,6 +59,11 @@ class ScanFilesTest extends TestCase { public function testRunWithoutUsers() { $this->config ->expects($this->at(0)) + ->method('getSystemValueBool') + ->with('files_no_background_scan', false) + ->will($this->returnValue(false)); + $this->config + ->expects($this->at(1)) ->method('getAppValue') ->with('files', 'cronjob_scan_files', 0) ->will($this->returnValue(50)); @@ -73,7 +78,7 @@ class ScanFilesTest extends TestCase { ->with('', 500) ->will($this->returnValue([])); $this->config - ->expects($this->at(1)) + ->expects($this->at(2)) ->method('setAppValue') ->with('files', 'cronjob_scan_files', 500); @@ -84,6 +89,11 @@ class ScanFilesTest extends TestCase { $fakeUser = $this->createMock(IUser::class); $this->config ->expects($this->at(0)) + ->method('getSystemValueBool') + ->with('files_no_background_scan', false) + ->will($this->returnValue(false)); + $this->config + ->expects($this->at(1)) ->method('getAppValue') ->with('files', 'cronjob_scan_files', 0) ->will($this->returnValue(50)); @@ -95,7 +105,7 @@ class ScanFilesTest extends TestCase { $fakeUser ])); $this->config - ->expects($this->at(1)) + ->expects($this->at(2)) ->method('setAppValue') ->with('files', 'cronjob_scan_files', 550); $this->scanFiles @@ -109,6 +119,11 @@ class ScanFilesTest extends TestCase { public function testRunWithUsersAndOffsetAtEndOfUserList() { $this->config ->expects($this->at(0)) + ->method('getSystemValueBool') + ->with('files_no_background_scan', false) + ->will($this->returnValue(false)); + $this->config + ->expects($this->at(1)) ->method('getAppValue') ->with('files', 'cronjob_scan_files', 0) ->will($this->returnValue(50)); @@ -123,7 +138,7 @@ class ScanFilesTest extends TestCase { ->with('', 500) ->will($this->returnValue([])); $this->config - ->expects($this->at(1)) + ->expects($this->at(2)) ->method('setAppValue') ->with('files', 'cronjob_scan_files', 500); $this->scanFiles