[Api] Refactor fetching id from IRI converter to direct object

Sylius · Feb 26, 2021 · e7d1f2d · e7d1f2d
1 parent 6070216
commit e7d1f2d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/src/Sylius/Bundle/ApiBundle/test/src/Tests/FooTest.php b/src/Sylius/Bundle/ApiBundle/test/src/Tests/FooTest.php
@@ -66,7 +66,10 @@ public function it_allows_to_get_collection_as_a_visitor(): void
      */
     public function it_allows_to_get_item_by_iri(): void
     {
-        $response = static::createClient()->request('GET', $this->findIriBy(Foo::class, ['name' => 'Foo0']));
+        /** @var Foo $foo */
+        $foo = $this->objects['foo1'];
+
+        $response = static::createClient()->request('GET', 'api/v2/foos/' . $foo->getId());
 
         $this->assertResponseIsSuccessful();
         $this->assertResponseHeaderSame('content-type', 'application/ld+json; charset=utf-8');

diff --git a/src/Sylius/Bundle/ApiBundle/test/src/Tests/SetUpTestsTrait.php b/src/Sylius/Bundle/ApiBundle/test/src/Tests/SetUpTestsTrait.php
@@ -23,6 +23,9 @@ trait SetUpTestsTrait
     /** @var array */
     private $fixturesFiles;
 
+    /** @var array */
+    private $objects;
+
     public function setFixturesFiles(array $fixturesFiles)
     {
         $this->fixturesFiles = array_merge(
@@ -43,9 +46,9 @@ public function setUpTest(): void
         /** @var JWTTokenManagerInterface $JWTManager */
         $JWTManager = $container->get('lexik_jwt_authentication.jwt_manager');
 
-        $objects = $loader->load($this->fixturesFiles, [], [], PurgeMode::createDeleteMode());
+        $this->objects = $loader->load($this->fixturesFiles, [], [], PurgeMode::createDeleteMode());
 
-        $adminUser = $objects['admin'];
+        $adminUser = $this->objects['admin'];
 
         $this->JWTAdminUserToken = $JWTManager->create($adminUser);
     }