Commit 6a46e15c authored by Jeremy Mikola's avatar Jeremy Mikola

Assertion args are expected and actual, respectively

parent ad506578
...@@ -961,7 +961,7 @@ class DocumentationExamplesTest extends FunctionalTestCase ...@@ -961,7 +961,7 @@ class DocumentationExamplesTest extends FunctionalTestCase
'ns' => (object) ['db' => 'phplib_test', 'coll' => 'inventory'], 'ns' => (object) ['db' => 'phplib_test', 'coll' => 'inventory'],
'documentKey' => (object) ['_id' => $insertedId] 'documentKey' => (object) ['_id' => $insertedId]
]; ];
$this->assertEquals($current, $expectedChange); $this->assertEquals($expectedChange, $current);
// Start Changestream Example 3 // Start Changestream Example 3
$resumeToken = ($current !== null) ? $current->_id : null; $resumeToken = ($current !== null) ? $current->_id : null;
...@@ -981,7 +981,7 @@ class DocumentationExamplesTest extends FunctionalTestCase ...@@ -981,7 +981,7 @@ class DocumentationExamplesTest extends FunctionalTestCase
'ns' => (object) ['db' => 'phplib_test', 'coll' => 'inventory'], 'ns' => (object) ['db' => 'phplib_test', 'coll' => 'inventory'],
'documentKey' => (object) ['_id' => $insertedId] 'documentKey' => (object) ['_id' => $insertedId]
]; ];
$this->assertEquals($cursor->current(), $expectedChange); $this->assertEquals($expectedChange, $cursor->current());
// Start Changestream Example 4 // Start Changestream Example 4
$pipeline = [['$match' => ['$or' => [['fullDocument.username' => 'alice'], ['operationType' => 'delete']]]]]; $pipeline = [['$match' => ['$or' => [['fullDocument.username' => 'alice'], ['operationType' => 'delete']]]]];
......
...@@ -43,7 +43,7 @@ class WatchFunctionalTest extends FunctionalTestCase ...@@ -43,7 +43,7 @@ class WatchFunctionalTest extends FunctionalTestCase
'ns' => (object) ['db' => 'phplib_test', 'coll' => 'WatchFunctionalTest.e68b9f01'], 'ns' => (object) ['db' => 'phplib_test', 'coll' => 'WatchFunctionalTest.e68b9f01'],
'documentKey' => (object) ['_id' => 2] 'documentKey' => (object) ['_id' => 2]
]); ]);
$this->assertEquals($changeStream->current(), $expectedResult); $this->assertEquals($expectedResult, $changeStream->current());
$operation = new DatabaseCommand($this->getDatabaseName(), ["killCursors" => $this->getCollectionName(), "cursors" => [$changeStream->getCursorId()]]); $operation = new DatabaseCommand($this->getDatabaseName(), ["killCursors" => $this->getCollectionName(), "cursors" => [$changeStream->getCursorId()]]);
$operation->execute($this->getPrimaryServer()); $operation->execute($this->getPrimaryServer());
...@@ -58,7 +58,7 @@ class WatchFunctionalTest extends FunctionalTestCase ...@@ -58,7 +58,7 @@ class WatchFunctionalTest extends FunctionalTestCase
'ns' => (object) ['db' => 'phplib_test', 'coll' => 'WatchFunctionalTest.e68b9f01'], 'ns' => (object) ['db' => 'phplib_test', 'coll' => 'WatchFunctionalTest.e68b9f01'],
'documentKey' => (object) ['_id' => 3] 'documentKey' => (object) ['_id' => 3]
]); ]);
$this->assertEquals($changeStream->current(), $expectedResult); $this->assertEquals($expectedResult, $changeStream->current());
} }
public function testNoChangeAfterResumeBeforeInsert() public function testNoChangeAfterResumeBeforeInsert()
...@@ -81,7 +81,7 @@ class WatchFunctionalTest extends FunctionalTestCase ...@@ -81,7 +81,7 @@ class WatchFunctionalTest extends FunctionalTestCase
'ns' => (object) ['db' => 'phplib_test', 'coll' => 'WatchFunctionalTest.4a554985'], 'ns' => (object) ['db' => 'phplib_test', 'coll' => 'WatchFunctionalTest.4a554985'],
'documentKey' => (object) ['_id' => 2] 'documentKey' => (object) ['_id' => 2]
]); ]);
$this->assertEquals($changeStream->current(), $expectedResult); $this->assertEquals($expectedResult, $changeStream->current());
$operation = new DatabaseCommand($this->getDatabaseName(), ["killCursors" => $this->getCollectionName(), "cursors" => [$changeStream->getCursorId()]]); $operation = new DatabaseCommand($this->getDatabaseName(), ["killCursors" => $this->getCollectionName(), "cursors" => [$changeStream->getCursorId()]]);
$operation->execute($this->getPrimaryServer()); $operation->execute($this->getPrimaryServer());
...@@ -99,7 +99,7 @@ class WatchFunctionalTest extends FunctionalTestCase ...@@ -99,7 +99,7 @@ class WatchFunctionalTest extends FunctionalTestCase
'ns' => (object) ['db' => 'phplib_test', 'coll' => 'WatchFunctionalTest.4a554985'], 'ns' => (object) ['db' => 'phplib_test', 'coll' => 'WatchFunctionalTest.4a554985'],
'documentKey' => (object) ['_id' => 3] 'documentKey' => (object) ['_id' => 3]
]); ]);
$this->assertEquals($changeStream->current(), $expectedResult); $this->assertEquals($expectedResult, $changeStream->current());
} }
public function testResumeAfterKillThenNoOperations() public function testResumeAfterKillThenNoOperations()
...@@ -171,7 +171,7 @@ class WatchFunctionalTest extends FunctionalTestCase ...@@ -171,7 +171,7 @@ class WatchFunctionalTest extends FunctionalTestCase
'_id' => $changeStream->current()->_id, '_id' => $changeStream->current()->_id,
'foo' => [0] 'foo' => [0]
]); ]);
$this->assertEquals($changeStream->current(), $expectedResult); $this->assertEquals($expectedResult, $changeStream->current());
} }
public function testCursorWithEmptyBatchNotClosed() public function testCursorWithEmptyBatchNotClosed()
......
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