Replace assertions involving calls to empty() with assertEmpty()/assertNotEmpty()/assertArrayNotHasKey()

Created on 19 May 2020, almost 5 years ago
Updated 13 February 2025, about 2 months ago

Problem/Motivation

As title.

Proposed resolution

For example:

-    $this->assertFalse(!empty($view->build_info['pre_render_called']), 'Make sure hook_views_pre_render is not called for the cached view.');
+    $this->assertEmpty($view->build_info['pre_render_called']);

Not sure about whether to keep or drop the $message arguments here, though.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

πŸ“Œ Task
Status

Needs review

Version

4.0

Component

Code

Created by

πŸ‡ΊπŸ‡¦Ukraine alexshipilov

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

Merge Requests

Comments & Activities

Not all content is available!

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

Production build 0.71.5 2024