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.

CommentFileSizeAuthor
#4 1066822-test-include-cleanup.patch18.54 KBdave reid
#1 1066822-test-include-cleanup.patch43.32 KBdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
StatusFileSize
new 1066822-test-include-cleanup.patch43.32 KB
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Assigned: Unassigned » dave reid
pillarsdotnet’s picture

pillarsdotnet commented

Offtopic 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.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
StatusFileSize
new 1066822-test-include-cleanup.patch18.54 KB

There you go. :)

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Fixed

Committed to Git.
http://drupalcode.org/project/feeds.git/commit/98bacbd

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Committed to 6.x-1.x as well:
http://drupalcode.org/project/feeds.git/commit/976e8dd

twistor’s picture

twistor commented

The last patch adds a few dpm calls to FeedsNodeProcessor.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.