add enum for api type values

Author: JoMessina

src/Builder/Client.php

@@ -7,6 +7,7 @@
 use Diglin\Sylius\ApiClient\SyliusAdminClientBuilder;
 use Diglin\Sylius\ApiClient\SyliusShopClientBuilder;
 use Kiboko\Plugin\Sylius\MissingAuthenticationMethodException;
+use Kiboko\Plugin\Sylius\Validator\ApiType;
 use PhpParser\Builder;
 use PhpParser\Node;
 
@@ -20,7 +21,7 @@ final class Client implements Builder
     private ?Node\Expr $httpRequestFactory = null;
     private ?Node\Expr $httpStreamFactory = null;
     private ?Node\Expr $fileSystem = null;
-    private string $apiType = '';
+    private string $apiType;
 
     public const API_ADMIN_KEY = 'admin';
     public const API_SHOP_KEY = 'shop';
@@ -135,9 +136,9 @@ public function getNode(): Node\Expr\MethodCall
     private function getClientBuilderNode(): Node\Expr\MethodCall
     {
         $className = match ($this->apiType) {
-            self::API_ADMIN_KEY => SyliusAdminClientBuilder::class,
-            self::API_SHOP_KEY => SyliusShopClientBuilder::class,
-            self::API_LEGACY_KEY => 'Diglin\\Sylius\\ApiClient\\SyliusClientBuilder'
+            ApiType::ADMIN->value => SyliusAdminClientBuilder::class,
+            ApiType::SHOP->value => SyliusShopClientBuilder::class,
+            ApiType::LEGACY->value => 'Diglin\\Sylius\\ApiClient\\SyliusClientBuilder',
         };
 
         return new Node\Expr\MethodCall(

src/Validator/ApiType.php

@@ -0,0 +1,12 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Kiboko\Plugin\Sylius\Validator;
+
+enum ApiType: string
+{
+    case ADMIN = 'admin';
+    case SHOP = 'shop';
+    case LEGACY = 'legacy';
+}

src/Validator/ExtractorConfigurationValidator.php

@@ -459,15 +459,15 @@ class ExtractorConfigurationValidator implements ConfigurationValidatorInterface
     public static function validate(array $item): array
     {
         switch($item['api_type']) {
-            case 'admin':
+            case ApiType::ADMIN->value:
                 $endpoints = self::$endpointsAdmin;
                 $doubleEndpoints = self::$doubleEndpointsAdmin;
                 break;
-            case 'shop':
+            case ApiType::SHOP->value:
                 $endpoints = self::$endpointsShop;
                 $doubleEndpoints = self::$doubleEndpointsShop;
                 break;
-            case 'legacy':
+            case ApiType::LEGACY->value:
                 $endpoints = self::$endpointsLegacy;
                 $doubleEndpoints = self::$doubleEndpointsLegacy;
                 break;

src/Validator/LoaderConfigurationValidator.php

@@ -252,9 +252,9 @@ class LoaderConfigurationValidator implements ConfigurationValidatorInterface
     public static function validate(array $item): array
     {
         $endpoints = match ($item['api_type']) {
-            'admin' => self::$endpointsAdmin,
-            'shop' => self::$endpointsShop,
-            'legacy' => self::$endpointsLegacy
+            ApiType::ADMIN->value => self::$endpointsAdmin,
+            ApiType::SHOP->value => self::$endpointsShop,
+            ApiType::LEGACY->value => self::$endpointsLegacy
         };
         if (!\in_array($item['type'], array_keys($endpoints))) {
             throw new \InvalidArgumentException(sprintf('the value should be one of [%s], got %s', implode(', ', array_keys($endpoints)), json_encode($item['type'], \JSON_THROW_ON_ERROR)));