Problem/Motivation

As title

Proposed resolution

Remove redundant assertion messages at the same time.

Example:

-      $this->assertTrue(is_object($node), 'Node is an object');
+      $this->assertIsObject($node);

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 purge-replace_is_object_with_assertIsObject-3139376-2.patch4.22 KBalexshipilov

Issue fork purge-3139376

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

alexshipilov created an issue. See original summary.

alexshipilov’s picture

alexshipilov commented
Title: Replace assertions involving calls to is_object() with assertIsObject()/assertIsObject() » Replace assertions involving calls to is_object() with assertIsObject()/assertIsNotObject()
StatusFileSize
new purge-replace_is_object_with_assertIsObject-3139376-2.patch4.22 KB
alexshipilov’s picture

alexshipilov commented
Assigned: alexshipilov » Unassigned
Status: Active » Needs review

japerry made their first commit to this issue’s fork.

japerry opened merge request !72

  • japerry committed 8b44dbdd on 8.x-3.x 
    Resolve #3139376 "Replace assertions involving isObject"
japerry’s picture

japerry
Location KOMK
commented
Status: Needs review » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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