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

CommentFileSizeAuthor
#7 403test.patch2.44 KBcatch
#5 403_functionality_test.patch2.36 KBswentel
#3 403_functionality_test.patch2.36 KBswentel

Comments

cwgordon7’s picture

cwgordon7 commented

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

chx’s picture

chx commented

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

swentel’s picture

swentel commented
Status: Active » Needs review
StatusFileSize
new 403_functionality_test.patch2.36 KB

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

catch’s picture

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

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

swentel’s picture

swentel commented
Status: Needs work » Needs review
StatusFileSize
new 403_functionality_test.patch2.36 KB

Ah crap! Fixed that :)

catch’s picture

catch
he/him
Primary language English
commented
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.

catch’s picture

catch
he/him
Primary language English
commented
StatusFileSize
new 403test.patch2.44 KB

And the patch.

lilou’s picture

lilou commented

Patch still applies.

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

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Committed! Thanks. :)

Anonymous’s picture

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

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