Simpletest Guidelines [1] say:
' * Test descriptions should use active tense whenever possible. "Tests creation of a new node." not "Tests for creating a new node."'

...which is all wrong.
(1) the active is a voice not a tense; the distinction between the two examples is not active vs. passive but abstract noun vs. specific verb; and it is ambiguous because it apparently refers only to inline comments, not to the descriptions actually defined in each test.
(2) the guideline is not followed in core, where there is little consistency [2], despite having been in the docs since last October.
(3) the guideline doesn't necessarily result in good descriptions: you would get 'Tests enabling of a menu link' instead of, as now 'Enable a menu link' or, as a verb, 'Tests enabling a menu link'.

So I suggest removing the sentence for now, although in due course creating an actual standard could be considered by the relevant people, given the range of examples at [2].

[1] http://drupal.org/node/325974
[2] these are the tests from menu.module:
Login users, add menus and menu links, and test menu functionality through the admin and user interfaces.
Test standard menu functionality using navigation menu.
Test custom menu functionality using navigation menu.
Delete custom menu.
Test menu functionality using navigation menu.
Add a menu link using the menu module UI.
Attempt to add menu link with invalid path or no access permission.
Verify a menu link using the menu module UI.
Modify a menu link using the menu module UI.
Reset a standard menu link using the menu module UI.
Delete a menu link using the menu module UI.
Alternately disable and enable a menu link.
Disable a menu link.
Enable a menu link.
Get standard menu link.
Verify the logged in user has the desired access to the various menu nodes.

Comments

leehunter’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

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