fix(test): stop the positional path overriding the curated unit-only suite (#221)

`composer test:unit-only` ran `phpunit tests -c tests/phpunit-unit-only.xml`.
A positional path OVERRIDES the configured testsuite entirely, so the curated
directory list in that config was decorative — the script swept all of tests/
regardless, which is precisely what the curation exists to prevent.

Measured today:

    with `tests`      572 tests, 7 errors
    without it        565 tests, 0 errors

The 7 are unit/Repair. They need Doctrine\DBAL\ParameterType, which only
resolves through the Nextcloud server's 3rdparty autoload — exactly the
dependency a suite named "no Nextcloud bootstrap" exists to avoid.

Dropping the argument makes the config govern, which is what its comment always
claimed. unit/Repair is not lost: the main phpunit.xml suite sweeps all of
tests/ and is what CI executes in-container, where doctrine resolves.

The config's comment now says the list only governs when nothing is passed
positionally, and records that unit/Repair is absent ON PURPOSE — otherwise the
next person to notice the gap re-adds it and reintroduces the errors.

Author: rubenvdlinde

composer.json

@@ -26,7 +26,7 @@
     "cs:fix": "./vendor/bin/php-cs-fixer fix",
     "psalm": "./vendor/bin/psalm --threads=1 --no-cache",
     "test:unit": "./vendor/bin/phpunit -c phpunit.xml --colors=always --fail-on-warning --fail-on-risky",
-    "test:unit-only": "./vendor/bin/phpunit tests -c tests/phpunit-unit-only.xml --colors=always",
+    "test:unit-only": "./vendor/bin/phpunit -c tests/phpunit-unit-only.xml --colors=always",
     "openapi": "./vendor/bin/generate-spec",
     "rector": "./vendor/bin/rector && composer cs:fix",
     "check:strict": "E=0; for CMD in lint cs:check psalm; do echo; echo \"=== $CMD ===\"; composer $CMD || E=1; done; echo; if [ $E -eq 0 ]; then echo \"ALL CHECKS PASSED\"; else echo \"SOME CHECKS FAILED (see above)\"; fi; exit $E"

tests/phpunit-unit-only.xml

@@ -14,6 +14,19 @@
 			unit/Service (root-level InstallerService*/Installer* files),
 			unit/Sections, and unit/Settings were fully green but had been left
 			out of this curated list.
+
+			⚠️ THIS LIST ONLY GOVERNS IF NOTHING IS PASSED POSITIONALLY.
+			`composer test:unit-only` used to run `phpunit tests -c <this file>`,
+			and a positional path OVERRIDES the configured testsuite entirely —
+			so the curation was decorative and the script swept all of tests/
+			anyway. Measured 2026-08-24: 572 tests / 7 errors with the argument,
+			565 / 0 without it. The 7 were unit/Repair, which needs
+			Doctrine\DBAL\ParameterType from the server's 3rdparty autoload —
+			exactly the dependency this suite exists to avoid.
+
+			unit/Repair is therefore absent ON PURPOSE, not by oversight. It runs
+			in the main phpunit.xml suite, which sweeps all of tests/ and is what
+			CI executes in-container where doctrine resolves.
 		-->
 		<directory suffix="Test.php">unit/Service</directory>
 		<directory suffix="Test.php">unit/Controller</directory>