This is an to issue to catch various slightly off-topic or follow up sub-issues from the initiative to clean up core by removing t() from test assert messages.

#7 1797940-7-language-test-doc.patch841 bytesdcam
PASSED: [[SimpleTest]]: [MySQL] 40,349 pass(es).
[ View ]
#1 follow-up-1797940-1.patch1.43 KBLars Toomre
PASSED: [[SimpleTest]]: [MySQL] 41,704 pass(es).
[ View ]


new1.43 KB
PASSED: [[SimpleTest]]: [MySQL] 41,704 pass(es).
[ View ]

This is follow up to review of #1797360: Remove t() from assertion messages in tests for the language module. LanguageUrlRewritingTest::checkURL() unexpectedly used two assert messages and the helper function in that test did not have any @param directives. The attached patch rectifies that.

Status:Active» Needs review

Helps to set the right status!

In reading through many test files, I have also noticed inconsistent formatting as well as the use of t() in getInfo() functions. This is invariance with and should be addressed once the bulk of the remove t() assert issues have been committed.

Rather than filing one followup, each unrelated change should have its own issue with a descriptive title. So the docs fix in #1 for example can be filed as an independent documentation issue, and removing t() from getInfo() should be another separate issue, etc.

OK @xjm... I find it helpful to catch all for notes while doing the primary issue and then when have a little more time, to move them out to full follow up issues.

Title:Follow up to remove t() from test assertsFix docs for helper method in language tests
Version:8.x-dev» 7.x-dev
Component:base system» documentation
Status:Needs review» Patch (to be ported)

Yes, each follow-up needs its own issue. Fixing the title for this one, and I've committed it to 8.x. Presumably this method also exists in 7.x somewhere and this needs to be ported?

Status:Patch (to be ported)» Needs review
new841 bytes
PASSED: [[SimpleTest]]: [MySQL] 40,349 pass(es).
[ View ]

Backported #1 to D7.

Assigned:Unassigned» jhodgdon
Status:Needs review» Reviewed & tested by the community

Thanks! I'll get this committed ASAP.

Assigned:jhodgdon» Unassigned
Status:Reviewed & tested by the community» Fixed

Committed to 7.x -- thanks again!

Status:Fixed» Closed (fixed)

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