The base test class has a helper function for installing the various Commerce modules, and it groups them into API and UI groups. Currently it puts Cart / Checkout into the API group, but really these are UI modules. If we move them to the UI group, we can also put Commerce UI in the UI group. You just need to test to make sure the change doesn't affect any existing tests.

Comments

recidive’s picture

recidive commented
Assigned: Unassigned » recidive

Doing that, as I'm getting up to speed with test stuff.

rszrama’s picture

rszrama commented

Did you actually have a patch for this? It should just be a simple moving around of the items in the arrays and then a run through all the tests to make sure nothing breaks...

recidive’s picture

recidive commented

Yeah, I'll make sure it doesn't break any tests and will push patch to githb.

recidive’s picture

recidive commented
Status: Active » Reviewed & tested by the community

Ok, I've tested and moving UI modules doesn't break any tests.

https://github.com/recidive/drupalcommerce/commit/311fb7d19d95d283be2574...

rszrama’s picture

rszrama commented
Status: Reviewed & tested by the community » Fixed

Excellent. Committed.

Status: Fixed » Closed (fixed)
Issue tags: -low-hanging fruit

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

Issue tags: +low-hanging fruit

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.