The attached patch allows for integration with the spikephpcoverage framework. Basically, it checks to see if a subdirectory of the simpletest module "spikephpcoverage" exists, and if it does, starts it up, and has it generate a report on which lines of code were called (and which were not) during the test. Requires the spikephpcoverage package from http://developer.spikesource.com/wiki/index.php/Projects:phpcoverage to be installed and set up properly.

I was not able to run all tests locally without apache crashing, but a sample of the report generated by a single test can be found at http://cwgordon.com/simpletest/coverage_example/details/index.html.

There should be no effect upon anyone not attempting to determine the test coverage.

CommentFileSizeAuthor
#5 newhook.patch1.28 KBRok Žlender
#5 php_coverage.zip607.97 KBRok Žlender
#2 simpletest_coverage_02.patch1.9 KBcwgordon7
simpletest_coverage.patch1.77 KBcwgordon7

Comments

moshe weitzman’s picture

moshe weitzman commented

Looks nice. Uses xdebug for the heavy lifting. Thats good.

cwgordon7’s picture

cwgordon7 commented
StatusFileSize
new simpletest_coverage_02.patch1.9 KB

Coding standards.

**hangs head in shame**. SHAME!!!

Rok Žlender’s picture

Rok Žlender commented

What if we create a ST hook that would be run before test starts and after with run and stop params. This way we could move stuff like this which is not needed for regular tests into a diff module. This module can still be included with simpletest.

cwgordon7’s picture

cwgordon7 commented

Oh that would be completely AWESOME! Complete +1!

Rok Žlender’s picture

Rok Žlender commented
StatusFileSize
new php_coverage.zip607.97 KB
new newhook.patch1.28 KB

Ok code for new hook as a patch and new module that includes phpcoverage as a tar ball

boombatower’s picture

boombatower commented
Status: Needs review » Closed (duplicate)

http://drupal.org/node/252292