This causes a small slowdown, but allows us to remove the Node module dependency from Testing profile and each and every other presumption that's still baked in there.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch, drupal8.test-profile.0.patch, failed testing.

sun’s picture

Status: Needs work » Needs review
FileSize
4.19 KB

Good to see that this also improves test coverage of the module EnableDisableTest (which does not test the enabling + disabling of Node module currently).

Status: Needs review » Needs work

The last submitted patch, drupal8.test-profile.2.patch, failed testing.

sun’s picture

This is the effective alternative direction to #1541298: Remove Node module dependency from Testing profile and is only possible due to #632100: Use Stark as default theme for Minimal install profile

While the direction seems to make sense, I'm not sure whether it is a good idea to add back a major dependency on Minimal profile and its (minimal) assumptions for all web tests.

xjm’s picture

I've opened #1811016: [meta] Decouple tests from Node module. We'll probably be cleaning up at least some of the node dependencies for VDC.

sun’s picture

sun’s picture

Status: Needs work » Closed (duplicate)

Meanwhile, I don't think that this workaround is a good idea, since it would disallow us from tweaking Minimal profile for its target audience; site builders.

Instead, I've re-opened #1541298: Remove Node module dependency from Testing profile.