properly skip tests

jsteemann · jsteemann · commit 9b2326c2fd19 · 2017-09-21T12:55:26.000+02:00
diff --git a/tests/CollectionBasicTest.php b/tests/CollectionBasicTest.php
@@ -137,6 +137,7 @@ public function testCreateCollectionWithKeyOptionsAndVerifyProperties()
     {
         if (isCluster($this->connection)) {
             // don't execute this test in a cluster
+            $this->markTestSkipped("test is only meaningful in single server");
             return;
         }
 
@@ -193,6 +194,7 @@ public function testCreateCollectionWithKeyOptionsCluster()
     {
         if (!isCluster($this->connection)) {
             // don't execute this test in a non-cluster
+            $this->markTestSkipped("test is only meaningful in cluster");
             return;
         }
 
@@ -229,6 +231,7 @@ public function testCreateCollectionWithNumberOfShardsCluster()
     {
         if (!isCluster($this->connection)) {
             // don't execute this test in a non-cluster
+            $this->markTestSkipped("test is only meaningful in cluster");
             return;
         }
 
@@ -264,6 +267,7 @@ public function testCreateCollectionWithShardKeysCluster()
     {
         if (!isCluster($this->connection)) {
             // don't execute this test in a non-cluster
+            $this->markTestSkipped("test is only meaningful in cluster");
             return;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -137,6 +137,7 @@ public function testCreateCollectionWithKeyOptionsAndVerifyProperties()`
`137`	`137`	`{`
`138`	`138`	`if (isCluster($this->connection)) {`
`139`	`139`	`// don't execute this test in a cluster`
	`140`	`+ $this->markTestSkipped("test is only meaningful in single server");`
`140`	`141`	`return;`
`141`	`142`	`}`
`142`	`143`
`@@ -193,6 +194,7 @@ public function testCreateCollectionWithKeyOptionsCluster()`
`193`	`194`	`{`
`194`	`195`	`if (!isCluster($this->connection)) {`
`195`	`196`	`// don't execute this test in a non-cluster`
	`197`	`+ $this->markTestSkipped("test is only meaningful in cluster");`
`196`	`198`	`return;`
`197`	`199`	`}`
`198`	`200`
`@@ -229,6 +231,7 @@ public function testCreateCollectionWithNumberOfShardsCluster()`
`229`	`231`	`{`
`230`	`232`	`if (!isCluster($this->connection)) {`
`231`	`233`	`// don't execute this test in a non-cluster`
	`234`	`+ $this->markTestSkipped("test is only meaningful in cluster");`
`232`	`235`	`return;`
`233`	`236`	`}`
`234`	`237`
`@@ -264,6 +267,7 @@ public function testCreateCollectionWithShardKeysCluster()`
`264`	`267`	`{`
`265`	`268`	`if (!isCluster($this->connection)) {`
`266`	`269`	`// don't execute this test in a non-cluster`
	`270`	`+ $this->markTestSkipped("test is only meaningful in cluster");`
`267`	`271`	`return;`
`268`	`272`	`}`
`269`	`273`