Minor cleanup of linter testing code.

Summary:
Just some minor cleaning up, including:

- Removing old annotations for Diviner.
- Renaming some `lint-test` files so that the directory name bears a closer resemblance to that of the linter.
- Remove some useless `return` statements.

Test Plan: `arc unit`.

Reviewers: epriestley, #blessed_reviewers

Reviewed By: epriestley, #blessed_reviewers

Subscribers: epriestley, Korvin

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

src/lint/linter/__tests__/ArcanistArcanistLinterTestCase.php

@@ -1,8 +1,5 @@
 <?php
 
-/**
- * @group testcase
- */
 abstract class ArcanistArcanistLinterTestCase extends ArcanistLinterTestCase {
 
   protected function getLink($method) {

src/lint/linter/__tests__/ArcanistCpplintLinterTestCase.php

@@ -1,15 +1,11 @@
 <?php
-/**
+
- * Test cases for @{class:ArcanistCpplintLinter}.
- *
- * @group testcase
- */
 final class ArcanistCpplintLinterTestCase
   extends ArcanistArcanistLinterTestCase {
 
   public function testCpplintLint() {
     return $this->executeTestsInDirectory(
-      dirname(__FILE__).'/cpp/',
+      dirname(__FILE__).'/cpplint/',
       new ArcanistCpplintLinter());
   }
 

src/lint/linter/__tests__/ArcanistFlake8LinterTestCase.php

@@ -1,14 +1,10 @@
 <?php
-/**
+
- * Test cases for @{class:ArcanistFlake8Linter}.
- *
- * @group testcase
- */
 final class ArcanistFlake8LinterTestCase
   extends ArcanistArcanistLinterTestCase {
 
   public function testFlake8Lint() {
-    return $this->executeTestsInDirectory(
+    $this->executeTestsInDirectory(
       dirname(__FILE__).'/flake8/',
       new ArcanistFlake8Linter());
   }

src/lint/linter/__tests__/ArcanistLinterTestCase.php

@@ -2,8 +2,6 @@
 
 /**
  * Facilitates implementation of test cases for @{class:ArcanistLinter}s.
- *
- * @group testcase
  */
 abstract class ArcanistLinterTestCase extends ArcanistPhutilTestCase {
 
@@ -154,7 +152,7 @@ abstract class ArcanistLinterTestCase extends ArcanistPhutilTestCase {
       $char = $message->getChar();
       $code = $message->getCode();
       $name = $message->getName();
-      $message_key = $sev.":".$line.":".$char;
+      $message_key = $sev.':'.$line.':'.$char;
       $message_map[$message_key] = $message;
       $seen[] = $message_key;
       $raised[] = "  {$sev} at line {$line}, char {$char}: {$code} {$name}";
@@ -173,7 +171,7 @@ abstract class ArcanistLinterTestCase extends ArcanistPhutilTestCase {
     $seen   = array_fill_keys($seen, true);
 
     if (!$raised) {
-      $raised = array("No messages.");
+      $raised = array('No messages.');
     }
     $raised = "Actually raised:\n".implode("\n", $raised);
 
@@ -186,7 +184,6 @@ abstract class ArcanistLinterTestCase extends ArcanistPhutilTestCase {
     }
 
     foreach (array_diff_key($seen, $expect) as $surprising => $ignored) {
-
       $message = $message_map[$surprising];
       $message_info = $message->getDescription();
 
@@ -205,6 +202,6 @@ abstract class ArcanistLinterTestCase extends ArcanistPhutilTestCase {
     $this->assertEqual(
       $expected,
       $actual,
-      "File as patched by lint did not match the expected patched file.");
+      'File as patched by lint did not match the expected patched file.');
   }
 }

src/lint/linter/__tests__/ArcanistNoLintTestCase.php

@@ -1,13 +1,6 @@
 <?php
 
 /**
- * Test for @{class:ArcanistNoLintLinter}.
+ * Not a real test... meant to fail lint if @nolint is not respected.
- *
- * Not a real test... meant to fail lint
- * if @nolint is not respected.
- *
- * @group testcase
  */
-final class ArcanistNoLintTestCaseMisnamed extends ArcanistLinterTestCase {
+final class ArcanistNoLintTestCaseMisnamed extends ArcanistLinterTestCase {}
-
-}

src/lint/linter/__tests__/ArcanistPHPCSLinterTestCase.php

@@ -3,7 +3,7 @@
 final class ArcanistPHPCSLinterTestCase extends ArcanistArcanistLinterTestCase {
 
   public function testPHPCSLint() {
-    return $this->executeTestsInDirectory(
+    $this->executeTestsInDirectory(
       dirname(__FILE__).'/phpcs/',
       new ArcanistPhpcsLinter());
   }

src/lint/linter/__tests__/ArcanistPhutilXHPASTLinterTestCase.php

@@ -1,8 +1,5 @@
 <?php
 
-/**
- * @group testcase
- */
 final class ArcanistPhutilXHPASTLinterTestCase
   extends ArcanistArcanistLinterTestCase {
 
@@ -12,7 +9,7 @@ final class ArcanistPhutilXHPASTLinterTestCase
       'deprecated_function' => 'This function is most likely deprecated.',
     ));
 
-    return $this->executeTestsInDirectory(
+    $this->executeTestsInDirectory(
       dirname(__FILE__).'/phlxhp/',
       $linter);
   }

src/lint/linter/__tests__/ArcanistPuppetLintLinterTestCase.php

@@ -4,7 +4,7 @@ final class ArcanistPuppetLintLinterTestCase
   extends ArcanistArcanistLinterTestCase {
 
   public function testPuppetLintLinter() {
-    return $this->executeTestsInDirectory(
+    $this->executeTestsInDirectory(
       dirname(__FILE__).'/puppet-lint/',
       new ArcanistPuppetLintLinter());
   }

src/lint/linter/__tests__/ArcanistPyFlakesLinterTestCase.php

@@ -4,7 +4,7 @@ final class ArcanistPyFlakesLinterTestCase
   extends ArcanistArcanistLinterTestCase {
 
   public function testPyflakesLinter() {
-    return $this->executeTestsInDirectory(
+    $this->executeTestsInDirectory(
       dirname(__FILE__).'/pyflakes/',
       new ArcanistPyFlakesLinter());
   }

src/lint/linter/__tests__/ArcanistRubyLinterTestCase.php

@@ -1,14 +1,9 @@
 <?php
 
-/**
- * Test cases for @{class:ArcanistRubyLinter}.
- *
- * @group testcase
- */
 final class ArcanistRubyLinterTestCase extends ArcanistArcanistLinterTestCase {
 
   public function testRubyLint() {
-    return $this->executeTestsInDirectory(
+    $this->executeTestsInDirectory(
       dirname(__FILE__).'/ruby/',
       new ArcanistRubyLinter());
   }

src/lint/linter/__tests__/ArcanistSpellingLinterTestCase.php

@@ -1,10 +1,5 @@
 <?php
 
-/**
- * Test cases for @{class:ArcanistSpellingLinter}.
- *
- * @group testcase
- */
 final class ArcanistSpellingLinterTestCase
   extends ArcanistArcanistLinterTestCase {
 
@@ -13,7 +8,7 @@ final class ArcanistSpellingLinterTestCase
     $linter->addPartialWordRule('supermn', 'superman');
     $linter->addWholeWordRule('batmn', 'batman');
 
-    return $this->executeTestsInDirectory(
+    $this->executeTestsInDirectory(
       dirname(__FILE__).'/spelling/',
       $linter);
   }

src/lint/linter/__tests__/ArcanistTextLinterTestCase.php

@@ -1,14 +1,9 @@
 <?php
 
-/**
- * Test cases for @{class:ArcanistTextLinter}.
- *
- * @group testcase
- */
 final class ArcanistTextLinterTestCase extends ArcanistArcanistLinterTestCase {
 
   public function testTextLint() {
-    return $this->executeTestsInDirectory(
+    $this->executeTestsInDirectory(
       dirname(__FILE__).'/text/',
       new ArcanistTextLinter());
   }

src/lint/linter/__tests__/ArcanistXHPASTLinterTestCase.php

@@ -1,10 +1,5 @@
 <?php
 
-/**
- * Tests for @{class:ArcanistXHPASTLinter}.
- *
- * @group testcase
- */
 final class ArcanistXHPASTLinterTestCase
   extends ArcanistArcanistLinterTestCase {
 
@@ -17,7 +12,7 @@ final class ArcanistXHPASTLinterTestCase
           => ArcanistLintSeverity::SEVERITY_WARNING,
       ));
 
-    return $this->executeTestsInDirectory(
+    $this->executeTestsInDirectory(
       dirname(__FILE__).'/xhpast/',
       $linter);
   }