TestingParty08: 403 page needs a test

chx - August 17, 2008 - 08:10
Project:Drupal
Version:7.x-dev
Component:tests
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed
Description

Set a 403 page and check that it appears. Assert that blocks and tabs do not appear.

#1

cwgordon7 - August 18, 2008 - 00:32

Hang on - blocks *do* appear on 403 pages. Are you saying this is a bug?

#2

chx - August 18, 2008 - 07:14

Sorry I was mistaken. 404 pages do not have blocks. 403 have.

#3

swentel - August 28, 2008 - 11:51
Status:active» needs review

403 functionality, copied most of 404 patch, easy :)

AttachmentSize
403_functionality_test.patch 2.36 KB

#4

catch - September 18, 2008 - 23:55
Status:needs review» needs work

Copied it a little too much, the function needs renaming to something more accurate than pageNotFound() ;)

#5

swentel - September 19, 2008 - 09:08
Status:needs work» needs review

Ah crap! Fixed that :)

AttachmentSize
403_functionality_test.patch 2.36 KB

#6

catch - September 19, 2008 - 09:41
Status:needs review» reviewed & tested by the community

Didn't apply for some reason due to whitespace changes in HEAD, otherwise RTBC. I've re-rolled for whitespace only, so please don't credit on commit.

#7

catch - September 19, 2008 - 09:42

And the patch.

AttachmentSize
403test.patch 2.44 KB

#8

lilou - October 10, 2008 - 17:06

Patch still applies.

Is this issue is a duplicate of this one : #316186: Test for Custom Access Denied ?

#9

webchick - October 11, 2008 - 18:17
Status:reviewed & tested by the community» fixed

Committed! Thanks. :)

#10

Anonymous (not verified) - October 25, 2008 - 18:23
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.