Firstly, providing a return message is pointless, since one never gets to see it. Secondly, there's no acknowledgement of the fact that any tests have been skipped.

Call diag() first, then return.