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

CommentFileSizeAuthor
drupal_get_headers_test_1.patch4.25 KBgrndlvl

Comments

grndlvl’s picture

grndlvl commented
Priority: Normal » Critical

Just getting same priority as parent task.

damien tournoud’s picture

damien tournoud commented
Status: Needs review » Needs work

- There is a DrupalCustomAccessDeniedTestCase, I guess it's a mistake.
- I'm not convinced of the usefulness of checking drupal_get_headers() indirectly. Maybe we could add a menu callback to the system_test mock module, and check various cases from here.

catch’s picture

catch
he/him
Primary language English
commented
Component: tests » base system
Category: bug » task
Priority: Critical » Normal

Moving this out of the critical bugs queue - see #607038: Meta issue: fix gaps in code coverage.

grndlvl’s picture

grndlvl commented
Assigned: grndlvl » Unassigned

I have not checked if there is still a need for this.

Status: Needs work » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.