mirror of
https://we.phorge.it/source/arcanist.git
synced 2024-11-25 16:22:42 +01:00
Minor linter fixes
Summary: Self explanatory/ Test Plan: `arc lint` Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: aurelijus, Korvin, epriestley Differential Revision: https://secure.phabricator.com/D10943
This commit is contained in:
parent
4ca3463df6
commit
565a96e0ee
3 changed files with 3 additions and 6 deletions
|
@ -267,8 +267,7 @@ final class PhpunitTestEngine extends ArcanistUnitTestEngine {
|
||||||
if ($bin) {
|
if ($bin) {
|
||||||
if (Filesystem::binaryExists($bin)) {
|
if (Filesystem::binaryExists($bin)) {
|
||||||
$this->phpunitBinary = $bin;
|
$this->phpunitBinary = $bin;
|
||||||
}
|
} else {
|
||||||
else {
|
|
||||||
$this->phpunitBinary = Filesystem::resolvePath($bin, $project_root);
|
$this->phpunitBinary = Filesystem::resolvePath($bin, $project_root);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -121,8 +121,7 @@ final class PytestTestEngine extends ArcanistUnitTestEngine {
|
||||||
|
|
||||||
if (intval($line->getAttribute('hits')) == 0) {
|
if (intval($line->getAttribute('hits')) == 0) {
|
||||||
$coverage .= 'U';
|
$coverage .= 'U';
|
||||||
}
|
} else if (intval($line->getAttribute('hits')) > 0) {
|
||||||
else if (intval($line->getAttribute('hits')) > 0) {
|
|
||||||
$coverage .= 'C';
|
$coverage .= 'C';
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -68,8 +68,7 @@ EOTEXT
|
||||||
if (!$commits) {
|
if (!$commits) {
|
||||||
throw new ArcanistUsageException(
|
throw new ArcanistUsageException(
|
||||||
'This revision has not been committed yet!');
|
'This revision has not been committed yet!');
|
||||||
}
|
} else if (count($commits) > 1) {
|
||||||
else if (count($commits) > 1) {
|
|
||||||
throw new ArcanistUsageException(
|
throw new ArcanistUsageException(
|
||||||
'The revision you provided has multiple commits!');
|
'The revision you provided has multiple commits!');
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue