TYPO3 Core

I'd say: It depends ... I guess there are situations where firing a query to assert DB state is fine.

But I've also seen places - especially in extbase tests like persistence stuff - that would benefit when they assert csv files instead, and I remember I already changed various of them towards this. Please go ahead and do more.