Since some recent commit, the MonitoringUITest testSensorListLimit is broken...
https://qa.drupal.org/pifr/test/742098
Fatal error: Call to undefined method Drupal\monitoring\Plugin\monitoring\SensorPlugin\ContentEntityAggregatorSensorPlugin::id() in /var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules/monitoring/src/Plugin/monitoring/SensorPlugin/ContentEntityAggregatorSensorPlugin.php on line 292
FATAL Drupal\monitoring\Tests\MonitoringUITest: test runner returned a non-zero error code (255).
Confusion: People claim this was passing... ;-)
Investigate and fix that.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | fix_test_fail-2560609-3.patch | 527 bytes | giancarlosotelo |
Comments
Comment #2
miro_dietikerComment #3
giancarlosotelo commentedI found the problem.
We are setting the default configuration(entity type => 'node') for the Content Entity Aggregator just when the ajax is called ( one of the recent commits) so for the UI works well always but trying to create a Content Entity sensor by coding doesn't works because the entity type is missing so I am adding the entity type in the test and now the UI test is green again :)
Comment #4
LKS90 commentedYep, the tests needed an update after the recent commits. I wish we had automated testing again, but what can we do... :D.
Comment #5
miro_dietikerFixed. Yeah, it was me... My local system currently doesn't like to run tests and i was too lazy to wait... :-)
Comment #9
juanse254 commentedComment #11
giancarlosotelo commented