- Issue created by @mxr576
- 🇺🇸United States smustgrave
Seems like something that probably should have test coverage.
- Status changed to Needs work
3 days ago 2:05pm 21 August 2025 - 🇭🇺Hungary mxr576 Hungary
@smustgrave good idea, but unfortunately as I can tell, the Recipe Unpack feature was merged without covering this assertion originally, probably because assertions are for developers and testing assert()-s became significantly complicated recently: https://github.com/sebastianbergmann/phpunit/issues/5062
Admittedly I haven't checked if Drupal core has introduced any trick for covering any existing assert()-s with tests since this change, I remember and saw that @alexpot was also commented this GH issue.
So currently I am unsure if coverage is necessary or not, open for inputs.
- 🇭🇺Hungary mxr576 Hungary
FTR, the current test failure is unrelated: 🐛 [10.5.x] test DB transaction isolation set to read-committed breaks tests Active