Contrib.social

Replace assert* involving an instanceof operator with assertInstanceOf()/assertNotInstanceOf()

Open on Drupal.org →
Created on 22 May 2020, almost 5 years ago
Updated 26 September 2023, over 1 year ago

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

📌 Task
Status

Fixed

Version

3.0

Component

Code

Created by

🇺🇦Ukraine alexshipilov

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024