Merge pull request #10 from zumba/avoid-insert-empty

Avoiding issues when the fixture is empty
zumba · Feb 17, 2017 · 7862bf1 · 7862bf1
2 parents 4e99268 + 0c37b11
commit 7862bf1
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/examples/PizzaTraitTest.php b/examples/PizzaTraitTest.php
@@ -32,7 +32,10 @@ class PizzaTraitTest extends \PHPUnit_Framework_TestCase {
 		'orders' => [
 			['size' => 'large', 'toppings' => ['cheese', 'ham']],
 			['size' => 'medium', 'toppings' => ['cheese']]
-		]
+		],
+		'carts' => [
+			// Intentionally empty
+		],
 	];
 
 	/**

diff --git a/src/DataSet/DataSet.php b/src/DataSet/DataSet.php
@@ -66,7 +66,9 @@ public function dropAllCollections() {
 	 */
 	public function buildCollections() {
 		foreach ($this->fixture as $collection => $data) {
-			$this->connection->collection($collection)->insertMany($data);
+			if (!empty($data)) {
+				$this->connection->collection($collection)->insertMany($data);
+			}
 		}
 		return $this;
 	}