[Export] Fix and test serializations

Author: NoResponseMate

.github/workflows/build.yaml

@@ -11,16 +11,18 @@ jobs:
     tests:
         runs-on: ubuntu-latest
 
-        name: "PHP ${{ matrix.php }}, Symfony ${{ matrix.symfony }}"
+        name: "PHP ${{ matrix.php }}, Symfony ${{ matrix.symfony }}, MySQL ${{ matrix.mysql }}"
 
         strategy:
             fail-fast: false
             matrix:
                 php: ["8.3"]
                 symfony: ["^6.4", "^7.3"]
+                mysql: ["8.4"]
 
         env:
-            APP_ENV: test
+            APP_ENV: test_minimal
+            DATABASE_URL: "mysql://root:root@127.0.0.1/sylius?serverVersion=${{ matrix.mysql }}"
 
         steps:
             -
@@ -35,6 +37,17 @@ jobs:
                     tools: symfony
                     coverage: none
 
+            -
+                name: Shutdown default MySQL
+                run: sudo service mysql stop
+
+            -
+                name: Setup MySQL
+                uses: mirromutth/mysql-action@v1.1
+                with:
+                    mysql version: "${{ matrix.mysql }}"
+                    mysql root password: "root"
+
             -
                 name: Get Composer cache directory
                 id: composer-cache
@@ -74,6 +87,24 @@ jobs:
                 name: Run PHPStan
                 run: vendor/bin/phpstan analyse
 
-#            -
-#                name: Run PHPUnit
-#                run: vendor/bin/phpunit --colors=always
+            -
+                name: Warmup cache
+                run: vendor/bin/console cache:warmup
+
+            -
+                name: Validate container
+                run: vendor/bin/console lint:container
+
+            -
+                name: Warmup cache
+                run: vendor/bin/console cache:warmup
+
+            -
+                name: Setup database
+                run: |
+                    vendor/bin/console doctrine:database:create
+                    vendor/bin/console doctrine:schema:create
+
+            -
+                name: Run PHPUnit
+                run: vendor/bin/phpunit --testsuite=functional --colors=always

composer.json

@@ -10,6 +10,14 @@
             "Sylius\\ImportExport\\": "src/"
         }
     },
+    "autoload-dev": {
+        "psr-4": {
+            "Tests\\Sylius\\ImportExport\\": [
+                "tests/",
+                "tests/TestApplication/src/"
+            ]
+        }
+    },
     "require": {
         "php": "^8.2",
         "doctrine/orm": "^2.18 || ^3.3",
@@ -24,12 +32,12 @@
         "symfony/uid": "^6.4 || ^7.3"
     },
     "require-dev": {
+        "phpstan/phpstan": "^1.12",
+        "phpstan/phpstan-doctrine": "^1.5",
         "phpunit/phpunit": "^10.5",
-        "webmozart/assert": "^1.11",
         "sylius-labs/coding-standard": "^4.4",
         "sylius/test-application": "*",
-        "phpstan/phpstan": "^1.12",
-        "phpstan/phpstan-doctrine": "^1.5"
+        "webmozart/assert": "^1.11"
     },
     "config": {
         "allow-plugins": {

ecs.php

@@ -9,6 +9,8 @@
 return function (ECSConfig $ecsConfig): void {
     $ecsConfig->paths([
         __DIR__ . '/src',
+        __DIR__ . '/tests/Functional',
+        __DIR__ . '/tests/TestApplication/src',
     ]);
 
     $ecsConfig->import('vendor/sylius-labs/coding-standard/ecs.php');

phpunit.xml.dist

@@ -1,23 +1,24 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
-<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-         xsi:noNamespaceSchemaLocation="http://schema.phpunit.de/8.5/phpunit.xsd"
-         colors="true"
-         bootstrap="tests/Application/config/bootstrap.php">
+<phpunit
+    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+    xsi:noNamespaceSchemaLocation="http://schema.phpunit.de/8.5/phpunit.xsd"
+    colors="true"
+    bootstrap="vendor/sylius/test-application/config/bootstrap.php"
+>
     <testsuites>
-        <testsuite name="AcmeSyliusExamplePlugin Test Suite">
-            <directory>tests/Integration</directory>
-            <directory>tests/Api</directory>
-            <directory>tests/Unit</directory>
+        <testsuite name="functional">
+            <directory>tests/Functional</directory>
         </testsuite>
     </testsuites>
 
     <php>
         <ini name="error_reporting" value="-1" />
 
-        <server name="KERNEL_CLASS_PATH" value="/tests/Application/Kernel.php" />
+        <server name="KERNEL_CLASS" value="Sylius\TestApplication\Kernel" />
         <server name="IS_DOCTRINE_ORM_SUPPORTED" value="true" />
-        <env name="APP_ENV" value="test"/>
+
+        <env name="APP_ENV" value="test_minimal" />
         <env name="SHELL_VERBOSITY" value="-1" />
     </php>
 </phpunit>

src/DependencyInjection/Configuration.php

@@ -59,7 +59,7 @@ private function addExportConfiguration(ArrayNodeDefinition $node): void
                                 ->end()
                                 ->children()
                                     ->scalarNode('serialization_group')
-                                        ->isRequired()
+                                        ->cannotBeEmpty()
                                         ->defaultValue(DefaultSerializationGroups::EXPORT_GROUP)
                                     ->end()
                                     ->scalarNode('provider')

src/Provider/ResourceData/GridResourceDataProvider.php

@@ -60,7 +60,7 @@ public function getData(MetadataInterface $resource, string $gridCode, array $re
         /** @phpstan-ignore-next-line */
         return $this->serializer->normalize($rawData, context: [
             ExportAwareItemNormalizer::EXPORT_CONTEXT_KEY => true,
-            'groups' => $parameters['serialization_groups'], [DefaultSerializationGroups::EXPORT_GROUP],
+            'groups' => $parameters['serialization_groups'] ?? [DefaultSerializationGroups::EXPORT_GROUP],
         ]);
     }
 }