Add support for getting Database Stub using Database::getConnection()

Open on Drupal.org →

Created on 22 June 2024, 6 days ago

Problem/Motivation

Now the Database service stub can be initiated by calling the database service.

But some code uses the static calls Database::getConnection() to get the database service.
Because this is a static call, we can't simply replace it with a stub class.

So will be good to invent something to make this way of getting a db connection from a static class works well too in Test Helpers.

✨ Feature request

Status

Fixed

Version

1.4

Component

Code

Created by

🇦🇲Armenia Murz Yerevan, Armenia

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!97Add support for getting Database Stub using Database::getConnection()
Merged
🇦🇲Armenia Murz
updated 6 days ago

Comments & Activities

Issue created by @Murz
Status changed to Needs review 6 days ago10:30am 22 June 2024
Comment 6 days ago →
🇦🇲Armenia Murz Yerevan, Armenia
I committed here https://git.drupalcode.org/issue/test_helpers-3456398/-/tree/3456398-dat... the base implementation that works well for a single connection. Needs review and testing. Will try to extend it to support multiple connections.
Merge request !97Issue #3456398: Rework ConnectionStub class to match the Drupal Core database driver → (Merged) created by Murz
Pipeline finished with Skipped
6 days ago
#205505

Comment 6 days ago →

Murz → committed 26f61511 on 1.4.x

Issue #3456398: Add support for getting Database Stub using Database::...

Comment 6 days ago →
🇦🇲Armenia Murz Yerevan, Armenia
Implemented.
Status changed to Fixed 6 days ago11:48am 22 June 2024
Comment 6 days ago →
🇦🇲Armenia Murz Yerevan, Armenia

contrib.social Blog FAQ Discussions

Production build 0.69.0 2024