Description: ------------ A case of "useless tests" that I see from time to time is that developers write tests that execution SUT code without asserting -- thus not actually testing -- anything. The sniff should raise a warning when the following conditions are met: - Class has PHPUnit_Framework_TestCase as an ancestor - Method is public and has name test*() or the @test annotation - The method itself (and other methods called from it) does not call $this->assert*(), self::assert*(), or PHPUnit_Framework_Assert::assert*() and does not set up a mock object with expectations. Test script: --------------- . Expected result: ---------------- . Actual result: -------------- .

Would this be easier to implement in PHPUnit itself? IE, PHPUnit's already got easy ways to tell when you are running a method named test, count assertations, etc.

I have no implemented this in PHPUnit 3.5, so not sure if anyone wants this still implementing in PHP_CodeSniffer?

No need to add to PHPCS if PHPUnit does it. Thanks for sending that info through.