StatementWrapperLegacyTest::testClientStatementMethod() should be less specific.

I see the same sqlite failure on php 8.1 at #3217699: Convert select query extenders to backend-overrideable services, good to know it’s unrelated

+1 for RTBC. I have run the test with the MR on my local machine with SQL Server and it passes all but the test StatementWrapperLegacyTest::testMissingMethod(). To pass that test we need to change the code for the method Drupal\Core\Database\StatementWrapper::__call() to:

  public function __call($method, $arguments) {
    try {
      if (is_callable([$this->getClientStatement(), $method])) {
        @trigger_error("StatementWrapper::{$method} should not be called in drupal:9.1.0 and will error in drupal:10.0.0. Access the client-level statement object via ::getClientStatement(). See https://www.drupal.org/node/3177488", E_USER_DEPRECATED);
        return call_user_func_array([$this->getClientStatement(), $method], $arguments);
      }
    }
    catch (\PDOException $e) {
      // Do nothing.
    }
    throw new \BadMethodCallException($method);
  }

This is needed because the code is_callable([$this->getClientStatement(), $method]) will trigger a PDOException on the SQL Server.

@Beakerboy: Thank you for the info!

Committed and pushed 8512e0e93c to 9.4.x and 9d6d526458 to 9.3.x. Thanks!

Backported to 9.3.x as a test only fix.