Currently, our pathauto tests are for the 'administrator' role. We should create some similar tests using the 'editor' role because they don't have access to the 'Permalink' field, which used to cause differences in the way aliases worked until we fixed: #2283531: Path alias isn't preserved if the user doesn't have "create url alias" permission

CommentFileSizeAuthor
#1 editor-pathauto-2334143-1.patch10.03 KBjessehs

Comments

jessehs’s picture

jessehs
he/him
Primary language English
Location Taos, New Mexico
commented
Status: Active » Needs review
StatusFileSize
new editor-pathauto-2334143-1.patch10.03 KB

This allows editors to create panopoly_test_page content (and exports these permissions) and tests whether path aliases change when title is changed.

Editors cannot create custom paths, so this new test removes that check.

The pathauto.feature is renamed pathauto_administrator.feature.

  • dsnopek committed 82aa315 on 7.x-1.x 
    Update Panopoly Test for #2334143 by jessehs | dsnopek: Write Behat...
dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Needs review » Fixed

Thanks! Patch/test looks good. I tested locally and everything worked fine, so: committed. Hopefully Travis-CI will agree:

https://travis-ci.org/panopoly/panopoly/builds/40468257

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented

Found a minor bug in this when testing panopoly_test with a child distribution: #2377713: panopoly_test shouldn't put "use text format panopoly_(html|wysiwyg)_text" in user_permission

Status: Fixed » Closed (fixed)

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