feat(adr-008): add SettingsServiceTest with all branches

Adds PHPUnit coverage for SettingsService, mirroring the shape of the
existing SettingsControllerTest. All 6 dependencies (IAppConfig,
IAppManager, ContainerInterface, IGroupManager, IUserSession,
LoggerInterface) are mocked and every method is exercised:

- isOpenRegisterAvailable() — both installed and not-installed branches
- getSettings() — isAdmin=true (admin user), isAdmin=false (non-admin),
  and the no-user/guest branch that must skip groupManager->isAdmin()
- updateSettings() — persists known CONFIG_KEYS via IAppConfig, ignores
  unknown keys
- loadConfiguration() — success path (force=true), OpenRegister-missing
  short-circuit, and the Throwable-caught branch per ADR-005 (logs the
  real exception server-side, returns a static generic message)

10 test methods / 32 assertions — well above the ADR-008 floor of >= 3
methods per logical unit.

Also extends tests/bootstrap-unit.php to autoload NC core classes
(OCP\IAppConfig, OCP\IRequest, etc.) without triggering OC::init().
This allows the existing SettingsControllerTest to run too (previously
it failed with "Class OCP\\IRequest does not exist") so both test
classes now execute: 15 tests, 49 assertions, all passing.

Author: rubenvdlinde

tests/bootstrap-unit.php

@@ -8,16 +8,31 @@
 // Include Composer's autoloader.
 require_once __DIR__ . '/../vendor/autoload.php';
 
-// Bootstrap Nextcloud — since we run inside the Docker container,
-// the full environment (including \OC::$server) is available.
-if (file_exists(__DIR__ . '/../../../lib/base.php')) {
-    require_once __DIR__ . '/../../../lib/base.php';
+// Register NC core autoloader (classes only — no OC::init / DB boot).
+//
+// Pure unit tests mock everything with PHPUnit createMock(), but the mock
+// generator still needs the target interfaces (OCP\IAppConfig, OCP\IRequest,
+// etc.) to be autoloadable. We point Composer's ClassLoader at NC's core
+// classmap if the server is reachable at one of the expected layouts.
+$ncRoots = [
+    __DIR__ . '/../../..',                     // apps-extra/{app} next to server/lib
+    __DIR__ . '/../../../..',                  // apps-extra/nested/{app}
+    '/var/www/html',                           // running inside NC container
+];
+foreach ($ncRoots as $ncRoot) {
+    if (is_file($ncRoot . '/lib/composer/autoload.php')) {
+        require_once $ncRoot . '/lib/composer/autoload.php';
+        break;
+    }
 }
 
-// Register Test\ namespace for NC test classes.
-$serverTestsLib = __DIR__ . '/../../../tests/lib/';
-if (is_dir($serverTestsLib)) {
-    $loader = new \Composer\Autoload\ClassLoader();
-    $loader->addPsr4('Test\\', $serverTestsLib);
-    $loader->register(true);
+// Register Test\ namespace for NC test classes (used by some test bases).
+foreach ($ncRoots as $ncRoot) {
+    $serverTestsLib = $ncRoot . '/tests/lib/';
+    if (is_dir($serverTestsLib)) {
+        $loader = new \Composer\Autoload\ClassLoader();
+        $loader->addPsr4('Test\\', $serverTestsLib);
+        $loader->register(true);
+        break;
+    }
 }