Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Several test cases refer to Simpletest, but we have now migrated to PHPUnit.
Steps to reproduce
Proposed resolution
Replace or remove the references.
Remaining tasks
User interface changes
API changes
Data model changes
Release notes snippet
Issue fork drupal-3293216
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #3
longwaveComment #4
alexpottNice this will help us not have to think as much. +1
Comment #5
longwaveComment #7
SpokjeI like this clean up :)
One thing that I've noticed is that, if I grep the codebase with the MR applied, there's still a lot of references left.
I've already incorporated the fact that we (most probably) can't rename the test-output folder from
sites/simpletest
to something else without starting The Apocalypse.Also (almost?) all of the references are in comments or doc-blocks, so maybe that's something for a follow-up?
Comment #8
longwaveI left anything to do with the sites/simpletest directory or the prefixed environment variables (which I think have to be handled in separate issues). I probably missed some references in comments that still could be cleaned up, happy to have another pass at those.
Comment #9
longwaveRemoved another 40 or so references, mostly from comments.
Comment #10
SpokjeTook me a while to finally get back to reviewing this one. Sorry about that.
All looks great, so this is RTBC for me.
Comment #12
catchI grepped for simpletest with this applied, and still found a dozen or so references, however this is a big improvement, so let's open a follow-up to remove the remaining stragglers.
Committed/pushed to 10.1.x and cherry-picked to 10.0.x, thanks!