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.Currently the logic in scheduler_node_presave() is not fully tested. We should add a unit test for this.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 7x_hook_node_presave_coverage.png | 197.14 KB | jonathan1055 |
| #1 | 2112867-unit-test-for-saving-scheduld-notes-0.patch | 5.25 KB | legovaer |
Comments
Comment #1
legovaerI have a patch which covers 95% of that function. Definitely needs a review!
Comment #3
jonathan1055 CreditAttribution: jonathan1055 as a volunteer commentedHi legovaer,
Since Oct 2013 (when this issue was raised) we have added quite a few more tests and using your excellent prototype test coverage analysis it seems that only two lines of hook_node_preave() are not currently tested:
However, if we are going to write unit tests we should probably start it in 8.x first.