Remove support for skipping tests

Summary: Almost revert D2673 but leave the support for 'repeat'.

Test Plan: `arc help unit`.

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D2689

src/unit/engine/ArcanistBaseUnitTestEngine.php

@@ -25,7 +25,6 @@ abstract class ArcanistBaseUnitTestEngine {
 
   private $workingCopy;
   private $paths;
-  private $skipFiles;
   private $arguments = array();
   protected $diffID;
   private $enableAsyncTests;
@@ -55,15 +54,6 @@ final public function getPaths() {
     return $this->paths;
   }
 
-  final public function setSkipFiles(array $paths) {
-    $this->skipFiles = $paths;
-    return $this;
-  }
-
-  final public function getSkipFiles() {
-    return $this->skipFiles;
-  }
-
   final public function setArguments(array $arguments) {
     $this->arguments = $arguments;
     return $this;

src/unit/engine/PhutilUnitTestEngine.php

@@ -77,13 +77,6 @@ public function run() {
       } while ($library_path != '.');
     }
 
-    $skip = array();
-    foreach ($this->getSkipFiles() as $skip_path) {
-      $skip_path = Filesystem::resolvePath($skip_path);
-      $skip_path = Filesystem::readablePath($skip_path, $library_root);
-      $skip[$skip_path] = true;
-    }
-
     // Look for any class that extends ArcanistPhutilTestCase inside a
     // __tests__ directory in any parent directory of every affected file.
     //
@@ -107,9 +100,6 @@ public function run() {
         ->selectAndLoadSymbols();
 
       foreach ($symbols as $symbol) {
-        if (isset($skip[$symbol['where']])) {
-          continue;
-        }
         $run_tests[$symbol['name']] = true;
       }
     }

src/workflow/ArcanistUnitWorkflow.php

@@ -68,11 +68,6 @@ public function getArguments() {
         'help' =>
           "Override configured unit engine for this project."
       ),
-      'skip' => array(
-        'param' => 'path',
-        'help' => "Specify file paths that will be ignored.",
-        'repeat' => true,
-      ),
       'coverage' => array(
         'help' => 'Always enable coverage information.',
         'conflicts' => array(
@@ -132,7 +127,6 @@ public function run() {
     $this->engine = newv($engine_class, array());
     $this->engine->setWorkingCopy($working_copy);
     $this->engine->setPaths($paths);
-    $this->engine->setSkipFiles($this->getArgument('skip'));
     $this->engine->setArguments($this->getPassthruArgumentsAsMap('unit'));
 
     $enable_coverage = null; // Means "default".