I've just posted on the devs list in case anyone else wants to weigh in (or at least vote) on the issue... there seems to be enough good debate here that it could set a precedent one way or the other.

For my part, I probably should have left out the docblock comments in this test class, most because of the repetitive nature of those private members rather than because they're in test classes. However, for the main classes, I prefer good docblock comments. As @dbu pointed out, they do help IDEs (and possibly other code analysis tools?) to quickly spot errors: just as unit tests help you find errors sooner, IDEs can help you find them even sooner still.

However, I would not wish for too much strictness here, some leeway should be allowed for common sense and overall readability. So a mild +1 for me, but I'd probably take them out of this PR.