Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Problem/Motivation
Because of #3083275: [meta] Update tests that rely on Classy to not rely on it anymore and Classy being deprecated in Drupal 9 + removed in Drupal 10,: Tests that aren't specifically testing Classy yet declare $defaultTheme = 'classy'; should be refactored to use Stark as the default theme instead.
Proposed resolution
Change all tests in this module to use Stark as the default theme, and refactor the tests where needed so they continue to function properly.
Remaining tasks
User interface changes
API changes
Data model changes
Release notes snippet
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 3272354-2.patch | 6.64 KB | danflanagan8 |
Comments
Comment #2
danflanagan8This one only has a single test with a single line that needed to be changed. Sooooo, I went ahead and updated all the status message assertions I could find to use the new WebAssert methods. That gives us a little more value from this patch without making the review significantly harder.
Comment #4
danflanagan8That was a random quickedit failure. This is ready for review.
Comment #5
larowlanNice cleanup
Comment #6
alexpottCommitted and pushed ba111c104d to 10.0.x and d9a2db8b8d to 9.5.x and 10ed5d1d26 to 9.4.x. Thanks!