From fba62c0d9d6403ae549d7e0698e43c69f88333fc Mon Sep 17 00:00:00 2001 From: Joshua Spence Date: Sat, 3 Jan 2015 23:52:45 +1100 Subject: [PATCH] Remove `arcconfig` settings from `ArcanistLinterTestCase` Summary: This doesn't make too much sense anymore as all modern linters are configured via the `.arclint` file rather than `.arcconfig`. Test Plan: `grep`ped to make sure this option isn't used. Reviewers: epriestley, #blessed_reviewers Reviewed By: epriestley, #blessed_reviewers Subscribers: Korvin, epriestley Differential Revision: https://secure.phabricator.com/D11173 --- src/lint/linter/__tests__/ArcanistLinterTestCase.php | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/src/lint/linter/__tests__/ArcanistLinterTestCase.php b/src/lint/linter/__tests__/ArcanistLinterTestCase.php index e60bd0d9..b933171b 100644 --- a/src/lint/linter/__tests__/ArcanistLinterTestCase.php +++ b/src/lint/linter/__tests__/ArcanistLinterTestCase.php @@ -49,7 +49,6 @@ abstract class ArcanistLinterTestCase extends ArcanistPhutilTestCase { 'hook' => 'optional bool', 'config' => 'optional wild', 'path' => 'optional string', - 'arcconfig' => 'optional map', )); $exception = null; @@ -65,15 +64,10 @@ abstract class ArcanistLinterTestCase extends ArcanistPhutilTestCase { $dir = dirname($full_path); $path = basename($full_path); - $config_file = null; - $arcconfig = idx($config, 'arcconfig'); - if ($arcconfig) { - $config_file = json_encode($arcconfig); - } $working_copy = ArcanistWorkingCopyIdentity::newFromRootAndConfigFile( $dir, - $config_file, + null, 'Unit Test'); $configuration_manager = new ArcanistConfigurationManager(); $configuration_manager->setWorkingCopyIdentity($working_copy);