Hello,

This morning the continuous integration showed up an error with the new beta 11.

It seems that the following line is not right:

module_load_include('test', 'feeds', 'test/feeds');

It should be replaced by:

module_load_include('test', 'feeds', 'tests/feeds');

I made a patch who fix this. I hope it will work.

CommentFileSizeAuthor
#3 0001-Fix-paths-to-parent-test-classes.patch8.27 KBcarlos8f
#1 fix_simpletests_includes-1203578.patch7.95 KBPol
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Pol’s picture

Pol
the/that
Location Brussels
CreditAttribution: Pol commented
FileSize
fix_simpletests_includes-1203578.patch7.95 KB

Here is the patch. Tested and working.

carlos8f’s picture

carlos8f CreditAttribution: carlos8f commented
Status: Active » Reviewed & tested by the community

yep, that's better.

carlos8f’s picture

carlos8f CreditAttribution: carlos8f commented
FileSize
0001-Fix-paths-to-parent-test-classes.patch8.27 KB

actually here's the properly rolled git patch.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Reviewed & tested by the community » Fixed

#3 was missing a chunk - I already had #1 applied locally, so committed to Git.
http://drupalcode.org/project/feeds.git/commit/dab7051

Status: Fixed » Closed (fixed)

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