Define a menu item without an access callback and with a permission as the access argument. Check that users with that permission can access the page and without it can't access.

CommentFileSizeAuthor
#7 menu_default_access_callback_7.patch2.67 KBmikey_p
#5 menu_default_access_callback_5.patch3.14 KBmikey_p
#1 menu_default_access_callback_1.patch3.35 KBmikey_p

Comments

mikey_p’s picture

mikey_p commented
Status: Active » Needs review
StatusFileSize
new menu_default_access_callback_1.patch3.35 KB

(trying to think of something witty to say for my first test)

After this, we should rename hook_menu.module to reflect the standards for mock module names. See #320363: Rename mock modules.

Status: Needs review » Needs work

The last submitted patch failed testing.

lilou’s picture

lilou commented
Status: Needs work » Needs review

See: #335122: Test clean HEAD after every commit and http://pastebin.ca/1258476

catch’s picture

catch
he/him
Primary language English
commented
Category: bug » task
Status: Needs review » Needs work

We no longer have phpdoc for setUp(), getInfo() or tearDown()
http://drupal.org/node/325974

mikey_p’s picture

mikey_p commented
Status: Needs work » Needs review
StatusFileSize
new menu_default_access_callback_5.patch3.14 KB

No more comments for getInfo(), setUp().

catch’s picture

catch
he/him
Primary language English
commented
Component: tests » menu system
Status: Needs review » Needs work

No longer applies - hook_menu.module is now menu_test.module

mikey_p’s picture

mikey_p commented
Status: Needs work » Needs review
StatusFileSize
new menu_default_access_callback_7.patch2.67 KB

Doh, I filed the issue to rename the mock module!

Try again...

Status: Needs review » Needs work

The last submitted patch failed testing.

mikey_p’s picture

mikey_p commented
Status: Needs work » Needs review

retest

Status: Needs review » Needs work

The last submitted patch failed testing.

bradfordcp’s picture

bradfordcp commented
Status: Needs work » Closed (duplicate)

Duplicate of #296292: Menu access callbacks need a test

Code from mikey_p has been rolled in there....