test(phpstan): Ignore PHPStan warning directly in code with a comment (#1263)

Author: klausi

phpstan.neon

@@ -22,11 +22,6 @@ parameters:
   # yet, disable for now.
   checkGenericClassInNonGenericObjectType: false
   ignoreErrors:
-    # PHPStan bug, ignore for now.
-    -
-      message: "#^Unreachable statement \\- code above always terminates\\.$#"
-      count: 1
-      path: src/Entity/Server.php
     # @todo Ignore phpstan-drupal extension's rules for now, activate later.
     - '#\Drupal calls should be avoided in classes, use dependency injection instead#'
     # new static() is a best practice in Drupal, so we cannot fix that.

src/Entity/Server.php

@@ -526,6 +526,9 @@ protected function getValidationRules() {
         return [];
       }
 
+      // PHPStan thinks this is unreachable code because of the wrongly
+      // documented $params->queryId.
+      // @phpstan-ignore-next-line
       $rules = array_values(DocumentValidator::defaultRules());
       if ($this->getDisableIntrospection()) {
         $rules[] = new DisableIntrospection();