Configure new phpstan version and fix phpcs config

Signed-off-by: Anton Fedurtsya <[email protected]>

.editorconfig

@@ -13,7 +13,7 @@ indent_size = 4
 trim_trailing_whitespace = true
 
 # Unix-style newlines with a newline ending every file
-[*.yaml]
+[{*.yml,*.yaml}]
 indent_size = 2
 
 # Tab indentation (no size specified)

.github/workflows/development.yml

@@ -203,6 +203,11 @@ jobs:
           make down
           sleep 2
 
+      - name: Cleanup style reports
+        run: |
+            perl -pi -e 's#/var/www/test-module/#./#g;' source/test-module/tests/reports/phpstan.report.json
+            perl -pi -e 's#/var/www/test-module/#./#g;' source/test-module/tests/reports/phpmd.report.json
+
       - name: Upload log artifact
         if: always()
         uses: actions/upload-artifact@v3
@@ -264,7 +269,6 @@ jobs:
 
   codeception_twig:
     needs: [ install_shop_with_module ]
-    if: ${{ inputs.template_engine == 'both' || inputs.template_engine == 'twig' }}
     runs-on: ubuntu-latest
     steps:
       - name: Load current installation from cache
@@ -320,7 +324,7 @@ jobs:
           sleep 2
 
   sonarcloud:
-    needs: [ unit_integration_tests ]
+    needs: [ styles, unit_integration_tests ]
     if: ${{ always() && inputs.report_sonarcloud }}
     runs-on: ubuntu-latest
     steps:
@@ -332,16 +336,29 @@ jobs:
         uses: actions/download-artifact@v3
         with:
           name: ModuleIntegrationTestsLog-${{ inputs.php }}-${{ inputs.mysql }}-${{ inputs.template_engine }}
-          path: ./tests/
+          path: ./testsResults/
 
-      - name: Debug
+      - name: Download StylesLog artifacts
+        continue-on-error: true
+        uses: actions/download-artifact@v3
+        with:
+          name: StylesLog-${{ inputs.php }}
+          path: ./stylesResults/
+
+      - name: Debug tests
         run: |
           ls -alh
-          cd tests
+          cd testsResults
           ls -alh
           cd source
           ls -alh
 
+      - name: Debug styles
+        run: |
+            ls -alh
+            cd stylesResults
+            ls -alh
+
       - name: SonarCloud Scan
         uses: sonarsource/sonarcloud-github-action@master
         env:
@@ -354,6 +371,6 @@ jobs:
             -Dsonar.sources=src
             -Dsonar.tests=tests
             -Dsonar.sourceEncoding=UTF-8
-            -Dsonar.php.coverage.reportPaths=tests/coverage.xml
+            -Dsonar.php.coverage.reportPaths=testsResults/source/coverage.xml
             -Dsonar.cpd.php.minimumTokens=25
-            -Dsonar.cpd.php.minimumLines=5
+            -Dsonar.cpd.php.minimumLines=5

composer.json

@@ -18,14 +18,14 @@
     },
     "require-dev": {
         "phpunit/phpunit": "^9.5",
-        "phpstan/phpstan": "^0.12.26",
         "squizlabs/php_codesniffer": "3.*",
         "phpmd/phpmd": "^2.11",
         "oxid-esales/oxideshop-ce": "dev-b-7.0.x",
         "qossmic/deptrac-shim": "^0.24.0",
         "codeception/module-rest": "*",
         "codeception/module-phpbrowser": "*",
-        "codeception/module-db": "*"
+        "codeception/module-db": "*",
+        "phpstan/phpstan": "^1.10"
     },
     "minimum-stability": "dev",
     "prefer-stable": true,

tests/PhpStan/phpstan-bootstrap.php

@@ -0,0 +1,8 @@
+<?php
+
+/**
+ * Copyright © OXID eSales AG. All rights reserved.
+ * See LICENSE file for license details.
+ */
+
+declare(strict_types=1);

tests/PhpStan/phpstan.neon

@@ -0,0 +1,8 @@
+parameters:
+    checkMissingIterableValueType: false
+    bootstrapFiles:
+        - phpstan-bootstrap.php
+    level: 8
+    scanFiles:
+        - ../../vendor/oxid-esales/oxideshop-ce/source/oxfunctions.php
+        - ../../vendor/oxid-esales/oxideshop-ce/source/overridablefunctions.php

tests/phpcs.xml

@@ -5,10 +5,6 @@
     <!-- Paths to check -->
     <file>../src/</file>
     <file>./</file>
-    <!--
-    <file>../translations</file>
-    -->
-    <file>../migration</file>
     <exclude-pattern>./tests/Codeception/Config</exclude-pattern>
 
     <!-- display progress -->