diff options
author | Joas Schilling <coding@schilljs.com> | 2024-10-04 16:24:22 +0200 |
---|---|---|
committer | Joas Schilling <coding@schilljs.com> | 2024-10-21 11:28:55 +0200 |
commit | 57069e35c7666b4240db80cefa702333b2852024 (patch) | |
tree | b402bc5efc25edf3fcf312619cc125c96aaae76a /tests | |
parent | decc236f9829333ec98acf8518ae5e74fca6df41 (diff) | |
download | nextcloud-server-57069e35c7666b4240db80cefa702333b2852024.tar.gz nextcloud-server-57069e35c7666b4240db80cefa702333b2852024.zip |
test(settings): Correctly only return bool for Symfony Input::hasParameterOption callsdependabot/composer/stable30/symfony-6.4
Signed-off-by: Joas Schilling <coding@schilljs.com>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/Core/Command/User/SettingTest.php | 24 |
1 files changed, 17 insertions, 7 deletions
diff --git a/tests/Core/Command/User/SettingTest.php b/tests/Core/Command/User/SettingTest.php index 433348a8b98..4dd78708bbd 100644 --- a/tests/Core/Command/User/SettingTest.php +++ b/tests/Core/Command/User/SettingTest.php @@ -193,7 +193,16 @@ class SettingTest extends TestCase { ->willReturnMap($options); $this->consoleInput->expects($this->any()) ->method('hasParameterOption') - ->willReturnMap($parameterOptions); + ->willReturnCallback(function (string|array $config, bool $default = false) use ($parameterOptions): bool { + foreach ($parameterOptions as $parameterOption) { + if ($config === $parameterOption[0] + // Check the default value if the maps has 3 entries + && (!isset($parameterOption[2]) || $default === $parameterOption[1])) { + return end($parameterOption); + } + } + return false; + }); if ($user !== false) { $this->userManager->expects($this->once()) @@ -401,15 +410,16 @@ class SettingTest extends TestCase { if ($defaultValue === null) { $this->consoleInput->expects($this->atLeastOnce()) ->method('hasParameterOption') - ->willReturnMap([ - ['--default-value', false], - ]); + ->willReturn(false); } else { $this->consoleInput->expects($this->atLeastOnce()) ->method('hasParameterOption') - ->willReturnMap([ - ['--default-value', false, true], - ]); + ->willReturnCallback(function (string|array $config, bool $default = false): bool { + if ($config === '--default-value' && $default === false) { + return true; + } + return false; + }); $this->consoleInput->expects($this->once()) ->method('getOption') ->with('default-value') |