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.
While working on #3057148: Add deletePath() method to CollectPathsEvent class we found that Some static event classes test coverage was not as big as desired.
So here i'm opening this issue to increae the test coverage of those classes.
Working on it while we are in Dev Days Transylvania.
Comment | File | Size | Author |
---|---|---|---|
#4 | interdiff-3060959-3-4.txt | 2.37 KB | samuel.mortenson |
#4 | 3060959-4.patch | 15.81 KB | samuel.mortenson |
|
Comments
Comment #2
plopescHere is a first patch adding tests to Tome Static events following the same pattern in
CollectPathsEventTest
class.Comment #3
plopescoh, it seems that
Drupal\Tests\token\Kernel\KernelTestBase
autocomplete issue bit me again. :/Comment #4
samuel.mortensonFixed up some coding standards - but I like this! @plopesc are you planning on doing any additional work on this? If not I'll commit before 1.0. Thanks!
Comment #5
plopescYour fixes look totally legit. I just made some mistakes copying & pasting stuff :)
I had no plan to add more test related to event, so from my poin of view, this isue is ready to go.
Thanks!
Comment #7
samuel.mortenson