At first I thought that putting tests inside app logic is a no no but on the other hand it could be lowering the bar to test things for more junior devs.
I see it as kind of the same thing as putting server actions inside app logic which is something we now apparently do regularly and people love it (see tanstack/start)