I've started putting some work into creating some SimpleTest tests for the CustomError module. Attached is a patch for the module which implements hook_simpletest() and an initial test which simply tests that the 404 message is returned when expected. The test file should be renamed to pagenotfound.test and placed into a tests directory in the customerror directory.

CommentFileSizeAuthor
#4 pagenotfound.patch3.6 KBgeodaniel
#4 accessdenied.test_.txt1.92 KBgeodaniel
#1 pagenotfound.txt1.96 KBgeodaniel
customerror_simpletest.patch719 bytesgeodaniel
pagenotfound.txt1.94 KBgeodaniel

Comments

geodaniel’s picture

geodaniel commented
StatusFileSize
new pagenotfound.txt1.96 KB

Cleaning up spacing in test file.

kbahey’s picture

kbahey commented
Status: Needs review » Fixed

Committed to HEAD and 5.x.

geodaniel’s picture

geodaniel commented

Thanks Khalid

geodaniel’s picture

geodaniel commented
Status: Fixed » Needs review
StatusFileSize
new accessdenied.test_.txt1.92 KB
new pagenotfound.patch3.6 KB

Attaching a patch to fix up the page not found test (spacing and also adding a user login so that we're not relying on the anonymous user having access content privileges) and also adding a test for access denied.

kbahey’s picture

kbahey commented
Status: Needs review » Fixed

Committed to HEAD and 5.x.

Please do not reopen fixed issue for new features. Create a new one.

geodaniel’s picture

geodaniel commented

Thanks, I will do that in the future.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

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

  • Commit cd73dae on 6.x-1.x, master, 8.x-1.x by kbahey: 
    #204246 by geodaniel, adding simpletest for page not found.