Download & Extend
Issues

TestingParty08: menu access callback default need a test

Posted by chx on August 17, 2008 at 8:10am
Project:Drupal core
Version:7.x-dev
Component:menu system
Category:task
Priority:critical
Assigned:Unassigned
Status:closed (duplicate)

Issue Summary

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.

Login or register to post comments

Comments

#1

Posted by mikey_p on October 12, 2008 at 9:52pm
Status:active» needs review

(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.

AttachmentSizeStatusTest resultOperations
menu_default_access_callback_1.patch3.35 KBIdleFailed: Failed to apply patch.View details | Re-test

#2

Posted by System Message on November 16, 2008 at 9:50pm
Status:needs review» needs work

The last submitted patch failed testing.

#3

Posted by lilou on November 17, 2008 at 2:16pm
Status:needs work» needs review

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

#4

Posted by catch on November 19, 2008 at 1:18am
Category:bug report» task
Status:needs review» needs work

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

#5

Posted by mikey_p on November 24, 2008 at 9:22pm
Status:needs work» needs review

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

AttachmentSizeStatusTest resultOperations
menu_default_access_callback_5.patch3.14 KBIdleFailed: Failed to apply patch.View details | Re-test

#6

Posted by catch on January 11, 2009 at 5:35pm
Component:tests» menu system
Status:needs review» needs work

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

#7

Posted by mikey_p on January 12, 2009 at 8:44am
Status:needs work» needs review

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

Try again...

AttachmentSizeStatusTest resultOperations
menu_default_access_callback_7.patch2.67 KBIdleFailed: Failed to apply patch.View details | Re-test

#8

Posted by System Message on January 15, 2009 at 8:45am
Status:needs review» needs work

The last submitted patch failed testing.

#9

Posted by mikey_p on January 16, 2009 at 4:34pm
Status:needs work» needs review

retest

#10

Posted by System Message on February 5, 2009 at 7:55am
Status:needs review» needs work

The last submitted patch failed testing.

#11

Posted by bradfordcp on April 21, 2009 at 1:41am
Status:needs work» closed (duplicate)

Duplicate of #296292: Menu access callbacks need a test

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

nobody click here