Merge pull request #476 from derrabus/bugfix/loader-test

Use stub implementations instead of mocks
doctrine · May 5, 2024 · dab8064 · dab8064
2 parents 7a3052e + dcef9ac
commit dab8064
Showing 1 changed file with 33 additions and 10 deletions.
diff --git a/tests/Common/DataFixtures/LoaderTest.php b/tests/Common/DataFixtures/LoaderTest.php
@@ -6,7 +6,9 @@
 
 use Doctrine\Common\DataFixtures\FixtureInterface;
 use Doctrine\Common\DataFixtures\Loader;
+use Doctrine\Common\DataFixtures\ReferenceRepository;
 use Doctrine\Common\DataFixtures\SharedFixtureInterface;
+use Doctrine\Persistence\ObjectManager;
 use TestFixtures\MyFixture1;
 use TestFixtures\NotAFixture;
 
@@ -18,11 +20,9 @@ class LoaderTest extends BaseTestCase
     public function testLoadFromDirectory(): void
     {
         $loader = new Loader();
-        $loader->addFixture($this->getMockBuilder(FixtureInterface::class)->setMockClassName('Mock1')->getMock());
-        $loader->addFixture($this->getMockBuilder(FixtureInterface::class)->setMockClassName('Mock2')->getMock());
-        $loader->addFixture(
-            $this->getMockBuilder(SharedFixtureInterface::class)->setMockClassName('Mock3')->getMock(),
-        );
+        $loader->addFixture(new DummyFixtureOne());
+        $loader->addFixture(new DummyFixtureTwo());
+        $loader->addFixture(new SharedDummyFixture());
 
         $this->assertCount(3, $loader->getFixtures());
 
@@ -35,11 +35,9 @@ public function testLoadFromDirectory(): void
     public function testLoadFromFile(): void
     {
         $loader = new Loader();
-        $loader->addFixture($this->getMockBuilder(FixtureInterface::class)->setMockClassName('Mock1')->getMock());
-        $loader->addFixture($this->getMockBuilder(FixtureInterface::class)->setMockClassName('Mock2')->getMock());
-        $loader->addFixture(
-            $this->getMockBuilder(SharedFixtureInterface::class)->setMockClassName('Mock3')->getMock(),
-        );
+        $loader->addFixture(new DummyFixtureOne());
+        $loader->addFixture(new DummyFixtureTwo());
+        $loader->addFixture(new SharedDummyFixture());
 
         $this->assertCount(3, $loader->getFixtures());
 
@@ -63,3 +61,28 @@ public function testGetFixture(): void
         $this->assertInstanceOf(MyFixture1::class, $fixture);
     }
 }
+
+final class DummyFixtureOne implements FixtureInterface
+{
+    public function load(ObjectManager $manager): void
+    {
+    }
+}
+
+final class DummyFixtureTwo implements FixtureInterface
+{
+    public function load(ObjectManager $manager): void
+    {
+    }
+}
+
+final class SharedDummyFixture implements SharedFixtureInterface
+{
+    public function load(ObjectManager $manager): void
+    {
+    }
+
+    public function setReferenceRepository(ReferenceRepository $referenceRepository): void
+    {
+    }
+}