Skip to content

Commit bf9978f

Browse files
committed
Resolve "client" operation arg as a common option
This is presently only used for test runner ops, but the spec does define it as a common option that should always be resolved (like "session").
1 parent 3f5de67 commit bf9978f

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

tests/UnifiedSpecTests/Operation.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -651,11 +651,6 @@ private function executeForTestRunner()
651651
{
652652
$args = $this->prepareArguments();
653653

654-
if (array_key_exists('client', $args)) {
655-
assertIsString($args['client']);
656-
$args['client'] = $this->entityMap->getClient($args['client']);
657-
}
658-
659654
switch ($this->name) {
660655
case 'assertCollectionExists':
661656
assertArrayHasKey('databaseName', $args);
@@ -786,6 +781,11 @@ private function prepareArguments() : array
786781
{
787782
$args = $this->arguments;
788783

784+
if (array_key_exists('client', $args)) {
785+
assertIsString($args['client']);
786+
$args['client'] = $this->entityMap->getClient($args['client']);
787+
}
788+
789789
if (array_key_exists('session', $args)) {
790790
assertIsString($args['session']);
791791
$args['session'] = $this->entityMap->getSession($args['session']);

0 commit comments

Comments
 (0)