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

Add $projectRoot to PytestTestEngine

Browse source 
Summary:
Fixes T8912. Property `$project_root` was missing in `PytestTestEngine` class, resulting in
broken py.test wrapper. Also renaming the property so the linter is happy.

Test Plan: `arc unit --everything`

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: kparal, epriestley, Korvin

Maniphest Tasks: T8912

Differential Revision: https://secure.phabricator.com/D13698
This commit is contained in:
Lukas Sparrow 2015-07-24 05:11:32 -07:00 committed by epriestley
parent bd1da9da6c
commit 5fcf7b5a3b
1 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/unit/engine/PytestTestEngine.php
View file

@ -5,9 +5,11 @@
*/
final class PytestTestEngine extends ArcanistUnitTestEngine {
private $projectRoot;
public function run() {
$working_copy = $this->getWorkingCopy();
$this->project_root = $working_copy->getProjectRoot();
$this->projectRoot = $working_copy->getProjectRoot();
$junit_tmp = new TempFile();
$cover_tmp = new TempFile();
@ -25,7 +27,7 @@ final class PytestTestEngine extends ArcanistUnitTestEngine {
}
$future = new ExecFuture('coverage xml -o %s', $cover_tmp);
$future->setCWD($this->project_root);
$future->setCWD($this->projectRoot);
$future->resolvex();
return $this->parseTestResults($junit_tmp, $cover_tmp);
@ -39,7 +41,7 @@ final class PytestTestEngine extends ArcanistUnitTestEngine {
if ($this->getEnableCoverage() !== false) {
$cmd_line = csprintf(
'coverage run --source %s -m %C',
$this->project_root,
$this->projectRoot,
$cmd_line);
}