revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-11-22 14:52:40 +01:00
Code Issues Releases Wiki Activity

PHPUnit test name handling improvements

Browse source 
Summary:
Better test name handling for tests with data sets.
Instead of showing test name with full data set, show it's id/name,
e.g. `testConstructor with data set #1`

Test Plan: - Try running tests with data sets

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Koolvin

Differential Revision: https://secure.phabricator.com/D2535
This commit is contained in:
Aurelijus 2012-05-22 17:24:01 +02:00 committed by epriestley
parent 4f739014ae
commit 6e87de7304
1 changed files with 14 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
src/unit/engine/phpunit/PhpunitTestEngine.php
View file

@ -188,7 +188,14 @@ final class PhpunitTestEngine extends ArcanistBaseUnitTestEngine {
}
}
$name = substr($event->test, strlen($event->suite) + 2);
$pos = strpos($event->suite, '::');
if ($pos === false) {
$name = substr($event->test, strlen($event->suite) + 2);
} else {
$name = substr($event->test, $pos + 2);
$name = preg_replace('/ \(array\(.*\)\)/', '', $name);
}
$result = new ArcanistUnitTestResult();
$result->setName($name);
$result->setResult($status);
@ -212,7 +219,12 @@ final class PhpunitTestEngine extends ArcanistBaseUnitTestEngine {
private function readCoverage($path) {
$test_results = Filesystem::readFile($path);
if (empty($test_results)) {
throw new Exception('Clover coverage XML report file is empty');
throw new Exception('Clover coverage XML report file is empty, '
. 'it probably means that phpunit failed to run tests. '
. 'Try running arc unit with --trace option and then run '
. 'generated phpunit command yourself, you might get the '
. 'answer.'
);
}
$coverage_dom = new DOMDocument();