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.
Comment | File | Size | Author |
---|---|---|---|
#8 | convert_user_tests-1711434-8.patch | 9.72 KB | Albert Volkman |
#6 | convert_user_tests-1711434-6.patch | 9.72 KB | Albert Volkman |
#4 | user-1711434-4.patch | 9.72 KB | tim.plunkett |
#2 | drupal-1711434-2.patch | 9.04 KB | tim.plunkett |
Comments
Comment #1
tim.plunkettComment #2
tim.plunkettWoo!
Comment #3
xjmSee the summary of #1711070: Convert tests to use ::$modules property instead of parent::setUp($modules) for the docblocks that should be added.
Comment #4
tim.plunkettRerolled
Comment #5
xjmDo we want a blank line after the setup and before the inline comment (as in the other patches)?
Comment #6
Albert Volkman CreditAttribution: Albert Volkman commentedNow with more spaces.
Comment #7
tim.plunkettStill missing the newline?
Comment #8
Albert Volkman CreditAttribution: Albert Volkman commentedComment #9
tim.plunkettLooks good. Thanks for cleaning that up!
Comment #10
ZenDoodles CreditAttribution: ZenDoodles commentedNow that timplunkett rolled them into one, this is a duplicate of the parent issue: #1711070: Convert tests to use ::$modules property instead of parent::setUp($modules)