Currently, the setUp() function in FeedsTamperUnitTestCase assumes that plugins exist inside the feeds tamper module directory hierarchy:
$file = drupal_get_path('module', 'feeds_tamper') . '/plugins/' . $this->plugin_id . '.inc';
I maintain feeds_tamper_conditional and would like to extend FeedsTamperUnitTestCase but it errors out because the file path feeds_tamper/plugins/feeds_tamper_conditional.inc does not exist.
I propose changing the code to account for the plugin existing in a separate module:
if(module_exists($this->plugin_id)) {
$file = drupal_get_path('module', $this->plugin_id) . '/plugins/' . $this->plugin_id . '.inc';
}
else {
$file = drupal_get_path('module', 'feeds_tamper') . '/plugins/' . $this->plugin_id . '.inc';
}
Patch to follow.
Comment | File | Size | Author |
---|---|---|---|
#2 | feeds_tamper-extend-feedstamperunittestcase-2824742-1.patch | 773 bytes | littledynamo |
Comments
Comment #2
littledynamo CreditAttribution: littledynamo commentedComment #3
littledynamo CreditAttribution: littledynamo commented