Add interfaces to services

src/Command/ImportModListsCommand.php

@@ -4,7 +4,7 @@
 
 namespace App\Command;
 
-use App\Service\LegacyModListImport\ModListImport;
+use App\Service\LegacyModListImport\ModListImportInterface;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
@@ -13,7 +13,7 @@
 class ImportModListsCommand extends Command
 {
     public function __construct(
-        private ModListImport $modListImport
+        private ModListImportInterface $modListImport
     ) {
         parent::__construct();
     }

src/Controller/HomeController.php

@@ -4,7 +4,7 @@
 
 namespace App\Controller;
 
-use App\Service\Mission\MissionClient;
+use App\Service\Mission\MissionClientInterface;
 use Psr\Log\LoggerInterface;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
 use Symfony\Component\HttpFoundation\Response;
@@ -16,7 +16,7 @@
 class HomeController extends AbstractController
 {
     public function __construct(
-        private MissionClient $missionClient,
+        private MissionClientInterface $missionClient,
         private LoggerInterface $logger
     ) {
     }

src/Controller/ModListPublicController.php

@@ -8,7 +8,7 @@
 use App\Repository\Mod\ModRepository;
 use App\Repository\ModList\ModListRepository;
 use App\Security\Enum\PermissionsEnum;
-use App\Service\Mission\MissionClient;
+use App\Service\Mission\MissionClientInterface;
 use Psr\Log\LoggerInterface;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\IsGranted;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
@@ -25,7 +25,7 @@ public function __construct(
         private LoggerInterface $logger,
         private ModRepository $modRepository,
         private ModListRepository $modListRepository,
-        private MissionClient $missionClient,
+        private MissionClientInterface $missionClient,
     ) {
     }
 

src/Form/Dlc/DataTransformer/DlcFormDtoDataTransformer.php

@@ -10,13 +10,13 @@
 use App\Form\FormDtoInterface;
 use App\Form\RegisteredDataTransformerInterface;
 use App\Service\Steam\Helper\SteamHelper;
-use App\Service\Steam\SteamApiClient;
+use App\Service\Steam\SteamApiClientInterface;
 use Ramsey\Uuid\Uuid;
 
 class DlcFormDtoDataTransformer implements RegisteredDataTransformerInterface
 {
     public function __construct(
-        private SteamApiClient $steamApiClient
+        private SteamApiClientInterface $steamApiClient
     ) {
     }
 

src/Form/Mod/DataTransformer/ModFormDtoDataTransformer.php

@@ -15,13 +15,13 @@
 use App\Form\Mod\Dto\ModFormDto;
 use App\Form\RegisteredDataTransformerInterface;
 use App\Service\Steam\Helper\SteamHelper;
-use App\Service\Steam\SteamApiClient;
+use App\Service\Steam\SteamApiClientInterface;
 use Ramsey\Uuid\Uuid;
 
 class ModFormDtoDataTransformer implements RegisteredDataTransformerInterface
 {
     public function __construct(
-        private SteamApiClient $steamApiClient
+        private SteamApiClientInterface $steamApiClient
     ) {
     }
 

src/Service/LegacyModListImport/DtoToEntityConverter.php

@@ -14,7 +14,7 @@
 /**
  * @see https://github.com/ArmaForces/modlist/blob/master/src/util/mods.js
  */
-class DtoToEntityConverter
+class DtoToEntityConverter implements DtoToEntityConverterInterface
 {
     public function convert(ModCsvEntryDto $modCsvEntryDto): AbstractMod
     {

src/Service/LegacyModListImport/DtoToEntityConverterInterface.php

@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Service\LegacyModListImport;
+
+use App\Entity\Mod\AbstractMod;
+use App\Service\LegacyModListImport\Dto\ModCsvEntryDto;
+
+interface DtoToEntityConverterInterface
+{
+    public function convert(ModCsvEntryDto $modCsvEntryDto): AbstractMod;
+}

src/Service/LegacyModListImport/ModListCsvReader.php

@@ -7,7 +7,7 @@
 use App\Service\LegacyModListImport\Dto\ModCsvEntryDto;
 use League\Csv\Reader;
 
-class ModListCsvReader
+class ModListCsvReader implements ModListCsvReaderInterface
 {
     protected const COLUMN_INDEX_ID = 0;
     protected const COLUMN_INDEX_NAME = 1;

src/Service/LegacyModListImport/ModListCsvReaderInterface.php

@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Service\LegacyModListImport;
+
+interface ModListCsvReaderInterface
+{
+    public function readCsvRow(string $path, string $delimiter = ';'): \Generator;
+}

src/Service/LegacyModListImport/ModListImport.php

@@ -13,12 +13,12 @@
 use Ramsey\Uuid\Uuid;
 use Symfony\Component\Finder\Finder;
 
-class ModListImport
+class ModListImport implements ModListImportInterface
 {
     public function __construct(
         private EntityManagerInterface $entityManager,
-        private ModListCsvReader $modListCsvReader,
-        private DtoToEntityConverter $dtoToEntityConverter,
+        private ModListCsvReaderInterface $modListCsvReader,
+        private DtoToEntityConverterInterface $dtoToEntityConverter,
         private SteamWorkshopModRepository $steamWorkshopModRepository,
         private DirectoryModRepository $directoryModRepository
     ) {

src/Service/LegacyModListImport/ModListImportInterface.php

@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Service\LegacyModListImport;
+
+interface ModListImportInterface
+{
+    public function importFromDirectory(string $path, string $extension = '.csv'): void;
+}

src/Service/Mission/MissionClient.php

@@ -10,7 +10,7 @@
 use Symfony\Component\HttpClient\ScopingHttpClient;
 use Symfony\Contracts\HttpClient\HttpClientInterface;
 
-class MissionClient
+class MissionClient implements MissionClientInterface
 {
     public function __construct(
         private HttpClientInterface $client,

src/Service/Mission/MissionClientInterface.php

@@ -0,0 +1,20 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Service\Mission;
+
+use App\Service\Mission\Dto\MissionDto;
+
+interface MissionClientInterface
+{
+    public function getMissions(bool $includeArchive = true, int $ttl = 600): \Generator;
+
+    public function getNextUpcomingMission(): ?MissionDto;
+
+    public function getCurrentMission(): ?MissionDto;
+
+    public function getArchivedMissions(): array;
+
+    public function getUpcomingMissions(): array;
+}

src/Service/Steam/SteamApiClient.php

@@ -11,7 +11,7 @@
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Contracts\HttpClient\HttpClientInterface;
 
-class SteamApiClient
+class SteamApiClient implements SteamApiClientInterface
 {
     public function __construct(
         private HttpClientInterface $httpClient

src/Service/Steam/SteamApiClientInterface.php

@@ -0,0 +1,15 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Service\Steam;
+
+use App\Service\Steam\Dto\AppInfoDto;
+use App\Service\Steam\Dto\WorkshopItemInfoDto;
+
+interface SteamApiClientInterface
+{
+    public function getWorkshopItemInfo(int $itemId): WorkshopItemInfoDto;
+
+    public function getAppInfo(int $appId): AppInfoDto;
+}

src/Service/Version/VersionProvider.php

@@ -4,7 +4,7 @@
 
 namespace App\Service\Version;
 
-class VersionProvider
+class VersionProvider implements VersionProviderInterface
 {
     public function __construct(
         private string $projectDir,

src/Service/Version/VersionProviderInterface.php

@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Service\Version;
+
+interface VersionProviderInterface
+{
+    public function getVersion(): string;
+}

src/Twig/VersionExtension.php

@@ -4,14 +4,14 @@
 
 namespace App\Twig;
 
-use App\Service\Version\VersionProvider;
+use App\Service\Version\VersionProviderInterface;
 use Twig\Extension\AbstractExtension;
 use Twig\TwigFunction;
 
 class VersionExtension extends AbstractExtension
 {
     public function __construct(
-        private VersionProvider $version
+        private VersionProviderInterface $version
     ) {
     }
 

src/Validator/Dlc/SteamStoreArma3DlcUrlValidator.php

@@ -8,7 +8,7 @@
 use App\Service\Steam\Exception\AppNotFoundException;
 use App\Service\Steam\Helper\Exception\InvalidAppUrlFormatException;
 use App\Service\Steam\Helper\SteamHelper;
-use App\Service\Steam\SteamApiClient;
+use App\Service\Steam\SteamApiClientInterface;
 use App\Validator\AbstractValidator;
 use Doctrine\ORM\EntityManagerInterface;
 use Symfony\Component\Validator\Constraint;
@@ -20,7 +20,7 @@ class SteamStoreArma3DlcUrlValidator extends AbstractValidator
 
     public function __construct(
         EntityManagerInterface $entityManager,
-        private SteamApiClient $steamApiClient
+        private SteamApiClientInterface $steamApiClient
     ) {
         parent::__construct($entityManager);
     }

src/Validator/Mod/SteamWorkshopArma3ModUrlValidator.php

@@ -8,7 +8,7 @@
 use App\Service\Steam\Exception\WorkshopItemNotFoundException;
 use App\Service\Steam\Helper\Exception\InvalidWorkshopItemUrlFormatException;
 use App\Service\Steam\Helper\SteamHelper;
-use App\Service\Steam\SteamApiClient;
+use App\Service\Steam\SteamApiClientInterface;
 use App\Validator\AbstractValidator;
 use Doctrine\ORM\EntityManagerInterface;
 use Symfony\Component\Validator\Constraint;
@@ -20,7 +20,7 @@ class SteamWorkshopArma3ModUrlValidator extends AbstractValidator
 
     public function __construct(
         EntityManagerInterface $entityManager,
-        private SteamApiClient $steamApiClient
+        private SteamApiClientInterface $steamApiClient
     ) {
         parent::__construct($entityManager);
     }