🧪 test(private-space): robustesse, pertinence métier et documentation des tests UserRepository

[ronan-develop]

Cette PR fiabilise et documente la couverture de tests sur la feature PrivateSpace :

Remplacement des mocks PrivateSpace par de vraies entités Doctrine dans les tests UserRepositoryTest
Ajout des champs obligatoires sur PrivateSpace pour éviter les erreurs d’intégrité
Ajout d’un contrôle métier explicite dans upgradePassword (username requis, lève LogicException)
Documentation de la couverture pertinente (README_UserRepositoryTest.md) : pas de test artificiel du constructeur, coverage à 50% assumé et expliqué
Respect strict de la pertinence métier pour chaque test, pas de test « pour le coverage »
Synchronisation vérifiée de la documentation, des schémas (classes.puml) et des endpoints (api_endpoints.md) avec l’état du code

[Copilot]

Pull Request Overview

Cette PR améliore la robustesse et la documentation des tests pour la feature PrivateSpace en remplaçant les mocks par de vraies entités Doctrine et en ajoutant une validation métier explicite dans UserRepository.

• Remplacement des mocks PrivateSpace par de vraies entités Doctrine dans les tests
• Ajout d'un contrôle métier dans upgradePassword pour valider la présence du username
• Documentation explicite de la stratégie de couverture de tests (50% assumé et justifié)

Reviewed Changes

Copilot reviewed 21 out of 24 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
lenouvel.me/tests/bootstrap.php	Configuration bootstrap pour les tests PHPUnit
lenouvel.me/tests/Repository/UserRepositoryTest.php	Tests du repository User avec vraies entités Doctrine
lenouvel.me/tests/Repository/README_UserRepositoryTest.md	Documentation de la stratégie de couverture de tests
lenouvel.me/src/Repository/UserRepository.php	Ajout d'une validation métier dans upgradePassword
lenouvel.me/src/Entity/User.php	Ajout de la relation OneToOne avec PrivateSpace
lenouvel.me/src/Entity/PrivateSpace.php	Création de l'entité PrivateSpace avec relation User

[Copilot]

[nitpick] Le test vérifie le hash CRC32C du mot de passe, mais cette logique de hashage dans __serialize() pourrait être testée de manière plus robuste en vérifiant que la sérialisation/désérialisation préserve l'intégrité des données plutôt que de tester l'implémentation interne du hash.

Suggested change

	$this->assertSame(hash('crc32c', 'secret'), $data["\0App\\Entity\\User\0password"]);
	// Serialize and then unserialize the user object
	$serialized = serialize($user);
	$unserializedUser = unserialize($serialized);
	// Check that the unserialized object has the same properties
	$this->assertInstanceOf(User::class, $unserializedUser);
	$this->assertSame($user->getUsername(), $unserializedUser->getUsername());
	$this->assertSame($user->getEmail(), $unserializedUser->getEmail());
	$this->assertSame($user->getPassword(), $unserializedUser->getPassword());

[Copilot]

L'utilisation de CRC32C pour hasher le mot de passe dans la sérialisation est problématique pour la sécurité. CRC32C n'est pas cryptographiquement sûr et peut permettre la récupération du mot de passe original. Considérez utiliser un hash cryptographique sécurisé ou éviter de sérialiser le mot de passe.

Suggested change

	$data["\0" . self::class . "\0password"] = $this->password !== null ? hash('crc32c', $this->password) : null;
	$data["\0" . self::class . "\0password"] = $this->password !== null ? hash('sha256', $this->password) : null;