Commit ffc47d4f authored by Katherine Walker's avatar Katherine Walker

PHPLIB-315: Remove default for mapReduce verbose option

parent 7231d874
...@@ -121,7 +121,7 @@ class MapReduce implements Executable ...@@ -121,7 +121,7 @@ class MapReduce implements Executable
* applied to the returned Cursor (it is not sent to the server). * applied to the returned Cursor (it is not sent to the server).
* *
* * verbose (boolean): Specifies whether to include the timing information * * verbose (boolean): Specifies whether to include the timing information
* in the result information. The default is true. * in the result information.
* *
* * writeConcern (MongoDB\Driver\WriteConcern): Write concern. This only * * writeConcern (MongoDB\Driver\WriteConcern): Write concern. This only
* applies when results are output to a collection. * applies when results are output to a collection.
...@@ -143,10 +143,6 @@ class MapReduce implements Executable ...@@ -143,10 +143,6 @@ class MapReduce implements Executable
throw InvalidArgumentException::invalidType('$out', $out, 'string or array or object'); throw InvalidArgumentException::invalidType('$out', $out, 'string or array or object');
} }
$options += [
'verbose' => true,
];
if (isset($options['bypassDocumentValidation']) && ! is_bool($options['bypassDocumentValidation'])) { if (isset($options['bypassDocumentValidation']) && ! is_bool($options['bypassDocumentValidation'])) {
throw InvalidArgumentException::invalidType('"bypassDocumentValidation" option', $options['bypassDocumentValidation'], 'boolean'); throw InvalidArgumentException::invalidType('"bypassDocumentValidation" option', $options['bypassDocumentValidation'], 'boolean');
} }
......
...@@ -201,7 +201,6 @@ class CollectionFunctionalTest extends FunctionalTestCase ...@@ -201,7 +201,6 @@ class CollectionFunctionalTest extends FunctionalTestCase
$this->assertGreaterThanOrEqual(0, $result->getExecutionTimeMS()); $this->assertGreaterThanOrEqual(0, $result->getExecutionTimeMS());
$this->assertNotEmpty($result->getCounts()); $this->assertNotEmpty($result->getCounts());
$this->assertNotEmpty($result->getTiming());
} }
/** /**
......
...@@ -93,7 +93,6 @@ class MapReduceFunctionalTest extends FunctionalTestCase ...@@ -93,7 +93,6 @@ class MapReduceFunctionalTest extends FunctionalTestCase
$this->assertInstanceOf('MongoDB\MapReduceResult', $result); $this->assertInstanceOf('MongoDB\MapReduceResult', $result);
$this->assertGreaterThanOrEqual(0, $result->getExecutionTimeMS()); $this->assertGreaterThanOrEqual(0, $result->getExecutionTimeMS());
$this->assertNotEmpty($result->getCounts()); $this->assertNotEmpty($result->getCounts());
$this->assertNotEmpty($result->getTiming());
} }
public function testResultDoesNotIncludeTimingWithoutVerboseOption() public function testResultDoesNotIncludeTimingWithoutVerboseOption()
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment