Problem/Motivation

Replace the deprecated assertEqual with assertEquals()

Proposed resolution

Replace assertEqual with assertEquals as per the drupal standards.

CommentFileSizeAuthor
#7 3045247-7.patch12.24 KBidebr
#5 deprecated-3045247-5.patch5.87 KBthalles
#2 deprecated-3045247.patch12.09 KBmithun s

Comments

Mithun S created an issue. See original summary.

mithun s’s picture

mithun s
Primary language English
Location Bangalore
commented
Status: Active » Needs review
StatusFileSize
new deprecated-3045247.patch12.09 KB

Please review the patch applied.

Status: Needs review » Needs work

The last submitted patch, 2: deprecated-3045247.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

voleger’s picture

voleger
he/his
Primary language Ukrainian
Location Ukraine, Rivne
commented
Parent issue: » #3042568: Drupal 9 Deprecated Code Report for Token
thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
Status: Needs work » Needs review
StatusFileSize
new deprecated-3045247-5.patch5.87 KB

Apparently some assertEqual still should not be replaced

Status: Needs review » Needs work

The last submitted patch, 5: deprecated-3045247-5.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

idebr’s picture

idebr commented
Status: Needs work » Needs review
StatusFileSize
new 3045247-7.patch12.24 KB

New patch after #3061605: Convert simpletest tests to phpunit/browser tests was committed.

  • Berdir committed 7613a7f on 8.x-1.x authored by idebr 
    Issue #3045247 by Mithun S, thalles, idebr: Replace deprecated...
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Fixed

Thanks, looks good now that we actually have phpunit tests ;)

Status: Fixed » Closed (fixed)

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