Problem/Motivation

As title.

Proposed resolution

For example:


- $this->assertTrue(stristr($top_form_elements[0]->getText(), 'Title field is required.') !== FALSE);
+ $this->assertStringContainsString('Title field is required.', $top_form_elements[0]->getText());

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 3155761-2.patch968 bytesHardik_Patel_12
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Hardik_Patel_12 created an issue. See original summary.

Hardik_Patel_12’s picture

Status: Active » Needs review
FileSize
968 bytes

Kindly review a patch.

longwave’s picture

Status: Needs review » Reviewed & tested by the community

Only one instance of stristr() in assertions and this one doesn't look to need to be case insensitive, so this is good to go.

  • catch committed 207fb2b on 9.1.x
    Issue #3155761 by Hardik_Patel_12, longwave: Replace assertions...
catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 9.1.x, thanks!

Status: Fixed » Closed (fixed)

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