This is a sub-task of #500866: [META] remove t() from assert message focused on the Update module.

In D8 per http://drupal.org/simpletest-tutorial-drupal7#t, best practice is to remove t() from assert messages in tests. When necessary, t() should be replaced with format_string().

There are approximately 23 changes needed in 4 Test files. This issue includes conversion of all format_patch() occurances as well (none were needed).

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Lars Toomre’s picture

Status: Active » Needs review
FileSize
10.54 KB

Here is an initial untested patch for this issue.

dcam’s picture

Status: Needs review » Reviewed & tested by the community

#1 looks good to me. I don't notice any additional t()'s around assert messages.

jhodgdon’s picture

Version: 8.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Thanks! Committed to 8.x. Ready for backport...

dcam’s picture

Status: Patch (to be ported) » Needs review
FileSize
8.93 KB

Backported #1 to D7.

dcam’s picture

#4: 1798384-4-t-update.patch queued for re-testing.

dcam’s picture

Issue tags: +Novice

Tagging as Novice.

Mile23’s picture

Status: Needs review » Reviewed & tested by the community

The patch in #4 applies cleanly to 7.x-dev, and I don't see any t()-wrapped assertion messages.

jhodgdon’s picture

Status: Reviewed & tested by the community » Fixed

Thanks, and sorry for the delay -- committed to 7.x.

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

Anonymous’s picture

Issue summary: View changes

Modified initial counts.