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.
D7 has a class registry. There's no need to stuff test helpers in .inc files and try to include them. Just name them .test files and be done with it. This makes it way harder for everyone else to run tests.
Comment | File | Size | Author |
---|---|---|---|
#4 | 1066822-test-include-cleanup.patch | 18.54 KB | Dave Reid |
#1 | 1066822-test-include-cleanup.patch | 43.32 KB | Dave Reid |
Comments
Comment #1
Dave ReidComment #2
Dave ReidComment #3
pillarsdotnet CreditAttribution: pillarsdotnet commentedOfftopic I know, but somebody oughtta extend the format of patch/diff to indicate a file rename as a single operation rather than a line-by-line removal from one name and addition to another.
Comment #4
Dave ReidThere you go. :)
Comment #5
Dave ReidCommitted to Git.
http://drupalcode.org/project/feeds.git/commit/98bacbd
Comment #6
Dave ReidCommitted to 6.x-1.x as well:
http://drupalcode.org/project/feeds.git/commit/976e8dd
Comment #7
twistor CreditAttribution: twistor commentedThe last patch adds a few dpm calls to FeedsNodeProcessor.