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
#2791163: Random automatic testing failures on SQLite with PHP 5.5 broke simpletest_clean_database() - opps.
Proposed resolution
Clean tables like test* instead of simpletest*
Remaining tasks
User interface changes
API changes
None
Data model changes
None
Comment | File | Size | Author |
---|---|---|---|
#11 | 2807171-11.patch | 1.07 KB | alexpott |
#11 | 8-11-interdiff.txt | 599 bytes | alexpott |
#8 | 3-8-interdiff.txt | 983 bytes | alexpott |
#8 | 2807171-8.patch | 1.08 KB | alexpott |
#3 | 2807171-2.patch | 1 KB | alexpott |
Comments
Comment #2
alexpottI think we should commit this before 8.2.0 since this is not run-time code and only test code plus it might allow us to fix #2806697: Random fail for AlreadyInstalledException
Comment #3
alexpottComment #4
alexpottComment #5
cilefen CreditAttribution: cilefen commentedWrong now, yes?
Comment #6
cilefen CreditAttribution: cilefen commentedActually, it is a useless comment.
Comment #7
dawehnerIMHO the comment is still valid, just written a bit weird. Well, we don't want to delete the simpletest table, but yeah this is not a problem with this regex at all.
Comment #8
alexpottActually the current function is broken on prefixed sites. Here's a patch that is more reliable. I've tested this on both a prefixed db and a non prefixed db. Unfortunately this method is hard to test using DrupalCI.
Comment #9
klausiTested the patch manually, I can confirm that now any table prefixed with "test" is removed as it should.
Comment #10
klausi"example" twice in the comment, can be fixed on commit.
Comment #11
alexpottComment #14
catchCommitted/pushed to 8.3.x and cherry-picked to 8.2.x. Thanks!