diff --git a/spec/ValueHandler/FileAttributeValueHandlerSpec.php b/spec/ValueHandler/FileAttributeValueHandlerSpec.php index b5925d1c..c52e68d6 100644 --- a/spec/ValueHandler/FileAttributeValueHandlerSpec.php +++ b/spec/ValueHandler/FileAttributeValueHandlerSpec.php @@ -17,6 +17,7 @@ use Sylius\Component\Core\Model\ProductInterface; use Sylius\Component\Core\Model\ProductVariantInterface; use Symfony\Component\Filesystem\Filesystem; +use Webgriffe\SyliusAkeneoPlugin\TemporaryFilesManagerInterface; use Webgriffe\SyliusAkeneoPlugin\ValueHandler\FileAttributeValueHandler; use Webgriffe\SyliusAkeneoPlugin\ValueHandlerInterface; use Webmozart\Assert\InvalidArgumentException; @@ -31,7 +32,8 @@ public function let( MediaFileApiInterface $productMediaFileApi, Filesystem $filesystem, ProductVariantInterface $productVariant, - ProductInterface $product + ProductInterface $product, + TemporaryFilesManagerInterface $temporaryFilesManager, ): void { $commerceChannel = new Channel(); $commerceChannel->setCode('ecommerce'); @@ -43,7 +45,8 @@ public function let( $apiClient->getProductMediaFileApi()->willReturn($productMediaFileApi); $productMediaFileApi->download(Argument::type('string'))->willReturn(new Response(200, [], '__FILE_CONTENT__')); $attributeApi->get('allegato_1')->willReturn(['type' => 'pim_catalog_file']); - $this->beConstructedWith($apiClient, $filesystem, 'allegato_1', 'public/media/attachment/product/'); + $temporaryFilesManager->generateTemporaryFilePath()->willReturn('tempfile'); + $this->beConstructedWith($apiClient, $filesystem, $temporaryFilesManager, 'allegato_1', 'public/media/attachment/product/'); } public function it_is_initializable(): void diff --git a/spec/ValueHandler/ImageValueHandlerSpec.php b/spec/ValueHandler/ImageValueHandlerSpec.php index 587556d8..437e0303 100644 --- a/spec/ValueHandler/ImageValueHandlerSpec.php +++ b/spec/ValueHandler/ImageValueHandlerSpec.php @@ -19,6 +19,7 @@ use Sylius\Component\Resource\Factory\FactoryInterface; use Sylius\Component\Resource\Repository\RepositoryInterface; use Webgriffe\SyliusAkeneoPlugin\ApiClientInterface; +use Webgriffe\SyliusAkeneoPlugin\TemporaryFilesManagerInterface; use Webgriffe\SyliusAkeneoPlugin\ValueHandler\ImageValueHandler; class ImageValueHandlerSpec extends ObjectBehavior @@ -45,7 +46,8 @@ public function let( ResponseInterface $downloadResponse, StreamInterface $responseBody, ProductVariantInterface $productVariant, - ProductInterface $product + ProductInterface $product, + TemporaryFilesManagerInterface $temporaryFilesManager, ): void { $productImageFactory->createNew()->willReturn($productImage); $apiClient->getProductMediaFileApi()->willReturn($productMediaFileApi); @@ -66,10 +68,12 @@ public function let( $productImageRepository ->findBy(['owner' => $product, 'type' => self::SYLIUS_IMAGE_TYPE]) ->willReturn(new ArrayCollection([])); + $temporaryFilesManager->generateTemporaryFilePath()->willReturn('tempfile'); $this->beConstructedWith( $productImageFactory, $productImageRepository, $apiClient, + $temporaryFilesManager, self::AKENEO_ATTRIBUTE_CODE, self::SYLIUS_IMAGE_TYPE ); diff --git a/src/ValueHandler/ImageValueHandler.php b/src/ValueHandler/ImageValueHandler.php index cb5e30f4..fc2c360b 100644 --- a/src/ValueHandler/ImageValueHandler.php +++ b/src/ValueHandler/ImageValueHandler.php @@ -15,7 +15,7 @@ use Sylius\Component\Resource\Repository\RepositoryInterface; use Symfony\Component\HttpFoundation\File\File; use Symfony\Component\HttpKernel\Exception\HttpException; -use Webgriffe\SyliusAkeneoPlugin\TemporaryFilesManager; +use Webgriffe\SyliusAkeneoPlugin\TemporaryFilesManagerInterface; use Webgriffe\SyliusAkeneoPlugin\ValueHandlerInterface; use Webmozart\Assert\Assert; @@ -25,7 +25,7 @@ public function __construct( private FactoryInterface $productImageFactory, private RepositoryInterface $productImageRepository, private AkeneoPimClientInterface $apiClient, - private TemporaryFilesManager $temporaryFilesManager, + private TemporaryFilesManagerInterface $temporaryFilesManager, private string $akeneoAttributeCode, private string $syliusImageType, ) {