refactor: use test setup from 'Leitfaden'

teamneusta · Jul 19, 2023 · d4d6f3f · d4d6f3f
1 parent 2e62425
commit d4d6f3f
Show file tree

Hide file tree

Showing 32 changed files with 146 additions and 742 deletions.
diff --git a/.editorconfig b/.editorconfig
@@ -8,11 +8,11 @@ indent_style = space
 insert_final_newline = true
 trim_trailing_whitespace = true
 
-[*.yaml]
+[*.{json,yaml,yml}]
 indent_size = 2
 
-[*.neon]
-indent_style = tab
-
 [*.md]
 trim_trailing_whitespace = false
+
+[*.neon]
+indent_style = tab
diff --git a/.gitattributes b/.gitattributes
@@ -1,12 +1,10 @@
-/docs/* export-ignore
-/.github export-ignore
 /.gitattributes export-ignore
+/.github/ export-ignore
 /.gitignore export-ignore
 /.php-cs-fixer.php export-ignore
-/docker-compose.yaml export-ignore
-/phpstan.neon export-ignore
+/compose.yaml export-ignore
+/docs/ export-ignore
 /phpstan-baseline.neon export-ignore
+/phpstan.neon export-ignore
 /phpunit.xml.dist export-ignore
-/run-tests.sh export-ignore
-/tests export-ignore
-/wait-for-it.sh export-ignore
+/tests/ export-ignore
diff --git a/.gitignore b/.gitignore
@@ -7,3 +7,5 @@
 
 # PHPUnit
 .phpunit.result.cache
+/reports/
+/.phpunit.cache/
diff --git a/.php-cs-fixer.php b/.php-cs-fixer.php
@@ -12,20 +12,28 @@
             __DIR__ . '/src',
             __DIR__ . '/tests',
         ])
-        ->notPath('DependencyInjection/Configuration.php')
+        ->notPath(['DependencyInjection/Configuration.php', 'app/var'])
     )
     ->setRiskyAllowed(true)
     ->setRules([
         '@Symfony' => true,
+        '@Symfony:risky' => true,
 
         // declare strict types must be on first line after opening tag
         'blank_line_after_opening_tag' => false, // overwrite @Symfony
         'linebreak_after_opening_tag' => false, // overwrite @Symfony
         'declare_strict_types' => true, // custom
 
+        // allow throw's in multiple lines, so message can be a long string
+        'single_line_throw' => false, // overwrite @Symfony
+
         // we want spaces
         'concat_space' => ['spacing' => 'one'],
 
-        // allow both styles
+        // we want to leave the choice to the developer,
+        // because some people have their own style of naming test methods
         'php_unit_method_casing' => false,
+
+        // we want to leave the choice to the developer
+        'php_unit_test_annotation' => false,
     ]);
diff --git a/README.md b/README.md
@@ -10,21 +10,19 @@ This bundle reads standard symfony translation files and migrates them to Pimcor
 
 ## Installation
 
-Require via Composer
+1. **Require the bundle**
 
-```shell
-composer require teamneusta/pimcore-translation-migration-bundle
-```
+   ```shell script
+   composer require teamneusta/pimcore-translation-migration-bundle
+   ```
 
-Enable the bundle via the [Symfony Bundle System](https://symfony.com/doc/current/bundles.html).
+2. **Enable the bundle**
 
-```php
-// config/bundles.php
-return [
-    ...
-    Neusta\Pimcore\TranslationMigrationBundle\NeustaPimcoreTranslationMigrationBundle::class => ['all' => true],
-];
-```
+   Add the Elastic Bundle and the Translation Migration Bundle to your `config/bundles.php`:
+
+   ```php
+   Neusta\Pimcore\TranslationMigrationBundle\NeustaPimcoreTranslationMigrationBundle::class => ['all' => true],
+   ```
 
 ## Usage
 
@@ -46,24 +44,23 @@ Feel free to open issues for any bug, feature request, or other ideas.
 
 Please remember to create an issue before creating large pull requests.
 
-### Running tests for development
+### Local Development
+
+To develop on local machine, the vendor dependencies are required.
 
 ```shell
-./run-tests.sh
+bin/composer install
 ```
 
-Only supported on Linux.
+We use composer scripts for our main quality tools. They can be executed via the `bin/composer` file as well.
 
-### Further development
-
-Pipelines will tell you, when code does not meet our standards. To use the same tools in local development, take the Docker command from above with other scripts from the `composer.json`. For example:
+```shell
+bin/composer cs:fix
+bin/composer phpstan
+```
 
-* cs:check
-* phpstan
+For the tests there is a different script, that includes a database setup.
 
 ```shell
-docker run -it --rm -v $(pwd):/app -w /app pimcore/pimcore:PHP8.1-cli composer install --ignore-platform-reqs
-docker run -it --rm -v $(pwd):/app -w /app pimcore/pimcore:PHP8.1-cli composer <composer-script>
+bin/run-tests
 ```
-
-Only supported on Linux.
diff --git a/bin/composer b/bin/composer
@@ -0,0 +1,3 @@
+#!/bin/sh
+
+exec docker compose run --rm --user "$(id -u):$(id -g)" --no-deps php composer "$@"
diff --git a/bin/run-tests b/bin/run-tests
@@ -0,0 +1,4 @@
+#!/bin/sh
+
+docker compose run --rm --user "$(id -u):$(id -g)" php composer tests
+docker compose down
diff --git a/compose.yaml b/compose.yaml
@@ -0,0 +1,26 @@
+services:
+  database:
+    image: mariadb:10.11.4
+    command: ["mysqld", "--character-set-server=utf8mb4", "--collation-server=utf8mb4_unicode_ci"]
+    environment:
+      MYSQL_ALLOW_EMPTY_PASSWORD: yes
+      MYSQL_DATABASE: pimcore
+      MYSQL_PASSWORD: pimcore
+      MYSQL_USER: pimcore
+    tmpfs:
+      - /tmp/
+      - /var/lib/mysql/
+    healthcheck:
+      test: ["CMD", "mysqladmin" ,"ping", "-h", "localhost"]
+      interval: 5s
+      timeout: 10s
+
+  php:
+    image: pimcore/pimcore:php8.1-v1.3
+    volumes:
+      - ./:/var/www/html/
+    environment:
+      MYSQL_SERVER_VERSION: mariadb-10.11.4
+    depends_on:
+      database:
+        condition: service_healthy
diff --git a/composer.json b/composer.json
@@ -11,23 +11,31 @@
       "role": "Developer"
     }
   ],
+  "config": {
+    "sort-packages": true,
+    "allow-plugins": {
+      "phpstan/extension-installer": true
+    }
+  },
   "require": {
     "php": "^8.0",
     "pimcore/pimcore": "^10.0",
-    "symfony/console": "^5.4",
-    "symfony/finder": "^5.4",
-    "symfony/framework-bundle": "^5.0|^6.0",
-    "symfony/yaml": "^5.0|^6.0"
+    "symfony/console": "^5.4 || ^6.0",
+    "symfony/finder": "^5.4 || ^6.0",
+    "symfony/framework-bundle": "^5.0 || ^6.0",
+    "symfony/yaml": "^5.0 || ^6.0"
   },
   "require-dev": {
+    "friendsofphp/php-cs-fixer": "^3.11",
+    "phpspec/prophecy": "^1.11",
     "phpspec/prophecy-phpunit": "^2.0",
+    "phpstan/extension-installer": "^1.1",
+    "phpstan/phpstan": "^1.9",
+    "phpstan/phpstan-symfony": "^1.2",
     "phpunit/phpunit": "^9.5",
     "spatie/phpunit-snapshot-assertions": "^4.2",
     "symfony/filesystem": "^5.4",
-    "friendsofphp/php-cs-fixer": "^3.11",
-    "phpstan/phpstan": "^1.8",
-    "phpstan/phpstan-phpunit": "^1.1",
-    "phpstan/phpstan-symfony": "^1.2"
+    "teamneusta/pimcore-testing-framework": "^0.11.0"
   },
   "autoload": {
     "psr-4": {
@@ -36,24 +44,24 @@
   },
   "autoload-dev": {
     "psr-4": {
-      "Neusta\\Pimcore\\TranslationMigrationBundle\\Tests\\Functional\\": "tests/functional/",
-      "Neusta\\Pimcore\\TranslationMigrationBundle\\Tests\\Unit\\": "tests/unit/"
+      "Neusta\\Pimcore\\TranslationMigrationBundle\\Tests\\": "tests/"
     },
     "classmap": [
       "tests/app/TestKernel.php"
     ]
   },
   "scripts": {
-    "cs:check": "php-cs-fixer fix -v --diff --dry-run",
-    "cs:fix": "php-cs-fixer fix -v --diff",
-    "phpstan": "phpstan analyse --level=${PHPSTAN_LEVEL:-8} -c phpstan.neon",
-    "phpstan:baseline": "phpstan analyse --level=${PHPSTAN_LEVEL:-8} -c phpstan.neon --generate-baseline",
+    "cs:check": "@cs:fix --dry-run",
+    "cs:fix": "php-cs-fixer fix --ansi --verbose --diff",
+    "phpstan": "phpstan analyse --ansi --no-interaction",
+    "phpstan:baseline": "phpstan analyse --generate-baseline",
     "tests": "phpunit"
   },
   "scripts-descriptions": {
     "cs:check": "Checks code style (but doesn't fix anything)",
     "cs:fix": "Checks and fixes code style",
-    "phpstan": "Checks code against phpstan rules",
-    "tests": "runs all the phpunit tests"
+    "phpstan": "Checks for code smells",
+    "phpstan:baseline": "Creates a baseline for phpstan",
+    "tests": "Run all phpunit tests"
   }
 }
diff --git a/docker-compose.yaml b/docker-compose.yaml
diff --git a/phpstan-baseline.neon b/phpstan-baseline.neon
@@ -1,17 +1,17 @@
 parameters:
 	ignoreErrors:
 		-
-			message: "#^Argument of an invalid type array\\|bool\\|float\\|int\\|string\\|UnitEnum\\|null supplied for foreach, only iterables are supported\\.$#"
+			message: "#^Argument of an invalid type array\\|bool\\|float\\|int\\|string\\|null supplied for foreach, only iterables are supported\\.$#"
 			count: 1
 			path: src/DependencyInjection/NeustaPimcoreTranslationMigrationExtension.php
 
 		-
-			message: "#^Binary operation \"\\.\" between array\\|bool\\|float\\|int\\|string\\|UnitEnum\\|null and '/Resources…' results in an error\\.$#"
+			message: "#^Binary operation \"\\.\" between array\\|bool\\|float\\|int\\|string\\|null and '/Resources…' results in an error\\.$#"
 			count: 1
 			path: src/DependencyInjection/NeustaPimcoreTranslationMigrationExtension.php
 
 		-
-			message: "#^Binary operation \"\\.\" between array\\|bool\\|float\\|int\\|string\\|UnitEnum\\|null and '/translations' results in an error\\.$#"
+			message: "#^Binary operation \"\\.\" between array\\|bool\\|float\\|int\\|string\\|null and '/translations' results in an error\\.$#"
 			count: 1
 			path: src/DependencyInjection/NeustaPimcoreTranslationMigrationExtension.php
 

diff --git a/phpstan.neon b/phpstan.neon
@@ -1,11 +1,14 @@
 includes:
-    - phpstan-baseline.neon
+	- phpstan-baseline.neon
 
 parameters:
-    paths:
-        - src
 
-    bootstrapFiles:
-        - vendor/pimcore/pimcore/stubs/dynamic-constants.php
+	level: 8
 
-    checkGenericClassInNonGenericObjectType: false
+	paths:
+		- src
+
+	bootstrapFiles:
+		- vendor/pimcore/pimcore/stubs/dynamic-constants.php
+
+	checkGenericClassInNonGenericObjectType: false
diff --git a/phpunit.xml.dist b/phpunit.xml.dist
@@ -1,27 +1,29 @@
 <?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"
-        backupGlobals="false"
-        colors="true"
-        bootstrap="tests/bootstrap.php"
-        failOnRisky="true"
-        failOnWarning="true">
-
-    <coverage>
-        <include>
-            <directory>./src</directory>
-        </include>
-    </coverage>
 
+<phpunit
+    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+    xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
+    bootstrap="tests/bootstrap.php"
+    cacheResultFile=".phpunit.cache/test-results"
+    colors="true"
+    failOnRisky="true"
+    failOnWarning="true"
+>
     <php>
+        <ini name="display_startup_errors" value="On"/>
+        <ini name="display_errors" value="On"/>
         <ini name="error_reporting" value="-1"/>
-        <ini name="memory_limit" value="-1" />
     </php>
 
     <testsuites>
-        <testsuite name="tests">
+        <testsuite name="All tests">
             <directory>./tests/</directory>
         </testsuite>
     </testsuites>
+
+    <coverage cacheDirectory=".phpunit.cache/code-coverage" processUncoveredFiles="true">
+        <include>
+            <directory>./src</directory>
+        </include>
+    </coverage>
 </phpunit>
diff --git a/run-tests.sh b/run-tests.sh
diff --git a/src/Command/TranslationsMigrateCommand.php b/src/Command/TranslationsMigrateCommand.php
@@ -54,7 +54,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
         if ($isVerbose) {
             $output->writeln('Reading from directories:');
-            $this->io->listing(\array_map(
+            $this->io->listing(array_map(
                 fn (string $path): string => $this->stripProjectPrefix($path),
                 $this->sourceProvider->getDirectories(),
             ));
@@ -102,7 +102,7 @@ function (FileCannotBeLoaded $event) use (&$ioTableRows): void {
     private function stripProjectPrefix(string $string): string
     {
         return str_starts_with($string, self::PROJECT_ROOT)
-            ? substr($string, strlen(self::PROJECT_ROOT))
+            ? substr($string, \strlen(self::PROJECT_ROOT))
             : $string;
     }
 }