ContentEntityBaseUnitTest.php includes a function call to "assertTRUE()" which should be "assertTrue()"

CommentFileSizeAuthor
#2 drupal-assert_true_trivial_patch-2850170.patch660 bytesaaronbauman

Comments

aaronbauman created an issue. See original summary.

aaronbauman’s picture

aaronbauman
he/him
commented
Status: Active » Needs review
StatusFileSize
new drupal-assert_true_trivial_patch-2850170.patch660 bytes
dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

This is indeed the only instance of assertTRUE or assertFALSE.

  • xjm committed 43de9e0 on 8.4.x 
    Issue #2850170 by aaronbauman, dawehner: Capitalization error for...

  • xjm committed 0b6be28 on 8.3.x 
    Issue #2850170 by aaronbauman, dawehner: Capitalization error for...
xjm’s picture

xjm
she/her
Primary language English
commented
Version: 8.4.x-dev » 8.3.x-dev
Status: Reviewed & tested by the community » Fixed

Thanks @dawehner for confirming that there are not other instances! Saved me checking that. :)

Since this is a simple no-op coding standards and testing fix, I also backported it to 8.3.x to avoid unnecessary merge conflicts.

Status: Fixed » Closed (fixed)

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