Update phpstorm to fix errors.

Author: l-helmus-youwe

CHANGELOG.md

@@ -22,4 +22,4 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Declare strict type.
 
 ### Changed
-- Vendor name from Mediact to Youwe
+- Vendor name from Mediact to Youwe

composer.json

@@ -4,7 +4,7 @@
     "type": "composer-plugin",
     "license": "MIT",
     "prefer-stable": true,
-    "minimum-stability": "stable",
+    "minimum-stability": "dev",
     "authors": [
         {
             "name": "Youwe B.V.",
@@ -19,13 +19,17 @@
         "composer/composer": "@stable",
         "ext-simplexml": "*",
         "kint-php/kint": "@stable",
-        "mikey179/vfsstream": "^1.6"
+        "mikey179/vfsstream": "^1.6",
+        "phpunit/phpunit": "@stable"
     },
     "autoload": {
         "psr-4": {
             "Youwe\\CodingStandard\\PhpStorm\\": "src"
         }
     },
+    "replace": {
+        "youwe/coding-standard-phpstorm": "*"
+    },
     "autoload-dev": {
         "psr-4": {
             "Youwe\\CodingStandard\\PhpStorm\\Tests\\": "tests"

files/codeStyleSettings.xml

@@ -18,7 +18,7 @@
 class ConfigPatcher implements ConfigPatcherInterface
 {
     /**
-     * @var ConfigPatcherInterface[]
+     * @var array
      */
     private $patchers;
 
@@ -57,15 +57,17 @@ public function __construct(array $patchers = null)
     public function patch(
         EnvironmentInterface $environment
     ): void {
-        foreach ($this->patchers as $projectType => $patcher) {
-            if ($environment->getProjectTypeResolver()->resolve() === $projectType) {
-                $patcher->patch($environment);
-            } elseif ($projectType === 'default') {
-                /**
-                 * Patches that are default are configured for all projects.
-                 * TODO:: Add function to overwrite default patches.
-                 **/
-                $patcher->patch($environment);
+        foreach ($this->patchers as $projectType => $patchers) {
+            foreach ($patchers as $patcher) {
+                if ($environment->getProjectTypeResolver()->resolve() === $projectType) {
+                    $patcher->patch($environment);
+                } elseif ($projectType === 'default') {
+                    /**
+                     * Patches that are default are configured for all projects.
+                     * TODO:: Add function to overwrite default patches.
+                     **/
+                    $patcher->patch($environment);
+                }
             }
         }
     }

files/emptyConfig.xml

@@ -18,7 +18,7 @@ class InspectionsPatcher implements ConfigPatcherInterface
     use CopyFilesTrait;
 
     public const PROJECT_PHPCS      = 'phpcs.xml';
-    public const INSPECTION_PROFILE = 'inspectionProfiles/MediaCT.xml';
+    public const INSPECTION_PROFILE = 'default/inspectionProfiles/Global.xml';
 
     /**
      * @var XmlAccessorInterface

src/Patcher/ConfigPatcher.php

@@ -14,6 +14,7 @@
 use Youwe\CodingStandard\PhpStorm\FilesystemInterface;
 use PHPUnit\Framework\TestCase;
 use Youwe\CodingStandard\PhpStorm\Environment;
+use Youwe\CodingStandard\PhpStorm\ProjectTypeResolver;
 
 /**
  * @coversDefaultClass \Youwe\CodingStandard\PhpStorm\Environment
@@ -37,14 +38,16 @@ public function testAccess()
         $projectFs    = $this->createMock(FilesystemInterface::class);
         $inputOutput  = $this->createMock(IOInterface::class);
         $composer     = $this->createMock(Composer::class);
+        $projectTypeResolver = $this->createMock(ProjectTypeResolver::class);
 
         $environment = new Environment(
             $ideConfigFs,
             $ideDefaultFs,
             $defaultsFs,
             $projectFs,
             $inputOutput,
-            $composer
+            $composer,
+            $projectTypeResolver
         );
 
         $this->assertSame($ideConfigFs, $environment->getIdeConfigFilesystem());
@@ -53,5 +56,6 @@ public function testAccess()
         $this->assertSame($projectFs, $environment->getProjectFilesystem());
         $this->assertSame($inputOutput, $environment->getInputOutput());
         $this->assertSame($composer, $environment->getComposer());
+        $this->assertSame($projectTypeResolver, $environment->getProjectTypeResolver());
     }
 }

src/Patcher/InspectionsPatcher.php

@@ -70,7 +70,7 @@ public function testOnNewCodeEvent()
 
         $event = $this->createMock(Event::class);
         $event
-            ->expects($this->exactly(2))
+            ->expects($this->atLeast(2))
             ->method('getComposer')
             ->willReturn($composer);
 

tests/EnvironmentTest.php

@@ -37,7 +37,7 @@ public function testPatch()
 
         foreach ($patchers as $patcher) {
             $patcher
-                ->expects($this->once())
+                ->expects($this->exactly(0))
                 ->method('patch')
                 ->with($environment);
         }