update tests

Author: Nikolay Shapovalov

tests/Drupal/Attributes/ValidHookNameUnitTest.inc

@@ -6,8 +6,14 @@
  */
 
 /**
- * Implements hook_node_view().
- *
+ * Valid hook.
+ */
+#[Hook('valid')]
+function module_valid() {
+
+}
+
+/**
  * Single quotes.
  */
 #[Hook('hook_node_view')]
@@ -16,8 +22,6 @@ function module_node_view() {
 }
 
 /**
- * Implements hook_node_load().
- *
  * Double quotes.
  */
 #[Hook("hook_node_load")]
@@ -27,27 +31,25 @@ function module_node_load() {
 
 
 /**
- * Implements hook_node_delete().
+ * Attribute named argument.
  */
 #[Hook(hook: 'hook_node_delete')]
 function module_node_delete() {
 
 }
 
 /**
- * Implements hook_node_alter().
+ * Attribute named arguments.
  */
 #[Hook(hook: 'hook_node_alter', module: 'custom_module')]
 function module_node_alter() {
 
 }
 
 /**
- * Implements hook_piratehook_view().
- *
- * "hook" is a part of hook name. No warning.
+ * "hook" is a part of hook name.
  */
-#[Hook('piratehook_view')]
+#[Hook('hook_piratehook_view')]
 function module_piratehook_view() {
 
 }
@@ -57,14 +59,12 @@ function module_piratehook_view() {
  *
  * "hook" is a part of hook name. No warning.
  */
-#[Hook('hookpirate_view')]
+#[Hook('hook_hookpirate_view')]
 function module_hookpirate_view() {
 
 }
 
 /**
- * Implements hook_hook_info().
- *
  * "hook_info" is exception. No warning.
  */
 #[Hook('hook_info')]
@@ -73,12 +73,25 @@ function module_hook_info() {
 }
 
 /**
- * Implements hook_hook_info().
- *
- * "hook_info" is exception. No warning.
+ * Named argument. "hook_info" is exception. No warning.
  */
 #[Hook(hook: 'hook_info')]
-function module_custom_hook_info() {
+function mymodule_hook_info() {
+
+}
+
+/**
+ * Valid hook.
+ */
+#[Hook('valid', 'validMethod', 'module')]
+class ValidHooks {
+
+  /**
+   *
+   */
+  public function validMethod() {
+
+  }
 
 }
 
@@ -98,7 +111,7 @@ class Hooks {
 }
 
 /**
- *
+ * Named arguments, double quotes.
  */
 #[Hook(hook: "hook_user_login", method: "userLogin", module: "views")]
 class MyHooks {

tests/Drupal/Attributes/ValidHookNameUnitTest.inc.fixed

@@ -6,8 +6,14 @@
  */
 
 /**
- * Implements hook_node_view().
- *
+ * Valid hook.
+ */
+#[Hook('valid')]
+function module_valid() {
+
+}
+
+/**
  * Single quotes.
  */
 #[Hook('node_view')]
@@ -16,8 +22,6 @@ function module_node_view() {
 }
 
 /**
- * Implements hook_node_load().
- *
  * Double quotes.
  */
 #[Hook("node_load")]
@@ -26,25 +30,23 @@ function module_node_load() {
 }
 
 /**
- * Implements hook_node_delete().
+ * Attribute named argument.
  */
 #[Hook(hook: 'node_delete')]
 function module_node_delete() {
 
 }
 
 /**
- * Implements hook_node_alter().
+ * Attribute named arguments.
  */
 #[Hook(hook: 'node_alter', module: 'custom_module')]
 function module_node_alter() {
 
 }
 
 /**
- * Implements hook_piratehook_view().
- *
- * "hook" is a part of hook name. No warning.
+ * "hook" is a part of hook name.
  */
 #[Hook('piratehook_view')]
 function module_piratehook_view() {
@@ -62,8 +64,6 @@ function module_hookpirate_view() {
 }
 
 /**
- * Implements hook_hook_info().
- *
  * "hook_info" is exception. No warning.
  */
 #[Hook('hook_info')]
@@ -72,12 +72,25 @@ function module_hook_info() {
 }
 
 /**
- * Implements hook_hook_info().
- *
- * "hook_info" is exception. No warning.
+ * Named argument. "hook_info" is exception. No warning.
  */
 #[Hook(hook: 'hook_info')]
-function module_custom_hook_info() {
+function mymodule_hook_info() {
+
+}
+
+/**
+ * Valid hook.
+ */
+#[Hook('valid', 'validMethod', 'module')]
+class ValidHooks {
+
+  /**
+   *
+   */
+  public function validMethod() {
+
+  }
 
 }
 
@@ -97,7 +110,7 @@ class Hooks {
 }
 
 /**
- *
+ * Named arguments, double quotes.
  */
 #[Hook(hook: "user_login", method: "userLogin", module: "views")]
 class MyHooks {

tests/Drupal/Attributes/ValidHookNameUnitTest.php

@@ -38,12 +38,14 @@ protected function getErrorList(string $testFile): array
     protected function getWarningList(string $testFile): array
     {
         return [
-            13  => 1,
-            23  => 1,
-            32  => 1,
-            40  => 1,
-            88  => 1,
-            103 => 1,
+            19  => 1,
+            27  => 1,
+            36  => 1,
+            44  => 1,
+            52  => 1,
+            62  => 1,
+            101 => 1,
+            116 => 1,
         ];
 
     }//end getWarningList()