Minify profile pictures

app/Actions/Fortify/UpdateUserProfileInformation.php

@@ -2,6 +2,7 @@
 
 namespace App\Actions\Fortify;
 
+use App\Jobs\MinifyProfilePhoto;
 use App\Models\User;
 use Illuminate\Contracts\Auth\MustVerifyEmail;
 use Illuminate\Http\UploadedFile;
@@ -22,13 +23,15 @@ public function update(User $user, array $input): void
         Validator::make($input, [
             'name' => ['required', 'string', 'max:255'],
             'email' => ['required', 'email', 'max:255', Rule::unique('users')->ignore($user->id)],
-            'photo' => ['nullable', 'mimes:jpg,jpeg,png', 'max:1024'],
+            'photo' => ['nullable', 'mimes:jpg,jpeg,png', 'max:15360'], // 15MB
         ])->validateWithBag('updateProfileInformation');
 
         if (isset($input['photo'])) {
             /** @var UploadedFile $photo */
             $photo = $input['photo'];
             $user->updateProfilePhoto($photo);
+
+            MinifyProfilePhoto::dispatch($user);
         }
 
         if ($input['email'] !== $user->email &&

app/Actions/Photos/ExtractLocationFromPhotoAction.php

@@ -3,15 +3,14 @@
 namespace App\Actions\Photos;
 
 use Illuminate\Http\UploadedFile;
-use Intervention\Image\Drivers\Gd\Driver;
 use Intervention\Image\ImageManager;
 
 class ExtractLocationFromPhotoAction implements ExtractsLocationFromPhoto
 {
     public function run(UploadedFile $photo): array
     {
-        $manager = new ImageManager(new Driver());
-        $image = $manager->read($photo);
+        $image = ImageManager::gd()->read($photo);
+
         $exif = $image->exif('GPS');
 
         if ($this->isExifInvalid($exif)) {

app/Jobs/MinifyProfilePhoto.php

@@ -0,0 +1,37 @@
+<?php
+
+namespace App\Jobs;
+
+use App\Models\User;
+use Illuminate\Bus\Queueable;
+use Illuminate\Contracts\Queue\ShouldQueue;
+use Illuminate\Foundation\Bus\Dispatchable;
+use Illuminate\Queue\InteractsWithQueue;
+use Illuminate\Queue\SerializesModels;
+use Illuminate\Support\Facades\Storage;
+use Intervention\Image\ImageManager;
+
+class MinifyProfilePhoto implements ShouldQueue
+{
+    use Dispatchable;
+    use InteractsWithQueue;
+    use Queueable;
+    use SerializesModels;
+
+    public function __construct(public readonly User $user)
+    {
+    }
+
+    public function handle(): void
+    {
+        if ($this->user->profile_photo_path === null) {
+            return;
+        }
+
+        $photo = Storage::disk('public')->path($this->user->profile_photo_path);
+
+        $image = ImageManager::gd()->read($photo);
+        $image->scaleDown(height: 500);
+        $image->save(quality: 50);
+    }
+}

tests/Feature/Auth/ProfileInformationTest.php

@@ -1,16 +1,28 @@
 <?php
 
+use App\Jobs\MinifyProfilePhoto;
 use App\Models\User;
+use Illuminate\Http\UploadedFile;
+use Illuminate\Support\Facades\Queue;
+use Illuminate\Support\Facades\Storage;
 
 test('profile information can be updated', function () {
+    Queue::fake();
+    Storage::fake('public');
     $this->actingAs($user = User::factory()->create());
 
     $response = $this->put('/user/profile-information', [
         'name' => 'Test Name',
         'email' => '[email protected]',
+        'photo' => UploadedFile::fake()->image('photo.png'),
     ]);
 
+    $response->assertRedirect();
     expect($user->fresh())
         ->name->toEqual('Test Name')
-        ->email->toEqual('[email protected]');
+        ->email->toEqual('[email protected]')
+        ->profile_photo_path->not()->toBeNull();
+
+    Storage::disk('public')->assertExists($user->fresh()->profile_photo_path);
+    Queue::assertPushed(MinifyProfilePhoto::class);
 });

tests/Feature/Jobs/MinifyProfilePhotoTest.php

@@ -0,0 +1,29 @@
+<?php
+
+use App\Jobs\MinifyProfilePhoto;
+use App\Models\User;
+use Illuminate\Support\Facades\Storage;
+use Intervention\Image\Drivers\Gd\Driver;
+
+test('it minifies the profile photo', function () {
+    Storage::fake('public');
+    Storage::disk('public')->put(
+        'profile-photos/default.jpg',
+        Storage::disk('local')->get('default.jpg')
+    );
+    $user = User::factory()->create([
+        'profile_photo_path' => 'profile-photos/default.jpg',
+    ]);
+    $previousSize = Storage::disk('public')->size($user->profile_photo_path);
+
+    $action = new MinifyProfilePhoto($user);
+    $action->handle(new Driver());
+
+    $this->assertFileExists(
+        Storage::disk('public')->path($user->profile_photo_path)
+    );
+    $this->assertLessThan(
+        $previousSize,
+        Storage::disk('public')->size($user->profile_photo_path)
+    );
+})->group('slow');

tests/Feature/Photos/UploadPhotosTest.php

@@ -45,7 +45,7 @@
     $photo = $user->photos()->first();
     expect($photo->latitude)->toBe(40.053030045789)
         ->and($photo->longitude)->toBe(-77.15449870066);
-});
+})->group('slow');
 
 test('a photo can not be larger than 20MB', function () {
     Storage::fake('public');