Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.modules/php/php.test:
* incorrect array structure (fatal errors)
modules/simpletest/tests/xmlrpc.test:
* incorrectly labels an assert
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | 493296-fixes.patch | 764 bytes | boombatower |
| drupal_test_fixes.patch | 1.39 KB | jrchamp |
Comments
Comment #1
dries commentedHow come these fatal errors were not reported?
Comment #2
jrchamp commentedMaybe my setup is strange, but I enabled the simpletest module and tried to do a run through of all of the tests. It ran out of memory a couple times, so I attempted to continue from the last completed test.
Running the first PHP test without the array structure changes yields:
Fatal error: Unsupported operand types in modules/simpletest/drupal_web_test_case.php on line 679
Clicking continue yields the following (with no failed tests, just "The test run did not successfully finish."):
Undefined offset: 0 Notice drupal_web_test_case.php 679 DrupalWebTestCase->drupalCreateNode()
Comment #3
yched commentedIt's a know problem that fatal errors in tests result in tests simply being skipped (less 'passes'), but no fail or warning.
#443154: Fatal errors in tests not reported as failures
Comment #4
boombatower commentedSeems we should fix the issue referenced in #3 and re-evaluate this issue at that point.
Comment #5
jrchamp commented@boombatower: Alright, so you plan to use the existing PHP filter fatal error as the test mechanism for the issue referenced in #3?
Comment #6
boombatower commentedI suppose that is right, once other is ready it will fail due to these...so we can either patch them all in one shot in other issue or patch these now as long as it doesn't fail.
Let bot review...
Comment #8
boombatower commentedFirst part was already committed.
Comment #9
webchickCommitted. Thanks!