Skip to content

Commit aceae37

Browse files
committed
Move Manager::startSession() error test and delete obsolete test
The deleted error test was redundant. It was originally created in ca4c52e to work around changes in PHP 8, but 60febd0 subsequently reintroduced a portable version of the test case. Also adds missing SKIPIF to startSession test.
1 parent e1c4fe5 commit aceae37

File tree

2 files changed

+7
-26
lines changed

2 files changed

+7
-26
lines changed

tests/session/session_error-001.phpt renamed to tests/manager/manager-startSession_error-001.phpt

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,10 @@
11
--TEST--
2-
MongoDB\Driver\Session with wrong defaultTransactionOptions
2+
MongoDB\Driver\Manager::startSession() with wrong defaultTransactionOptions
3+
--SKIPIF--
4+
<?php require __DIR__ . "/../utils/basic-skipif.inc"; ?>
5+
<?php skip_if_not_libmongoc_crypto(); ?>
6+
<?php skip_if_not_live(); ?>
7+
<?php skip_if_server_version('<', '3.6'); ?>
38
--FILE--
49
<?php
510
require_once __DIR__ . "/../utils/basic.inc";
@@ -40,7 +45,7 @@ foreach ($options as $txnOptions) {
4045
$manager->startSession([
4146
'defaultTransactionOptions' => $txnOptions
4247
]);
43-
}, 'MongoDB\Driver\Exception\InvalidArgumentException'), "\n";
48+
}, MongoDB\Driver\Exception\InvalidArgumentException::class), "\n";
4449
}
4550

4651
echo raises(function() use ($manager) {

tests/session/session_error-002.phpt

Lines changed: 0 additions & 24 deletions
This file was deleted.

0 commit comments

Comments
 (0)