fix: test empty entities array

.travis.yml

@@ -5,6 +5,7 @@ php:
   - 5.6
   - 7.1
   - 7.2
+  - 7.3
 
 sudo: false
 
@@ -23,4 +24,6 @@ before_script:
 script:
   - composer code-style
   - vendor/bin/phpunit -c phpunit.xml --coverage-clover=build/logs/clover.xml --coverage-text --color=always
-  - sh -c 'if [ "$TRAVIS_PHP_VERSION" = "7.0" ]; then php vendor/bin/coveralls -v; fi;'
+
+after_success:
+  - sh -c 'if [ "$TRAVIS_PHP_VERSION" = "7.1" ]; then php vendor/bin/coveralls -v; fi;'

src/Testing/EntityFetcherMock/Result.php

@@ -93,7 +93,9 @@ public function matches($expression)
      */
     public function addEntities(Entity ...$entities)
     {
-        array_push($this->entities, ...$entities);
+        if (!empty($entities)) {
+            array_push($this->entities, ...$entities);
+        }
         return $this;
     }
 

tests/Testing/EntityFetcherMockTest.php

@@ -76,6 +76,19 @@ public function returnsAnEmptyArrayIfNoResultsMatch()
         self::assertEmpty($articles);
     }
 
+    /** @test */
+    public function returnsAnEmptyResultIfMatched()
+    {
+        $this->em->addResult(Article::class, new Article(['title' => 'Foo']));
+        $this->em->addResult(Article::class)->where('title', 'Bar');
+
+        $query = new EntityFetcherMock($this->em, Article::class);
+        $query->where('title', 'Bar');
+        $articles = $query->all();
+
+        self::assertEmpty($articles);
+    }
+
     /** @test */
     public function returnsEntitiesWithoutConditions()
     {