Merge pull request #8 from adriansuter/patch-phpcs

Add phpcs

Author: adriansuter

composer.json

@@ -26,7 +26,8 @@
   "autoload-dev": {
     "psr-4": {
       "My\\Integration\\": "tests/integration/src",
-      "AdrianSuter\\Autoload\\Override\\": "tests/integration/src-override"
+      "AdrianSuter\\Autoload\\Override\\": "tests/integration/src-override",
+      "AdrianSuter\\Autoload\\Override\\Tests\\": "tests"
     },
     "classmap": [
       "tests/integration/src/TestClassMapOverride"
@@ -37,12 +38,15 @@
     "nikic/php-parser": "^4.3"
   },
   "require-dev": {
-    "phpunit/phpunit": "^7.0"
+    "phpunit/phpunit": "^7.0",
+    "squizlabs/php_codesniffer": "^3.5"
   },
   "scripts": {
     "test": [
-      "@phpunit"
+      "@phpunit",
+      "@phpcs"
     ],
-    "phpunit": "phpunit"
+    "phpunit": "phpunit",
+    "phpcs": "phpcs"
   }
 }

phpcs.xml

@@ -0,0 +1,17 @@
+<?xml version="1.0"?>
+<ruleset name="PHP Autoload Override coding standard">
+    <description>PHP Autoload Override coding standard</description>
+
+    <!-- display progress -->
+    <arg value="p"/>
+    <!-- use colors in output -->
+    <arg name="colors"/>
+
+    <!-- inherit rules from: -->
+    <rule ref="PSR12"/>
+    <rule ref="Generic.Arrays.DisallowLongArraySyntax"/>
+
+    <!-- Paths to check -->
+    <file>src</file>
+    <file>tests</file>
+</ruleset>

src/AutoloadCollection.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * PHP Autoload Override (https://github.com/adriansuter/php-autoload-override)
  *

src/ClosureHandler.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * PHP Autoload Override (https://github.com/adriansuter/php-autoload-override)
  *

src/CodeConverter.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * PHP Autoload Override (https://github.com/adriansuter/php-autoload-override)
  *
@@ -75,10 +76,10 @@ public function __construct(
         ?NodeFinder $nodeFinder = null
     ) {
         $this->lexer = $lexer ?? new Emulative(
-                [
+            [
                     'usedAttributes' => ['comments', 'startLine', 'endLine', 'startTokenPos', 'endTokenPos'],
                 ]
-            );
+        );
 
         $this->parser = $parser ?? new Php7($this->lexer);
 

src/FileStreamWrapper.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * PHP Autoload Override (https://github.com/adriansuter/php-autoload-override)
  *
@@ -42,6 +43,7 @@
 use function touch;
 use function unlink;
 
+// phpcs:disable PSR1.Methods.CamelCapsMethodName
 class FileStreamWrapper
 {
     /**

src/Override.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * PHP Autoload Override (https://github.com/adriansuter/php-autoload-override)
  *

tests/AbstractIntegrationTest.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * PHP Autoload Override (https://github.com/adriansuter/php-autoload-override)
  *
@@ -7,6 +8,8 @@
 
 declare(strict_types=1);
 
+namespace AdrianSuter\Autoload\Override\Tests;
+
 use AdrianSuter\Autoload\Override\Override;
 use PHPUnit\Framework\TestCase;
 
@@ -24,7 +27,6 @@ public static function setUpBeforeClass()
     protected function setUp()
     {
         parent::setUp();
-
         if (!$this->overrideApplied) {
             $this->overrideApplied = true;
             Override::apply(self::$classLoader, $this->getOverrideDeclarations());

tests/AutoloadCollectionTest.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * PHP Autoload Override (https://github.com/adriansuter/php-autoload-override)
  *
@@ -7,6 +8,8 @@
 
 declare(strict_types=1);
 
+namespace AdrianSuter\Autoload\Override\Tests;
+
 use AdrianSuter\Autoload\Override\AutoloadCollection;
 use PHPUnit\Framework\TestCase;
 

tests/ClosureHandlerTest.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * PHP Autoload Override (https://github.com/adriansuter/php-autoload-override)
  *
@@ -7,8 +8,11 @@
 
 declare(strict_types=1);
 
+namespace AdrianSuter\Autoload\Override\Tests;
+
 use AdrianSuter\Autoload\Override\ClosureHandler;
 use PHPUnit\Framework\TestCase;
+use RuntimeException;
 
 final class ClosureHandlerTest extends TestCase
 {