Fixed some issues with retrieving dojo and character

config/api_platform/resources.xml

@@ -79,7 +79,6 @@
 <resource class="App\Entity\Dojo">
     <operations>
         <operation class="ApiPlatform\Metadata\Get" />
-        <operation class="ApiPlatform\Metadata\Get" uriTemplate="dojo" controller="App\Controller\GetUserDojo" />
         <operation class="ApiPlatform\Metadata\Post" processor="App\State\DojoPostProcessor"/>
         <operation class="ApiPlatform\Metadata\Patch" />
     </operations>

src/Controller/GetUserDojo.php

@@ -1,21 +0,0 @@
-<?php
-namespace App\Controller;
-
-use App\Entity\Dojo;
-use Doctrine\ORM\EntityManagerInterface;
-use Symfony\Bundle\SecurityBundle\Security;
-use Symfony\Component\HttpKernel\Attribute\AsController;
-
-#[AsController]
-final class GetUserDojo
-{
-
-    public function __construct(private Security $security)
-    {}
-
-    public function __invoke(EntityManagerInterface $em): ?Dojo
-    {
-        return $em->getRepository(Dojo::class)->findOneByOwner($this->security->getUser());
-    }
-}
-

src/Factory/CharacterFactory.php

@@ -2,7 +2,7 @@
 namespace App\Factory;
 
 use App\Entity\Character;
-use Zenstruck\Foundry\ModelFactory;
+use Zenstruck\Foundry\Persistence\PersistentProxyObjectFactory;
 
 /**
  *
@@ -24,7 +24,7 @@ use Zenstruck\Foundry\ModelFactory;
  * @method static Character[]|Proxy[] randomRange(int $min, int $max, array $attributes = [])
  * @method static Character[]|Proxy[] randomSet(int $number, array $attributes = [])
  */
-final class CharacterFactory extends ModelFactory
+final class CharacterFactory extends PersistentProxyObjectFactory
 {
 
     /**
@@ -44,7 +44,7 @@ final class CharacterFactory extends ModelFactory
      *
      * @todo add your default values here
      */
-    protected function getDefaults(): array
+    protected function defaults(): array
     {
         return [
             'name' => self::faker()->firstName(),
@@ -66,7 +66,7 @@ final class CharacterFactory extends ModelFactory
         // ->afterInstantiate(function(Character $character): void {})
     }
 
-    protected static function getClass(): string
+    public static function class(): string
     {
         return Character::class;
     }

src/Factory/DojoFactory.php

@@ -1,14 +1,11 @@
 <?php
-
 namespace App\Factory;
 
 use App\Entity\Dojo;
-use Doctrine\ORM\EntityRepository;
+use Zenstruck\Foundry\Persistence\PersistentProxyObjectFactory;
-use Zenstruck\Foundry\ModelFactory;
-use Zenstruck\Foundry\Proxy;
-use Zenstruck\Foundry\RepositoryProxy;
 
 /**
+ *
  * @extends ModelFactory<Dojo>
  *
  * @method Dojo|Proxy create(array|callable $attributes = [])
@@ -27,9 +24,11 @@ use Zenstruck\Foundry\RepositoryProxy;
  * @method static Dojo[]|Proxy[] randomRange(int $min, int $max, array $attributes = [])
  * @method static Dojo[]|Proxy[] randomSet(int $number, array $attributes = [])
  */
-final class DojoFactory extends ModelFactory
+final class DojoFactory extends PersistentProxyObjectFactory
 {
+
     /**
+     *
      * @see https://symfony.com/bundles/ZenstruckFoundryBundle/current/index.html#factories-as-services
      *
      * @todo inject services if required
@@ -40,29 +39,30 @@ final class DojoFactory extends ModelFactory
     }
 
     /**
+     *
      * @see https://symfony.com/bundles/ZenstruckFoundryBundle/current/index.html#model-factories
      *
      * @todo add your default values here
      */
-    protected function getDefaults(): array
+    protected function defaults(): array
     {
         return [
             'name' => self::faker()->text(),
-            'owner' => UserFactory::new(),
+            'owner' => UserFactory::new()
         ];
     }
 
     /**
+     *
      * @see https://symfony.com/bundles/ZenstruckFoundryBundle/current/index.html#initialization
      */
     protected function initialize(): self
     {
-        return $this
+        return $this;
         // ->afterInstantiate(function(Dojo $dojo): void {})
-        ;
     }
 
-    protected static function getClass(): string
+    public static function class(): string
     {
         return Dojo::class;
     }

src/Factory/FightFactory.php

@@ -1,14 +1,11 @@
 <?php
-
 namespace App\Factory;
 
 use App\Entity\Fight;
-use App\Repository\FightRepository;
+use Zenstruck\Foundry\Persistence\PersistentProxyObjectFactory;
-use Zenstruck\Foundry\ModelFactory;
-use Zenstruck\Foundry\Proxy;
-use Zenstruck\Foundry\RepositoryProxy;
 
 /**
+ *
  * @extends ModelFactory<Fight>
  *
  * @method Fight|Proxy create(array|callable $attributes = [])
@@ -27,9 +24,11 @@ use Zenstruck\Foundry\RepositoryProxy;
  * @method static Fight[]|Proxy[] randomRange(int $min, int $max, array $attributes = [])
  * @method static Fight[]|Proxy[] randomSet(int $number, array $attributes = [])
  */
-final class FightFactory extends ModelFactory
+final class FightFactory extends PersistentProxyObjectFactory
 {
+
     /**
+     *
      * @see https://symfony.com/bundles/ZenstruckFoundryBundle/current/index.html#factories-as-services
      *
      * @todo inject services if required
@@ -40,31 +39,32 @@ final class FightFactory extends ModelFactory
     }
 
     /**
+     *
      * @see https://symfony.com/bundles/ZenstruckFoundryBundle/current/index.html#model-factories
      *
      * @todo add your default values here
      */
-    protected function getDefaults(): array
+    protected function defaults(): array
     {
         return [
             'events' => [],
             'startDate' => self::faker()->dateTime(),
             'tournament' => TournamentFactory::new(),
-            'winner' => CharacterFactory::new(),
+            'winner' => CharacterFactory::new()
         ];
     }
 
     /**
+     *
      * @see https://symfony.com/bundles/ZenstruckFoundryBundle/current/index.html#initialization
      */
     protected function initialize(): self
     {
-        return $this
+        return $this;
         // ->afterInstantiate(function(Fight $fight): void {})
-        ;
     }
 
-    protected static function getClass(): string
+    public static function class(): string
     {
         return Fight::class;
     }

src/Factory/TechniqueFactory.php

@@ -2,7 +2,7 @@
 namespace App\Factory;
 
 use App\Entity\Technique;
-use Zenstruck\Foundry\ModelFactory;
+use Zenstruck\Foundry\Persistence\PersistentProxyObjectFactory;
 
 /**
  *
@@ -24,7 +24,7 @@ use Zenstruck\Foundry\ModelFactory;
  * @method static Technique[]|Proxy[] randomRange(int $min, int $max, array $attributes = [])
  * @method static Technique[]|Proxy[] randomSet(int $number, array $attributes = [])
  */
-final class TechniqueFactory extends ModelFactory
+final class TechniqueFactory extends PersistentProxyObjectFactory
 {
 
     /**
@@ -44,7 +44,7 @@ final class TechniqueFactory extends ModelFactory
      *
      * @todo add your default values here
      */
-    protected function getDefaults(): array
+    protected function defaults(): array
     {
         return [
             'name' => self::faker()->ean13(),
@@ -66,7 +66,7 @@ final class TechniqueFactory extends ModelFactory
         // ->afterInstantiate(function(Technique $technique): void {})
     }
 
-    protected static function getClass(): string
+    public static function class(): string
     {
         return Technique::class;
     }

src/Factory/TournamentFactory.php

@@ -1,14 +1,11 @@
 <?php
-
 namespace App\Factory;
 
 use App\Entity\Tournament;
-use App\Repository\TournamentRepository;
+use Zenstruck\Foundry\Persistence\PersistentProxyObjectFactory;
-use Zenstruck\Foundry\ModelFactory;
-use Zenstruck\Foundry\Proxy;
-use Zenstruck\Foundry\RepositoryProxy;
 
 /**
+ *
  * @extends ModelFactory<Tournament>
  *
  * @method Tournament|Proxy create(array|callable $attributes = [])
@@ -27,9 +24,11 @@ use Zenstruck\Foundry\RepositoryProxy;
  * @method static Tournament[]|Proxy[] randomRange(int $min, int $max, array $attributes = [])
  * @method static Tournament[]|Proxy[] randomSet(int $number, array $attributes = [])
  */
-final class TournamentFactory extends ModelFactory
+final class TournamentFactory extends PersistentProxyObjectFactory
 {
+
     /**
+     *
      * @see https://symfony.com/bundles/ZenstruckFoundryBundle/current/index.html#factories-as-services
      *
      * @todo inject services if required
@@ -40,29 +39,30 @@ final class TournamentFactory extends ModelFactory
     }
 
     /**
+     *
      * @see https://symfony.com/bundles/ZenstruckFoundryBundle/current/index.html#model-factories
      *
      * @todo add your default values here
      */
-    protected function getDefaults(): array
+    protected function defaults(): array
     {
         return [
             'name' => self::faker()->text(255),
-            'startDate' => self::faker()->dateTime(),
+            'startDate' => self::faker()->dateTime()
         ];
     }
 
     /**
+     *
      * @see https://symfony.com/bundles/ZenstruckFoundryBundle/current/index.html#initialization
      */
     protected function initialize(): self
     {
-        return $this
+        return $this;
         // ->afterInstantiate(function(Tournament $tournament): void {})
-        ;
     }
 
-    protected static function getClass(): string
+    public static function class(): string
     {
         return Tournament::class;
     }

src/Factory/UserFactory.php

@@ -2,7 +2,7 @@
 namespace App\Factory;
 
 use App\Entity\User;
-use Zenstruck\Foundry\ModelFactory;
+use Zenstruck\Foundry\Persistence\PersistentProxyObjectFactory;
 
 /**
  *
@@ -24,7 +24,7 @@ use Zenstruck\Foundry\ModelFactory;
  * @method static User[]|Proxy[] randomRange(int $min, int $max, array $attributes = [])
  * @method static User[]|Proxy[] randomSet(int $number, array $attributes = [])
  */
-final class UserFactory extends ModelFactory
+final class UserFactory extends PersistentProxyObjectFactory
 {
 
     /**
@@ -44,7 +44,7 @@ final class UserFactory extends ModelFactory
      *
      * @todo add your default values here
      */
-    protected function getDefaults(): array
+    protected function defaults(): array
     {
         return [
             'authName' => self::faker()->userName(),
@@ -62,7 +62,7 @@ final class UserFactory extends ModelFactory
         // ->afterInstantiate(function(User $user): void {})
     }
 
-    protected static function getClass(): string
+    public static function class(): string
     {
         return User::class;
     }

src/State/DojoPostProcessor.php

@@ -35,6 +35,8 @@ class DojoPostProcessor implements ProcessorInterface
 
     private function updateDojo(Dojo $dojo): void
     {
+        $user = $this->security->getUser();
         $dojo->setOwner($this->security->getUser());
+        $user->setDojo($dojo);
     }
 }

tests/CharacterTest.php

@@ -1,14 +1,30 @@
 <?php
 namespace App\Tests;
 
+use App\Entity\Dojo;
 use App\Factory\CharacterFactory;
 use App\Factory\DojoFactory;
 use App\Factory\TechniqueFactory;
 use App\Factory\UserFactory;
+use Zenstruck\Foundry\Proxy;
 
 class CharacterTest extends AbstractTest
 {
 
+    private function createCharacters(Proxy|Dojo $dojo): array
+    {
+        $technique = TechniqueFactory::createOne();
+
+        $characters = CharacterFactory::createMany(4, [
+            'dojo' => $dojo,
+            'techniques' => [
+                $technique
+            ]
+        ]);
+
+        return $characters;
+    }
+
     /**
      * Requirement: A user should be able see all characters from a dojo, but only the public fields!
      */
@@ -92,6 +108,29 @@ class CharacterTest extends AbstractTest
             ->toBase32(), $chars[0]['techniques'][0]);
     }
 
+    /**
+     * Regression test, to verify that the method will also work when multiple dojos are created.
+     */
+    public function testRetrieveCharactersFromOwnDojoOnly(): void
+    {
+        $dojo = DojoFactory::createOne([
+            'owner' => UserFactory::createOne()
+        ]);
+        $chars = $this->createCharacters($dojo);
+
+        $otherDojos = DojoFactory::createMany(10);
+        foreach ($otherDojos as $otherDojo) {
+            $this->createCharacters($otherDojo);
+        }
+
+        $response = static::createClientWithToken($dojo->getOwner()->authName)->request('GET', '/api/dojo/characters');
+        $this->assertResponseStatusCodeSame(200);
+
+        $readChars = $response->toArray();
+
+        $this->assertEquals(count($chars), count($readChars));
+    }
+
     /**
      * Requirement: MVP only (in the future the recuitment will be different).
      * A user should be able to create a single character.

tests/DojoTest.php

@@ -11,7 +11,7 @@ class DojoTest extends AbstractTest
     /**
      * Requirement: A user should be able to create a dojo!
      */
-    public function testCreateDojo(): void
+    public function testDojoCreate(): void
     {
         $userName = "FooBarFigher";
         $dojoName = "BigFightDojo";
@@ -27,58 +27,18 @@ class DojoTest extends AbstractTest
 
         $this->assertResponseStatusCodeSame(201);
 
-        $this->assertCount(1, $userRepository->findByAuthName($userName));
+        $user = $userRepository->findOneByAuthName($userName);
-    }
 
-    /**
+        $this->assertJsonContains([
-     * Requirement: A user should be request his own dojo!
-     */
-    public function testUserReadDojo(): void
-    {
-        $userName = "FooBarFigher";
-        $dojoName = "BigFightDojo";
-        $dojo = DojoFactory::createOne(
-            [
             'name' => $dojoName,
-                'owner' => UserFactory::createOne([
+            'owner' => $this->getIri($user)
-                    'authName' => $userName
-                ])
-            ]);
-
-        static::createClientWithToken($userName)->request('GET', '/api/dojo');
-
-        $this->assertResponseStatusCodeSame(200);
-
-        $this->assertJsonContains(
-            [
-                'name' => 'BigFightDojo',
-                'members' => [],
-                'owner' => $this->getIri($dojo->getOwner()),
-                'id' => $dojo->getId()
-                    ->toBase32()
         ]);
     }
 
-    /**
-     * Requirement: A user should be request his own dojo!
-     * Fails, if the dojo is not yet created.
-     */
-    public function testUserReadDojoNotYetCreated(): void
-    {
-        $userName = "FooBarFigher";
-        $userRepository = $this->getContainer()->get(UserRepository::class);
-
-        $this->assertCount(0, $userRepository->findByAuthName($userName));
-
-        static::createClientWithToken($userName)->request('GET', '/api/dojo');
-
-        $this->assertResponseStatusCodeSame(404);
-    }
-
     /**
      * Requirement: A user should NOT be able to create more than one dojos!
      */
-    public function testUserCannotCreateMultipleDojos(): void
+    public function testDojoUserCannotCreateMultipleDojos(): void
     {
         $userName = "FooBarFigher";
         $dojoName = "BigFightDojo";
@@ -102,7 +62,7 @@ class DojoTest extends AbstractTest
      * Requirement: A user should be able to change the dojos name!
      * FIXME: Add limitation so users will not do this frequently.
      */
-    public function testChangeDojoName(): void
+    public function testDojoChangeDojoName(): void
     {
         $userName = "FooBarFigher";
         $dojoName = "BigFightDojo";
@@ -127,5 +87,24 @@ class DojoTest extends AbstractTest
 
         $this->assertResponseStatusCodeSame(200);
     }
+
+    public function testDojoRetrieve(): void
+    {
+        $userName = "FooBarFigher";
+        $dojoName = "SmallFightDojo";
+        $dojo = DojoFactory::createOne(
+            [
+                'name' => $dojoName,
+                'owner' => UserFactory::createOne([
+                    'authName' => $userName
+                ])
+            ]);
+
+        static::createClientWithToken($userName)->request('GET', '/api/dojos/' . $dojo->getId());
+
+        $this->assertJsonContains([
+            'name' => $dojoName
+        ]);
+    }
 }
 

tests/UserTest.php

@@ -6,6 +6,18 @@ use App\Factory\UserFactory;
 class UserTest extends AbstractTest
 {
 
+    public function testUserGetByAuthname(): void
+    {
+        $requestUser = UserFactory::createOne();
+        UserFactory::createMany(5);
+
+        $response = static::createClientWithToken($requestUser->authName)->request('GET',
+            '/api/users/authName/' . $requestUser->authName);
+
+        $this->assertResponseStatusCodeSame(200);
+        $this->assertEquals($requestUser->getId(), $response->toArray()['id']);
+    }
+
     /**
      * Requirement: A user should be able to update his properties
      */
@@ -53,7 +65,7 @@ class UserTest extends AbstractTest
                 ]
             ]);
 
-        $this->assertResponseStatusCodeSame(405);
+        $this->assertResponseStatusCodeSame(200);
 
         $this->assertNotEquals("foo.bar", $response->toArray()['authName']);
     }