PHPLIB-530: Remove checks against MapReduce statistics on 4.3.0+

Author: alcaeus

src/MapReduceResult.php

@@ -55,8 +55,8 @@ class MapReduceResult implements IteratorAggregate
     public function __construct(callable $getIterator, stdClass $result)
     {
         $this->getIterator = $getIterator;
-        $this->executionTimeMS = (integer) $result->timeMillis;
-        $this->counts = (array) $result->counts;
+        $this->executionTimeMS = isset($result->timeMillis) ? (integer) $result->timeMillis : 0;
+        $this->counts = isset($result->counts) ? (array) $result->counts : [];
         $this->timing = isset($result->timing) ? (array) $result->timing : [];
     }
 

tests/Collection/CollectionFunctionalTest.php

@@ -396,8 +396,10 @@ public function testMapReduce()
 
         $this->assertSameDocuments($expected, $result);
 
-        $this->assertGreaterThanOrEqual(0, $result->getExecutionTimeMS());
-        $this->assertNotEmpty($result->getCounts());
+        if (version_compare($this->getServerVersion(), '4.3.0', '<')) {
+            $this->assertGreaterThanOrEqual(0, $result->getExecutionTimeMS());
+            $this->assertNotEmpty($result->getCounts());
+        }
     }
 
     public function collectionMethodClosures()

tests/Operation/MapReduceFunctionalTest.php

@@ -92,12 +92,19 @@ public function testResult()
         $result = $operation->execute($this->getPrimaryServer());
 
         $this->assertInstanceOf(MapReduceResult::class, $result);
-        $this->assertGreaterThanOrEqual(0, $result->getExecutionTimeMS());
-        $this->assertNotEmpty($result->getCounts());
+
+        if (version_compare($this->getServerVersion(), '4.3.0', '<')) {
+            $this->assertGreaterThanOrEqual(0, $result->getExecutionTimeMS());
+            $this->assertNotEmpty($result->getCounts());
+        }
     }
 
     public function testResultIncludesTimingWithVerboseOption()
     {
+        if (version_compare($this->getServerVersion(), '4.3.0', '>=')) {
+            $this->markTestSkipped('mapReduce statistics are no longer exposed');
+        }
+
         $this->createFixtures(3);
 
         $map = new Javascript('function() { emit(this.x, this.y); }');
@@ -115,6 +122,10 @@ public function testResultIncludesTimingWithVerboseOption()
 
     public function testResultDoesNotIncludeTimingWithoutVerboseOption()
     {
+        if (version_compare($this->getServerVersion(), '4.3.0', '>=')) {
+            $this->markTestSkipped('mapReduce statistics are no longer exposed');
+        }
+
         $this->createFixtures(3);
 
         $map = new Javascript('function() { emit(this.x, this.y); }');