Problem/Motivation

In #2578263: Add helper functions to MigrateDrupal7TestBase benjy said

I read the backscroll on IRC, chx mentioned his concerns with the large amount of $modules we're enabling on the base. I do see his point, couldn't we move any module setUp into the helpers methods?

svendecabooter tried to move the loading of the dependent modules to the helper function but found that the migration_templates aren't available in the helper function. He suggested that "only the migration templates for the enabled module should be created".

Read svendecabooter's test results.

Proposed resolution

None yet.

Remaining tasks

Make a plan
Code and test

User interface changes

N/A

API changes

N/A

Data model changes

N/A

Comments

quietone created an issue. See original summary.

Version: 8.0.x-dev » 8.1.x-dev

Drupal 8.0.6 was released on April 6 and is the final bugfix release for the Drupal 8.0.x series. Drupal 8.0.x will not receive any further development aside from security fixes. Drupal 8.1.0-rc1 is now available and sites should prepare to update to 8.1.0.

Bug reports should be targeted against the 8.1.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.2.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan at Virtuoso Performance commented
Status: Active » Closed (outdated)

This appears to have been addressed - MigrateDrupal7TestBase enables no modules itself (parent class MigrateDrupalTestBase just enables a few), and the individual module tests just enable what they need.