Based on the discussion on http://groups.drupal.org/node/7731, it appears that putting comments for inherited test methods like getInfo(), setUp() and tearDown() is generally confusing. Rather than treating them like hook implementations or having a generic comment being repeated, it would be cleaner to not have comments at all. This opinion is shared by many people including dww, moshe and drewish.

Here's a patch based on a regular expression to remove all comments from getInfo(), setUp() and tearDown() functions in .test files.

For the reference, here's the regexp:
In *.test files, replace

([^\n]*\/\*\*\n.*\n\s*\*\/\n)(\s*)function (getInfo|setUp|tearDown)

with

$2function $3
CommentFileSizeAuthor
test_comment_cleanup.patch50.49 KBfloretan

Comments

Anonymous’s picture

Status: Needs review » Needs work

The last submitted patch failed testing.

dww’s picture

FWIW, I totally support this change. ;)

naxoc’s picture

Component: tests » ajax system
Status: Needs work » Closed (fixed)

It seems that the generic comments have been removed. This produces nothing now:
grep -r '([^\n]*\/\*\*\n.*\n\s*\*\/\n)(\s*)function (getInfo|setUp|tearDown)' .

Closing the issue :)