[UPDATE] Better tests

phpstan.neon

@@ -5,10 +5,6 @@ parameters:
  - 'src'
  - 'tests'
  ignoreErrors:
- -
- message: "#^Cannot use array destructuring on array\\<int, string\\|null\\>\\|null\\.$#"
- count: 1
- path: src/DataCollector/GotenbergDataCollector.php
  -
  message: "#^Method Sensiolabs\\\\GotenbergBundle\\\\Tests\\\\Kernel\\:\\:configureContainer\\(\\) is unused\\.$#"
  count: 1

src/Builder/Pdf/AbstractChromiumPdfBuilder.php

@@ -3,7 +3,7 @@
 namespace Sensiolabs\GotenbergBundle\Builder\Pdf;
 
 use Sensiolabs\GotenbergBundle\Client\GotenbergClientInterface;
-use Sensiolabs\GotenbergBundle\Enum\PaperSize;
+use Sensiolabs\GotenbergBundle\Enum\EmulatedMediaType;
 use Sensiolabs\GotenbergBundle\Enum\PaperSizeInterface;
 use Sensiolabs\GotenbergBundle\Enum\Part;
 use Sensiolabs\GotenbergBundle\Enum\PdfFormat;
@@ -315,9 +315,9 @@ public function waitForExpression(string $expression): static
  *
  * @see https://gotenberg.dev/docs/routes#console-exceptions
  */
- public function emulatedMediaType(string $mediaType): static
+ public function emulatedMediaType(EmulatedMediaType $mediaType): static
  {
- $this->formFields['emulatedMediaType'] = $mediaType;
+ $this->formFields['emulatedMediaType'] = $mediaType->value;
 
  return $this;
  }
@@ -526,7 +526,7 @@ protected function withRenderedPart(Part $pdfPart, string $template, array $cont
  private function addConfiguration(string $configurationName, mixed $value): void
  {
  $splitAndParseStringWithUnit = static function (mixed $raw, callable $callback): void {
- [$value, $unit] = sscanf((string) $raw, '%d%s') ?? throw new \InvalidArgumentException(sprintf('Unexpected value "%s", expected format is "%%d%%s"', $raw));
+ [$value, $unit] = sscanf((string) $raw, '%f%s') ?? throw new \InvalidArgumentException(sprintf('Unexpected value "%s", expected format is "%%d%%s"', $raw));
 
  $callback((float) $value, Unit::tryFrom((string) $unit) ?? Unit::Inches);
  };
@@ -535,8 +535,8 @@ private function addConfiguration(string $configurationName, mixed $value): void
  'single_page' => $this->singlePage($value),
  'pdf_format' => $this->pdfFormat(PdfFormat::from($value)),
  'pdf_universal_access' => $this->pdfUniversalAccess($value),
- 'paper_width' => $this->paperWidth($value),
- 'paper_height' => $this->paperHeight($value),
+ 'paper_width' => $splitAndParseStringWithUnit($value, $this->paperWidth(...)),
+ 'paper_height' => $splitAndParseStringWithUnit($value, $this->paperHeight(...)),
  'margin_top' => $splitAndParseStringWithUnit($value, $this->marginTop(...)),
  'margin_bottom' => $splitAndParseStringWithUnit($value, $this->marginBottom(...)),
  'margin_left' => $splitAndParseStringWithUnit($value, $this->marginLeft(...)),
@@ -549,7 +549,7 @@ private function addConfiguration(string $configurationName, mixed $value): void
  'native_page_ranges' => $this->nativePageRanges($value),
  'wait_delay' => $this->waitDelay($value),
  'wait_for_expression' => $this->waitForExpression($value),
- 'emulated_media_type' => $this->emulatedMediaType($value),
+ 'emulated_media_type' => $this->emulatedMediaType(EmulatedMediaType::from($value)),
  'cookies' => $this->cookies($value),
  'extra_http_headers' => $this->extraHttpHeaders($value),
  'fail_on_http_status_codes' => $this->failOnHttpStatusCodes($value),

src/Builder/Pdf/AbstractPdfBuilder.php

@@ -51,7 +51,7 @@ public function __construct(
  return $this->encodeData('failOnHttpStatusCodes', $value);
  },
  'cookies' => function (mixed $value): array {
- return $this->encodeData('cookies', \array_values($value));
+ return $this->encodeData('cookies', array_values($value));
  },
  'metadata' => function (mixed $value): array {
  return $this->encodeData('metadata', $value);
@@ -164,12 +164,22 @@ private function addToMultipart(string $key, array|string|int|float|bool|DataPar
  ]];
  }
 
- if (\is_int($value) || \is_float($value)) {
+ if (\is_int($value)) {
  return [[
  $key => (string) $value,
  ]];
  }
 
+ if (\is_float($value)) {
+ [$left, $right] = sscanf((string) $value, '%d.%s');
+
+ $right ??= '0';
+
+ return [[
+ $key => "{$left}.{$right}",
+ ]];
+ }
+
  if (\is_array($value)) {
  $result = [];
  foreach ($value as $nestedValue) {

src/DependencyInjection/Configuration.php

@@ -9,7 +9,6 @@
 use Symfony\Component\Config\Definition\Builder\NodeDefinition;
 use Symfony\Component\Config\Definition\Builder\TreeBuilder;
 use Symfony\Component\Config\Definition\ConfigurationInterface;
-use function array_map;
 
 class Configuration implements ConfigurationInterface
 {
@@ -205,7 +204,7 @@ private function addChromiumPdfOptionsNode(NodeBuilder $treeBuilder): void
  ->end()
  ->enumNode('emulated_media_type')
  ->info('The media type to emulate, either "screen" or "print" - default "print". https://gotenberg.dev/docs/routes#emulated-media-type')
- ->values(array_map(static fn(EmulatedMediaType $case): string => $case->value, EmulatedMediaType::cases()))
+ ->values(\array_map(static fn (EmulatedMediaType $case): string => $case->value, EmulatedMediaType::cases()))
  ->defaultNull()
  ->end()
  ->arrayNode('cookies')
@@ -274,7 +273,7 @@ private function addChromiumPdfOptionsNode(NodeBuilder $treeBuilder): void
  ->end()
  ->enumNode('pdf_format')
  ->info('Convert the resulting PDF into the given PDF/A format - default None. https://gotenberg.dev/docs/routes#pdfa-chromium')
- ->values(array_map(static fn(PdfFormat $case): string => $case->value, PdfFormat::cases()))
+ ->values(\array_map(static fn (PdfFormat $case): string => $case->value, PdfFormat::cases()))
  ->defaultNull()
  ->end()
  ->booleanNode('pdf_universal_access')
@@ -302,7 +301,7 @@ private function addChromiumScreenshotOptionsNode(NodeBuilder $treeBuilder): voi
  ->end()
  ->enumNode('format')
  ->info('The image compression format, either "png", "jpeg" or "webp" - default png. https://gotenberg.dev/docs/routes#screenshots-route')
- ->values(array_map(static fn(ScreenshotFormat $case): string => $case->value, ScreenshotFormat::cases()))
+ ->values(\array_map(static fn (ScreenshotFormat $case): string => $case->value, ScreenshotFormat::cases()))
  ->defaultNull()
  ->end()
  ->integerNode('quality')
@@ -341,7 +340,7 @@ private function addChromiumScreenshotOptionsNode(NodeBuilder $treeBuilder): voi
  ->end()
  ->enumNode('emulated_media_type')
  ->info('The media type to emulate, either "screen" or "print" - default "print". https://gotenberg.dev/docs/routes#emulated-media-type')
- ->values(array_map(static fn(EmulatedMediaType $case): string => $case->value, EmulatedMediaType::cases()))
+ ->values(\array_map(static fn (EmulatedMediaType $case): string => $case->value, EmulatedMediaType::cases()))
  ->defaultNull()
  ->end()
  ->arrayNode('cookies')
@@ -438,7 +437,7 @@ private function addPdfOfficeNode(): NodeDefinition
  ->end()
  ->enumNode('pdf_format')
  ->info('Convert the resulting PDF into the given PDF/A format - default None. https://gotenberg.dev/docs/routes#pdfa-chromium')
- ->values(array_map(static fn(PdfFormat $case): string => $case->value, PdfFormat::cases()))
+ ->values(\array_map(static fn (PdfFormat $case): string => $case->value, PdfFormat::cases()))
  ->defaultNull()
  ->end()
  ->booleanNode('pdf_universal_access')

tests/Builder/AbstractBuilderTestCase.php

@@ -2,7 +2,12 @@
 
 namespace Sensiolabs\GotenbergBundle\Tests\Builder;
 
+use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
+use Sensiolabs\GotenbergBundle\Client\GotenbergClientInterface;
+use Sensiolabs\GotenbergBundle\Formatter\AssetBaseDirFormatter;
+use Symfony\Component\Filesystem\Filesystem;
+use Symfony\Component\Mime\Part\DataPart;
 use Twig\Environment;
 use Twig\Loader\FilesystemLoader;
 
@@ -12,8 +17,35 @@ abstract class AbstractBuilderTestCase extends TestCase
 
  protected static Environment $twig;
 
+ /**
+ * @var MockObject&GotenbergClientInterface
+ */
+ protected static GotenbergClientInterface $gotenbergClient;
+ protected static AssetBaseDirFormatter $assetBaseDirFormatter;
+
  public static function setUpBeforeClass(): void
  {
- self::$twig = new Environment(new FilesystemLoader(self::FIXTURE_DIR.'/templates'));
+ self::$twig = new Environment(new FilesystemLoader(self::FIXTURE_DIR));
+ self::$assetBaseDirFormatter = new AssetBaseDirFormatter(new Filesystem(), self::FIXTURE_DIR, self::FIXTURE_DIR);
+ }
+
+ protected function setUp(): void
+ {
+ self::$gotenbergClient = $this->createMock(GotenbergClientInterface::class);
+ }
+
+ /**
+ * @param array<mixed> $data
+ */
+ protected function assertFile(array $data, string $filename, string $expectedContent): void
+ {
+ self::assertArrayHasKey('files', $data);
+
+ $file = $data['files'];
+
+ self::assertInstanceOf(DataPart::class, $file);
+
+ self::assertSame($expectedContent, $file->getBody());
+ self::assertSame($filename, $file->getFilename());
  }
 }