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
We've had ResourceTestBase
for exactly a year now (since #2737719: EntityResource: Provide comprehensive test coverage: for every entity type, every format, every method).
In working on #2893804: Remove rest.module BC layers, it became clear that DbLogResourceTest
was still using the deprecated base test class. In fact, this issue blocks that issue.
Proposed resolution
Update DbLogResourceTest
.
Remaining tasks
None.
User interface changes
None.
API changes
None.
Data model changes
None.
Comment | File | Size | Author |
---|---|---|---|
#14 | 2927758-14.patch | 6.89 KB | Wim Leers |
#11 | 2927758-11.patch | 5.28 KB | dagmar |
#8 | interdiff-2927758-3-8.txt | 979 bytes | dagmar |
#8 | 2927758-8.patch | 5.28 KB | dagmar |
#3 | 2927758-3.patch | 5.38 KB | Wim Leers |
Comments
Comment #2
Wim LeersComment #3
Wim LeersI did not update the namespace despite moving the file. My bad.
Comment #4
dawehnerComment #7
Wim LeersThis is missing
'config:rest.settings'
. Then it should be green.Comment #8
dagmarThanks for working on this! I had to create this patch using
git diff -C -M20%
since there were so many changes and git wasn't able to detect the renamed file.Comment #10
Wim LeersLooks like your diff is going the wrong direction? :D
Comment #11
dagmarHah. Yes. Sorry for that.
Comment #12
Wim LeersI'd RTBC, but I've created most of this patch. @dagmar made a trivial change in #8. As the maintainer of this component, I think he's eligible to RTBC this patch.
Comment #13
dagmarThe patch looks good for me.
There is only one coding standard issue here:
Comment #14
Wim LeersFixed!
Comment #15
dagmarThanks!
Comment #17
Anonymous (not verified) CreditAttribution: Anonymous commentedRandom fail with memory.
Comment #19
Wim LeersComment #22
larowlanCommitted as 1aab6ab and pushed to 8.5.x.
Cheerily-picked as 28b52c3 and pushed to 8.4.x.
Thanks! the march continues simpletest-countdown.org
Comment #23
Wim Leers#22: hah! :D Thanks for that URL btw, I was trying to find it last week, but couldn't!