search

MTES-MCT / dialog

Intégration de la réglementation de circulation dans les solutions numériques
https://dialog.beta.gouv.fr
GNU Affero General Public License v3.0
8 stars 0 forks source link

Corrige les namespaces de certains fichiers de tests #824

Closed florimondmanca closed 2 weeks ago

florimondmanca commented 2 weeks ago

Certains tests utilisaient App\Test\... mais le mapping du dossier tests/ est vers App\Tests\...

La question de savoir pourquoi on n'a pas d'erreur en local ni sur la CI jusqu'à ce lundi reste entière

Edit : c'est lié à Composer 2.7.7 qui corrige un bug qui ignorait certaines violations https://github.com/MTES-MCT/dialog/pull/824#issuecomment-2160677821

mmarchois commented 2 weeks ago

C'est bizarre que ce ne soit pas détecté par le check psr :thinking:

florimondmanca commented 2 weeks ago

Sur la CI le dernier job main à avoir réussi utilisait Composer 2.7.2 mais désormais ça utilise 2.7.7 (les deux utilisent PHP 8.2.20)

En local j'ai Composer 2.7.2 aussi

https://github.com/composer/composer/releases/tag/2.7.7

Ça doit être ça

Fixed PSR violations for classes not matching the namespace of a rule being hidden, this may lead to new violations being shown (https://github.com/composer/composer/issues/11957)

florimondmanca commented 2 weeks ago

C'est bon https://github.com/MTES-MCT/dialog/actions/runs/9466601959/job/26078644940