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.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | help.test-251237-1.patch | 3.82 KB | floretan |
Comments
boombatower’s picture
Comment #2
boombatower commented Component: forum.module » tests
Change component is relation to http://drupal.org/node/253744.
boombatower’s picture
Comment #3
boombatower commentedThe 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
Comment #5
aspilicious commented 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.
Comment #1
floretan commentedThe 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.