Skip to content

Commit ded214e

Browse files
Fix existing tests
Signed-off-by: Daniel Bishop <[email protected]>
1 parent e40590c commit ded214e

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

apps/files_sharing/tests/Controller/ShareesAPIControllerTest.php

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -254,11 +254,11 @@ public function testSearch($getData, $apiSetting, $enumSetting, $remoteSharingEn
254254
/** @var IConfig|\PHPUnit\Framework\MockObject\MockObject $config */
255255
$config = $this->createMock(IConfig::class);
256256

257-
// TODO: Add exclude groups config request
258-
$config->expects($this->exactly(1))
257+
$config->expects($this->exactly(2))
259258
->method('getAppValue')
260259
->with($this->anything(), $this->anything(), $this->anything())
261260
->willReturnMap([
261+
['core', 'shareapi_exclude_groups', 'no', 'no'],
262262
['files_sharing', 'lookupServerEnabled', 'yes', 'yes'],
263263
]);
264264

@@ -360,8 +360,12 @@ public function testSearchInvalid($getData, $message) {
360360

361361
/** @var IConfig|\PHPUnit\Framework\MockObject\MockObject $config */
362362
$config = $this->createMock(IConfig::class);
363-
$config->expects($this->never())
364-
->method('getAppValue');
363+
$config->expects($this->exactly(1))
364+
->method('getAppValue')
365+
->with($this->anything(), $this->anything(), $this->anything())
366+
->willReturnMap([
367+
['core', 'shareapi_exclude_groups', 'no', 'no'],
368+
]);
365369

366370
/** @var string */
367371
$uid = 'test123';

0 commit comments

Comments
 (0)