Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
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... ;-)
Proposed resolution
Investigate and fix that.
Remaining tasks
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#3 | fix_test_fail-2560609-3.patch | 527 bytes | giancarlosotelo |
Comments
Comment #2
miro_dietikerComment #3
giancarlosotelo CreditAttribution: giancarlosotelo at MD Systems GmbH 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 CreditAttribution: 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 CreditAttribution: juanse254 at MD Systems GmbH commentedComment #11
giancarlosotelo CreditAttribution: giancarlosotelo at MD Systems GmbH commented