Upgrade to PHPUnit 11 (#76)

* Remove NamespaceSniff, we are way past PHPUnit 6
* Replace @Covers with attributes
* Remove AtCovers sniffs
* Add typehints to const
* Drop PHP 8.1 and 8.2 support, add 8.4 and 8.5
* Upgrade tot PHPUnit 11
* Updated deprecated syntax
* Remove referenced deprecated sniffs

Author: janlam7

.github/workflows/main.yaml

@@ -12,7 +12,7 @@ jobs:
 
     strategy:
       matrix:
-        php-versions: ['8.1','8.2','8.3']
+        php-versions: ['8.3','8.4','8.5']
     name: PHP ${{ matrix.php-versions }}
     steps:
       -   uses: actions/checkout@v2

composer.json

@@ -4,7 +4,7 @@
     "description": "PHP_CodeSniffer tokenises PHP, JavaScript and CSS files and detects violations of a defined set of coding standards. Hostnet version",
     "license":     "MIT",
     "require": {
-        "php":                              "^8.1",
+        "php":                              "^8.3",
         "composer-plugin-api":              "^2.0.0",
         "hostnet/path-composer-plugin-lib": "^1.0.5",
         "slevomat/coding-standard":         "^8.6.2",
@@ -13,7 +13,7 @@
     },
     "require-dev": {
         "composer/composer": "^2.1.14",
-        "phpunit/phpunit":   "^9.5.10"
+        "phpunit/phpunit":   "^11.5.55"
     },
     "autoload": {
         "psr-0": {

phpunit.xml.dist

@@ -1,13 +1,15 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd" colors="true" bootstrap="vendor/squizlabs/php_codesniffer/tests/bootstrap.php">
-  <coverage>
-    <include>
-      <directory>./src</directory>
-    </include>
-  </coverage>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/11.5/phpunit.xsd" colors="true" bootstrap="vendor/squizlabs/php_codesniffer/tests/bootstrap.php">
   <testsuites>
     <testsuite name="hostnet/phpcs-tool test suite">
       <directory>./test/</directory>
+      <exclude>./test/Hostnet/Tests/AbstractSniffUnitTest.php</exclude>
+      <exclude>./test/Hostnet/Tests/CopiedAbstractSniffUnitTest.php</exclude>
     </testsuite>
   </testsuites>
+  <source>
+    <include>
+      <directory>./src</directory>
+    </include>
+  </source>
 </phpunit>

src/Hostnet/Sniffs/Classes/CopyrightMustBeBeforeDeclareSniff.php

@@ -11,7 +11,7 @@
 
 class CopyrightMustBeBeforeDeclareSniff implements Sniff
 {
-    private const ERROR = 'declare(strict_types=1) should be after the copyright statement';
+    private const string ERROR = 'declare(strict_types=1) should be after the copyright statement';
 
     /**
      * {@inheritdoc}

src/Hostnet/Sniffs/Classes/VariableAndPropertyMustBeInSnakeCaseSniff.php

@@ -14,7 +14,7 @@
  */
 class VariableAndPropertyMustBeInSnakeCaseSniff implements Sniff
 {
-    private const SUPER_GLOBALS = [
+    private const array SUPER_GLOBALS = [
         'GLOBALS',
         '_SERVER',
         '_GET',

src/Hostnet/Sniffs/Commenting/AtCoversCounterPartSniff.php

@@ -15,9 +15,9 @@
  */
 class StrictSniff implements Sniff
 {
-    private const ERROR = 'declare(strict_types = 1) not found';
+    private const string ERROR = 'declare(strict_types = 1) not found';
 
-    private const R_VALUE = [
+    private const array R_VALUE = [
         T_LNUMBER,                    //integers
         T_STRING,                     //identifiers
         T_NUM_STRING,                 // numeric array index inside string