Skip to content

Commit 9f0c402

Browse files
cuppettclaude
andcommitted
test(provisioning_api): Revert AppConfigControllerTest data providers to match strict 'yes'-only validation
The verifyConfigKey check on core.encryption_enabled was reverted to master's strict $value !== 'yes' in 626fadd per review feedback, but the test data providers still asserted the broader truthy set (1/true/YES/on). Drop those entries so the tests match the controller. This is validation, not storage — IAppConfig::setValueBool's broader input handling is unrelated. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> Signed-off-by: Stephen Cuppett <steve@cuppett.com>
1 parent ae643d6 commit 9f0c402

1 file changed

Lines changed: 0 additions & 6 deletions

File tree

apps/provisioning_api/tests/Controller/AppConfigControllerTest.php

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -367,10 +367,6 @@ public static function dataVerifyConfigKey(): array {
367367
['dav', 'public_route', ''],
368368
['files', 'remote_route', ''],
369369
['core', 'encryption_enabled', 'yes'],
370-
['core', 'encryption_enabled', '1'],
371-
['core', 'encryption_enabled', 'true'],
372-
['core', 'encryption_enabled', 'YES'],
373-
['core', 'encryption_enabled', 'on'],
374370
];
375371
}
376372

@@ -388,8 +384,6 @@ public static function dataVerifyConfigKeyThrows(): array {
388384
['contacts', 'types', ''],
389385
['core', 'encryption_enabled', 'no'],
390386
['core', 'encryption_enabled', ''],
391-
['core', 'encryption_enabled', '0'],
392-
['core', 'encryption_enabled', 'false'],
393387
['core', 'public_files', ''],
394388
['core', 'public_dav', ''],
395389
['core', 'remote_files', ''],

0 commit comments

Comments
 (0)