fix: Use Illuminate\Http\UploadedFile to avoid conversion and ignore test property

homiedopie · homiedopie · commit 9e5c3f6d643d · 2026-02-16T06:04:26.000Z
diff --git a/src/Drivers/LaravelHttpServer.php b/src/Drivers/LaravelHttpServer.php
@@ -4,8 +4,6 @@
 
 namespace Pest\Browser\Drivers;
 
-use const UPLOAD_ERR_OK;
-
 use Amp\ByteStream\ReadableResourceStream;
 use Amp\Http\Cookie\RequestCookie;
 use Amp\Http\Server\DefaultErrorHandler;
@@ -21,6 +19,7 @@
 use Illuminate\Contracts\Http\Kernel as HttpKernel;
 use Illuminate\Foundation\Testing\Concerns\WithoutExceptionHandlingHandler;
 use Illuminate\Http\Request;
+use Illuminate\Http\UploadedFile;
 use Illuminate\Routing\UrlGenerator;
 use Illuminate\Support\Uri;
 use Pest\Browser\Contracts\HttpServer;
@@ -29,7 +28,6 @@
 use Pest\Browser\GlobalState;
 use Pest\Browser\Playwright\Playwright;
 use Psr\Log\NullLogger;
-use Symfony\Component\HttpFoundation\File\UploadedFile;
 use Symfony\Component\Mime\MimeTypes;
 use Throwable;
 
diff --git a/tests/Unit/Drivers/Laravel/LaravelHttpServerTest.php b/tests/Unit/Drivers/Laravel/LaravelHttpServerTest.php
@@ -120,6 +120,58 @@
     $page->assertSee('Empty file: ');
 });
 
+it('validates multipart pdf upload metadata', function (): void {
+    Route::get('favicon.ico', static fn (): string => '');
+    Route::get('/', static fn (): string => "
+        <html>
+        <head></head>
+        <body>
+            <form method='post' enctype='multipart/form-data' action='/form'>
+                <label for='pdf-file'>PDF file</label>
+                <input id='pdf-file' type='file' name='pdf_file'>
+
+                <button type='submit'>Send</button>
+            </form>
+        </body>
+        </html>
+    ");
+
+    $expectedPdfSize = filesize(fixture('example.pdf'));
+    assert($expectedPdfSize !== false);
+
+    Route::post('/form', static function (Request $request) use ($expectedPdfSize): string {
+        $pdf = $request->file('pdf_file');
+        $name = $pdf?->getClientOriginalName() ?? '';
+        $extension = $pdf?->getClientOriginalExtension() ?? '';
+        $valid = $pdf?->isValid() ? 'yes' : 'no';
+        $size = (string) ($pdf?->getSize() ?? '');
+
+        return "
+            <html>
+            <head></head>
+            <body>
+                <p>Name: $name</p>
+                <p>Extension: $extension</p>
+                <p>Valid: $valid</p>
+                <p>Size: $size</p>
+                <p>Expected size: $expectedPdfSize</p>
+            </body>
+            </html>
+        ";
+    });
+
+    $page = visit('/');
+
+    $page->attach('PDF file', fixture('example.pdf'));
+    $page->submit();
+
+    $page->assertSee('Name: example.pdf')
+        ->assertSee('Extension: pdf')
+        ->assertSee('Valid: yes')
+        ->assertSee('Expected size: '.$expectedPdfSize)
+        ->assertSee('Size: '.$expectedPdfSize);
+});
+
 it('parse a multipart body with nested fields', function (): void {
     Route::get('/', static fn (): string => "
         <html>
