* Assert that a select option in the current page is not checked.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

scor’s picture

Status: Active » Needs review
FileSize
507 bytes
mr.baileys’s picture

Status: Needs review » Needs work
Issue tags: +Quick fix

Even though most of the function descriptions in DrupalWebTestCase still use the old-style second person form, we should convert these to use the third-person form whenever we're making changes to a function's Doxygen block.

So in this case the first line should read: "Asserts that a select option in the current page is checked."

scor’s picture

Issue tags: +Novice

tagging for novice queue, that's an easy fix.

amateescu’s picture

Status: Needs work » Needs review
FileSize
4.51 KB

Here is a patch for that.

It fixes the description for assertOptionSelected() and other occurences of the second person form in this file.

scor’s picture

Status: Needs review » Reviewed & tested by the community

This patch looks good and includes the initial fix for assertOptionSelected().

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)
Issue tags: -Quick fix, -Novice

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