Fixed the failed tests

adlarge · Sep 8, 2019 · e332d7f · e332d7f
1 parent 687c511
commit e332d7f
Showing 1 changed file with 26 additions and 6 deletions.
diff --git a/tests/Model/DocumentationTest.php b/tests/Model/DocumentationTest.php
@@ -84,7 +84,7 @@ public function testAddFixtureWithMultidimensionalArray(): void
 
         $documentation = new Documentation([]);
 
-        $documentation->addFixture('fixtures', ['id' => 1, 'array' => ['name' => 'fixture1', 'color' => 'red']]);
+        $documentation->addFixture('fixtures', 1, ['id' => 1, 'array' => ['name' => 'fixture1', 'color' => 'red']]);
     }
 
     public function testAddFixtureEntity(): void
@@ -93,7 +93,7 @@ public function testAddFixtureEntity(): void
             ->makePartial();
         $mockFixture->shouldReceive('setLinks')
             ->once();
-            
+
         $mockDocumentation = Mockery::mock(
             Documentation::class,
             [
@@ -107,7 +107,8 @@ public function testAddFixtureEntity(): void
             ->with('Product', 'Product-1', [
                 'name' => 'product 1',
                 'category' => 'category 1'
-            ]);
+            ])
+            ->andReturn($mockFixture);
 
         $product = (new Product())
             ->setId(1)
@@ -120,6 +121,11 @@ public function testAddFixtureEntity(): void
 
     public function testAddFixtureEntityWithPublicProperties(): void
     {
+        $mockFixture = Mockery::mock(Fixture::class)
+            ->makePartial();
+        $mockFixture->shouldReceive('setLinks')
+            ->once();
+
         $mockDocumentation = Mockery::mock(
             Documentation::class,
             [
@@ -133,14 +139,16 @@ public function testAddFixtureEntityWithPublicProperties(): void
             ->with('ProductPublic', 'ProductPublic-1', [
                 'name' => 'product 1',
                 'category' => 'category 1'
-            ]);
+            ])
+            ->andReturn($mockFixture);
 
         $product = new ProductPublic();
         $product->id = 1;
         $product->name = 'product 1';
         $product->category = 'category 1';
 
         $mockDocumentation->addFixtureEntity($product);
+        // We expect 0 because if it has the good parameter it will be catch by the mock
         $this->assertCount(0, $mockDocumentation->getSections());
     }
 
@@ -151,6 +159,11 @@ public function testAddFixtureEntityWithComplexProperties(): void
         $mockFixture->shouldReceive('setLinks')
             ->once();
 
+        $mockFixture = Mockery::mock(Fixture::class)
+            ->makePartial();
+        $mockFixture->shouldReceive('setLinks')
+            ->once();
+
         $mockDocumentation = Mockery::mock(
             Documentation::class,
             [
@@ -168,7 +181,8 @@ public function testAddFixtureEntityWithComplexProperties(): void
                 'name' => 'product 1',
                 'category' => 'category name',
                 'tags' => 3
-            ]);
+            ])
+            ->andReturn($mockFixture);
 
         $category = new Category();
         $category->name = 'category name';
@@ -189,6 +203,11 @@ public function testAddFixtureEntityWithComplexProperties(): void
      */
     public function testAddFixtureEntityWithNonExistingProperty(): void
     {
+        $mockFixture = Mockery::mock(Fixture::class)
+            ->makePartial();
+        $mockFixture->shouldReceive('setLinks')
+            ->once();
+
         $mockDocumentation = Mockery::mock(
             Documentation::class,
             [
@@ -201,7 +220,8 @@ public function testAddFixtureEntityWithNonExistingProperty(): void
             ->once()
             ->with('Product', 'Product-1', [
                 'category' => 'category 1'
-            ]);
+            ])
+            ->andReturn($mockFixture);
 
         $product = (new Product())
             ->setId(1)