tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 12:05:37 +00:00
Code Issues Projects Releases Wiki Activity

BUGFIX Fixed regression of dev/tests/all running a coverage report instead of just unit tests (from r111091)

Browse Source 
git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/trunk@112892 467b73ca-7a2a-4603-9d3b-597d59a354a9
This commit is contained in:
Sam Minnee 2010-10-19 03:44:52 +00:00
parent 0200c75ca0
commit 0d7d983c14
1 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
dev/TestRunner.php
View File

@ -86,7 +86,7 @@ class TestRunner extends Controller {
* Run test classes that should be run with every commit.
* Currently excludes PhpSyntaxTest
*/
function all($coverage = false) {
function all($request, $coverage = false) {
ManifestBuilder::load_test_manifest();
$tests = ClassInfo::subclassesFor('SapphireTest');
array_shift($tests);
@ -100,7 +100,7 @@ class TestRunner extends Controller {
if(!$reflection->isInstantiable()) unset($tests[$class]);
}
$this->runTests($tests, $coverage);
$this->runTests($tests, false);
}
/**
@ -117,10 +117,8 @@ class TestRunner extends Controller {
}
$this->runTests($tests);
}
/**
* Browse all enabled test cases in the environment
*/
@ -156,9 +154,9 @@ class TestRunner extends Controller {
/**
* Run a coverage test across all modules
*/
function coverageAll() {
function coverageAll($request) {
ManifestBuilder::load_all_classes();
$this->all(true);
$this->all($request, true);
}
/**