0b974869ba
* FIX pre-commit PHPStan hook errors on commits with no file in scope phpstan.neon.dist only analyzes htdocs/ and scripts/, but the pre-commit wrapper passed every staged file straight to phpstan regardless of path. A commit touching only files outside that scope (e.g. test/phpunit/) made phpstan exit with "No files found to analyse", failing the hook for reasons unrelated to the change being committed. Filter to htdocs/ and scripts/ first, and skip cleanly when nothing remains. * NEW: Add phpunit test for Opensurveysondage class Add a CRUD test (create/fetch/update/delete) for the Opensurveysondage class, which had no test coverage yet. Note: create()'s return value is not used to identify the created record - Opensurveysondage::create() returns $this->id, but this class never sets $this->id (the real primary key is the caller- supplied $this->id_sondage string), so the return value carries no information. The test uses errors/re-fetch to check success instead. * NEW: Auto-activate module opensurvey in OpensurveysondageTest if needed Activate module opensurvey in setUpBeforeClass() when it is not already enabled, so the test does not depend on the environment's module configuration. Note: this activation is real and persists after the test run - it is not undone by the transaction rollback in tearDownAfterClass(). Activating a module re-runs its SQL install scripts (CREATE/ALTER TABLE), which causes an implicit commit in MySQL/InnoDB, same as an admin enabling it from Setup > Modules would do. |
||
|---|---|---|
| .. | ||
| allow_phpstan_in_precommit.sh | ||
| phpstan_v1_apstats.neon | ||