Decouple kernel tests from XbTestSetup

Created on 22 June 2025, 22 days ago

Overview

For velocity sake, quite a few of our kernel tests make use of XbTestSetup to setup a test environment.
This however makes the tests slow. Removing this dependency on XbTestSetup and having them only setup the things they need can speed them up significantly.

In 🐛 Page status changes from "Published" to "Changed" even when no actual changes are made Active I was working on fixing fails in \Drupal\Tests\experience_builder\Kernel\AutoSaveManagerTest and noticed the test was very slow. As I was already dramatically changing that test, I decoupled it from XbTestSetup so I could get faster feedback.

Before time to execute the 5 test methods is 3 mins 20 seconds
After - 28 sec

Here's the list of tests that are making use of it

tests/src/Kernel/AutoSaveManagerTest.php // 👈️ This one is updated in 
            
              
              
              🐛
              Page status changes from "Published" to "Changed" even when no actual changes are made
                Active
              
            
tests/src/Kernel/ComponentInputsFormTest.php
tests/src/Kernel/ClientServerConversionTraitTest.php
tests/src/Kernel/ApiLayoutControllerGetTest.php
tests/src/Kernel/ApiLayoutControllerPostTest.php
tests/src/Kernel/ComponentTreeLoaderTest.php
tests/src/Kernel/ApiLayoutControllerPatchTest.php
tests/src/Kernel/ClientDataToEntityConverterTest.php
tests/src/Kernel/ApiAutoSaveControllerTest.php

Proposed resolution

User interface changes

📌 Task

Status

Active

Version

0.0

Component

Miscellaneous

Created by

🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @larowlan
Comment 19 days ago →
🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
💯 — I've always been opposed to this.

Found this via https://git.drupalcode.org/project/experience_builder/-/merge_requests/1... btw, where you wrote:

ℹ️ Moved this away from XbTestSetup because this test was burning time working on fixing it.
See #3531679 for follow-up to do the rest of our Kernel tests in this boat. significant DX and velocity boost - before time to execute the 5 test methods is 3 mins 20 seconds, after is 28 sec

For context: this was the pragmatic choice at the time. 😞 The Page content entity type has not yet existed for that long, so for a long time you HAD to install the article node type with the right fields etc in just the right way to get something working to develop against.
Comment 19 days ago →
🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024