Merge pull request 'fix(import): extraire les images CBZ vers le stockage individuel' (#15) from fix/import-cbz-image-storage into main

Reviewed-on: #15

TASK.md

@@ -75,6 +75,47 @@
 
 ---
 
+## [Perf] Reader — Lazy-loading des pages (InfiniteReader)
+
+**Problème :** `readerStore.js` charge toutes les pages avec `itemsPerPage=9999`. `InfiniteReader.vue` monte tous les composants `ReaderPage` simultanément dans le DOM. Sur un chapitre de 200 pages, cela représente 200 composants actifs et autant d'images pré-chargées.
+
+- [ ] Implémenter un `IntersectionObserver` sur les wrappers de page pour ne charger les images qu'au moment où elles entrent dans le viewport (`loading="lazy"` ou src conditionnel)
+- [ ] Limiter le nombre de composants montés simultanément (virtualisation ou windowing) : ne rendre que les pages proches de la page courante (ex. fenêtre de ±3 pages)
+- [ ] Adapter `readerStore.js` : remplacer `itemsPerPage=9999` par la vraie pagination côté API si la virtualisation le justifie, sinon conserver le fetch unique mais différer le rendu
+- [ ] Vérifier que le mode `single` n'est pas impacté (il affiche déjà une seule page)
+
+---
+
+## [Bug] Reader — N+1 requêtes SQL dans `getChapterContext()`
+
+**Problème :** `LegacyChapterRepository::getChapterContext()` émet 5 requêtes SQL pour un seul chargement : la requête principale + 2 doublons dans `getPreviousChapterId()` / `getNextChapterId()` (chacune re-fetche le chapitre courant) + les 2 requêtes de navigation.
+
+- [ ] Refactorer `getPreviousChapterId()` et `getNextChapterId()` pour accepter l'entité `ChapterEntity` déjà chargée en paramètre (au lieu de re-fetcher par ID)
+- [ ] Appeler ces méthodes depuis `getChapterContext()` en passant l'entité déjà disponible
+- [ ] Résultat attendu : 3 requêtes maximum (1 pour le chapitre courant + 1 prev + 1 next), idéalement 1 seule avec une requête SQL combinée
+
+---
+
+## [Bug] Reader — Division par zéro dans `ChapterPagesResponse::getTotalPages()`
+
+**Problème :** `ceil($totalItems / $itemsPerPage)` crashe si `itemsPerPage = 0`. Le test existant documente le bug avec un TODO et assert un HTTP 500 au lieu de corriger.
+
+- [ ] Ajouter une validation dans `ChapterPagesProvider` : rejeter la requête avec HTTP 400 si `itemsPerPage <= 0`
+- [ ] Corriger le test `GetChapterPagesTest` pour vérifier HTTP 400 (et non 500)
+- [ ] Supprimer le commentaire TODO du test une fois corrigé
+
+---
+
+## [Bug] Reader — `totalPages` toujours égal à 0 dans `ChapterContext`
+
+**Problème :** `LegacyChapterRepository::getChapterContext()` hardcode `totalPages: 0`. La méthode `getTotalPagesForChapter()` existe mais n'est jamais appelée depuis `GetChapterContextHandler`.
+
+- [ ] Appeler `getTotalPagesForChapter()` dans `getChapterContext()` (ou dans le handler) pour calculer le vrai nombre de pages
+- [ ] Vérifier que la valeur est correctement sérialisée dans la réponse API Platform (`ChapterContextResponse`)
+- [ ] Adapter les tests existants qui pourraient asserter `totalPages: 0`
+
+---
+
 ## [Style] Page conversion CBR → CBZ — Simplification UI + notifications toast
 
 **Objectif :** Revoir le style de la page de conversion CBR → CBZ pour le simplifier, et remplacer le message statique "Conversion réussie" par les notifications toast de l'application.

assets/vue/app/domain/reader/presentation/components/ChapterReader.vue

@@ -22,6 +22,7 @@
                     :pages="store.pages"
                     :zoom="store.zoom"
                     :double-page-mode="store.effectiveDoublePageMode"
+                    :initial-page="store.currentPage"
                     @page-visible="store.handlePageVisible"
                     ref="infiniteReaderRef" />
             </template>

assets/vue/app/domain/reader/presentation/components/InfiniteReader.vue

@@ -1,10 +1,15 @@
 <template>
     <div class="infinite-reader" ref="containerRef">
-        <div v-for="(page, index) in pages" :key="index" class="page-wrapper">
-            <div v-if="!page?.url" class="loading">
-                <div class="animate-spin rounded-full h-12 w-12 border-b-2 border-blue-600"></div>
-            </div>
-            <ReaderPage v-else :page-data="page" :page-number="index + 1" :zoom="zoom" :double-page-mode="doublePageMode" loading="lazy" />
+        <div v-for="(page, index) in pages" :key="index" class="page-wrapper" :data-page-index="index">
+            <ReaderPage
+                v-if="isPageInWindow(index) && page?.url"
+                :page-data="page"
+                :page-number="index + 1"
+                :zoom="zoom"
+                :double-page-mode="doublePageMode"
+                loading="eager"
+            />
+            <div v-else class="page-placeholder" :style="getPlaceholderStyle(page)" />
         </div>
 
         <!-- Bouton flottant pour revenir en haut -->
@@ -37,6 +42,25 @@
 import { useHeaderStore } from '../../../../shared/stores/headerStore';
 import ReaderPage from './ReaderPage.vue';
 
+    const WINDOW_SIZE = 3;
+    const currentVisibleIndex = ref(0); // initialisé via prop initialPage dans onMounted
+
+    const isPageInWindow = (index) => Math.abs(index - currentVisibleIndex.value) <= WINDOW_SIZE;
+
+    const getPlaceholderStyle = (page) => {
+        if (page?.dimensions?.width && page?.dimensions?.height) {
+            const maxW = windowWidth.value < 1200
+                ? windowWidth.value * 0.95
+                : 1200;
+            return {
+                aspectRatio: `${page.dimensions.width} / ${page.dimensions.height}`,
+                width: '100%',
+                maxWidth: `${Math.min(page.dimensions.width, maxW)}px`,
+            };
+        }
+        return { height: '800px', width: '100%' };
+    };
+
     const props = defineProps({
         pages: {
             type: Array,
@@ -49,6 +73,10 @@ import ReaderPage from './ReaderPage.vue';
         doublePageMode: {
             type: String,
             required: true
+        },
+        initialPage: {
+            type: Number,
+            default: 0
         }
     });
 
@@ -68,12 +96,15 @@ import ReaderPage from './ReaderPage.vue';
     let scrollDirection = 'down';
 
     const observeIntersection = entries => {
-        entries.forEach(entry => {
-            if (entry.isIntersecting) {
-                const pageIndex = parseInt(entry.target.getAttribute('data-page-index'));
-                emit('pageVisible', pageIndex);
-            }
-        });
+        const intersectingIndices = entries
+            .filter(e => e.isIntersecting)
+            .map(e => parseInt(e.target.getAttribute('data-page-index')));
+
+        if (intersectingIndices.length > 0) {
+            const minIdx = Math.min(...intersectingIndices);
+            currentVisibleIndex.value = minIdx;
+            emit('pageVisible', minIdx);
+        }
     };
 
     const setupIntersectionObserver = () => {
@@ -89,8 +120,7 @@ import ReaderPage from './ReaderPage.vue';
         nextTick(() => {
             const pageElements = containerRef.value?.querySelectorAll('.page-wrapper');
             if (pageElements) {
-                pageElements.forEach((element, index) => {
-                    element.setAttribute('data-page-index', index);
+                pageElements.forEach(element => {
                     observer.value.observe(element);
                 });
             }
@@ -247,6 +277,7 @@ import ReaderPage from './ReaderPage.vue';
     };
 
     onMounted(() => {
+        currentVisibleIndex.value = props.initialPage;
         setupIntersectionObserver();
 
         // Activer l'auto-hide du header si la largeur < 1200px
@@ -303,29 +334,26 @@ import ReaderPage from './ReaderPage.vue';
         @apply mb-2 sm:mb-4 px-1 sm:px-4;
     }
 
-    .loading,
+    .page-placeholder {
+        @apply flex justify-center;
+        background: transparent;
+    }
+
     .error {
-        @apply flex items-center justify-center min-h-[400px];
-        /* Largeur adaptative selon la taille d'écran */
-        width: 95vw; /* Mobile : 95% de la largeur */
+        @apply text-red-500 text-xl bg-red-500/10 rounded-lg flex items-center justify-center min-h-[400px];
+        width: 95vw;
     }
 
     @screen sm {
-        .loading,
         .error {
-            width: 80vw; /* Tablette : 80% de la largeur */
+            width: 80vw;
         }
     }
 
     @screen lg {
-        .loading,
         .error {
-            width: 70vw; /* Desktop : 70% de la largeur */
+            width: 70vw;
         }
     }
 
-    .error {
-        @apply text-red-500 text-xl bg-red-500/10 rounded-lg;
-    }
-
 </style>

assets/vue/app/domain/reader/presentation/components/ReaderPage.vue

@@ -15,6 +15,7 @@
                     :alt="`Page ${pageNumber} (Double page)`"
                     class="page-image rotated"
                     :style="doublePageRotatedStyle"
+                    :loading="loading"
                     @load="handleImageLoad"
                     ref="imageRef" />
                 <div class="rotation-hint">
@@ -33,6 +34,7 @@
                         :alt="`Page ${pageNumber} (Double page)`"
                         class="page-image scrollable"
                         :style="doublePageScrollStyle"
+                        :loading="loading"
                         @load="handleImageLoad"
                         ref="imageRef" />
                 </div>
@@ -52,6 +54,7 @@
             :alt="`Page ${pageNumber}`"
             class="page-image"
             :style="imageStyle"
+            :loading="loading"
             @load="handleImageLoad"
             ref="imageRef" />
     </div>
@@ -78,6 +81,10 @@ import { useReaderStore } from '../../application/store/readerStore';
             type: String,
             default: 'rotate', // 'rotate', 'scroll', 'normal'
             validator: (value) => ['rotate', 'scroll', 'normal'].includes(value)
+        },
+        loading: {
+            type: String,
+            default: 'lazy',
         }
     });
 

config/services.yaml

@@ -126,10 +126,10 @@ services:
         tags:
             - { name: messenger.message_handler, bus: command.bus }
 
-    App\Domain\Scraping\Domain\Contract\Service\ImageStorageInterface:
-        alias: App\Domain\Scraping\Infrastructure\Service\LocalImageStorage
+    App\Domain\Shared\Domain\Contract\ImageStorageInterface:
+        alias: App\Domain\Shared\Infrastructure\Service\ImageStorageManager
 
-    App\Domain\Scraping\Infrastructure\Service\LocalImageStorage:
+    App\Domain\Shared\Infrastructure\Service\ImageStorageManager:
         arguments:
             $storagePath: '%kernel.project_dir%/public/images'
             

config/services_test.yaml

@@ -12,7 +12,7 @@ services:
         class: 'App\Tests\Domain\Scraping\Adapter\InMemoryScrapingJobRepository'
         public: true
 
-    App\Domain\Scraping\Domain\Contract\Service\ImageStorageInterface:
+    App\Domain\Shared\Domain\Contract\ImageStorageInterface:
         class: 'App\Tests\Domain\Scraping\Adapter\InMemoryImageStorage'
         public: true
 

src/Domain/Manga/Application/CommandHandler/ImportChapterHandler.php

@@ -6,13 +6,13 @@ use App\Domain\Manga\Application\Command\ImportChapter;
 use App\Domain\Manga\Domain\Contract\Repository\MangaRepositoryInterface;
 use App\Domain\Manga\Domain\Exception\MangaNotFoundException;
 use App\Domain\Manga\Domain\Exception\ChapterNotFoundException;
-use App\Domain\Shared\Domain\Contract\MangaPathManagerInterface;
+use App\Domain\Shared\Domain\Contract\ImageStorageInterface;
 
 readonly class ImportChapterHandler
 {
     public function __construct(
         private MangaRepositoryInterface $mangaRepository,
-        private MangaPathManagerInterface $pathManager
+        private ImageStorageInterface $imageStorage
     ) {
     }
 
@@ -39,11 +39,15 @@ readonly class ImportChapterHandler
             throw new ChapterNotFoundException("Chapter {$command->chapterNumber} not found for manga {$command->mangaId}");
         }
 
-        // 4. Save the CBZ file to storage using the path manager
-        $cbzPath = $this->saveCbzFile($command, $manga, $existingChapter);
+        // 4. Extract CBZ into individual images storage
+        $pagesDirectory = $this->imageStorage->extractFromCbz(
+            $existingChapter->getId(),
+            $command->fileBinary
+        );
+        $pageCount = $this->imageStorage->countCbzImages($command->fileBinary);
 
         // 5. Update existing chapter with new path through the aggregate
-        $manga->updateChapterPages($existingChapter, $cbzPath, $existingChapter->getPageCount());
+        $manga->updateChapterPages($existingChapter, $pagesDirectory, $pageCount);
         $this->mangaRepository->save($manga);
     }
 
@@ -53,21 +57,4 @@ readonly class ImportChapterHandler
 
         return strpos($fileBinary, $zipMagicNumber) === 0;
     }
-
-    private function saveCbzFile(ImportChapter $command, \App\Domain\Manga\Domain\Model\Manga $manga, \App\Domain\Manga\Domain\Model\Chapter $chapter): string
-    {
-        $volumeNumber = $chapter->getVolume() ?? 0;
-        $cbzPath = $this->pathManager->buildChapterCbzPath(
-            $manga->getTitle()->getValue(),
-            (string)$manga->getPublicationYear(),
-            $volumeNumber,
-            (string)$command->chapterNumber
-        );
-
-        if (!file_put_contents($cbzPath, $command->fileBinary)) {
-            throw new \RuntimeException('Failed to save CBZ file');
-        }
-
-        return $cbzPath;
-    }
 }

src/Domain/Manga/Application/CommandHandler/ImportVolumeHandler.php

@@ -5,13 +5,13 @@ namespace App\Domain\Manga\Application\CommandHandler;
 use App\Domain\Manga\Application\Command\ImportVolume;
 use App\Domain\Manga\Domain\Contract\Repository\MangaRepositoryInterface;
 use App\Domain\Manga\Domain\Exception\MangaNotFoundException;
-use App\Domain\Shared\Domain\Contract\MangaPathManagerInterface;
+use App\Domain\Shared\Domain\Contract\ImageStorageInterface;
 
 readonly class ImportVolumeHandler
 {
     public function __construct(
         private MangaRepositoryInterface $mangaRepository,
-        private MangaPathManagerInterface $pathManager
+        private ImageStorageInterface $imageStorage
     ) {
     }
 
@@ -40,12 +40,14 @@ readonly class ImportVolumeHandler
             );
         }
 
-        // 4. Save the CBZ file to storage using the path manager
-        $cbzPath = $this->saveCbzFile($command, $manga);
+        // 4. Extract CBZ into individual images storage (shared directory for all volume chapters)
+        $volumeDirectoryId = sprintf('volume_%s_%d', $command->mangaId, $command->volumeNumber);
+        $pagesDirectory = $this->imageStorage->extractFromCbz($volumeDirectoryId, $command->fileBinary);
+        $pageCount = $this->imageStorage->countCbzImages($command->fileBinary);
 
         // 5. Update all chapters with the volume path through the aggregate
         foreach ($chapters as $chapter) {
-            $manga->updateChapterPages($chapter, $cbzPath, $chapter->getPageCount());
+            $manga->updateChapterPages($chapter, $pagesDirectory, $pageCount);
         }
         $this->mangaRepository->save($manga);
     }
@@ -56,19 +58,4 @@ readonly class ImportVolumeHandler
 
         return strpos($fileBinary, $zipMagicNumber) === 0;
     }
-
-    private function saveCbzFile(ImportVolume $command, \App\Domain\Manga\Domain\Model\Manga $manga): string
-    {
-        $cbzPath = $this->pathManager->buildVolumeCbzPath(
-            $manga->getTitle()->getValue(),
-            (string)$manga->getPublicationYear(),
-            $command->volumeNumber
-        );
-
-        if (!file_put_contents($cbzPath, $command->fileBinary)) {
-            throw new \RuntimeException('Failed to save CBZ file');
-        }
-
-        return $cbzPath;
-    }
 }

src/Domain/Reader/Infrastructure/Persistence/LegacyChapterRepository.php

@@ -154,7 +154,7 @@ readonly class LegacyChapterRepository implements ChapterRepositoryInterface
             $pages[] = new Page(
                 basename($files[$i]),
                 new PageNumber($i + 1),
-                sprintf('/images/pages/%s/%s', $chapterId->getValue(), basename($files[$i])),
+                sprintf('/images/pages/%s/%s', basename($pagesDirectory), basename($files[$i])),
                 $imageSize[0],
                 $imageSize[1]
             );

src/Domain/Scraping/Application/CommandHandler/ScrapeChapterHandler.php

@@ -6,7 +6,7 @@ use App\Domain\Scraping\Application\Command\ScrapeChapter;
 use App\Domain\Scraping\Domain\Contract\Repository\ChapterRepositoryInterface;
 use App\Domain\Scraping\Domain\Contract\Repository\MangaRepositoryInterface;
 use App\Domain\Scraping\Domain\Contract\Repository\SourceRepositoryInterface;
-use App\Domain\Scraping\Domain\Contract\Service\ImageStorageInterface;
+use App\Domain\Shared\Domain\Contract\ImageStorageInterface;
 use App\Domain\Scraping\Domain\Contract\Service\ImageDownloaderInterface;
 use App\Domain\Scraping\Domain\Contract\Service\ScraperFactoryInterface;
 use App\Domain\Shared\Domain\Event\ChapterScraped;

src/Domain/Scraping/Domain/Contract/Service/ImageStorageInterface.php

@@ -1,16 +0,0 @@
-<?php
-
-namespace App\Domain\Scraping\Domain\Contract\Service;
-
-interface ImageStorageInterface
-{
-    /**
-     * Copies images to permanent storage. Returns the pagesDirectory path.
-     *
-     * @param string   $chapterId       The chapter UUID used as directory name
-     * @param string[] $localImagePaths Paths to the locally downloaded image files
-     *
-     * @return string Absolute path to the directory where images were stored
-     */
-    public function storeChapterImages(string $chapterId, array $localImagePaths): string;
-}

src/Domain/Scraping/Infrastructure/Service/LocalImageStorage.php

@@ -1,31 +0,0 @@
-<?php
-
-namespace App\Domain\Scraping\Infrastructure\Service;
-
-use App\Domain\Scraping\Domain\Contract\Service\ImageStorageInterface;
-
-readonly class LocalImageStorage implements ImageStorageInterface
-{
-    public function __construct(private string $storagePath)
-    {
-    }
-
-    public function storeChapterImages(string $chapterId, array $localImagePaths): string
-    {
-        $targetDir = $this->storagePath . '/pages/' . $chapterId;
-
-        if (!is_dir($targetDir)) {
-            mkdir($targetDir, 0755, true);
-        }
-
-        sort($localImagePaths);
-
-        foreach ($localImagePaths as $index => $localPath) {
-            $extension = pathinfo($localPath, PATHINFO_EXTENSION) ?: 'jpg';
-            $targetFile = sprintf('%s/%03d.%s', $targetDir, $index + 1, $extension);
-            copy($localPath, $targetFile);
-        }
-
-        return $targetDir;
-    }
-}

src/Domain/Shared/Domain/Contract/ImageStorageInterface.php

@@ -0,0 +1,28 @@
+<?php
+
+namespace App\Domain\Shared\Domain\Contract;
+
+interface ImageStorageInterface
+{
+    /**
+     * Store images from local file paths into the individual images storage.
+     * Used by the scraping flow.
+     *
+     * @param  string[] $localImagePaths
+     * @return string The directory path where images are stored (pagesDirectory)
+     */
+    public function storeChapterImages(string $targetId, array $localImagePaths): string;
+
+    /**
+     * Extract images from a CBZ binary into the individual images storage.
+     * Used by the import flow.
+     *
+     * @return string The directory path where images are stored (pagesDirectory)
+     */
+    public function extractFromCbz(string $targetId, string $cbzBinary): string;
+
+    /**
+     * Count images in a CBZ binary.
+     */
+    public function countCbzImages(string $cbzBinary): int;
+}

src/Domain/Shared/Infrastructure/Service/ImageStorageManager.php

@@ -0,0 +1,97 @@
+<?php
+
+namespace App\Domain\Shared\Infrastructure\Service;
+
+use App\Domain\Shared\Domain\Contract\ImageStorageInterface;
+use ZipArchive;
+
+class ImageStorageManager implements ImageStorageInterface
+{
+    public function __construct(private string $storagePath)
+    {
+    }
+
+    public function storeChapterImages(string $targetId, array $localImagePaths): string
+    {
+        $targetDir = $this->storagePath . '/pages/' . $targetId;
+
+        if (!is_dir($targetDir)) {
+            mkdir($targetDir, 0755, true);
+        }
+
+        sort($localImagePaths);
+
+        foreach ($localImagePaths as $index => $localPath) {
+            $extension = pathinfo($localPath, PATHINFO_EXTENSION) ?: 'jpg';
+            $targetFile = sprintf('%s/%03d.%s', $targetDir, $index + 1, $extension);
+            copy($localPath, $targetFile);
+        }
+
+        return $targetDir;
+    }
+
+    public function extractFromCbz(string $targetId, string $cbzBinary): string
+    {
+        $targetDir = $this->storagePath . '/pages/' . $targetId;
+
+        if (!is_dir($targetDir)) {
+            mkdir($targetDir, 0755, true);
+        }
+
+        $tmpFile = tempnam(sys_get_temp_dir(), 'cbz_');
+        file_put_contents($tmpFile, $cbzBinary);
+
+        $zip = new ZipArchive();
+        if ($zip->open($tmpFile) !== true) {
+            unlink($tmpFile);
+            throw new \RuntimeException('Failed to open CBZ file as ZIP archive');
+        }
+
+        $imageEntries = [];
+        for ($i = 0; $i < $zip->numFiles; $i++) {
+            $name = $zip->getNameIndex($i);
+            if (preg_match('/\.(jpg|jpeg|png|webp|gif)$/i', $name)) {
+                $imageEntries[] = ['index' => $i, 'name' => $name];
+            }
+        }
+
+        usort($imageEntries, fn ($a, $b) => strcmp($a['name'], $b['name']));
+
+        foreach ($imageEntries as $seq => $entry) {
+            $extension = strtolower(pathinfo($entry['name'], PATHINFO_EXTENSION)) ?: 'jpg';
+            $targetFile = sprintf('%s/%03d.%s', $targetDir, $seq + 1, $extension);
+            $content = $zip->getFromIndex($entry['index']);
+            file_put_contents($targetFile, $content);
+        }
+
+        $zip->close();
+        unlink($tmpFile);
+
+        return $targetDir;
+    }
+
+    public function countCbzImages(string $cbzBinary): int
+    {
+        $tmpFile = tempnam(sys_get_temp_dir(), 'cbz_');
+        file_put_contents($tmpFile, $cbzBinary);
+
+        $zip = new ZipArchive();
+        if ($zip->open($tmpFile) !== true) {
+            unlink($tmpFile);
+            throw new \RuntimeException('Failed to open CBZ file as ZIP archive');
+        }
+
+        $count = 0;
+        for ($i = 0; $i < $zip->numFiles; $i++) {
+            $name = $zip->getNameIndex($i);
+            if (preg_match('/\.(jpg|jpeg|png|webp|gif)$/i', $name)) {
+                $count++;
+            }
+        }
+
+        $zip->close();
+        unlink($tmpFile);
+
+        return $count;
+    }
+}

tests/Domain/Manga/Application/CommandHandler/ImportChapterHandlerTest.php

@@ -13,22 +13,22 @@ use App\Domain\Manga\Domain\Model\ValueObject\MangaId;
 use App\Domain\Manga\Domain\Model\ValueObject\MangaSlug;
 use App\Domain\Manga\Domain\Model\ValueObject\MangaTitle;
 use App\Tests\Domain\Manga\Adapter\InMemoryMangaRepository;
-use App\Tests\Domain\Manga\Adapter\InMemoryPathManager;
+use App\Tests\Domain\Scraping\Adapter\InMemoryImageStorage;
 use PHPUnit\Framework\TestCase;
 
 class ImportChapterHandlerTest extends TestCase
 {
     private InMemoryMangaRepository $mangaRepository;
-    private InMemoryPathManager $pathManager;
+    private InMemoryImageStorage $imageStorage;
     private ImportChapterHandler $handler;
 
     protected function setUp(): void
     {
         $this->mangaRepository = new InMemoryMangaRepository();
-        $this->pathManager = new InMemoryPathManager();
+        $this->imageStorage = new InMemoryImageStorage();
         $this->handler = new ImportChapterHandler(
             $this->mangaRepository,
-            $this->pathManager
+            $this->imageStorage
         );
     }
 

tests/Domain/Manga/Application/CommandHandler/ImportVolumeHandlerTest.php

@@ -12,22 +12,22 @@ use App\Domain\Manga\Domain\Model\ValueObject\MangaId;
 use App\Domain\Manga\Domain\Model\ValueObject\MangaSlug;
 use App\Domain\Manga\Domain\Model\ValueObject\MangaTitle;
 use App\Tests\Domain\Manga\Adapter\InMemoryMangaRepository;
-use App\Tests\Domain\Manga\Adapter\InMemoryPathManager;
+use App\Tests\Domain\Scraping\Adapter\InMemoryImageStorage;
 use PHPUnit\Framework\TestCase;
 
 class ImportVolumeHandlerTest extends TestCase
 {
     private InMemoryMangaRepository $mangaRepository;
-    private InMemoryPathManager $pathManager;
+    private InMemoryImageStorage $imageStorage;
     private ImportVolumeHandler $handler;
 
     protected function setUp(): void
     {
         $this->mangaRepository = new InMemoryMangaRepository();
-        $this->pathManager = new InMemoryPathManager();
+        $this->imageStorage = new InMemoryImageStorage();
         $this->handler = new ImportVolumeHandler(
             $this->mangaRepository,
-            $this->pathManager
+            $this->imageStorage
         );
     }
 

tests/Domain/Scraping/Adapter/InMemoryImageStorage.php

@@ -2,18 +2,31 @@
 
 namespace App\Tests\Domain\Scraping\Adapter;
 
-use App\Domain\Scraping\Domain\Contract\Service\ImageStorageInterface;
+use App\Domain\Shared\Domain\Contract\ImageStorageInterface;
 
 class InMemoryImageStorage implements ImageStorageInterface
 {
-    /** @var array<string, string> chapterId => pagesDirectory */
+    /** @var array<string, string> targetId => pagesDirectory */
     public array $stored = [];
 
-    public function storeChapterImages(string $chapterId, array $localImagePaths): string
+    public function storeChapterImages(string $targetId, array $localImagePaths): string
     {
-        $dir = '/fake/pages/' . $chapterId;
-        $this->stored[$chapterId] = $dir;
+        $dir = '/fake/pages/' . $targetId;
+        $this->stored[$targetId] = $dir;
 
         return $dir;
     }
+
+    public function extractFromCbz(string $targetId, string $cbzBinary): string
+    {
+        $dir = '/fake/pages/' . $targetId;
+        $this->stored[$targetId] = $dir;
+
+        return $dir;
+    }
+
+    public function countCbzImages(string $cbzBinary): int
+    {
+        return 0;
+    }
 }