revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2025-02-16 16:58:38 +01:00
Code Issues Releases Wiki Activity

Fix lint tests after D3756

Browse source 
Test Plan:
Threw normal exception, saw failed test.
Threw usage exception, saw skipped test.

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D3761
This commit is contained in:
vrana 2012-10-20 06:16:19 -07:00
parent 7cb3551dc7
commit 262423b5a8
1 changed files with 10 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
src/lint/linter/__tests__/ArcanistLinterTestCase.php
View file

@ -100,10 +100,18 @@ abstract class ArcanistLinterTestCase extends ArcanistTestCase {
} catch (ArcanistPhutilTestTerminatedException $ex) { } catch (ArcanistPhutilTestTerminatedException $ex) {
throw $ex; throw $ex;
} catch (ArcanistUsageException $ex) {
$this->assertSkipped($ex->getMessage());
} catch (Exception $exception) { } catch (Exception $exception) {
$caught_exception = true; $caught_exception = true;
if ($exception instanceof PhutilAggregateException) {
$caught_exception = false;
foreach ($exception->getExceptions() as $ex) {
if ($ex instanceof ArcanistUsageException) {
$this->assertSkipped($ex->getMessage());
} else {
$caught_exception = true;
}
}
}
$exception_message = $exception->getMessage()."\n\n". $exception_message = $exception->getMessage()."\n\n".
$exception->getTraceAsString(); $exception->getTraceAsString();
} }