Problem/Motivation

As title

Proposed resolution

Recommended regex for searching: assert\w+\(.+instanceof.+\)

Example:

-    $this->assertTrue($property instanceof StringInterface, 'Got the right wrapper fo the page.front property.');
+    $this->assertInstanceOf(StringInterface::class, $property, 'Got the right wrapper fo the page.front property.');

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alexshipilov created an issue. See original summary.

alexshipilov’s picture

alexshipilov’s picture

Assigned: alexshipilov » Unassigned
Status: Active » Needs review

  • bonus committed 3fe8231 on 8.x-2.x authored by alexshipilov
    Issue #3138281 by alexshipilov: Replace assert* involving an instanceof...
gg4’s picture

Status: Needs review » Fixed

Thanks!

Status: Fixed » Closed (fixed)

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