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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

littledynamo created an issue. See original summary.

littledynamo’s picture

littledynamo’s picture

Issue summary: View changes