Test for Custom Access Denied
grndlvl - October 2, 2008 - 11:32
| Project: | Drupal |
| Version: | 7.x-dev |
| Component: | tests |
| Category: | bug report |
| Priority: | critical |
| Assigned: | grndlvl |
| Status: | closed |
Description
This patch was originally inside of a really large patch, was suggested to split into smaller patches.
Tests needed: common.inc
http://drupal.org/node/276272
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| custom_access_denied_test_1.patch | 1.82 KB | Ignored | None | None |
#1
Just getting same priority as parent task.
#2
I think we should refactor this one with PageNotFoundTestCase from system.test. After all, those are exactly the same, the only thing that change is the target path.
#3
Per comment #2 suggestion this patch now applies this test to system.test rather than common.test.
#4
Looks good. I tested the patch, and it works for me.
Two suggestions:
Error reporting
Control how Drupal deals with errors including 403/404 errors as well as PHP error reporting.
#5
made suggested changes from comment #4.
#6
What's the difference with http://drupal.org/node/296306 ?
#7
@swentel : I think test in #296306: TestingParty08: 403 page needs a test is also include in the last patch of this issue.
#8
It seems that http://drupal.org/node/296306, contains all that this patch covered.
--project followup subject--
Automatically closed -- issue fixed for two weeks with no activity.
#9
Automatically closed -- issue fixed for two weeks with no activity.