help.test has several assertions commented out, notably to do with blog and poll modules. It's not clear whether these are broken bits of test or broken bits of code, but we should fix it.

CommentFileSizeAuthor
#1 help.test-251237-1.patch3.82 KBfloretan

Comments

floretan’s picture

Status: Active » Needs review
StatusFileSize
new3.82 KB

The existing test assumes that every enabled module implements hook_help(), and also uses assertions that could be influenced by external settings (site name, theme, etc).

The help module doesn't provide much besides the implementation of hook_help, and that hook has very loose requirements. The only way to figure out if a module will display help on a certain page is to call its help hook and see if something is returned. There is no guarantee that a module implementing hook_help will display anything on any page.

Here's a patch that works, but I'm still not satisfied with the approach. I'm working on a dummy module whose only purpose would be to test hooks, but that's a different issue.

boombatower’s picture

Component: forum.module » tests

Change component is relation to http://drupal.org/node/253744.

boombatower’s picture

The issues were stated here, but never fixed: http://drupal.org/node/242475.

catch’s picture

Status: Needs review » Needs work

No longer applies. 1 out of 2 hunks failed.

aspilicious’s picture

Component: tests » base system
Status: Needs work » Fixed

help.test is rewritten, no comment out assertions left

Status: Fixed » Closed (fixed)

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