php error in simpletest xml results [#2877081]

Some tests, like the OpenID module's OpenIDRegistrationTestCase in openid.test, have only a simple function name in the result function property. So this line
list($class, $name) = explode('->', $result->function, 2);
produces the error

Notice: Undefined offset: 1 in simpletest_script_reporter_write_xml_results() (line 616 of /path/to/docroot/scripts/run-tests.sh).

Comment	File	Size	Author
#2	simpletest-undefined-offset-2877081-1.patch	737 bytes	brad.bulger
#2	7.x: PHP 7.2 & MySQL 5.5 2,050 pass

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

9 May 2017 at 17:54

brad.bulger created an issue. See original summary.

Comment #2

brad.bulger CreditAttribution: brad.bulger commented 9 May 2017 at 17:57

File	Size
simpletest-undefined-offset-2877081-1.patch	737 bytes
7.x: PHP 7.2 & MySQL 5.5 2,050 pass

Of the variables being defined by that line of code, only one is used, and only in the subsequent line:
$case->setAttribute('name', $name);
So we can use an array variable instead of the list() assignment and avoid the error.