>>> [!note] Migrated issue <!-- Drupal.org comment --> <!-- Migrated from issue #3629965. --> Reported by: [matthand](https://www.drupal.org/user/171527) Related to !59 >>> <h3 id="summary-problem-motivation">Problem/Motivation</h3> <p>On Drupal 12.x, <code>SqlImportTest</code> fails in the <code>phpunit (next major)</code> job:</p> <pre> Drupal\Tests\mongodb_storage\Kernel\SqlImportTest::testImportActual with data set #0 Failed asserting that 4 matches expected 0. modules/mongodb_storage/tests/src/Kernel/SqlImportTest.php:171 </pre><p>Example job: <a href="https://git.drupalcode.org/issue/mongodb-3627745/-/jobs/12795721">https://git.drupalcode.org/issue/mongodb-3627745/-/jobs/12795721</a><br> The 8.x-2.x next major job passed on 2026-10-05: <a href="https://git.drupalcode.org/project/mongodb/-/jobs/12660720">https://git.drupalcode.org/project/mongodb/-/jobs/12660720</a></p> <p>The failure comes from a core change, not from a mongodb change. Core commit 5c8d732 (<a href="https://www.drupal.org/project/drupal/issues/3613347">#3613347: Re-use container in kernel tests</a>, committed 2026-10-10) removed the override in <code>KernelTestBase::register()</code> that replaced the <code>keyvalue</code> service with a <code>KeyValueMemoryFactory</code>. Kernel tests now use the real <code>keyvalue</code> service, which stores data in the SQL <code>key_value</code> table.</p> <p>When the test container is compiled, <code>HookCollectorKeyValueWritePass::process()</code> writes the <code>hook_data</code> collection through that service. This leaves 4 rows in <code>key_value</code> before the test runs:</p> <ul> <li><code>hook_data</code> / <code>hook_list</code></li> <li><code>hook_data</code> / <code>packed_order_operations</code></li> <li><code>hook_data</code> / <code>preprocess_for_suggestions</code></li> <li><code>hook_data</code> / <code>theme_hook_list</code></li> </ul> <p><code>KeyValueTestBase::setUp()</code> only deletes its own <code>mongodb.nonexistent</code> collection, so <code>testImportActual()</code> finds those rows and fails its <code>assertEquals(0, $actualPreDbCount)</code> check. Data set #1 (<code>key_value_expire</code>) still passes because nothing writes to that table.</p> <p>Only removing that assertion would not be enough. <code>SqlImport::import()</code> copies every row in the table, so the <code>hook_data</code> collection would also reach MongoDB and break the later "Collection names match" assertion.</p> <h4 id="summary-steps-reproduce">Steps to reproduce</h4> <ol> <li>Set up a site on Drupal core 12.x (<code>main</code>) at or after commit 5c8d732, with PHP 8.5 and ext-mongodb 2.x.</li> <li>Run the test against MySQL and a MongoDB server:<br> <code>SIMPLETEST_DB=mysql://... MONGODB_URI=mongodb://... vendor/bin/phpunit -c web/core/phpunit.xml.dist web/modules/contrib/mongodb/modules/mongodb_storage/tests/src/Kernel/SqlImportTest.php</code></li> <li><code>testImportActual with data set #0</code> fails with <code>Failed asserting that 4 matches expected 0.</code></li> </ol> <p>On core before 5c8d732, such as 12.0.0-beta1, the test passes.</p> <h3 id="summary-proposed-resolution">Proposed resolution</h3> <p>In <code>SqlImportTest::testImportActual()</code>, stop assuming that the SQL table starts empty. Read the rows that are already there and add them to the expected import result:</p> <pre> // Core may already have written rows, e.g. the hook_data collection // written by HookCollectorKeyValueWritePass on container compile: they // are imported too, so expect them. $preExisting = $this-&gt;database-&gt;select($table, 't') -&gt;fields('t', ['collection', 'name', 'value']) -&gt;execute() -&gt;fetchAll(); ... $expectedCollections = []; foreach ($preExisting as $row) { // @codingStandardsIgnoreStart $expectedCollections[$row-&gt;collection][$row-&gt;name] = unserialize($row-&gt;value); // @codingStandardsIgnoreEnd } foreach ($rows as $row) { ... </pre><p>This replaces the <code>countTable()</code> and <code>assertEquals(0, ...)</code> lines. The check that MongoDB starts with no collections stays as it is.</p> <p>This works on core both before and after 5c8d732. I tested it locally on 12.0.0-beta1, both with the core change simulated and without it.</p> <p>Emptying <code>key_value</code> in <code>setUp()</code> is not recommended, because core reads <code>hook_data</code> at runtime.</p> <h3 id="summary-remaining-tasks">Remaining tasks</h3> <ul> <li>Create an MR with the test change.</li> <li>Confirm that the <code>phpunit (next major)</code> job passes.</li> <li>Review.</li> </ul> <h3 id="summary-ui-changes">User interface changes</h3> <p>None.</p> <h3 id="summary-api-changes">API changes</h3> <p>None.</p> <h3 id="summary-data-model-changes">Data model changes</h3> <p>None.</p> <p>AI Disclosure: I used Claude Code to triage this issue and help write this bug report.</p> > Related issue: [Issue #3613347](https://www.drupal.org/node/3613347)