Is there a reason why the features_test module isn't hidden?

I haven't actually looked at the test so I'm not sure. But it does botter me to see a test module in the module listing :-/

Patching coming up.

CommentFileSizeAuthor
#1 1587200-feature-tests.patch723 bytesericduran

Comments

ericduran’s picture

ericduran commented
Status: Active » Needs review
StatusFileSize
new 1587200-feature-tests.patch723 bytes
bojanz’s picture

bojanz commented
Status: Needs review » Reviewed & tested by the community

Makes sense to me.

mpotter’s picture

mpotter commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Nice, I've wanted this too. Committed and pushed to 24247b4.

pfrenssen’s picture

pfrenssen
Location Sofia
commented

This causes the FeaturesEnableTestCase test to fail, see issue #1805432: Call to undefined function taxonomy_features_api() in FeaturesEnableTestCase.

pfrenssen’s picture

pfrenssen
Location Sofia
commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Patch (to be ported) » Needs work

This also causes the FeaturesUserTestCase to fail. It seems like Features can not handle hidden modules.

mpotter’s picture

mpotter commented
Status: Needs work » Fixed

Removed the patch from #1187232: Hidden and disabled features should not be displayed in UI and should not be considered for conflicts that seems to have caused Features to break on Hidden modules. Committed to 7.x-1.x in 05c9af6. Let me know if this fixes the SimpleText.

Status: Fixed » Closed (fixed)

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