Merge pull request #9 from PHPCSStandards/feature/further-slash-normalization

More slash normalization

Author: afilina

src/Parser/SniffParser.php

@@ -4,6 +4,7 @@
 namespace App\Parser;
 
 use App\Parser\Exception\NotASniffPath;
+use App\Util\Functions;
 use App\Value\Diff;
 use App\Value\Property;
 use App\Value\Sniff;
@@ -27,7 +28,7 @@ class SniffParser
 {
     public function parse(string $phpFilePath, SourceLocator $projectSourceLocator): Sniff
     {
-        $phpFilePath = $this->normalizeSlashes($phpFilePath);
+        $phpFilePath = Functions::normalizeSlashes($phpFilePath);
         $astLocator = (new BetterReflection())->astLocator();
         $reflector = new ClassReflector(
             new AggregateSourceLocator([
@@ -212,16 +213,4 @@ private function getUrls(ReflectionClass $classInfo, array $xmlUrls): UrlList
 
         return new UrlList(array_merge($urls, $xmlUrls));
     }
-
-    /**
-     * Normalizes all slashes in a file path to forward slashes.
-     *
-     * @param string $path File path.
-     *
-     * @return string The file path with normalized slashes.
-     */
-    private function normalizeSlashes($path)
-    {
-        return str_replace('\\', '/', $path);
-    }
 }

src/Parser/ViolationParser.php

@@ -4,6 +4,7 @@
 namespace App\Parser;
 
 use App\Parser\Exception\NotAViolationPath;
+use App\Util\Functions;
 use App\Value\Diff;
 use App\Value\Url;
 use App\Value\UrlList;
@@ -27,6 +28,7 @@ public function parse(string $xmlFilePath): Violation
 
     private function getErrorCode(string $xmlFilePath): string
     {
+        $xmlFilePath = Functions::normalizeSlashes($xmlFilePath);
         $part = '([^/]*)';
         preg_match("`$part/Docs/$part/{$part}Standard/$part\.xml$`", $xmlFilePath, $matches);
         if ($matches === []) {

src/Util/Functions.php

@@ -0,0 +1,20 @@
+<?php
+declare(strict_types=1);
+
+namespace App\Util;
+
+class Functions
+{
+
+    /**
+     * Normalizes all slashes in a file path to forward slashes.
+     *
+     * @param string $path File path.
+     *
+     * @return string The file path with normalized slashes.
+     */
+    public static function normalizeSlashes(string $path)
+    {
+        return str_replace('\\', '/', $path);
+    }
+}

tests/Util/FunctionsTest.php

@@ -0,0 +1,50 @@
+<?php
+declare(strict_types=1);
+
+namespace App\Tests\Utils;
+
+use App\Util\Functions;
+use PHPUnit\Framework\TestCase;
+
+/** @covers \App\Util\Functions */
+class FunctionsTest extends TestCase
+{
+
+    /**
+     * Verify slash normalization.
+     *
+     * @dataProvider dataNormalizeSlashes
+     *
+     * @param string $input    Path to normalize.
+     * @param string $expected Expected normalized path.
+     *
+     * @return void
+     */
+    public function testNormalizeSlashes($input, $expected)
+    {
+        self::assertSame($expected, Functions::normalizeSlashes($input));
+    }
+
+    /**
+     * Data provider.
+     *
+     * @return array
+     */
+    public function dataNormalizeSlashes()
+    {
+        return [
+            'unix_slashes' => [
+                'path/to/folder/filename.php',
+                'path/to/folder/filename.php',
+            ],
+            'windows_slashes' => [
+                'path\to\folder\filename.php',
+                'path/to/folder/filename.php',
+            ],
+            'mixed_slashes' => [
+                'path\to/folder\filename.php',
+                'path/to/folder/filename.php',
+            ],
+        ];
+    }
+}