Skip to content

Commit 8ead9df

Browse files
committedFeb 9, 2018
included coverage for tests run in php-cli process
1 parent 78b46f6 commit 8ead9df

File tree

1 file changed

+16
-16
lines changed

1 file changed

+16
-16
lines changed
 

‎tests/RESTFul/TestBase.php

+16-16
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,19 @@
99
abstract class TestBase extends \PHPUnit_Framework_TestCase {
1010

1111
private static $errorLogDir = null;
12+
private static $coverage = null;
1213

1314
static function setUpBeforeClass(){
1415
parent::setUpBeforeClass();
1516

17+
// generate coverage for tests run in php-cli process
18+
if(defined('COVERAGE_DIR')){
19+
$filter = new \PHP_CodeCoverage_Filter();
20+
$filter->addDirectoryToWhitelist(dirname(__FILE__).'/../../src');
21+
self::$coverage = new \PHP_CodeCoverage(null,$filter);
22+
self::$coverage->start('testiCircleAccounts');
23+
}
24+
1625
// create a temporary error log directory
1726
$errorLogDir = sys_get_temp_dir().'/php-platform/restful/errors/'.microtime(true);
1827
mkdir($errorLogDir,0777,true);
@@ -23,28 +32,12 @@ static function setUpBeforeClass(){
2332
// clear caches
2433
SettingsCache::getInstance()->reset();
2534

26-
// build the routes - with coverage
27-
if(defined('APP_COVERAGE') && APP_COVERAGE == "true"){
28-
$filter = new \PHP_CodeCoverage_Filter();
29-
$filter->addDirectoryToWhitelist(dirname(__FILE__).'/../../src');
30-
31-
$coverage = new \PHP_CodeCoverage(null,$filter);
32-
$coverage->start('testRESTful');
33-
}
34-
3535
Build::run();
3636

3737
/**
3838
* @desc HACK : same file is used by SettingsCache , tests are run from root user and apache is run from www-data , causing permission issues to access this shared cache file
3939
*/
4040
chmod(sys_get_temp_dir().'/settingscache236512233125', 0777);
41-
42-
if(isset($coverage)){
43-
$coverage->stop();
44-
$writer = new \PHP_CodeCoverage_Report_PHP();
45-
$coverageFileName = 'coverage_build.php';
46-
$writer->process($coverage, COVERAGE_DIR.'/'.$coverageFileName);
47-
}
4841
}
4942

5043
function setUp(){
@@ -88,6 +81,13 @@ function assertContainsAndClearLog($message){
8881
static function tearDownAfterClass(){
8982
// delete error log directory
9083
rmdir(self::$errorLogDir);
84+
85+
if(defined('COVERAGE_DIR')){
86+
self::$coverage->stop();
87+
$writer = new \PHP_CodeCoverage_Report_PHP();
88+
$coverageFileName = md5(microtime()).'.php';
89+
$writer->process(self::$coverage, COVERAGE_DIR.'/'.$coverageFileName);
90+
}
9191
}
9292

9393
}

0 commit comments

Comments
 (0)