Part of meta issue #500866: [META] remove t() from assert message.

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 38 changes needed in 4 Test files. This issue includes conversion of all format_patch() occurances as well.

CommentFileSizeAuthor
#7 statistics-1797520-7.patch13.54 KBdcam
#4 1797520-4-t-statistics.patch13.26 KBlars toomre
#2 1797520-2-t-statistics.patch13.87 KBlars toomre

Comments

lars toomre’s picture

lars toomre commented
Assigned: xjm » lars toomre

Patch forthcoming for this one!

lars toomre’s picture

lars toomre commented
Status: Active » Needs review
StatusFileSize
new 1797520-2-t-statistics.patch13.87 KB

Here is an initial patch for this issue. It includes conversion(s) to format_string().

Status: Needs review » Needs work

The last submitted patch, 1797520-2-t-statistics.patch, failed testing.

lars toomre’s picture

lars toomre commented
Status: Needs work » Needs review
StatusFileSize
new 1797520-4-t-statistics.patch13.26 KB

Whoops... Deleted one too many ) there. This should fix that.

xjm’s picture

xjm
she/her
Primary language English
commented
Assigned: lars toomre » jhodgdon
Status: Needs review » Reviewed & tested by the community

All correct.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Version: 8.x-dev » 7.x-dev
Assigned: jhodgdon » Unassigned
Status: Reviewed & tested by the community » Patch (to be ported)

Thanks! Committed to 8.x.

dcam’s picture

dcam commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new statistics-1797520-7.patch13.54 KB

Backported #4 to D7.

Status: Needs review » Needs work
Issue tags: -Needs backport to D7

The last submitted patch, statistics-1797520-7.patch, failed testing.

dcam’s picture

dcam commented
Status: Needs work » Needs review
Issue tags: +Needs backport to D7

#7: statistics-1797520-7.patch queued for re-testing.

lars toomre’s picture

lars toomre commented
Status: Needs review » Reviewed & tested by the community

I have checked all of the changes proposed in #7 and they are all correct. I did not check whether this covers all of the changes in the Tests for the Statistics module because I did not apply the patch to D7 install. FRom my perspective, this is RTBC!

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Assigned: Unassigned » jhodgdon

Tum te tum...

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Assigned: jhodgdon » Unassigned
Status: Reviewed & tested by the community » Fixed

Thanks! Committed to 7.x.

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

Added count details.