From ef598794a8123ea6dcd92374ad056b4d68e427e1 Mon Sep 17 00:00:00 2001 From: Joshua Spence Date: Thu, 9 Jan 2014 09:25:35 -0800 Subject: [PATCH] Skip unit tests if `ArcanistLinter::getCacheVersion` throws an `ArcanistUsageException` Summary: Fixes T4288 Test Plan: N/A Reviewers: epriestley, #blessed_reviewers Reviewed By: epriestley CC: Korvin, epriestley, aran Maniphest Tasks: T4288 Differential Revision: https://secure.phabricator.com/D7913 --- src/lint/linter/__tests__/ArcanistLinterTestCase.php | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/lint/linter/__tests__/ArcanistLinterTestCase.php b/src/lint/linter/__tests__/ArcanistLinterTestCase.php index 2d273eb0..24d385ff 100644 --- a/src/lint/linter/__tests__/ArcanistLinterTestCase.php +++ b/src/lint/linter/__tests__/ArcanistLinterTestCase.php @@ -119,6 +119,8 @@ abstract class ArcanistLinterTestCase extends ArcanistPhutilTestCase { $caught_exception = true; } } + } else if ($exception instanceof ArcanistUsageException) { + $this->assertSkipped($exception->getMessage()); } $exception_message = $exception->getMessage()."\n\n". $exception->getTraceAsString();