Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 292626

Replace assert*() involving equality comparison operators with assert(Not)(Equals|Same)

$
0
0

Problem/Motivation

As title

Proposed resolution

Regex for searching: >assert.*\(.*( == | === | != | !== )

example:

-    $this->assertTrue($output == $expected, new FormattableMarkup('Token recognized in string %string', ['%string' => $input]));
+    $this->assertEquals($expected, $output, "Token is expected to be replaced in string $input");

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet


Viewing all articles
Browse latest Browse all 292626

Trending Articles