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

BUGFIX Fixed regression of dev/tests/all running a coverage report instead of just unit tests

Browse Source 
git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/branches/2.4@111091 467b73ca-7a2a-4603-9d3b-597d59a354a9
This commit is contained in:
Sean Harvey 2010-09-22 02:12:45 +00:00 committed by Sam Minnee
parent aa5eacc6ff
commit 3c51c0cd71
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. * Run test classes that should be run with every commit.
* Currently excludes PhpSyntaxTest * Currently excludes PhpSyntaxTest
*/ */
function all($coverage = false) { function all($request, $coverage = false) {
ManifestBuilder::load_test_manifest(); ManifestBuilder::load_test_manifest();
$tests = ClassInfo::subclassesFor('SapphireTest'); $tests = ClassInfo::subclassesFor('SapphireTest');
array_shift($tests); array_shift($tests);
@ -100,7 +100,7 @@ class TestRunner extends Controller {
if(!$reflection->isInstantiable()) unset($tests[$class]); 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); $this->runTests($tests);
} }
/** /**
* Browse all enabled test cases in the environment * Browse all enabled test cases in the environment
*/ */
@ -156,9 +154,9 @@ class TestRunner extends Controller {
/** /**
* Run a coverage test across all modules * Run a coverage test across all modules
*/ */
function coverageAll() { function coverageAll($request) {
ManifestBuilder::load_all_classes(); ManifestBuilder::load_all_classes();
$this->all(true); $this->all($request, true);
} }
/** /**