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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

grndlvl’s picture

Priority: Normal » Critical

Just getting same priority as parent task.

Damien Tournoud’s picture

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

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

Assigned: grndlvl » Unassigned

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