Remove t() from default test assertions in WebTestBase class [#2195623]

Part of #500866: [META] remove t() from assert message
Follow-up to #1803674: Remove t() from default test assertions in TestBase class

The WebTestBase class should not contain calls to t() for format_plural().

All t() calls should be replaced with String::format().

Plain text strings without token replacements can be output directly.

Calls to format_plural() are to be replaced with English plural rules, which means this:

- $message = format_plural($emailCount, '1 e-mail was sent during this test.', '@count e-mails were sent during this test.');
+ $message = $emailCount == 1 ? '1 e-mail was sent during this test.' : $emailCount . ' e-mails were sent during this test.';

Comment	File	Size	Author
#2	drupal-2195623-2.patch	19.75 KB	olli
#2

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

jhodgdon

she/her

English

CreditAttribution: jhodgdon commented 13 February 2014 at 18:49

Issue tags:

+Novice

Possibly this would be a good Novice project, for a motivated novice? Or non-novice. Either way. :)

Comment #2

olli CreditAttribution: olli commented 14 February 2014 at 05:52

Status:

Active

» Needs review

File	Size
drupal-2195623-2.patch	19.75 KB

Comment #3

jhodgdon

she/her

English

CreditAttribution: jhodgdon commented 14 February 2014 at 17:08

Status:

Needs review

» Reviewed & tested by the community

Thanks!

This patch was very carefully done. I checked it over carefully, and am reasonably certain it is removing t() from the right places, and leaving it in where I believe we still want it (such as translating the submit button text in code like this:

    $this->drupalPostForm('user', $edit, t('Log in'));

I also verified there are no format_plural() calls in WebTestBase, and that it caught everything.

I especially love the irony that there were all these documentation comments in WebTestBase:

   *   (optional) A message to display with the assertion. Do not translate
   *   messages: use format_string() to embed variables in the message text, not
   *   t(). If left blank, a default message will be displayed.

followed just a few lines later by the default message being translated with t(). Um. :)

Great job on the patch!