Add phpstan, psalm and phan aliases tests and fix a little more (remove unecessary comments)

Author: SerafimArts

libs/phpdoc/tests/DocBlock/Grammar/DescriptionGrammarRuleTest.php

@@ -48,10 +48,6 @@ public function readsThePlainTrailingText(): void
         self::assertSame('Some description text', (string) $description);
     }
 
-    /**
-     * The description delegate resolves inline tags, so a `{@...}` becomes a
-     * {@see TaggedDescription}.
-     */
     #[Test]
     public function keepsInlineTags(): void
     {
@@ -61,9 +57,6 @@ public function keepsInlineTags(): void
         self::assertSame('see {@link X}', (string) $description);
     }
 
-    /**
-     * The whole remainder is consumed regardless of the whitespace it contains.
-     */
     #[Test]
     public function consumesTheEntireRemainder(): void
     {

libs/phpdoc/tests/DocBlock/Grammar/ReferenceGrammarRuleTest.php

@@ -25,9 +25,6 @@ protected function rule(): ReferenceCombinator
     }
 
     /**
-     * Each reference, mapped to the expected reference class and the string it
-     * stringifies back to.
-     *
      * @return iterable<string, array{string, class-string<ReferenceInterface>, string}>
      */
     public static function referenceDataProvider(): iterable
@@ -64,9 +61,6 @@ public function classMethodExposesClassAndName(): void
         self::assertSame('foo', $reference->name);
     }
 
-    /**
-     * Only the reference is consumed, the rest stays for the next rule.
-     */
     #[Test]
     public function stopsAtTheFirstWhitespace(): void
     {
@@ -78,10 +72,6 @@ public function stopsAtTheFirstWhitespace(): void
         self::assertSame(4, $cursor->offset);
     }
 
-    /**
-     * A multi-part reference is consumed up to its last token, leaving the
-     * trailing text untouched.
-     */
     #[Test]
     public function stopsAfterTheReference(): void
     {

libs/phpdoc/tests/DocBlock/Grammar/TypeGrammarRuleTest.php

@@ -31,9 +31,6 @@ public function producesANamedType(): void
         self::assertSame('array', (string) $statement->type->name);
     }
 
-    /**
-     * The parsed type keeps the exact text it was read from.
-     */
     #[Test]
     public function preservesTheSourceText(): void
     {
@@ -60,10 +57,6 @@ public function producesAUnionType(): void
         self::assertInstanceOf(UnionTypeNode::class, $statement->type);
     }
 
-    /**
-     * Only the type is consumed, the trailing description stays for the next
-     * rule.
-     */
     #[Test]
     public function stopsAfterTheType(): void
     {
@@ -75,10 +68,6 @@ public function stopsAfterTheType(): void
         self::assertSame(11, $cursor->offset);
     }
 
-    /**
-     * The consumed offset is rebased onto the source when the cursor does not
-     * start at zero.
-     */
     #[Test]
     public function respectsTheCursorBase(): void
     {

libs/phpdoc/tests/DocBlock/Grammar/VariableGrammarRuleTest.php

@@ -36,9 +36,6 @@ public function returnsTheNameWithoutTheDollar(string $input, string $expected):
         self::assertSame($expected, $name);
     }
 
-    /**
-     * Only the variable is consumed, the rest stays for the next rule.
-     */
     #[Test]
     public function stopsAtTheFirstWhitespace(): void
     {

libs/phpdoc/tests/DocBlock/Tag/ExampleTagTest.php

@@ -61,10 +61,6 @@ public function parsesLocationOnly(): void
         self::assertSame('@example demo.php', (string) $tag);
     }
 
-    /**
-     * Any ordinary word is a valid relative URI, so a leading word is captured
-     * as the location and the remainder becomes the description.
-     */
     #[Test]
     public function treatsLeadingWordAsLocation(): void
     {
@@ -77,10 +73,6 @@ public function treatsLeadingWordAsLocation(): void
         self::assertSame('@example demo.php the bundled snippet below.', (string) $tag);
     }
 
-    /**
-     * A location is mandatory, so an input whose first word cannot be a URI is
-     * malformed.
-     */
     #[Test]
     public function rejectsMissingLocation(): void
     {

libs/phpdoc/tests/DocBlock/Tag/LicenseTagTest.php

@@ -28,9 +28,6 @@ public function parsesUrlForm(): void
         self::assertSame('@license https://opensource.org/licenses/MIT MIT License', (string) $tag);
     }
 
-    /**
-     * A word without a scheme is not a URL, so it is parsed as a named license.
-     */
     #[Test]
     public function parsesNameForm(): void
     {

libs/phpdoc/tests/DocBlock/Tag/MethodTagTest.php

@@ -47,9 +47,6 @@ public function parsesLeadingReturnTypeAndStatic(): void
         self::assertSame('@method static ReturnType bar(U $u) An optional description.', (string) $tag);
     }
 
-    /**
-     * A signature that is not a callable is malformed.
-     */
     #[Test]
     public function rejectsNonCallableSignature(): void
     {

libs/phpdoc/tests/DocBlock/Tag/ReferenceTagTest.php

@@ -45,10 +45,6 @@ public function usedByResolvesThroughTheRealParser(): void
         self::assertSame('used-by', $block->tags[0]->name);
     }
 
-    /**
-     * Unlike "@see", the "@uses" tag references a code element only, so a URI
-     * is not a valid target.
-     */
     #[Test]
     public function uriIsNotAccepted(): void
     {

libs/phpdoc/tests/DocBlock/Tag/TagTest.php

@@ -10,10 +10,6 @@
 use TypeLang\PhpDoc\DocBlock\Tag\TagInterface;
 use TypeLang\PhpDoc\Tests\TestCase;
 
-/**
- * Tests the behaviour shared by every tag through the abstract {@see Tag} base
- * class, exercised here via a minimal concrete subclass.
- */
 final class TagTest extends TestCase
 {
     #[Test]

libs/phpdoc/tests/DocBlock/Tag/TemplateTagTest.php

@@ -61,10 +61,6 @@ public function parsesBoundWithoutDefault(): void
         self::assertNull($tag->default);
     }
 
-    /**
-     * A word that merely starts with "of" is not the "of" keyword, so it stays
-     * part of the description.
-     */
     #[Test]
     public function keywordRespectsWordBoundary(): void
     {