Support from Acquia helps fund testing for Drupal Acquia logo

Comments

xjm’s picture

Title: Remove t() from asserts messages in tests for the forum module » Remove t() from assertion messages in tests for the forum module
Assigned: xjm » Unassigned
Status: Active » Needs review
Issue tags: +Needs backport to D7
FileSize
13.74 KB

I had to reformat one of the format_string() replacement lines here because it had so many parens that I couldn't read it myself.

Lars Toomre’s picture

Status: Needs review » Reviewed & tested by the community

I have reviewed each of the changes in this patch and each one of the removal/conversion of t() from test asserts is correct.

With the bot being green, this issue is RTBC!

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 8.x. Thanks.

Status: Fixed » Closed (fixed)

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

David_Rothstein’s picture

Version: 8.x-dev » 7.x-dev
Status: Closed (fixed) » Patch (to be ported)

This never made it into Drupal 7.

dcam’s picture

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

Backported #1 to D7.

jhodgdon’s picture

#6: 1797286-6-forum-t.patch queued for re-testing.

lazysoundsystem’s picture

-    $this->assertRaw(theme('breadcrumb', array('breadcrumb' => $breadcrumb)), t('Breadcrumbs were displayed'));
+    $this->assertRaw(theme('breadcrumb', array('breadcrumb' => $breadcrumb)), ('Breadcrumbs were displayed'));

No need for parentheses here, otherwise it's all fine.

jhodgdon’s picture

Status: Needs review » Needs work
dcam’s picture

Status: Needs work » Needs review
FileSize
13.28 KB
540 bytes

Fixed the issue pointed out in #8.

lazysoundsystem’s picture

Status: Needs review » Reviewed & tested by the community

Thanks @dcam, this too is good to go. RTBC.

jhodgdon’s picture

Status: Reviewed & tested by the community » Fixed

Thanks all! This has been committed to 7.x. Only about 4 more to go!

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

Anonymous’s picture

Issue summary: View changes

Removing myself from the author field so I can unfollow. --xjm