Simpletest reports a fail for "Found the requested form at" yet it also passes for "Found the Submit button" on the same form. The first message should more accurately read something like "Found the requested fields." Thanks to dmitrig01 and catch for responding to this in #drupal.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | simpletest_forms_message_d7.patch | 940 bytes | ezra-g |
| #2 | simpletest_form_message.patch | 975 bytes | ezra-g |
Comments
Comment #1
catchBumping this to core because I think boombatower is doing backports to 6.x too. This message is definitely confusing, and fields seems more accurate.
Comment #2
ezra-g commentedI already rolled the patch against simpletest 6.2, so here it is for purposes of the backport. It changes the text in question to 'Failed to find the requested form fields at @path'
Comment #3
ezra-g commentedHere is the patch for Drupal 7.
Comment #4
boombatower commentedThis is related to [#http://drupal.org/node/293099] I think they should be merged.
Doesn't follow the convention:
We state things in the positive and let the assertion result explain the state.
Changed to:
Since other thread already existed and these are similar (and minor) patches I've merged to other thread.
And yes SimpleTest 6.x-2.x is a backport of 7.x stuff.