New Assertion: Check for test being found only once

Well done. It might be good to provide a test that actually uses this assertion though, so we're not just adding stuff that's never used. One use case that might be fitting is a test on form_clean_id which would check to see if multiple IDs of the same name become unique when being added.

#2: dealt with, lets see what bot says.

I wonder if we can think of a better real-world test. This test doesn't seem very useful. Do we have a _real_ use case for this function?

I've got one. :) #405086: Custom 404 pages displayed twice

A couple minor things:
- Comments should wrap at 80 characters. Also, when doing PHPDoc comments, they should be done like this:

/**
 * One sentence summary, < 80 characters.
 *
 * The rest of the details. This can be as long as you want, and
 * span many lines.
 * ...
 */
function...

- If assertUniqueTextHelper isn't intended to be called, how about rename it to _uniqueTextHelper or similar? Right now in IDEs and whatnot it would come up and look like another assertion function which is confusing.

- This should be drupalGet (weird; how is that not generating a fatal error?):

+    $this->drupalget('form_test/form_clean_id');

- +function form_test_form_clean_id_page() { and +function form_test_test_form(&$form_state) { both need a one-liner of PHPDoc to explain what they do.

Just make assertUniqueTextHelper private as that is one of the nice things about OO.

Fix code style. Add documentation to the mock module functions.

We don't use "_" conventions for determining visibility, but rather proper private/protected/public modifiers. In that case, assertUniqueTextHelper() is modelled against assertTextHelper(), which has exactly the same help text. We can't make those private, because they could be useful for a subclass. It is not dangerous to call those functions directly, it's just not recommended.

And fix a drupalget() -> drupalGet() and some whitespace nitpicking.

Ok, cool. There are some other things about this that I'd like to see fixed (the comments are SHOUTING AT ME, for instance) but what's there is consistent with the rest of DWTC so out of scope for this patch.

Committed to HEAD. Thanks!

Awww. :) Yay!! :D

May it be the first of many more. :)

This is wrong because we do not change the message based on the assert failing or not. If you read "found once" failed then you know it's not been found once. If you read "found more than once" and a FAIL what does that mean?

disregard me. i misunderstood.