Remove legacy change stream spec test runner

Author: jmikola

tests/SpecTests/ChangeStreamsSpecTest.php

@@ -70,20 +70,6 @@ private function __construct(array $events)
         }
     }
 
-    public static function fromChangeStreams(array $expectedEvents)
-    {
-        $o = new self($expectedEvents);
-
-        $o->ignoreCommandFailed = true;
-        $o->ignoreCommandSucceeded = true;
-        /* Change Streams spec tests do not include getMore commands in the
-         * list of expected events, so ignore any observed events beyond the
-         * number that are expected. */
-        $o->ignoreExtraEvents = true;
-
-        return $o;
-    }
-
     public static function fromClientSideEncryption(array $expectedEvents)
     {
         $o = new self($expectedEvents);

tests/SpecTests/CommandExpectations.php

@@ -57,27 +57,6 @@ private function __construct()
     {
     }
 
-    public static function fromChangeStreams(stdClass $result)
-    {
-        $o = new self();
-
-        if (isset($result->error->code)) {
-            $o->code = $result->error->code;
-            $o->isExpected = true;
-        }
-
-        if (isset($result->error->errorLabels)) {
-            if (! self::isArrayOfStrings($result->error->errorLabels)) {
-                throw InvalidArgumentException::invalidType('errorLabels', $result->error->errorLabels, 'string[]');
-            }
-
-            $o->includedLabels = $result->error->errorLabels;
-            $o->isExpected = true;
-        }
-
-        return $o;
-    }
-
     public static function fromClientSideEncryption(stdClass $operation)
     {
         return self::fromGenericOperation($operation);

tests/SpecTests/ErrorExpectation.php

@@ -83,45 +83,6 @@ private function __construct(stdClass $operation)
         }
     }
 
-    public static function fromChangeStreams(stdClass $operation)
-    {
-        $o = new self($operation);
-
-        /* Note: change streams only return majority-committed writes, so ensure
-         * each operation applies that write concern. This will avoid spurious
-         * test failures. */
-        $writeConcern = new WriteConcern(WriteConcern::MAJORITY);
-
-        // Expect all operations to succeed
-        $o->errorExpectation = ErrorExpectation::noError();
-
-        /* The Change Streams spec tests include a unique "rename" operation,
-         * which we should convert to a renameCollection command to be run
-         * against the admin database. */
-        if ($operation->name === 'rename') {
-            $o->object = self::OBJECT_SELECT_DATABASE;
-            $o->databaseName = 'admin';
-            $o->name = 'runCommand';
-            $o->arguments = [
-                'command' => [
-                    'renameCollection' => $operation->database . '.' . $operation->collection,
-                    'to' => $operation->database . '.' . $operation->arguments->to,
-                // Note: Database::command() does not inherit WC, so be explicit
-                    'writeConcern' => $writeConcern,
-                ],
-            ];
-
-            return $o;
-        }
-
-        $o->databaseName = $operation->database;
-        $o->collectionName = $operation->collection;
-        $o->collectionOptions = ['writeConcern' => $writeConcern];
-        $o->object = self::OBJECT_SELECT_COLLECTION;
-
-        return $o;
-    }
-
     public static function fromClientSideEncryption(stdClass $operation)
     {
         $o = new self($operation);