.env.example

@@ -12,9 +12,7 @@ APP_FALLBACK_LOCALE=en
 APP_FAKER_LOCALE=en_US
 
 APP_MAINTENANCE_DRIVER=file
-# APP_MAINTENANCE_STORE=database
+APP_MAINTENANCE_STORE=database
-
-PHP_CLI_SERVER_WORKERS=4
 
 BCRYPT_ROUNDS=12
 
@@ -41,7 +39,7 @@ FILESYSTEM_DISK=local
 QUEUE_CONNECTION=database
 
 CACHE_STORE=database
-# CACHE_PREFIX=
+CACHE_PREFIX=
 
 MEMCACHED_HOST=127.0.0.1
 
@@ -51,7 +49,6 @@ REDIS_PASSWORD=null
 REDIS_PORT=6379
 
 MAIL_MAILER=log
-MAIL_SCHEME=null
 MAIL_HOST=127.0.0.1
 MAIL_PORT=2525
 MAIL_USERNAME=null

.gitignore

@@ -21,5 +21,3 @@ yarn-error.log
 /.idea
 /.vscode
 ray.php
-/public/gpg.key
-/public/assets/img/favicon.png

.phpactor.json

@@ -1,5 +0,0 @@
-{
-    "$schema": "/Users/jonny/git/phpactor/phpactor.schema.json",
-    "language_server_phpstan.enabled": false,
-    "language_server_psalm.enabled": true
-}

app/Console/Commands/CopyMediaToLocal.php

@@ -1,69 +0,0 @@
-<?php
-
-namespace App\Console\Commands;
-
-use App\Models\Media;
-use Illuminate\Console\Command;
-use Illuminate\Support\Facades\Storage;
-
-class CopyMediaToLocal extends Command
-{
-    /**
-     * The name and signature of the console command.
-     *
-     * @var string
-     */
-    protected $signature = 'app:copy-media-to-local';
-
-    /**
-     * The console command description.
-     *
-     * @var string
-     */
-    protected $description = 'Copy any historic media saved to S3 to the local filesystem';
-
-    /**
-     * Execute the console command.
-     */
-    public function handle()
-    {
-        // Load all the Media records
-        $media = Media::all();
-
-        // Loop through each media record and copy the file from S3 to the local filesystem
-        foreach ($media as $mediaItem) {
-            $filename = $mediaItem->path;
-
-            $this->info('Processing: ' . $filename);
-
-            // If the file is already saved locally skip to next one
-            if (Storage::disk('local')->exists('public/' . $filename)) {
-                $this->info('File already exists locally, skipping');
-
-                continue;
-            }
-
-            // Copy the file from S3 to the local filesystem
-            if (! Storage::disk('s3')->exists($filename)) {
-                $this->error('File does not exist on S3');
-
-                continue;
-            }
-            $contents = Storage::disk('s3')->get($filename);
-            Storage::disk('local')->put('public/' . $filename, $contents);
-
-            // Copy -medium and -small versions if they exist
-            $filenameParts = explode('.', $filename);
-            $extension = array_pop($filenameParts);
-            $basename = trim(implode('.', $filenameParts), '.');
-            $mediumFilename = $basename . '-medium.' . $extension;
-            $smallFilename = $basename . '-small.' . $extension;
-            if (Storage::disk('s3')->exists($mediumFilename)) {
-                Storage::disk('local')->put('public/' . $mediumFilename, Storage::disk('s3')->get($mediumFilename));
-            }
-            if (Storage::disk('s3')->exists($smallFilename)) {
-                Storage::disk('local')->put('public/' . $smallFilename, Storage::disk('s3')->get($smallFilename));
-            }
-        }
-    }
-}

app/Http/Controllers/Admin/ContactsController.php

@@ -40,7 +40,7 @@ class ContactsController extends Controller
      */
     public function store(): RedirectResponse
     {
-        $contact = new Contact;
+        $contact = new Contact();
         $contact->name = request()->input('name');
         $contact->nick = request()->input('nick');
         $contact->homepage = request()->input('homepage');
@@ -79,7 +79,7 @@ class ContactsController extends Controller
         if (request()->hasFile('avatar') && (request()->input('homepage') != '')) {
             $dir = parse_url(request()->input('homepage'), PHP_URL_HOST);
             $destination = public_path() . '/assets/profile-images/' . $dir;
-            $filesystem = new Filesystem;
+            $filesystem = new Filesystem();
             if ($filesystem->isDirectory($destination) === false) {
                 $filesystem->makeDirectory($destination);
             }
@@ -139,7 +139,7 @@ class ContactsController extends Controller
             }
             if ($avatar !== null) {
                 $directory = public_path() . '/assets/profile-images/' . parse_url($contact->homepage, PHP_URL_HOST);
-                $filesystem = new Filesystem;
+                $filesystem = new Filesystem();
                 if ($filesystem->isDirectory($directory) === false) {
                     $filesystem->makeDirectory($directory);
                 }

app/Http/Controllers/Admin/PasskeysController.php

@@ -116,8 +116,8 @@ class PasskeysController extends Controller
             throw new WebAuthnException('No public key credential request options found');
         }
 
-        $attestationStatementSupportManager = new AttestationStatementSupportManager;
+        $attestationStatementSupportManager = new AttestationStatementSupportManager();
-        $attestationStatementSupportManager->add(new NoneAttestationStatementSupport);
+        $attestationStatementSupportManager->add(new NoneAttestationStatementSupport());
 
         $webauthnSerializer = (new WebauthnSerializerFactory(
             $attestationStatementSupportManager
@@ -133,12 +133,12 @@ class PasskeysController extends Controller
             throw new WebAuthnException('Invalid response type');
         }
 
-        $algorithmManager = new Manager;
+        $algorithmManager = new Manager();
-        $algorithmManager->add(new Ed25519);
+        $algorithmManager->add(new Ed25519());
-        $algorithmManager->add(new ES256);
+        $algorithmManager->add(new ES256());
-        $algorithmManager->add(new RS256);
+        $algorithmManager->add(new RS256());
 
-        $ceremonyStepManagerFactory = new CeremonyStepManagerFactory;
+        $ceremonyStepManagerFactory = new CeremonyStepManagerFactory();
         $ceremonyStepManagerFactory->setAlgorithmManager($algorithmManager);
         $ceremonyStepManagerFactory->setAttestationStatementSupportManager(
             $attestationStatementSupportManager
@@ -206,8 +206,8 @@ class PasskeysController extends Controller
             ], 400);
         }
 
-        $attestationStatementSupportManager = new AttestationStatementSupportManager;
+        $attestationStatementSupportManager = new AttestationStatementSupportManager();
-        $attestationStatementSupportManager->add(new NoneAttestationStatementSupport);
+        $attestationStatementSupportManager->add(new NoneAttestationStatementSupport());
 
         $webauthnSerializer = (new WebauthnSerializerFactory(
             $attestationStatementSupportManager
@@ -240,15 +240,15 @@ class PasskeysController extends Controller
             'json'
         );
 
-        $algorithmManager = new Manager;
+        $algorithmManager = new Manager();
-        $algorithmManager->add(new Ed25519);
+        $algorithmManager->add(new Ed25519());
-        $algorithmManager->add(new ES256);
+        $algorithmManager->add(new ES256());
-        $algorithmManager->add(new RS256);
+        $algorithmManager->add(new RS256());
 
-        $attestationStatementSupportManager = new AttestationStatementSupportManager;
+        $attestationStatementSupportManager = new AttestationStatementSupportManager();
-        $attestationStatementSupportManager->add(new NoneAttestationStatementSupport);
+        $attestationStatementSupportManager->add(new NoneAttestationStatementSupport());
 
-        $ceremonyStepManagerFactory = new CeremonyStepManagerFactory;
+        $ceremonyStepManagerFactory = new CeremonyStepManagerFactory();
         $ceremonyStepManagerFactory->setAlgorithmManager($algorithmManager);
         $ceremonyStepManagerFactory->setAttestationStatementSupportManager(
             $attestationStatementSupportManager

app/Http/Controllers/ContactsController.php

@@ -18,7 +18,7 @@ class ContactsController extends Controller
      */
     public function index(): View
     {
-        $filesystem = new Filesystem;
+        $filesystem = new Filesystem();
         $contacts = Contact::all();
         foreach ($contacts as $contact) {
             $contact->homepageHost = parse_url($contact->homepage, PHP_URL_HOST);
@@ -40,7 +40,7 @@ class ContactsController extends Controller
         $contact->homepageHost = parse_url($contact->homepage, PHP_URL_HOST);
         $file = public_path() . '/assets/profile-images/' . $contact->homepageHost . '/image';
 
-        $filesystem = new Filesystem;
+        $filesystem = new Filesystem();
         $image = ($filesystem->exists($file)) ?
             '/assets/profile-images/' . $contact->homepageHost . '/image'
         :

app/Http/Controllers/MicropubController.php

@@ -53,13 +53,13 @@ class MicropubController extends Controller
         try {
             $tokenData = $this->tokenService->validateToken($request->input('access_token'));
         } catch (RequiredConstraintsViolated|InvalidTokenStructure|CannotDecodeContent) {
-            $micropubResponses = new MicropubResponses;
+            $micropubResponses = new MicropubResponses();
 
             return $micropubResponses->invalidTokenResponse();
         }
 
         if ($tokenData->claims()->has('scope') === false) {
-            $micropubResponses = new MicropubResponses;
+            $micropubResponses = new MicropubResponses();
 
             return $micropubResponses->tokenHasNoScopeResponse();
         }
@@ -73,7 +73,7 @@ class MicropubController extends Controller
             }
 
             if (! in_array('create', $scopes)) {
-                $micropubResponses = new MicropubResponses;
+                $micropubResponses = new MicropubResponses();
 
                 return $micropubResponses->insufficientScopeResponse();
             }
@@ -91,7 +91,7 @@ class MicropubController extends Controller
                 $scopes = explode(' ', $scopes);
             }
             if (! in_array('create', $scopes)) {
-                $micropubResponses = new MicropubResponses;
+                $micropubResponses = new MicropubResponses();
 
                 return $micropubResponses->insufficientScopeResponse();
             }
@@ -109,7 +109,7 @@ class MicropubController extends Controller
                 $scopes = explode(' ', $scopes);
             }
             if (! in_array('update', $scopes)) {
-                $micropubResponses = new MicropubResponses;
+                $micropubResponses = new MicropubResponses();
 
                 return $micropubResponses->insufficientScopeResponse();
             }
@@ -136,7 +136,7 @@ class MicropubController extends Controller
         try {
             $tokenData = $this->tokenService->validateToken($request->input('access_token'));
         } catch (RequiredConstraintsViolated|InvalidTokenStructure) {
-            return (new MicropubResponses)->invalidTokenResponse();
+            return (new MicropubResponses())->invalidTokenResponse();
         }
 
         if ($request->input('q') === 'syndicate-to') {

app/Http/Controllers/MicropubMediaController.php

@@ -39,13 +39,13 @@ class MicropubMediaController extends Controller
         try {
             $tokenData = $this->tokenService->validateToken($request->input('access_token'));
         } catch (RequiredConstraintsViolated|InvalidTokenStructure) {
-            $micropubResponses = new MicropubResponses;
+            $micropubResponses = new MicropubResponses();
 
             return $micropubResponses->invalidTokenResponse();
         }
 
         if ($tokenData->claims()->has('scope') === false) {
-            $micropubResponses = new MicropubResponses;
+            $micropubResponses = new MicropubResponses();
 
             return $micropubResponses->tokenHasNoScopeResponse();
         }
@@ -55,7 +55,7 @@ class MicropubMediaController extends Controller
             $scopes = explode(' ', $scopes);
         }
         if (! in_array('create', $scopes)) {
-            $micropubResponses = new MicropubResponses;
+            $micropubResponses = new MicropubResponses();
 
             return $micropubResponses->insufficientScopeResponse();
         }
@@ -111,13 +111,13 @@ class MicropubMediaController extends Controller
         try {
             $tokenData = $this->tokenService->validateToken($request->input('access_token'));
         } catch (RequiredConstraintsViolated|InvalidTokenStructure) {
-            $micropubResponses = new MicropubResponses;
+            $micropubResponses = new MicropubResponses();
 
             return $micropubResponses->invalidTokenResponse();
         }
 
         if ($tokenData->claims()->has('scope') === false) {
-            $micropubResponses = new MicropubResponses;
+            $micropubResponses = new MicropubResponses();
 
             return $micropubResponses->tokenHasNoScopeResponse();
         }
@@ -127,7 +127,7 @@ class MicropubMediaController extends Controller
             $scopes = explode(' ', $scopes);
         }
         if (! in_array('create', $scopes)) {
-            $micropubResponses = new MicropubResponses;
+            $micropubResponses = new MicropubResponses();
 
             return $micropubResponses->insufficientScopeResponse();
         }
@@ -140,10 +140,7 @@ class MicropubMediaController extends Controller
             ], 400);
         }
 
-        /** @var UploadedFile $file */
+        if ($request->file('file')->isValid() === false) {
-        $file = $request->file('file');
-
-        if ($file->isValid() === false) {
             return response()->json([
                 'response' => 'error',
                 'error' => 'invalid_request',
@@ -151,7 +148,7 @@ class MicropubMediaController extends Controller
             ], 400);
         }
 
-        $filename = Storage::disk('local')->putFile('media', $file);
+        $filename = $this->saveFile($request->file('file'));
 
         /** @var ImageManager $manager */
         $manager = resolve(ImageManager::class);
@@ -165,11 +162,18 @@ class MicropubMediaController extends Controller
 
         $media = Media::create([
             'token' => $request->bearerToken(),
-            'path' => $filename,
+            'path' => 'media/' . $filename,
             'type' => $this->getFileTypeFromMimeType($request->file('file')->getMimeType()),
             'image_widths' => $width,
         ]);
 
+        // put the file on S3 initially, the ProcessMedia job may edit this
+        Storage::disk('s3')->putFileAs(
+            'media',
+            new File(storage_path('app') . '/' . $filename),
+            $filename
+        );
+
         ProcessMedia::dispatch($filename);
 
         return response()->json([
@@ -233,7 +237,7 @@ class MicropubMediaController extends Controller
      *
      * @throws Exception
      */
-    private function saveFileToLocal(UploadedFile $file): string
+    private function saveFile(UploadedFile $file): string
     {
         $filename = Uuid::uuid4()->toString() . '.' . $file->extension();
         Storage::disk('local')->putFileAs('', $file, $filename);

app/Http/Controllers/NotesController.php

@@ -67,7 +67,7 @@ class NotesController extends Controller
      */
     public function redirect(int $decId): RedirectResponse
     {
-        return redirect(config('app.url') . '/notes/' . (new Numbers)->numto60($decId));
+        return redirect(config('app.url') . '/notes/' . (new Numbers())->numto60($decId));
     }
 
     /**

app/Jobs/DownloadWebMention.php

@@ -38,7 +38,7 @@ class DownloadWebMention implements ShouldQueue
         //4XX and 5XX responses should get Guzzle to throw an exception,
         //Laravel should catch and retry these automatically.
         if ($response->getStatusCode() === 200) {
-            $filesystem = new FileSystem;
+            $filesystem = new FileSystem();
             $filename = storage_path('HTML') . '/' . $this->createFilenameFromURL($this->source);
             //backup file first
             $filenameBackup = $filename . '.' . date('Y-m-d') . '.backup';

app/Jobs/ProcessMedia.php

@@ -32,38 +32,35 @@ class ProcessMedia implements ShouldQueue
      */
     public function handle(ImageManager $manager): void
     {
-        // Load file
+        //open file
-        $file = Storage::disk('local')->get('media/' . $this->filename);
-
-        // Open file
         try {
-            $image = $manager->read($file);
+            $image = $manager->read(storage_path('app') . '/' . $this->filename);
         } catch (DecoderException) {
             // not an image; delete file and end job
-            Storage::disk('local')->delete('media/' . $this->filename);
+            unlink(storage_path('app') . '/' . $this->filename);
 
             return;
         }
-
+        //create smaller versions if necessary
-        // Save the file publicly
-        Storage::disk('public')->put('media/' . $this->filename, $file);
-
-        // Create smaller versions if necessary
         if ($image->width() > 1000) {
             $filenameParts = explode('.', $this->filename);
             $extension = array_pop($filenameParts);
             // the following achieves this data flow
             // foo.bar.png => ['foo', 'bar', 'png'] => ['foo', 'bar'] => foo.bar
-            $basename = trim(implode('.', $filenameParts), '.');
+            $basename = ltrim(array_reduce($filenameParts, function ($carry, $item) {
-
+                return $carry . '.' . $item;
-            $medium = $image->resize(width: 1000);
+            }, ''), '.');
-            Storage::disk('public')->put('media/' . $basename . '-medium.' . $extension, (string) $medium->encode());
+            $medium = $image->resize(1000, null, function ($constraint) {
-
+                $constraint->aspectRatio();
-            $small = $image->resize(width: 500);
+            });
-            Storage::disk('public')->put('media/' . $basename . '-small.' . $extension, (string) $small->encode());
+            Storage::disk('s3')->put('media/' . $basename . '-medium.' . $extension, (string) $medium->encode());
+            $small = $image->resize(500, null, function ($constraint) {
+                $constraint->aspectRatio();
+            });
+            Storage::disk('s3')->put('media/' . $basename . '-small.' . $extension, (string) $small->encode());
         }
 
-        // Now we can delete the locally saved image
+        // now we can delete the locally saved image
-        Storage::disk('local')->delete('media/' . $this->filename);
+        unlink(storage_path('app') . '/' . $this->filename);
     }
 }

app/Jobs/ProcessWebMention.php

@@ -44,7 +44,7 @@ class ProcessWebMention implements ShouldQueue
         try {
             $response = $guzzle->request('GET', $this->source);
         } catch (RequestException $e) {
-            throw new RemoteContentNotFoundException;
+            throw new RemoteContentNotFoundException();
         }
         $this->saveRemoteContent((string) $response->getBody(), $this->source);
         $microformats = Mf2\parse((string) $response->getBody(), $this->source);
@@ -85,7 +85,7 @@ class ProcessWebMention implements ShouldQueue
         }// foreach
 
         // no webmention in the db so create new one
-        $webmention = new WebMention;
+        $webmention = new WebMention();
         $type = $parser->getMentionType($microformats); // throw error here?
         dispatch(new SaveProfileImage($microformats));
         $webmention->source = $this->source;

app/Jobs/SendWebMentions.php

@@ -108,7 +108,7 @@ class SendWebMentions implements ShouldQueue
         }
 
         $urls = [];
-        $dom = new \DOMDocument;
+        $dom = new \DOMDocument();
         $dom->loadHTML($html);
         $anchors = $dom->getElementsByTagName('a');
         foreach ($anchors as $anchor) {

app/Models/Article.php

@@ -58,10 +58,10 @@ class Article extends Model
     {
         return Attribute::get(
             get: function () {
-                $environment = new Environment;
+                $environment = new Environment();
-                $environment->addExtension(new CommonMarkCoreExtension);
+                $environment->addExtension(new CommonMarkCoreExtension());
-                $environment->addRenderer(FencedCode::class, new FencedCodeRenderer);
+                $environment->addRenderer(FencedCode::class, new FencedCodeRenderer());
-                $environment->addRenderer(IndentedCode::class, new IndentedCodeRenderer);
+                $environment->addRenderer(IndentedCode::class, new IndentedCodeRenderer());
                 $markdownConverter = new MarkdownConverter($environment);
 
                 return $markdownConverter->convert($this->main)->getContent();

app/Models/Media.php

@@ -33,7 +33,7 @@ class Media extends Model
                     return $attributes['path'];
                 }
 
-                return config('app.url') . '/storage/' . $attributes['path'];
+                return config('filesystems.disks.s3.url') . '/' . $attributes['path'];
             }
         );
     }
@@ -78,7 +78,7 @@ class Media extends Model
         $basename = $this->getBasename($path);
         $extension = $this->getExtension($path);
 
-        return config('app.url') . '/storage/' . $basename . '-' . $size . '.' . $extension;
+        return config('filesystems.disks.s3.url') . '/' . $basename . '-' . $size . '.' . $extension;
     }
 
     private function getBasename(string $path): string

app/Models/Note.php

@@ -271,7 +271,7 @@ class Note extends Model
             ]);
 
             if ($oEmbed->httpstatus >= 400) {
-                throw new Exception;
+                throw new Exception();
             }
         } catch (Exception $e) {
             return null;
@@ -388,18 +388,18 @@ class Note extends Model
                 'mentions_handle' => [
                     'prefix' => '@',
                     'pattern' => '([\w@.])+(\b)',
-                    'generator' => new MentionGenerator,
+                    'generator' => new MentionGenerator(),
                 ],
             ],
         ];
 
         $environment = new Environment($config);
-        $environment->addExtension(new CommonMarkCoreExtension);
+        $environment->addExtension(new CommonMarkCoreExtension());
-        $environment->addExtension(new AutolinkExtension);
+        $environment->addExtension(new AutolinkExtension());
-        $environment->addExtension(new MentionExtension);
+        $environment->addExtension(new MentionExtension());
-        $environment->addRenderer(Mention::class, new MentionRenderer);
+        $environment->addRenderer(Mention::class, new MentionRenderer());
-        $environment->addRenderer(FencedCode::class, new FencedCodeRenderer);
+        $environment->addRenderer(FencedCode::class, new FencedCodeRenderer());
-        $environment->addRenderer(IndentedCode::class, new IndentedCodeRenderer);
+        $environment->addRenderer(IndentedCode::class, new IndentedCodeRenderer());
         $markdownConverter = new MarkdownConverter($environment);
 
         return $markdownConverter->convert($note)->getContent();

app/Models/WebMention.php

@@ -42,7 +42,7 @@ class WebMention extends Model
                     return null;
                 }
 
-                $authorship = new Authorship;
+                $authorship = new Authorship();
                 $hCard = $authorship->findAuthor(json_decode($attributes['mf2'], true));
 
                 if ($hCard === false) {
@@ -140,7 +140,7 @@ class WebMention extends Model
             return $profile_image;
         }
 
-        $filesystem = new Filesystem;
+        $filesystem = new Filesystem();
         if ($filesystem->exists(public_path() . '/assets/profile-images/' . $host . '/image')) {
             return '/assets/profile-images/' . $host . '/image';
         }

app/Providers/AppServiceProvider.php

@@ -88,9 +88,9 @@ class AppServiceProvider extends ServiceProvider
         $this->app->bind('Lcobucci\JWT\Configuration', function () {
             $key = InMemory::plainText(config('app.key'));
 
-            $config = Configuration::forSymmetricSigner(new Sha256, $key);
+            $config = Configuration::forSymmetricSigner(new Sha256(), $key);
 
-            $config->setValidationConstraints(new SignedWith(new Sha256, $key));
+            $config->setValidationConstraints(new SignedWith(new Sha256(), $key));
 
             return $config;
         });
@@ -98,7 +98,7 @@ class AppServiceProvider extends ServiceProvider
         // Configure HtmlSanitizer
         $this->app->bind(HtmlSanitizer::class, function () {
             return new HtmlSanitizer(
-                (new HtmlSanitizerConfig)
+                (new HtmlSanitizerConfig())
                     ->allowSafeElements()
                     ->forceAttribute('a', 'rel', 'noopener nofollow')
             );

app/Services/BookmarkService.php

@@ -62,7 +62,7 @@ class BookmarkService extends Service
             $response = $client->request('GET', 'https://web.archive.org/save/' . $url);
         } catch (ClientException $e) {
             //throw an exception to be caught
-            throw new InternetArchiveException;
+            throw new InternetArchiveException();
         }
         if ($response->hasHeader('Content-Location')) {
             if (Str::startsWith(Arr::get($response->getHeader('Content-Location'), 0), '/web')) {
@@ -71,6 +71,6 @@ class BookmarkService extends Service
         }
 
         //throw an exception to be caught
-        throw new InternetArchiveException;
+        throw new InternetArchiveException();
     }
 }

app/Services/Micropub/UpdateService.php

@@ -83,7 +83,7 @@ class UpdateService
                 if ($property === 'photo') {
                     foreach ($value as $photoURL) {
                         if (Str::startsWith($photoURL, 'https://')) {
-                            $media = new Media;
+                            $media = new Media();
                             $media->path = $photoURL;
                             $media->type = 'image';
                             $media->save();

app/Services/PlaceService.php

@@ -25,7 +25,7 @@ class PlaceService
             $data['latitude'] = $matches[0][0];
             $data['longitude'] = $matches[0][1];
         }
-        $place = new Place;
+        $place = new Place();
         $place->name = $data['name'];
         $place->description = $data['description'];
         $place->latitude = $data['latitude'];
@@ -53,7 +53,7 @@ class PlaceService
         if (Arr::has($checkin, 'properties.latitude') === false) {
             throw new \InvalidArgumentException('Missing required longitude/latitude');
         }
-        $place = new Place;
+        $place = new Place();
         $place->name = Arr::get($checkin, 'properties.name.0');
         $place->external_urls = Arr::get($checkin, 'properties.url.0');
         $place->latitude = Arr::get($checkin, 'properties.latitude.0');

app/Services/TokenService.php

@@ -19,7 +19,7 @@ class TokenService
         $config = resolve(Configuration::class);
 
         $token = $config->builder()
-            ->issuedAt(new DateTimeImmutable)
+            ->issuedAt(new DateTimeImmutable())
             ->withClaim('client_id', $data['client_id'])
             ->withClaim('me', $data['me'])
             ->withClaim('scope', $data['scope'])

composer.json

@@ -1,8 +1,7 @@
 {
-    "$schema": "https://getcomposer.org/schema.json",
     "name": "jonnybarnes/jonnybarnes.uk",
     "type": "project",
-    "description": "The code for jonnybarnes.uk, based on Laravel 11",
+    "description": "The code for jonnybarnes.uk, based on Laravel 10",
     "keywords": ["laravel", "framework", "indieweb"],
     "license": "CC0-1.0",
     "require": {
@@ -12,14 +11,14 @@
         "ext-json": "*",
         "ext-pgsql": "*",
         "ext-sodium": "*",
-        "cviebrock/eloquent-sluggable": "^12.0",
+        "cviebrock/eloquent-sluggable": "^11.0",
         "guzzlehttp/guzzle": "^7.2",
         "indieauth/client": "^1.1",
         "intervention/image": "^3",
         "jonnybarnes/indieweb": "~0.2",
         "jonnybarnes/webmentions-parser": "~0.5",
         "jublonet/codebird-php": "4.0.0-beta.1",
-        "laravel/framework": "^12.0",
+        "laravel/framework": "^11.0",
         "laravel/horizon": "^5.0",
         "laravel/sanctum": "^4.0",
         "laravel/scout": "^10.1",
@@ -41,15 +40,16 @@
         "barryvdh/laravel-ide-helper": "^3.0",
         "fakerphp/faker": "^1.9.2",
         "laravel/dusk": "^8.0",
-        "laravel/pail": "^1.2",
         "laravel/pint": "^1.0",
         "laravel/sail": "^1.18",
         "mockery/mockery": "^1.4.4",
         "nunomaduro/collision": "^8.1",
         "openai-php/client": "^0.10.1",
-        "phpunit/php-code-coverage": "^11.0",
+        "phpunit/php-code-coverage": "^10.0",
-        "phpunit/phpunit": "^11.0",
+        "phpunit/phpunit": "^10.1",
-        "spatie/laravel-ray": "^1.12"
+        "psalm/plugin-laravel": "^2.8",
+        "spatie/laravel-ray": "^1.12",
+        "vimeo/psalm": "^5.0"
     },
     "autoload": {
         "psr-4": {
@@ -78,13 +78,7 @@
             "@php -r \"file_exists('.env') || copy('.env.example', '.env');\""
         ],
         "post-create-project-cmd": [
-            "@php artisan key:generate --ansi",
+            "@php artisan key:generate --ansi"
-            "@php -r \"file_exists('database/database.sqlite') || touch('database/database.sqlite');\"",
-            "@php artisan migrate --graceful --ansi"
-        ],
-        "dev": [
-            "Composer\\Config::disableProcessTimeout",
-            "npx concurrently -c \"#93c5fd,#c4b5fd,#fb7185,#fdba74\" \"php artisan serve\" \"php artisan queue:listen --tries=1\" \"php artisan pail --timeout=0\" \"npm run dev\" --names=server,queue,logs,vite"
         ]
     },
     "extra": {

config/app.php

@@ -65,7 +65,7 @@ return [
     |
     */
 
-    'timezone' => 'UTC',
+    'timezone' => env('APP_TIMEZONE', 'UTC'),
 
     /*
     |--------------------------------------------------------------------------

config/database.php

@@ -37,9 +37,6 @@ return [
             'database' => env('DB_DATABASE', database_path('database.sqlite')),
             'prefix' => '',
             'foreign_key_constraints' => env('DB_FOREIGN_KEYS', true),
-            'busy_timeout' => null,
-            'journal_mode' => null,
-            'synchronous' => null,
         ],
 
         'mysql' => [
@@ -148,7 +145,6 @@ return [
         'options' => [
             'cluster' => env('REDIS_CLUSTER', 'redis'),
             'prefix' => env('REDIS_PREFIX', Str::slug(env('APP_NAME', 'laravel'), '_').'_database_'),
-            'persistent' => env('REDIS_PERSISTENT', false),
         ],
 
         'default' => [

config/filesystems.php

@@ -32,10 +32,8 @@ return [
 
         'local' => [
             'driver' => 'local',
-            'root' => storage_path('app/private'),
+            'root' => storage_path('app'),
-            'serve' => true,
             'throw' => false,
-            'report' => false,
         ],
 
         'public' => [
@@ -44,7 +42,6 @@ return [
             'url' => env('APP_URL').'/storage',
             'visibility' => 'public',
             'throw' => false,
-            'report' => false,
         ],
 
         's3' => [
@@ -57,7 +54,6 @@ return [
             'endpoint' => env('AWS_ENDPOINT'),
             'use_path_style_endpoint' => env('AWS_USE_PATH_STYLE_ENDPOINT', false),
             'throw' => false,
-            'report' => false,
         ],
 
     ],

config/mail.php

@@ -38,14 +38,14 @@ return [
 
         'smtp' => [
             'transport' => 'smtp',
-            'scheme' => env('MAIL_SCHEME'),
             'url' => env('MAIL_URL'),
             'host' => env('MAIL_HOST', '127.0.0.1'),
             'port' => env('MAIL_PORT', 2525),
+            'encryption' => env('MAIL_ENCRYPTION', 'tls'),
             'username' => env('MAIL_USERNAME'),
             'password' => env('MAIL_PASSWORD'),
             'timeout' => null,
-            'local_domain' => env('MAIL_EHLO_DOMAIN', parse_url(env('APP_URL', 'http://localhost'), PHP_URL_HOST)),
+            'local_domain' => env('MAIL_EHLO_DOMAIN'),
         ],
 
         'ses' => [

config/session.php

@@ -32,7 +32,7 @@ return [
     |
     */
 
-    'lifetime' => (int) env('SESSION_LIFETIME', 120),
+    'lifetime' => env('SESSION_LIFETIME', 120),
 
     'expire_on_close' => env('SESSION_EXPIRE_ON_CLOSE', false),
 

database/seeders/ContactsTableSeeder.php

@@ -27,7 +27,7 @@ class ContactsTableSeeder extends Seeder
             'homepage' => 'https://aaronparecki.com',
             'facebook' => '123456',
         ]);
-        $fs = new FileSystem;
+        $fs = new FileSystem();
         if (! $fs->exists(public_path('assets/profile-images/aaronparecki.com'))) {
             $fs->makeDirectory(public_path('assets/profile-images/aaronparecki.com'));
         }

database/seeders/LikesTableSeeder.php

@@ -20,7 +20,7 @@ class LikesTableSeeder extends Seeder
         Like::factory(10)->create();
 
         $now = Carbon::now()->subDays(rand(3, 6));
-        $faker = new Generator;
+        $faker = new Generator();
         $faker->addProvider(new \Faker\Provider\en_US\Person($faker));
         $faker->addProvider(new \Faker\Provider\Lorem($faker));
         $faker->addProvider(new \Faker\Provider\Internet($faker));

database/seeders/NotesTableSeeder.php

@@ -154,7 +154,7 @@ class NotesTableSeeder extends Seeder
             ->update(['updated_at' => $now->toDateTimeString()]);
 
         $now = Carbon::now()->subHours(5);
-        $noteJustCheckin = new Note;
+        $noteJustCheckin = new Note();
         $noteJustCheckin->setCreatedAt($now);
         $place = Place::find(1);
         $noteJustCheckin->place()->associate($place);
@@ -164,12 +164,12 @@ class NotesTableSeeder extends Seeder
             ->update(['updated_at' => $now->toDateTimeString()]);
 
         $now = Carbon::now()->subHours(4);
-        $media = new Media;
+        $media = new Media();
         $media->path = 'media/f1bc8faa-1a8f-45b8-a9b1-57282fa73f87.jpg';
         $media->type = 'image';
         $media->image_widths = '3648';
         $media->save();
-        $noteWithOnlyImage = new Note;
+        $noteWithOnlyImage = new Note();
         $noteWithOnlyImage->setCreatedAt($now);
         $noteWithOnlyImage->setUpdatedAt($now);
         $noteWithOnlyImage->save();

database/seeders/PlacesTableSeeder.php

@@ -14,7 +14,7 @@ class PlacesTableSeeder extends Seeder
      */
     public function run(): void
     {
-        $place = new Place;
+        $place = new Place();
         $place->name = 'The Bridgewater Pub';
         $place->description = 'A lovely local pub with a decent selection of cask ales';
         $place->latitude = 53.4983;

database/seeders/WebMentionsTableSeeder.php

@@ -41,14 +41,5 @@ class WebMentionsTableSeeder extends Seeder
             'type' => 'repost-of',
             'mf2' => '{"rels": [], "items": [{"type": ["h-entry"], "properties": {"url": ["https://barryfrost.com/reposts/1"], "name": ["Kagi is the best"], "author": [{"type": ["h-card"], "value": "Barry Frost", "properties": {"url": ["https://barryfrost.com/"], "name": ["Barry Frost"], "photo": ["https://barryfrost.com/barryfrost.jpg"]}}], "content": [{"html": "Kagi is the Best", "value": "Kagi is the Best"}], "published": ["' . date(DATE_W3C) . '"], "u-repost-of": ["' . config('app.url') . '/notes/C"]}}]}',
         ]);
-        // WebMention like from Bluesky
-        WebMention::create([
-            'source' => 'https://brid.gy/like/bluesky/did:plc:n3jhgiq2ykctnpgzlm6p6b25/at%253A%252F%252Fdid%253Aplc%253An3jhgiq2ykctnpgzlm6p6b25%252Fapp.bsky.feed.post%252F3lalppbcyuc2w/did%253Aplc%253Aia23nh3t37r2lydmmqsixrps',
-            'target' => config('app.url') . '/notes/B',
-            'commentable_id' => '11',
-            'commentable_type' => 'App\Models\Note',
-            'type' => 'like-of',
-            'mf2' => '{"rels": [], "items": [{"type": ["h-entry"], "properties": {"uid": ["tag:bsky.app,2013:at://did:plc:n3jhgiq2ykctnpgzlm6p6b25/app.bsky.feed.post/3lalppbcyuc2w_liked_by_did:plc:ia23nh3t37r2lydmmqsixrps"], "url": ["https://bsky.app/profile/jonnybarnes.uk/post/3lalppbcyuc2w#liked_by_did:plc:ia23nh3t37r2lydmmqsixrps"], "name": [""], "author": [{"type": ["h-card"], "value": "bsky.app/profile/little... littledawg13.bsky.social", "properties": {"uid": ["tag:bsky.app,2013:did:plc:ia23nh3t37r2lydmmqsixrps"], "url": ["https://bsky.app/profile/littledawg13.bsky.social", "https://bsky.app/profile/did:plc:ia23nh3t37r2lydmmqsixrps"], "photo": [{"alt": "", "value": "https://cdn.bsky.app/img/avatar/plain/did:plc:ia23nh3t37r2lydmmqsixrps/bafkreifh7ydbyq7qe4maorornocksfnahijxqgx2i5zvvyq6y4i4mydfau@jpeg"}], "nickname": ["littledawg13.bsky.social"]}}], "like-of": ["https://bsky.app/profile/jonnybarnes.uk/post/3lalppbcyuc2w", "http://jonnybarnes.localhost/notes/B"]}}], "rel-urls": []}',
-        ]);
     }
 }

package.json

@@ -7,11 +7,11 @@
     "license": "CC0-1.0",
     "devDependencies": {
         "@eslint/js": "^9.6.0",
-        "@stylistic/eslint-plugin": "^3.0.0",
+        "@stylistic/eslint-plugin": "^2.3.0",
         "eslint": "^9.7.0",
         "globals": "^15.8.0",
         "stylelint": "^16.7.0",
-        "stylelint-config-standard": "^37.0.0"
+        "stylelint-config-standard": "^36.0.1"
     },
     "scripts": {
         "eslint": "eslint public/assets/js/*.js",

phpunit.xml

@@ -24,7 +24,7 @@
         <env name="CACHE_STORE" value="array"/>
         <!-- <env name="DB_CONNECTION" value="sqlite"/> -->
         <!-- <env name="DB_DATABASE" value=":memory:"/> -->
-        <env name="DB_DATABASE" value="jbuk_dev_testing"/>
+        <env name="DB_DATABASE" value="jbukdev_testing"/>
         <env name="MAIL_MAILER" value="array"/>
         <env name="PULSE_ENABLED" value="false"/>
         <env name="QUEUE_CONNECTION" value="sync"/>

public/assets/jonnybarnes-public-key-ecc.asc

@@ -0,0 +1,21 @@
+-----BEGIN PGP PUBLIC KEY BLOCK-----
+
+mG8EV0W04xMFK4EEACIDAwSZOjA8NdI6UvbI/Sqw8LfpckfDXMuiowrVgcANjhDr
+vQtvr0bYm7RnNlbiuwTQHQ064H3pwjJJYC12I5B6q1Is7h4PYzU4/ahtisb03U/Q
+ThDDuWxDKQq2hcyfrNI02KO0I0pvbm55IEJhcm5lcyA8am9ubnlAam9ubnliYXJu
+ZXMudWs+iLYEExMKAD4CGwMFCwkIBwMFFQoJCAsFFgMCAQACHgECF4AWIQSEGbWh
+2ITK9LCvj7MbLHavsWyFmwUCWSgPSgUJA8ON5wAKCRAbLHavsWyFm9hAAX9ymfnT
+CUQDBqHmSR+YJ7RkNNFRdq4J1ABsvaRnpRynIE60dde1WqX62CvOkQDyY3sBgLJp
+3KCNjB9VRoHHL3Gk1X78gxntU01wP+oYotA7tJescf34oM4CfzHoz4UdUTPK3Iif
+BBMTCgAnBQJXRbTjAhsDBQkB4TOABQsJCAcDBRUKCQgLBRYDAgEAAh4BAheAAAoJ
+EBssdq+xbIWb0qwBfijJLkE/QUH7iAASLDtD3pUGs13TUynrzl2n8NdOnwmJTDdB
+6eFT5+XxAgmuw+o28wF+MMEqzf+ELqjEyk/DZaZ3Kg9cJQAm3ybc0wFKy/kqQ2HY
+TaFwEG3dAyfFDddkuaDEuHMEV0W04xIFK4EEACIDAwR0mRtrTq604CFiA8OdQR77
+AVc8lRrNxFPwo7uJWsIEPBNVTHasC4OCXAvGgm9bPggQUNoOQ4fUCNmAgsVeGpYQ
+/b67m0ydqjcrHpd0fIRbK5kyWYvYPRsZ6mTgiKosrZIDAQkJiJ4EGBMKACYCGwwW
+IQSEGbWh2ITK9LCvj7MbLHavsWyFmwUCWSgPZQUJA8OOAgAKCRAbLHavsWyFm1fN
+AYDMf1p4GegE1FHiUZo4m4Y5iQfbxT9Nmlgaopbmq+BxJRwPMxVzJOvKXo4DiUd0
+nncBgOJUJ8esy6WGw+lUfkfvRNkhPw9CVt1GifjG4axGHGaDyDQdFdRcIeFyu0Fs
+7HsLmg==
+=sdL6
+-----END PGP PUBLIC KEY BLOCK-----

public/index.php

@@ -1,20 +1,55 @@
 <?php
 
-use Illuminate\Foundation\Application;
+use Illuminate\Contracts\Http\Kernel;
 use Illuminate\Http\Request;
 
 define('LARAVEL_START', microtime(true));
 
-// Determine if the application is in maintenance mode...
+/*
+|--------------------------------------------------------------------------
+| Check If The Application Is Under Maintenance
+|--------------------------------------------------------------------------
+|
+| If the application is in maintenance / demo mode via the "down" command
+| we will load this file so that any pre-rendered content can be shown
+| instead of starting the framework, which could cause an exception.
+|
+*/
+
 if (file_exists($maintenance = __DIR__.'/../storage/framework/maintenance.php')) {
     require $maintenance;
 }
 
-// Register the Composer autoloader...
+/*
+|--------------------------------------------------------------------------
+| Register The Auto Loader
+|--------------------------------------------------------------------------
+|
+| Composer provides a convenient, automatically generated class loader for
+| this application. We just need to utilize it! We'll simply require it
+| into the script here so we don't need to manually load our classes.
+|
+*/
+
 require __DIR__.'/../vendor/autoload.php';
 
-// Bootstrap Laravel and handle the request...
+/*
-/** @var Application $app */
+|--------------------------------------------------------------------------
+| Run The Application
+|--------------------------------------------------------------------------
+|
+| Once we have the application, we can handle the incoming request using
+| the application's HTTP kernel. Then, we will send the response back
+| to this client's browser, allowing them to enjoy our application.
+|
+*/
+
 $app = require_once __DIR__.'/../bootstrap/app.php';
 
-$app->handleRequest(Request::capture());
+$kernel = $app->make(Kernel::class);
+
+$response = $kernel->handle(
+    $request = Request::capture()
+)->send();
+
+$kernel->terminate($request, $response);

resources/views/master.blade.php

@@ -19,12 +19,8 @@
         <link rel="token_endpoint" href="{{ route('indieauth.token') }}">
         <link rel="micropub" href="{{ route('micropub-endpoint') }}">
         <link rel="webmention" href="{{ config('app.url') }}/webmention">
-        @if (File::exists(public_path('assets/img/favicon.png')))
+        <link rel="shortcut icon" href="{{ config('app.url') }}/assets/img/memoji-orange-bg-small-fs8.png">
-            <link rel="icon" href="{{ config('app.url') }}/assets/img/favicon.png">
+        <link rel="pgpkey" href="/assets/jonnybarnes-public-key-ecc.asc">
-        @endif
-        @if (File::exists(public_path('gpg.key')))
-            <link rel="pgpkey" href="{{ config('app.url')}}/gpg.key">
-        @endif
     </head>
     <body class="grid">
         <header id="site-header">
@@ -72,16 +68,6 @@
         <!--scripts go here when needed-->
         @section('scripts')
             <script type="module" src="/assets/js/app.js"></script>
-
-            <!-- Snow fall -->
-            <script type="module" src="/assets/frontend/is-land.js"></script>
-            <script type="module" src="/assets/frontend/snow-fall.js"></script>
-            <is-land on:media="(prefers-reduced-motion: no-preference)">
-                <snow-fall
-                    count="240"
-                    style="--snow-fall-color: rebeccapurple; --snow-fall-size: 8px"
-                ></snow-fall>
-            </is-land>
         @show
     </body>
 </html>

resources/views/notes/show.blade.php

@@ -13,7 +13,7 @@
                             @if (array_key_exists('photo', $reply['author']['properties']))
                                 <img src="{{ $reply['author']['properties']['photo'][0] }}" alt="" class="photo u-photo logo">
                             @endif
-                            <span class="fn">{{ $reply['author']['properties']['name'][0] ?? $reply['author']['properties']['nickname'][0] ?? 'unknown' }}</span>
+                            <span class="fn">{{ $reply['author']['properties']['name'][0] }}</span>
                         </a>
                     @else
                         Unknown author
@@ -33,7 +33,7 @@
             return ($webmention->type === 'like-of');
         }) as $like)
             <a href="{{ $like['author']['properties']['url'][0] }}">
-                <img src="{{ $like['author']['properties']['photo'][0] }}" alt="profile picture of {{ $like['author']['properties']['name'][0] ?? $like['author']['properties']['nickname'][0] ?? 'unknown' }}" class="like-photo">
+                <img src="{{ $like['author']['properties']['photo'][0] }}" alt="profile picture of {{ $like['author']['properties']['name'][0] }}" class="like-photo">
             </a>
         @endforeach
     </div>
@@ -47,7 +47,7 @@
             return ($webmention->type == 'repost-of');
         }) as $repost)
             <a href="{{ $repost['source'] }}">
-                <img src="{{ $repost['author']['properties']['photo'][0] }}" alt="{{ $repost['author']['properties']['name'][0] ?? $repost['author']['properties']['nickname'][0] ?? 'unknown' }} reposted this at {{ $repost['published'] }}">
+                <img src="{{ $repost['author']['properties']['photo'][0] }}" alt="{{ $repost['author']['properties']['name'][0] }} reposted this at {{ $repost['published'] }}">
             </a>
         @endforeach
     </div>

scripts/compress.sh

@@ -1,41 +1,16 @@
 #!/usr/bin/env zsh
 
-if ! (( $+commands[fd] )) &> /dev/null; then
+if ! type brotli &> /dev/null; then
-    echo "fd not installed"
-    exit 1
-fi
-
-if ! (( $+commands[brotli] )) &> /dev/null; then
     echo "brotli not installed"
     exit 1
 fi
 
-if ! (( $+commands[zstd] )) &> /dev/null; then
+for file in ./public/assets/css/*.css
-    echo "zstd not installed"
+do
-    exit 1
+    brotli --force --quality=11 --output=$file.br -- $file
-fi
+done
-
+
-# Make .br files
+for file in ./public/assets/js/*.js
-fd --extension css --extension js --search-path ./public/assets --type f --exec brotli --force --best --output={}.br {}
+do
-# Make .zst files
+    brotli --force --quality=11 --output=$file.br -- $file
-fd --extension css --extension js --search-path ./public/assets --type f --exec zstd --quiet --force --ultra -22 --exclude-compressed {} -o {}.zst
-
-# Remove files that actually got bigger!
-fd --extension br --extension zst --search-path ./public/assets --type f --exec sh -c '
-for file; do
-  src="${file%.br}"
-  src="${src%.zst}"
-  if [ -f "$src" ]; then
-    # Get file sizes using stat with cross-platform compatibility
-    if [ "$(uname)" = "Darwin" ]; then
-      file_size=$(stat -f%z "$file")
-      src_size=$(stat -f%z "$src")
-    else
-      file_size=$(stat -c%s "$file")
-      src_size=$(stat -c%s "$src")
-    fi
-    # Compare sizes and rm compressed file if larger than the source
-    [ "$file_size" -ge "$src_size" ] && rm "$file"
-  fi
 done
-' _ {}

storage/app/.gitignore

@@ -1,4 +1,3 @@
 *
-!private/
 !public/
 !.gitignore

storage/app/private/.gitignore

@@ -1,2 +0,0 @@
-*
-!.gitignore

tests/Browser/ExampleTest.php

@@ -12,7 +12,7 @@ class ExampleTest extends DuskTestCase
      *
      * @return void
      */
-    public function test_basic_example()
+    public function testBasicExample()
     {
         $this->browse(function (Browser $browser) {
             $browser->visit('/')

tests/DuskTestCase.php

@@ -32,7 +32,7 @@ abstract class DuskTestCase extends BaseTestCase
     {
         $desiredCapabilities = DesiredCapabilities::chrome();
 
-        $options = new ChromeOptions;
+        $options = new ChromeOptions();
         $options->addArguments([
             'headless',
             'disable-gpu',

tests/Feature/Admin/AdminHomeControllerTest.php

@@ -6,14 +6,13 @@ namespace Tests\Feature\Admin;
 
 use App\Models\User;
 use Illuminate\Foundation\Testing\RefreshDatabase;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class AdminHomeControllerTest extends TestCase
 {
     use RefreshDatabase;
 
-    #[Test]
+    /** @test */
     public function adminHomepageLoads(): void
     {
         $user = User::factory()->make();

tests/Feature/Admin/AdminTest.php

@@ -5,26 +5,25 @@ declare(strict_types=1);
 namespace Tests\Feature\Admin;
 
 use App\Models\User;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class AdminTest extends TestCase
 {
-    #[Test]
+    /** @test */
     public function adminPageRedirectsUnauthorisedUsersToLoginPage(): void
     {
         $response = $this->get('/admin');
         $response->assertRedirect('/login');
     }
 
-    #[Test]
+    /** @test */
     public function loginPageLoads(): void
     {
         $response = $this->get('/login');
         $response->assertViewIs('login');
     }
 
-    #[Test]
+    /** @test */
     public function loginAttemptWithBadCredentialsFails(): void
     {
         $response = $this->post('/login', [
@@ -34,7 +33,7 @@ class AdminTest extends TestCase
         $response->assertRedirect('/login');
     }
 
-    #[Test]
+    /** @test */
     public function loginSucceeds(): void
     {
         User::factory([
@@ -50,7 +49,7 @@ class AdminTest extends TestCase
         $response->assertRedirect('/admin');
     }
 
-    #[Test]
+    /** @test */
     public function whenLoggedInRedirectsToAdminPage(): void
     {
         $user = User::factory()->create();
@@ -58,14 +57,14 @@ class AdminTest extends TestCase
         $response->assertRedirect('/');
     }
 
-    #[Test]
+    /** @test */
     public function loggedOutUsersSimplyRedirected(): void
     {
         $response = $this->get('/logout');
         $response->assertRedirect('/');
     }
 
-    #[Test]
+    /** @test */
     public function loggedInUsersShownLogoutForm(): void
     {
         $user = User::factory()->create();
@@ -73,7 +72,7 @@ class AdminTest extends TestCase
         $response->assertViewIs('logout');
     }
 
-    #[Test]
+    /** @test */
     public function loggedInUsersCanLogout(): void
     {
         $user = User::factory()->create();

tests/Feature/Admin/ArticlesTest.php

@@ -9,14 +9,13 @@ use App\Models\User;
 use Faker\Factory;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Illuminate\Http\UploadedFile;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class ArticlesTest extends TestCase
 {
     use RefreshDatabase;
 
-    #[Test]
+    /** @test */
     public function adminArticlesPageLoads(): void
     {
         $user = User::factory()->make();
@@ -26,7 +25,7 @@ class ArticlesTest extends TestCase
         $response->assertSeeText('Select article to edit:');
     }
 
-    #[Test]
+    /** @test */
     public function adminCanLoadFormToCreateArticle(): void
     {
         $user = User::factory()->make();
@@ -36,7 +35,7 @@ class ArticlesTest extends TestCase
         $response->assertSeeText('Title (URL)');
     }
 
-    #[Test]
+    /** @test */
     public function admiNCanCreateNewArticle(): void
     {
         $user = User::factory()->make();
@@ -49,7 +48,7 @@ class ArticlesTest extends TestCase
         $this->assertDatabaseHas('articles', ['title' => 'Test Title']);
     }
 
-    #[Test]
+    /** @test */
     public function adminCanCreateNewArticleWithFile(): void
     {
         $user = User::factory()->make();
@@ -74,7 +73,7 @@ class ArticlesTest extends TestCase
         ]);
     }
 
-    #[Test]
+    /** @test */
     public function articleCanLoadFormToEditArticle(): void
     {
         $user = User::factory()->make();
@@ -87,7 +86,7 @@ class ArticlesTest extends TestCase
         $response->assertSeeText('This is *my* new blog. It uses `Markdown`.');
     }
 
-    #[Test]
+    /** @test */
     public function adminCanEditArticle(): void
     {
         $user = User::factory()->make();
@@ -105,7 +104,7 @@ class ArticlesTest extends TestCase
         ]);
     }
 
-    #[Test]
+    /** @test */
     public function adminCanDeleteArticle(): void
     {
         $user = User::factory()->make();

tests/Feature/Admin/BioTest.php

@@ -7,14 +7,13 @@ namespace Tests\Feature\Admin;
 use App\Models\Bio;
 use App\Models\User;
 use Illuminate\Foundation\Testing\RefreshDatabase;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class BioTest extends TestCase
 {
     use RefreshDatabase;
 
-    #[Test]
+    /** @test */
     public function adminBiosPageLoads(): void
     {
         $user = User::factory()->make();
@@ -24,7 +23,7 @@ class BioTest extends TestCase
         $response->assertSeeText('Edit bio');
     }
 
-    #[Test]
+    /** @test */
     public function adminCanCreateBio(): void
     {
         $user = User::factory()->make();
@@ -37,7 +36,7 @@ class BioTest extends TestCase
         $this->assertDatabaseHas('bios', ['content' => 'Bio content']);
     }
 
-    #[Test]
+    /** @test */
     public function adminCanLoadExistingBio(): void
     {
         $user = User::factory()->make();
@@ -50,7 +49,7 @@ class BioTest extends TestCase
         $response->assertSeeText('This is <em>my</em> bio. It uses <strong>HTML</strong>.');
     }
 
-    #[Test]
+    /** @test */
     public function adminCanEditBio(): void
     {
         $user = User::factory()->make();

tests/Feature/Admin/ClientsTest.php

@@ -7,14 +7,13 @@ namespace Tests\Feature\Admin;
 use App\Models\MicropubClient;
 use App\Models\User;
 use Illuminate\Foundation\Testing\RefreshDatabase;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class ClientsTest extends TestCase
 {
     use RefreshDatabase;
 
-    #[Test]
+    /** @test */
     public function clientsPageLoads(): void
     {
         $user = User::factory()->make();
@@ -24,7 +23,7 @@ class ClientsTest extends TestCase
         $response->assertSeeText('Clients');
     }
 
-    #[Test]
+    /** @test */
     public function adminCanLoadFormToCreateClient(): void
     {
         $user = User::factory()->make();
@@ -34,7 +33,7 @@ class ClientsTest extends TestCase
         $response->assertSeeText('New Client');
     }
 
-    #[Test]
+    /** @test */
     public function adminCanCreateNewClient(): void
     {
         $user = User::factory()->make();
@@ -50,7 +49,7 @@ class ClientsTest extends TestCase
         ]);
     }
 
-    #[Test]
+    /** @test */
     public function adminCanLoadEditFormForClient(): void
     {
         $user = User::factory()->make();
@@ -63,7 +62,7 @@ class ClientsTest extends TestCase
         $response->assertSee('https://jbl5.dev/notes/new');
     }
 
-    #[Test]
+    /** @test */
     public function adminCanEditClient(): void
     {
         $user = User::factory()->make();
@@ -81,7 +80,7 @@ class ClientsTest extends TestCase
         ]);
     }
 
-    #[Test]
+    /** @test */
     public function adminCanDeleteClient(): void
     {
         $user = User::factory()->make();

tests/Feature/Admin/ContactsTest.php

@@ -12,7 +12,6 @@ use GuzzleHttp\HandlerStack;
 use GuzzleHttp\Psr7\Response;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Illuminate\Http\UploadedFile;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class ContactsTest extends TestCase
@@ -28,7 +27,7 @@ class ContactsTest extends TestCase
         parent::tearDown();
     }
 
-    #[Test]
+    /** @test */
     public function contactIndexPageLoads(): void
     {
         $user = User::factory()->make();
@@ -37,7 +36,7 @@ class ContactsTest extends TestCase
         $response->assertViewIs('admin.contacts.index');
     }
 
-    #[Test]
+    /** @test */
     public function contactCreatePageLoads(): void
     {
         $user = User::factory()->make();
@@ -46,7 +45,7 @@ class ContactsTest extends TestCase
         $response->assertViewIs('admin.contacts.create');
     }
 
-    #[Test]
+    /** @test */
     public function adminCanCreateNewContact(): void
     {
         $user = User::factory()->make();
@@ -63,7 +62,7 @@ class ContactsTest extends TestCase
         ]);
     }
 
-    #[Test]
+    /** @test */
     public function adminCanSeeFormToEditContact(): void
     {
         $user = User::factory()->make();
@@ -73,7 +72,7 @@ class ContactsTest extends TestCase
         $response->assertViewIs('admin.contacts.edit');
     }
 
-    #[Test]
+    /** @test */
     public function adminCanUpdateContact(): void
     {
         $user = User::factory()->make();
@@ -92,7 +91,7 @@ class ContactsTest extends TestCase
         ]);
     }
 
-    #[Test]
+    /** @test */
     public function adminCanEditContactAndUploadAvatar(): void
     {
         copy(__DIR__ . '/../../aaron.png', sys_get_temp_dir() . '/tantek.png');
@@ -115,7 +114,7 @@ class ContactsTest extends TestCase
         );
     }
 
-    #[Test]
+    /** @test */
     public function adminCanDeleteContact(): void
     {
         $user = User::factory()->make();
@@ -133,7 +132,7 @@ class ContactsTest extends TestCase
         ]);
     }
 
-    #[Test]
+    /** @test */
     public function adminCanTriggerRetrievalOfRemoteAvatar(): void
     {
         $html = <<<'HTML'
@@ -162,7 +161,7 @@ class ContactsTest extends TestCase
         );
     }
 
-    #[Test]
+    /** @test */
     public function gettingRemoteAvatarFailsGracefullyWithRemoteNotFound(): void
     {
         $mock = new MockHandler([
@@ -179,7 +178,7 @@ class ContactsTest extends TestCase
         $response->assertRedirect('/admin/contacts/' . $contact->id . '/edit');
     }
 
-    #[Test]
+    /** @test */
     public function gettingRemoteAvatarFailsGracefullyWithRemoteError(): void
     {
         $html = <<<'HTML'
@@ -202,7 +201,7 @@ class ContactsTest extends TestCase
         $response->assertRedirect('/admin/contacts/' . $contact->id . '/edit');
     }
 
-    #[Test]
+    /** @test */
     public function gettingRemoteAvatarFailsGracefullyForContactWithNoHompage(): void
     {
         $contact = Contact::create([

tests/Feature/Admin/LikesTest.php

@@ -9,14 +9,13 @@ use App\Models\Like;
 use App\Models\User;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Illuminate\Support\Facades\Queue;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class LikesTest extends TestCase
 {
     use RefreshDatabase;
 
-    #[Test]
+    /** @test */
     public function likesPageLoads(): void
     {
         $user = User::factory()->make();
@@ -26,7 +25,7 @@ class LikesTest extends TestCase
         $response->assertSeeText('Likes');
     }
 
-    #[Test]
+    /** @test */
     public function likeCreateFormLoads(): void
     {
         $user = User::factory()->make();
@@ -36,7 +35,7 @@ class LikesTest extends TestCase
         $response->assertSeeText('New Like');
     }
 
-    #[Test]
+    /** @test */
     public function adminCanCreateLike(): void
     {
         Queue::fake();
@@ -52,7 +51,7 @@ class LikesTest extends TestCase
         Queue::assertPushed(ProcessLike::class);
     }
 
-    #[Test]
+    /** @test */
     public function likeEditFormLoads(): void
     {
         $user = User::factory()->make();
@@ -63,7 +62,7 @@ class LikesTest extends TestCase
         $response->assertSee('Edit Like');
     }
 
-    #[Test]
+    /** @test */
     public function adminCanEditLike(): void
     {
         Queue::fake();
@@ -81,7 +80,7 @@ class LikesTest extends TestCase
         Queue::assertPushed(ProcessLike::class);
     }
 
-    #[Test]
+    /** @test */
     public function adminCanDeleteLike(): void
     {
         $like = Like::factory()->create();

tests/Feature/Admin/NotesTest.php

@@ -9,14 +9,13 @@ use App\Models\Note;
 use App\Models\User;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Illuminate\Support\Facades\Queue;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class NotesTest extends TestCase
 {
     use RefreshDatabase;
 
-    #[Test]
+    /** @test */
     public function notesPageLoads(): void
     {
         $user = User::factory()->make();
@@ -25,7 +24,7 @@ class NotesTest extends TestCase
         $response->assertViewIs('admin.notes.index');
     }
 
-    #[Test]
+    /** @test */
     public function noteCreatePageLoads(): void
     {
         $user = User::factory()->make();
@@ -34,7 +33,7 @@ class NotesTest extends TestCase
         $response->assertViewIs('admin.notes.create');
     }
 
-    #[Test]
+    /** @test */
     public function adminCanCreateNewNote(): void
     {
         $user = User::factory()->make();
@@ -48,7 +47,7 @@ class NotesTest extends TestCase
         ]);
     }
 
-    #[Test]
+    /** @test */
     public function noteEditFormLoads(): void
     {
         $user = User::factory()->make();
@@ -58,7 +57,7 @@ class NotesTest extends TestCase
         $response->assertViewIs('admin.notes.edit');
     }
 
-    #[Test]
+    /** @test */
     public function adminCanEditNote(): void
     {
         Queue::fake();
@@ -77,7 +76,7 @@ class NotesTest extends TestCase
         Queue::assertPushed(SendWebMentions::class);
     }
 
-    #[Test]
+    /** @test */
     public function adminCanDeleteNote(): void
     {
         $user = User::factory()->make();

tests/Feature/Admin/PlacesTest.php

@@ -7,14 +7,13 @@ namespace Tests\Feature\Admin;
 use App\Models\Place;
 use App\Models\User;
 use Illuminate\Foundation\Testing\RefreshDatabase;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class PlacesTest extends TestCase
 {
     use RefreshDatabase;
 
-    #[Test]
+    /** @test */
     public function placesPageLoads(): void
     {
         $user = User::factory()->make();
@@ -23,7 +22,7 @@ class PlacesTest extends TestCase
         $response->assertViewIs('admin.places.index');
     }
 
-    #[Test]
+    /** @test */
     public function createPlacePageLoads(): void
     {
         $user = User::factory()->make();
@@ -32,7 +31,7 @@ class PlacesTest extends TestCase
         $response->assertViewIs('admin.places.create');
     }
 
-    #[Test]
+    /** @test */
     public function adminCanCreateNewPlace(): void
     {
         $user = User::factory()->make();
@@ -49,7 +48,7 @@ class PlacesTest extends TestCase
         ]);
     }
 
-    #[Test]
+    /** @test */
     public function editPlacePageLoads(): void
     {
         $user = User::factory()->make();
@@ -59,7 +58,7 @@ class PlacesTest extends TestCase
         $response->assertViewIs('admin.places.edit');
     }
 
-    #[Test]
+    /** @test */
     public function adminCanUpdatePlace(): void
     {
         $user = User::factory()->make();

tests/Feature/ArticlesTest.php

@@ -7,21 +7,20 @@ namespace Tests\Feature;
 use App\Models\Article;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Jonnybarnes\IndieWeb\Numbers;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class ArticlesTest extends TestCase
 {
     use RefreshDatabase;
 
-    #[Test]
+    /** @test */
     public function articlesPageLoads(): void
     {
         $response = $this->get('/blog');
         $response->assertViewIs('articles.index');
     }
 
-    #[Test]
+    /** @test */
     public function singleArticlePageLoads()
     {
         $article = Article::factory()->create();
@@ -29,7 +28,7 @@ class ArticlesTest extends TestCase
         $response->assertViewIs('articles.show');
     }
 
-    #[Test]
+    /** @test */
     public function wrongDateInUrlRedirectsToCorrectDate()
     {
         $article = Article::factory()->create();
@@ -37,7 +36,7 @@ class ArticlesTest extends TestCase
         $response->assertRedirect('/blog/' . date('Y') . '/' . date('m') . '/' . $article->titleurl);
     }
 
-    #[Test]
+    /** @test */
     public function oldUrlsWithIdAreRedirected()
     {
         $article = Article::factory()->create();
@@ -46,21 +45,21 @@ class ArticlesTest extends TestCase
         $response->assertRedirect($article->link);
     }
 
-    #[Test]
+    /** @test  */
     public function unknownSlugGetsNotFoundResponse()
     {
         $response = $this->get('/blog/' . date('Y') . '/' . date('m') . '/unknown-slug');
         $response->assertNotFound();
     }
 
-    #[Test]
+    /** @test */
     public function unknownArticleIdGetsNotFoundResponse()
     {
         $response = $this->get('/blog/s/22');
         $response->assertNotFound();
     }
 
-    #[Test]
+    /** @test */
     public function someUrlsDoNotParseCorrectly(): void
     {
         $response = $this->get('/blog/feed.js');

tests/Feature/BookmarksTest.php

@@ -8,7 +8,6 @@ use App\Jobs\ProcessBookmark;
 use App\Models\Bookmark;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Illuminate\Support\Facades\Queue;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 use Tests\TestToken;
 
@@ -16,14 +15,14 @@ class BookmarksTest extends TestCase
 {
     use RefreshDatabase, TestToken;
 
-    #[Test]
+    /** @test */
     public function bookmarksPageLoadsWithoutError(): void
     {
         $response = $this->get('/bookmarks');
         $response->assertViewIs('bookmarks.index');
     }
 
-    #[Test]
+    /** @test */
     public function singleBookmarkPageLoadsWithoutError(): void
     {
         $bookmark = Bookmark::factory()->create();
@@ -31,7 +30,7 @@ class BookmarksTest extends TestCase
         $response->assertViewIs('bookmarks.show');
     }
 
-    #[Test]
+    /** @test */
     public function whenBookmarkIsAddedUsingHttpSyntaxCheckJobToTakeScreenshotIsInvoked(): void
     {
         Queue::fake();
@@ -49,7 +48,7 @@ class BookmarksTest extends TestCase
         $this->assertDatabaseHas('bookmarks', ['url' => 'https://example.org/blog-post']);
     }
 
-    #[Test]
+    /** @test */
     public function whenBookmarkIsAddedUsingJsonSyntaxCheckJobToTakeScreenshotIsInvoked(): void
     {
         Queue::fake();
@@ -69,7 +68,7 @@ class BookmarksTest extends TestCase
         $this->assertDatabaseHas('bookmarks', ['url' => 'https://example.org/blog-post']);
     }
 
-    #[Test]
+    /** @test */
     public function whenTheBookmarkIsCreatedCheckNecessaryTagsAreAlsoCreated(): void
     {
         Queue::fake();

tests/Feature/ContactsTest.php

@@ -6,7 +6,6 @@ namespace Tests\Feature;
 
 use App\Models\Contact;
 use Illuminate\Foundation\Testing\RefreshDatabase;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class ContactsTest extends TestCase
@@ -15,8 +14,9 @@ class ContactsTest extends TestCase
 
     /**
      * Check the `/contacts` page gives a good response.
+     *
+     * @test
      */
-    #[Test]
     public function contactsPageLoadsWithoutError(): void
     {
         $response = $this->get('/contacts');
@@ -25,8 +25,9 @@ class ContactsTest extends TestCase
 
     /**
      * Test an individual contact page with default profile image.
+     *
+     * @test
      */
-    #[Test]
     public function contactPageShouldFallbackToDefaultProfilePic(): void
     {
         Contact::factory()->create([
@@ -38,8 +39,9 @@ class ContactsTest extends TestCase
 
     /**
      * Test an individual contact page with a specific profile image.
+     *
+     * @test
      */
-    #[Test]
     public function contactPageShouldUseSpecificProfilePicIfPresent(): void
     {
         Contact::factory()->create([
@@ -50,7 +52,7 @@ class ContactsTest extends TestCase
         $response->assertViewHas('image', '/assets/profile-images/aaronparecki.com/image');
     }
 
-    #[Test]
+    /** @test */
     public function unknownContactReturnsNotFoundResponse(): void
     {
         $response = $this->get('/contacts/unknown');

tests/Feature/CorsHeadersTest.php

@@ -4,7 +4,6 @@ declare(strict_types=1);
 
 namespace Tests\Feature;
 
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 use Tests\TestToken;
 
@@ -12,7 +11,7 @@ class CorsHeadersTest extends TestCase
 {
     use TestToken;
 
-    #[Test]
+    /** @test */
     public function checkCorsHeadersOnMediaEndpoint(): void
     {
         $response = $this->call(
@@ -26,7 +25,7 @@ class CorsHeadersTest extends TestCase
         $response->assertHeader('Access-Control-Allow-Origin', '*');
     }
 
-    #[Test]
+    /** @test */
     public function checkForNoCorsHeaderOnNonMediaEndpointLinks(): void
     {
         $response = $this->get('/blog');

tests/Feature/FeedsTest.php

@@ -8,7 +8,6 @@ use App\Models\Article;
 use App\Models\Note;
 use App\Models\Place;
 use Illuminate\Foundation\Testing\RefreshDatabase;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class FeedsTest extends TestCase
@@ -17,8 +16,9 @@ class FeedsTest extends TestCase
 
     /**
      * Test the blog RSS feed.
+     *
+     * @test
      */
-    #[Test]
     public function blogRssFeedIsPresent(): void
     {
         Article::factory()->count(3)->create();
@@ -29,8 +29,9 @@ class FeedsTest extends TestCase
 
     /**
      * Test the notes RSS feed.
+     *
+     * @test
      */
-    #[Test]
     public function notesRssFeedIsPresent(): void
     {
         Note::factory()->count(3)->create();
@@ -41,8 +42,9 @@ class FeedsTest extends TestCase
 
     /**
      * Test the blog RSS feed.
+     *
+     * @test
      */
-    #[Test]
     public function blogAtomFeedIsPresent(): void
     {
         Article::factory()->count(3)->create();
@@ -51,7 +53,7 @@ class FeedsTest extends TestCase
         $response->assertOk();
     }
 
-    #[Test]
+    /** @test */
     public function blogJf2FeedIsPresent(): void
     {
         Article::factory()->count(3)->create();
@@ -75,8 +77,9 @@ class FeedsTest extends TestCase
 
     /**
      * Test the notes RSS feed.
+     *
+     * @test
      */
-    #[Test]
     public function notesAtomFeedIsPresent(): void
     {
         Note::factory()->count(3)->create();
@@ -87,8 +90,9 @@ class FeedsTest extends TestCase
 
     /**
      * Test the blog JSON feed.
+     *
+     * @test
      */
-    #[Test]
     public function blogJsonFeedIsPresent(): void
     {
         Article::factory()->count(3)->create();
@@ -99,8 +103,9 @@ class FeedsTest extends TestCase
 
     /**
      * Test the notes JSON feed.
+     *
+     * @test
      */
-    #[Test]
     public function notesJsonFeedIsPresent(): void
     {
         Note::factory()->count(3)->create();
@@ -109,7 +114,7 @@ class FeedsTest extends TestCase
         $response->assertOk();
     }
 
-    #[Test]
+    /** @test */
     public function notesJf2FeedIsPresent(): void
     {
         Note::factory()->count(3)->create();
@@ -134,8 +139,9 @@ class FeedsTest extends TestCase
     /**
      * Each JSON feed item must have one of `content_text` or `content_html`,
      * and whichever one they have can’t be `null`.
+     *
+     * @test
      */
-    #[Test]
     public function jsonFeedsHaveRequiredAttributes(): void
     {
         Note::factory()->count(3)->create();
@@ -155,7 +161,7 @@ class FeedsTest extends TestCase
         }
     }
 
-    #[Test]
+    /** @test */
     public function jsonNoteFeedLoadsPlaceDataWithoutLazyLoading(): void
     {
         $place = Place::factory()->create();

tests/Feature/FrontPageTest.php

@@ -7,14 +7,13 @@ use App\Models\Bookmark;
 use App\Models\Like;
 use App\Models\Note;
 use Illuminate\Foundation\Testing\RefreshDatabase;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class FrontPageTest extends TestCase
 {
     use RefreshDatabase;
 
-    #[Test]
+    /** @test */
     public function frontPageLoadsAllContent(): void
     {
         Note::factory()->create(['note' => 'Note 1']);

tests/Feature/HorizonTest.php

@@ -3,15 +3,15 @@
 namespace Tests\Feature;
 
 use App\Models\User;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class HorizonTest extends TestCase
 {
     /**
      * Horizon has its own test suite, here we just test it has been installed successfully.
+     *
+     * @test
      */
-    #[Test]
     public function horizonIsInstalled(): void
     {
         $user = User::factory()->create([

tests/Feature/LikesTest.php

@@ -14,7 +14,6 @@ use GuzzleHttp\Psr7\Response;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Illuminate\Support\Facades\Queue;
 use Jonnybarnes\WebmentionsParser\Authorship;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 use Tests\TestToken;
 
@@ -23,14 +22,14 @@ class LikesTest extends TestCase
     use RefreshDatabase;
     use TestToken;
 
-    #[Test]
+    /** @test */
     public function likesPageHasCorrectView(): void
     {
         $response = $this->get('/likes');
         $response->assertViewIs('likes.index');
     }
 
-    #[Test]
+    /** @test */
     public function singleLikePageHasCorrectView(): void
     {
         $like = Like::factory()->create();
@@ -38,7 +37,7 @@ class LikesTest extends TestCase
         $response->assertViewIs('likes.show');
     }
 
-    #[Test]
+    /** @test */
     public function checkLikeCreatedFromMicropubApiRequests(): void
     {
         Queue::fake();
@@ -58,7 +57,7 @@ class LikesTest extends TestCase
         $this->assertDatabaseHas('likes', ['url' => 'https://example.org/blog-post']);
     }
 
-    #[Test]
+    /** @test */
     public function checkLikeCreatedFromMicropubWebRequests(): void
     {
         Queue::fake();
@@ -76,10 +75,10 @@ class LikesTest extends TestCase
         $this->assertDatabaseHas('likes', ['url' => 'https://example.org/blog-post']);
     }
 
-    #[Test]
+    /** @test */
     public function likeWithSimpleAuthor(): void
     {
-        $like = new Like;
+        $like = new Like();
         $like->url = 'http://example.org/note/id';
         $like->save();
         $id = $like->id;
@@ -108,17 +107,17 @@ class LikesTest extends TestCase
         $this->app->bind(Client::class, function () use ($client) {
             return $client;
         });
-        $authorship = new Authorship;
+        $authorship = new Authorship();
 
         $job->handle($client, $authorship);
 
         $this->assertEquals('Fred Bloggs', Like::find($id)->author_name);
     }
 
-    #[Test]
+    /** @test */
     public function likeWithHCard(): void
     {
-        $like = new Like;
+        $like = new Like();
         $like->url = 'http://example.org/note/id';
         $like->save();
         $id = $like->id;
@@ -151,17 +150,17 @@ class LikesTest extends TestCase
         $this->app->bind(Client::class, function () use ($client) {
             return $client;
         });
-        $authorship = new Authorship;
+        $authorship = new Authorship();
 
         $job->handle($client, $authorship);
 
         $this->assertEquals('Fred Bloggs', Like::find($id)->author_name);
     }
 
-    #[Test]
+    /** @test */
     public function likeWithoutMicroformats(): void
     {
-        $like = new Like;
+        $like = new Like();
         $like->url = 'http://example.org/note/id';
         $like->save();
         $id = $like->id;
@@ -187,17 +186,17 @@ class LikesTest extends TestCase
         $this->app->bind(Client::class, function () use ($client) {
             return $client;
         });
-        $authorship = new Authorship;
+        $authorship = new Authorship();
 
         $job->handle($client, $authorship);
 
         $this->assertNull(Like::find($id)->author_name);
     }
 
-    #[Test]
+    /** @test */
     public function likeThatIsATweet(): void
     {
-        $like = new Like;
+        $like = new Like();
         $like->url = 'https://twitter.com/jonnybarnes/status/1050823255123251200';
         $like->save();
         $id = $like->id;
@@ -220,23 +219,24 @@ class LikesTest extends TestCase
             'author_url' => 'https://twitter.com/jonnybarnes',
             'html' => '<div>HTML of the tweet embed</div>',
         ];
-        $codebirdMock = $this->createPartialMock(Codebird::class, ['__call']);
+        $codebirdMock = $this->getMockBuilder(Codebird::class)
-        $codebirdMock->method('__call')
+            ->addMethods(['statuses_oembed'])
-            ->with('statuses_oembed', $this->anything())
+            ->getMock();
+        $codebirdMock->method('statuses_oembed')
             ->willReturn($info);
         $this->app->instance(Codebird::class, $codebirdMock);
 
-        $authorship = new Authorship;
+        $authorship = new Authorship();
 
         $job->handle($client, $authorship);
 
         $this->assertEquals('Jonny Barnes', Like::find($id)->author_name);
     }
 
-    #[Test]
+    /** @test */
     public function noErrorForFailureToPosseWithBridgy(): void
     {
-        $like = new Like;
+        $like = new Like();
         $like->url = 'https://twitter.com/jonnybarnes/status/1050823255123251200';
         $like->save();
         $id = $like->id;
@@ -257,20 +257,21 @@ class LikesTest extends TestCase
             'author_url' => 'https://twitter.com/jonnybarnes',
             'html' => '<div>HTML of the tweet embed</div>',
         ];
-        $codebirdMock = $this->createPartialMock(Codebird::class, ['__call']);
+        $codebirdMock = $this->getMockBuilder(Codebird::class)
-        $codebirdMock->method('__call')
+            ->addMethods(['statuses_oembed'])
-            ->with('statuses_oembed', $this->anything())
+            ->getMock();
+        $codebirdMock->method('statuses_oembed')
             ->willReturn($info);
         $this->app->instance(Codebird::class, $codebirdMock);
 
-        $authorship = new Authorship;
+        $authorship = new Authorship();
 
         $job->handle($client, $authorship);
 
         $this->assertEquals('Jonny Barnes', Like::find($id)->author_name);
     }
 
-    #[Test]
+    /** @test */
     public function unknownLikeGivesNotFoundResponse(): void
     {
         $response = $this->get('/likes/202');

tests/Feature/MicropubControllerTest.php

@@ -15,7 +15,6 @@ use Carbon\Carbon;
 use Faker\Factory;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Illuminate\Support\Facades\Queue;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 use Tests\TestToken;
 
@@ -24,7 +23,7 @@ class MicropubControllerTest extends TestCase
     use RefreshDatabase;
     use TestToken;
 
-    #[Test]
+    /** @test */
     public function micropubGetRequestWithoutTokenReturnsErrorResponse(): void
     {
         $response = $this->get('/api/post');
@@ -32,7 +31,7 @@ class MicropubControllerTest extends TestCase
         $response->assertJsonFragment(['error_description' => 'No access token was provided in the request']);
     }
 
-    #[Test]
+    /** @test */
     public function micropubGetRequestWithoutValidTokenReturnsErrorResponse(): void
     {
         $response = $this->get('/api/post', ['HTTP_Authorization' => 'Bearer abc123']);
@@ -43,8 +42,9 @@ class MicropubControllerTest extends TestCase
     /**
      * Test a GET request for the micropub endpoint with a valid token gives a
      * 200 response. Check token information is also returned in the response.
+     *
+     * @test
      */
-    #[Test]
     public function micropubGetRequestWithValidTokenReturnsOkResponse(): void
     {
         $response = $this->get('/api/post', ['HTTP_Authorization' => 'Bearer ' . $this->getToken()]);
@@ -52,14 +52,14 @@ class MicropubControllerTest extends TestCase
         $response->assertJsonFragment(['response' => 'token']);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientsCanRequestSyndicationTargetsCanBeEmpty(): void
     {
         $response = $this->get('/api/post?q=syndicate-to', ['HTTP_Authorization' => 'Bearer ' . $this->getToken()]);
         $response->assertJsonFragment(['syndicate-to' => []]);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientsCanRequestSyndicationTargetsPopulatesFromModel(): void
     {
         $syndicationTarget = SyndicationTarget::factory()->create();
@@ -67,7 +67,7 @@ class MicropubControllerTest extends TestCase
         $response->assertJsonFragment(['uid' => $syndicationTarget->uid]);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientsCanRequestKnownNearbyPlaces(): void
     {
         Place::factory()->create([
@@ -80,6 +80,8 @@ class MicropubControllerTest extends TestCase
     }
 
     /**
+     * @test
+     *
      * @todo Add uncertainty parameter
      *
     public function micropubClientsCanRequestKnownNearbyPlacesWithUncertaintyParameter(): void
@@ -87,21 +89,22 @@ class MicropubControllerTest extends TestCase
         $response = $this->get('/api/post?q=geo:53.5,-2.38', ['HTTP_Authorization' => 'Bearer ' . $this->getToken()]);
         $response->assertJson(['places' => [['slug' => 'the-bridgewater-pub']]]);
     }*/
-    #[Test]
+
+    /** @test */
     public function returnEmptyResultWhenMicropubClientRequestsKnownNearbyPlaces(): void
     {
         $response = $this->get('/api/post?q=geo:1.23,4.56', ['HTTP_Authorization' => 'Bearer ' . $this->getToken()]);
         $response->assertJson(['places' => []]);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientCanRequestEndpointConfig(): void
     {
         $response = $this->get('/api/post?q=config', ['HTTP_Authorization' => 'Bearer ' . $this->getToken()]);
         $response->assertJsonFragment(['media-endpoint' => route('media-endpoint')]);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientCanCreateNewNote(): void
     {
         $faker = Factory::create();
@@ -120,7 +123,7 @@ class MicropubControllerTest extends TestCase
         $this->assertDatabaseHas('notes', ['note' => $note]);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientCanRequestTheNewNoteIsSyndicatedToMastodonAndBluesky(): void
     {
         Queue::fake();
@@ -154,7 +157,7 @@ class MicropubControllerTest extends TestCase
         Queue::assertPushed(SyndicateNoteToBluesky::class);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientsCanCreateNewPlaces(): void
     {
         $response = $this->post(
@@ -170,7 +173,7 @@ class MicropubControllerTest extends TestCase
         $this->assertDatabaseHas('places', ['slug' => 'the-barton-arms']);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientsCanCreateNewPlacesWithOldLocationSyntax(): void
     {
         $response = $this->post(
@@ -187,7 +190,7 @@ class MicropubControllerTest extends TestCase
         $this->assertDatabaseHas('places', ['slug' => 'the-barton-arms']);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientWebRequestWithInvalidTokenReturnsErrorResponse(): void
     {
         $response = $this->post(
@@ -202,7 +205,7 @@ class MicropubControllerTest extends TestCase
         $response->assertJson(['error' => 'invalid_token']);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientWebRequestWithTokenWithoutAnyScopesReturnsErrorResponse(): void
     {
         $response = $this->post(
@@ -217,7 +220,7 @@ class MicropubControllerTest extends TestCase
         $response->assertJson(['error_description' => 'The provided token has no scopes']);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientWebRequestWithTokenWithoutCreateScopesReturnsErrorResponse(): void
     {
         $response = $this->post(
@@ -235,8 +238,9 @@ class MicropubControllerTest extends TestCase
 
     /**
      * Test a valid micropub requests using JSON syntax creates a new note.
+     *
+     * @test
      */
-    #[Test]
     public function micropubClientApiRequestCreatesNewNote(): void
     {
         Queue::fake();
@@ -282,11 +286,12 @@ class MicropubControllerTest extends TestCase
     /**
      * Test a valid micropub requests using JSON syntax creates a new note with
      * existing self-created place.
+     *
+     * @test
      */
-    #[Test]
     public function micropubClientApiRequestCreatesNewNoteWithExistingPlaceInLocationData(): void
     {
-        $place = new Place;
+        $place = new Place();
         $place->name = 'Test Place';
         $place->latitude = 1.23;
         $place->longitude = 4.56;
@@ -312,8 +317,9 @@ class MicropubControllerTest extends TestCase
     /**
      * Test a valid micropub requests using JSON syntax creates a new note with
      * a new place defined in the location block.
+     *
+     * @test
      */
-    #[Test]
     public function micropubClientApiRequestCreatesNewNoteWithNewPlaceInLocationData(): void
     {
         $faker = Factory::create();
@@ -347,8 +353,9 @@ class MicropubControllerTest extends TestCase
     /**
      * Test a valid micropub requests using JSON syntax creates a new note without
      * a new place defined in the location block if there is missing data.
+     *
+     * @test
      */
-    #[Test]
     public function micropubClientApiRequestCreatesNewNoteWithoutNewPlaceInLocationData(): void
     {
         $faker = Factory::create();
@@ -380,8 +387,9 @@ class MicropubControllerTest extends TestCase
     /**
      * Test a micropub requests using JSON syntax without a token returns an
      * error. Also check the message.
+     *
+     * @test
      */
-    #[Test]
     public function micropubClientApiRequestWithoutTokenReturnsError(): void
     {
         $faker = Factory::create();
@@ -406,8 +414,9 @@ class MicropubControllerTest extends TestCase
     /**
      * Test a micropub requests using JSON syntax without a valid token returns
      * an error. Also check the message.
+     *
+     * @test
      */
-    #[Test]
     public function micropubClientApiRequestWithTokenWithInsufficientPermissionReturnsError(): void
     {
         $faker = Factory::create();
@@ -430,7 +439,7 @@ class MicropubControllerTest extends TestCase
             ->assertStatus(401);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientApiRequestForUnsupportedPostTypeReturnsError(): void
     {
         $response = $this->postJson(
@@ -451,7 +460,7 @@ class MicropubControllerTest extends TestCase
             ->assertStatus(500);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientApiRequestCreatesNewPlace(): void
     {
         $faker = Factory::create();
@@ -471,7 +480,7 @@ class MicropubControllerTest extends TestCase
             ->assertStatus(201);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientApiRequestCreatesNewPlaceWithUncertaintyParameter(): void
     {
         $faker = Factory::create();
@@ -491,7 +500,7 @@ class MicropubControllerTest extends TestCase
             ->assertStatus(201);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientApiRequestUpdatesExistingNote(): void
     {
         $note = Note::factory()->create();
@@ -511,7 +520,7 @@ class MicropubControllerTest extends TestCase
             ->assertStatus(200);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientApiRequestUpdatesNoteSyndicationLinks(): void
     {
         $note = Note::factory()->create();
@@ -538,7 +547,7 @@ class MicropubControllerTest extends TestCase
         ]);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientApiRequestAddsImageToNote(): void
     {
         $note = Note::factory()->create();
@@ -561,7 +570,7 @@ class MicropubControllerTest extends TestCase
         ]);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientApiRequestReturnsErrorTryingToUpdateNonNoteModel(): void
     {
         $response = $this->postJson(
@@ -580,7 +589,7 @@ class MicropubControllerTest extends TestCase
             ->assertStatus(500);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientApiRequestReturnsErrorTryingToUpdateNonExistingNote(): void
     {
         $response = $this->postJson(
@@ -599,7 +608,7 @@ class MicropubControllerTest extends TestCase
             ->assertStatus(404);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientApiRequestReturnsErrorWhenTryingToUpdateUnsupportedProperty(): void
     {
         $note = Note::factory()->create();
@@ -619,7 +628,7 @@ class MicropubControllerTest extends TestCase
             ->assertStatus(500);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientApiRequestWithTokenWithInsufficientScopeReturnsError(): void
     {
         $response = $this->postJson(
@@ -638,7 +647,7 @@ class MicropubControllerTest extends TestCase
             ->assertJson(['error' => 'insufficient_scope']);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientApiRequestCanReplaceNoteSyndicationTargets(): void
     {
         $note = Note::factory()->create();
@@ -665,7 +674,7 @@ class MicropubControllerTest extends TestCase
         ]);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientWebRequestCanEncodeTokenWithinTheForm(): void
     {
         $faker = Factory::create();
@@ -683,7 +692,7 @@ class MicropubControllerTest extends TestCase
         $this->assertDatabaseHas('notes', ['note' => $note]);
     }
 
-    #[Test]
+    /** @test */
     public function micropubClientApiRequestCreatesArticlesWhenItIncludesTheNameProperty(): void
     {
         $faker = Factory::create();

tests/Feature/MicropubMediaTest.php

@@ -10,8 +10,6 @@ use Illuminate\Foundation\Testing\RefreshDatabase;
 use Illuminate\Http\UploadedFile;
 use Illuminate\Support\Facades\Queue;
 use Illuminate\Support\Facades\Storage;
-use Illuminate\Support\Str;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 use Tests\TestToken;
 
@@ -20,11 +18,13 @@ class MicropubMediaTest extends TestCase
     use RefreshDatabase;
     use TestToken;
 
-    #[Test]
+    /** @test */
     public function emptyResponseForLastUploadWhenNoneFound(): void
     {
         // Make sure there’s no media
-        $this->assertCount(0, Media::all());
+        Media::all()->each(function ($media) {
+            $media->delete();
+        });
 
         $response = $this->get(
             '/api/media?q=last',
@@ -34,7 +34,7 @@ class MicropubMediaTest extends TestCase
         $response->assertJson(['url' => null]);
     }
 
-    #[Test]
+    /** @test */
     public function getRequestWithInvalidTokenReturnsErrorResponse(): void
     {
         $response = $this->get(
@@ -45,7 +45,7 @@ class MicropubMediaTest extends TestCase
         $response->assertJsonFragment(['error_description' => 'The provided token did not pass validation']);
     }
 
-    #[Test]
+    /** @test */
     public function getRequestWithTokenWithoutScopeReturnsErrorResponse(): void
     {
         $response = $this->get(
@@ -56,7 +56,7 @@ class MicropubMediaTest extends TestCase
         $response->assertJsonFragment(['error_description' => 'The provided token has no scopes']);
     }
 
-    #[Test]
+    /** @test */
     public function getRequestWithTokenWithInsufficientScopeReturnsErrorResponse(): void
     {
         $response = $this->get(
@@ -67,7 +67,7 @@ class MicropubMediaTest extends TestCase
         $response->assertJsonFragment(['error_description' => 'The token’s scope does not have the necessary requirements.']);
     }
 
-    #[Test]
+    /** @test */
     public function emptyGetRequestWithTokenReceivesOkResponse(): void
     {
         $response = $this->get(
@@ -78,12 +78,14 @@ class MicropubMediaTest extends TestCase
         $response->assertJson(['status' => 'OK']);
     }
 
-    #[Test]
+    /** @test */
     public function clientCanListLastUpload(): void
     {
         Queue::fake();
+        Storage::fake('s3');
         $file = __DIR__ . '/../aaron.png';
         $token = $this->getToken();
+        config(['filesystems.disks.s3.url' => 'https://s3.example.com']);
 
         $response = $this->post(
             '/api/media',
@@ -93,8 +95,12 @@ class MicropubMediaTest extends TestCase
             ['HTTP_Authorization' => 'Bearer ' . $token]
         );
 
+        $location = $response->headers->get('Location');
+
+        $this->assertStringStartsWith('https://s3.example.com/', $location);
+
         $path = parse_url($response->headers->get('Location'), PHP_URL_PATH);
-        $filename = Str::chopStart($path, '/storage/media/');
+        $filename = substr($path, 7);
 
         $lastUploadResponse = $this->get(
             '/api/media?q=last',
@@ -103,14 +109,14 @@ class MicropubMediaTest extends TestCase
         $lastUploadResponse->assertJson(['url' => $response->headers->get('Location')]);
 
         // now remove file
-        unlink(storage_path('app/private/media/') . $filename);
+        unlink(storage_path('app/') . $filename);
-        $this->removeDirIfEmpty(storage_path('app/private/media'));
     }
 
-    #[Test]
+    /** @test */
     public function clientCanSourceUploads(): void
     {
         Queue::fake();
+        Storage::fake('s3');
         $file = __DIR__ . '/../aaron.png';
         $token = $this->getToken();
 
@@ -123,7 +129,7 @@ class MicropubMediaTest extends TestCase
         );
 
         $path = parse_url($response->headers->get('Location'), PHP_URL_PATH);
-        $filename = Str::chopStart($path, '/storage/media/');
+        $filename = substr($path, 7);
 
         $sourceUploadResponse = $this->get(
             '/api/media?q=source',
@@ -135,14 +141,14 @@ class MicropubMediaTest extends TestCase
         ]]]);
 
         // now remove file
-        unlink(storage_path('app/private/media/') . $filename);
+        unlink(storage_path('app/') . $filename);
-        $this->removeDirIfEmpty(storage_path('app/private/media'));
     }
 
-    #[Test]
+    /** @test */
     public function clientCanSourceUploadsWithLimit(): void
     {
         Queue::fake();
+        Storage::fake('s3');
         $file = __DIR__ . '/../aaron.png';
         $token = $this->getToken();
 
@@ -155,7 +161,7 @@ class MicropubMediaTest extends TestCase
         );
 
         $path = parse_url($response->headers->get('Location'), PHP_URL_PATH);
-        $filename = Str::chopStart($path, '/storage/media/');
+        $filename = substr($path, 7);
 
         $sourceUploadResponse = $this->get(
             '/api/media?q=source&limit=1',
@@ -169,11 +175,10 @@ class MicropubMediaTest extends TestCase
         $this->assertCount(1, json_decode($sourceUploadResponse->getContent(), true)['items']);
 
         // now remove file
-        unlink(storage_path('app/private/media/') . $filename);
+        unlink(storage_path('app/') . $filename);
-        $this->removeDirIfEmpty(storage_path('app/private/media'));
     }
 
-    #[Test]
+    /** @test */
     public function mediaEndpointUploadRequiresFile(): void
     {
         $response = $this->post(
@@ -189,7 +194,7 @@ class MicropubMediaTest extends TestCase
         ]);
     }
 
-    #[Test]
+    /** @test */
     public function errorResponseForUnknownQValue(): void
     {
         $response = $this->get(
@@ -200,7 +205,7 @@ class MicropubMediaTest extends TestCase
         $response->assertJson(['error' => 'invalid_request']);
     }
 
-    #[Test]
+    /** @test */
     public function optionsRequestReturnsCorsResponse(): void
     {
         $response = $this->options('/api/media');
@@ -209,7 +214,7 @@ class MicropubMediaTest extends TestCase
         $response->assertHeader('access-control-allow-origin', '*');
     }
 
-    #[Test]
+    /** @test */
     public function mediaEndpointRequestWithInvalidTokenReturns400Response(): void
     {
         $response = $this->post(
@@ -221,7 +226,7 @@ class MicropubMediaTest extends TestCase
         $response->assertJsonFragment(['error_description' => 'The provided token did not pass validation']);
     }
 
-    #[Test]
+    /** @test */
     public function mediaEndpointRequestWithTokenWithNoScopeReturns400Response(): void
     {
         $response = $this->post(
@@ -233,7 +238,7 @@ class MicropubMediaTest extends TestCase
         $response->assertJsonFragment(['error_description' => 'The provided token has no scopes']);
     }
 
-    #[Test]
+    /** @test */
     public function mediaEndpointRequestWithInsufficientTokenScopesReturns401Response(): void
     {
         $response = $this->post(
@@ -247,10 +252,11 @@ class MicropubMediaTest extends TestCase
         ]);
     }
 
-    #[Test]
+    /** @test */
     public function mediaEndpointUploadFile(): void
     {
         Queue::fake();
+        Storage::fake('s3');
         $file = __DIR__ . '/../aaron.png';
 
         $response = $this->post(
@@ -262,18 +268,18 @@ class MicropubMediaTest extends TestCase
         );
 
         $path = parse_url($response->headers->get('Location'), PHP_URL_PATH);
-        $filename = Str::chopStart($path, '/storage/');
+        $filename = substr($path, 7);
         Queue::assertPushed(ProcessMedia::class);
         Storage::disk('local')->assertExists($filename);
         // now remove file
-        unlink(storage_path('app/private/') . $filename);
+        unlink(storage_path('app/') . $filename);
-        $this->removeDirIfEmpty(storage_path('app/private/media'));
     }
 
-    #[Test]
+    /** @test */
     public function mediaEndpointUploadAudioFile(): void
     {
         Queue::fake();
+        Storage::fake('s3');
         $file = __DIR__ . '/../audio.mp3';
 
         $response = $this->post(
@@ -285,18 +291,18 @@ class MicropubMediaTest extends TestCase
         );
 
         $path = parse_url($response->headers->get('Location'), PHP_URL_PATH);
-        $filename = Str::chopStart($path, '/storage/');
+        $filename = substr($path, 7);
         Queue::assertPushed(ProcessMedia::class);
         Storage::disk('local')->assertExists($filename);
         // now remove file
-        unlink(storage_path('app/private/') . $filename);
+        unlink(storage_path('app/') . $filename);
-        $this->removeDirIfEmpty(storage_path('app/private/media'));
     }
 
-    #[Test]
+    /** @test */
     public function mediaEndpointUploadVideoFile(): void
     {
         Queue::fake();
+        Storage::fake('s3');
         $file = __DIR__ . '/../video.ogv';
 
         $response = $this->post(
@@ -308,18 +314,18 @@ class MicropubMediaTest extends TestCase
         );
 
         $path = parse_url($response->headers->get('Location'), PHP_URL_PATH);
-        $filename = Str::chopStart($path, '/storage/');
+        $filename = substr($path, 7);
         Queue::assertPushed(ProcessMedia::class);
         Storage::disk('local')->assertExists($filename);
         // now remove file
-        unlink(storage_path('app/private/') . $filename);
+        unlink(storage_path('app/') . $filename);
-        $this->removeDirIfEmpty(storage_path('app/private/media'));
     }
 
-    #[Test]
+    /** @test */
     public function mediaEndpointUploadDocumentFile(): void
     {
         Queue::fake();
+        Storage::fake('s3');
 
         $response = $this->post(
             '/api/media',
@@ -330,15 +336,14 @@ class MicropubMediaTest extends TestCase
         );
 
         $path = parse_url($response->headers->get('Location'), PHP_URL_PATH);
-        $filename = Str::chopStart($path, '/storage/');
+        $filename = substr($path, 7);
         Queue::assertPushed(ProcessMedia::class);
         Storage::disk('local')->assertExists($filename);
         // now remove file
-        unlink(storage_path('app/private/') . $filename);
+        unlink(storage_path('app/') . $filename);
-        $this->removeDirIfEmpty(storage_path('app/private/media'));
     }
 
-    #[Test]
+    /** @test */
     public function mediaEndpointUploadInvalidFileReturnsError(): void
     {
         Queue::fake();

tests/Feature/NotesControllerTest.php

@@ -6,7 +6,6 @@ namespace Tests\Feature;
 
 use App\Models\Note;
 use Illuminate\Foundation\Testing\RefreshDatabase;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class NotesControllerTest extends TestCase
@@ -16,8 +15,9 @@ class NotesControllerTest extends TestCase
     /**
      * Test the `/notes` page returns 200, this should
      * mean the database is being hit.
+     *
+     * @test
      */
-    #[Test]
     public function notesPageLoads(): void
     {
         $response = $this->get('/notes');
@@ -26,8 +26,9 @@ class NotesControllerTest extends TestCase
 
     /**
      * Test a specific note.
+     *
+     * @test
      */
-    #[Test]
     public function specificNotePageLoads(): void
     {
         $note = Note::factory()->create();
@@ -45,8 +46,9 @@ class NotesControllerTest extends TestCase
 
     /**
      * Test that `/note/{decID}` redirects to `/notes/{nb60id}`.
+     *
+     * @test
      */
-    #[Test]
     public function oldNoteUrlsRedirect(): void
     {
         $note = Note::factory()->create();
@@ -56,22 +58,23 @@ class NotesControllerTest extends TestCase
 
     /**
      * Visit the tagged page and check the tag view data.
+     *
+     * @test
      */
-    #[Test]
     public function taggedNotesPageLoads(): void
     {
         $response = $this->get('/notes/tagged/beer');
         $response->assertViewHas('tag', 'beer');
     }
 
-    #[Test]
+    /** @test */
     public function unknownNoteGives404(): void
     {
         $response = $this->get('/notes/112233');
         $response->assertNotFound();
     }
 
-    #[Test]
+    /** @test */
     public function checkNoteIdNotOutOfRange(): void
     {
         $response = $this->get('/notes/photou-photologo');

tests/Feature/OwnYourGramTest.php

@@ -6,7 +6,6 @@ namespace Tests\Feature;
 
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Illuminate\Support\Carbon;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 use Tests\TestToken;
 
@@ -15,7 +14,7 @@ class OwnYourGramTest extends TestCase
     use RefreshDatabase;
     use TestToken;
 
-    #[Test]
+    /** @test */
     public function postingInstagramUrlSavesMediaPath(): void
     {
         $response = $this->json(

tests/Feature/ParseCachedWebMentionsTest.php

@@ -9,7 +9,6 @@ use Illuminate\FileSystem\FileSystem;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Illuminate\Support\Carbon;
 use Illuminate\Support\Facades\Artisan;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class ParseCachedWebMentionsTest extends TestCase
@@ -26,7 +25,7 @@ class ParseCachedWebMentionsTest extends TestCase
         copy(__DIR__ . '/../tantek.html', storage_path('HTML') . '/http/tantek.com/index.html');
     }
 
-    #[Test]
+    /** @test */
     public function parseWebmentionHtml(): void
     {
         $webmentionAaron = WebMention::factory()->create([
@@ -61,7 +60,7 @@ class ParseCachedWebMentionsTest extends TestCase
 
     protected function tearDown(): void
     {
-        $fs = new FileSystem;
+        $fs = new FileSystem();
         if ($fs->exists(storage_path() . '/HTML/https')) {
             $fs->deleteDirectory(storage_path() . '/HTML/https');
         }

tests/Feature/PlacesTest.php

@@ -6,7 +6,6 @@ namespace Tests\Feature;
 
 use App\Models\Place;
 use Illuminate\Foundation\Testing\RefreshDatabase;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class PlacesTest extends TestCase
@@ -15,8 +14,9 @@ class PlacesTest extends TestCase
 
     /**
      * Test the `/places` page for OK response.
+     *
+     * @test
      */
-    #[Test]
     public function placesPageLoads(): void
     {
         $response = $this->get('/places');
@@ -25,8 +25,9 @@ class PlacesTest extends TestCase
 
     /**
      * Test a specific place.
+     *
+     * @test
      */
-    #[Test]
     public function singlePlacePageLoads(): void
     {
         $place = Place::factory()->create();
@@ -34,7 +35,7 @@ class PlacesTest extends TestCase
         $response->assertViewHas('place', $place);
     }
 
-    #[Test]
+    /** @test */
     public function unknownPlaceGives404()
     {
         $response = $this->get('/places/unknown');

tests/Feature/ReDownloadWebMentionsTest.php

@@ -9,14 +9,13 @@ use App\Models\WebMention;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Illuminate\Support\Facades\Artisan;
 use Illuminate\Support\Facades\Queue;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class ReDownloadWebMentionsTest extends TestCase
 {
     use RefreshDatabase;
 
-    #[Test]
+    /** @test */
     public function downloadJobGetsQueued(): void
     {
         Queue::fake();

tests/Feature/SearchTest.php

@@ -5,12 +5,11 @@ declare(strict_types=1);
 namespace Tests\Feature;
 
 use App\Models\Note;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class SearchTest extends TestCase
 {
-    #[Test]
+    /** @test */
     public function searchEndpointReturnsResults(): void
     {
         Note::factory(10)->create();

tests/Feature/ShortURLsControllerTest.php

@@ -4,33 +4,32 @@ declare(strict_types=1);
 
 namespace Tests\Feature;
 
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class ShortURLsControllerTest extends TestCase
 {
-    #[Test]
+    /** @test */
     public function shortDomainRedirectsToLongDomain(): void
     {
         $response = $this->get('https://' . config('url.shorturl'));
         $response->assertRedirect(config('app.url'));
     }
 
-    #[Test]
+    /** @test */
     public function shortDomainSlashAtRedirectsToTwitter(): void
     {
         $response = $this->get('https://' . config('url.shorturl') . '/@');
         $response->assertRedirect('https://twitter.com/jonnybarnes');
     }
 
-    #[Test]
+    /** @test */
     public function shortDomainSlashTRedirectsToLongDomainSlashNotes(): void
     {
         $response = $this->get('https://' . config('url.shorturl') . '/t/E');
         $response->assertRedirect(config('app.url') . '/notes/E');
     }
 
-    #[Test]
+    /** @test */
     public function shortDomainSlashBRedirectsToLongDomainSlashBlog(): void
     {
         $response = $this->get('https://' . config('url.shorturl') . '/b/1');

tests/Feature/SwarmTest.php

@@ -8,7 +8,6 @@ use App\Jobs\SendWebMentions;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Illuminate\Support\Carbon;
 use Illuminate\Support\Facades\Queue;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 use Tests\TestToken;
 
@@ -19,8 +18,9 @@ class SwarmTest extends TestCase
 
     /**
      * Given a check in to Foursquare, this is the content Ownyourswarm will post to us.
+     *
+     * @test
      */
-    #[Test]
     public function mockedOwnyourswarmRequestWithFoursquare(): void
     {
         Queue::fake();
@@ -66,8 +66,9 @@ class SwarmTest extends TestCase
     /**
      * This request would actually come from another client than OwnYourSwarm, but we’re testing
      * OpenStreetMap data.
+     *
+     * @test
      */
-    #[Test]
     public function mockedOwnyourswarmRequestWithOsm(): void
     {
         Queue::fake();
@@ -108,8 +109,9 @@ class SwarmTest extends TestCase
 
     /**
      * This request would actually come from another client than OwnYourSwarm, as that would include a Foursquare URL
+     *
+     * @test
      */
-    #[Test]
     public function mockedOwnyourswarmRequestWithoutKnownExternalUrl(): void
     {
         Queue::fake();
@@ -148,7 +150,7 @@ class SwarmTest extends TestCase
         Queue::assertPushed(SendWebMentions::class);
     }
 
-    #[Test]
+    /** @test */
     public function mockedOwnyourswarmRequestWithNoTextContent(): void
     {
         $response = $this->json(
@@ -185,7 +187,7 @@ class SwarmTest extends TestCase
         $this->get($response->__get('headers')->get('location'))->assertSee('📍');
     }
 
-    #[Test]
+    /** @test */
     public function mockedOwnyourswarmRequestSavesJustThePostWhenAnErrorOccursInTheCheckinData(): void
     {
         Queue::fake();
@@ -223,7 +225,7 @@ class SwarmTest extends TestCase
         Queue::assertPushed(SendWebMentions::class);
     }
 
-    #[Test]
+    /** @test */
     public function mockedOwnyourswarmRequestWithHAdrLocation(): void
     {
         Queue::fake();
@@ -269,7 +271,7 @@ class SwarmTest extends TestCase
         Queue::assertPushed(SendWebMentions::class);
     }
 
-    #[Test]
+    /** @test */
     public function ownyourswarmCheckinTestUsingRealData(): void
     {
         $response = $this->json(

tests/Feature/TokenServiceTest.php

@@ -9,7 +9,6 @@ use DateTimeImmutable;
 use Lcobucci\JWT\Configuration;
 use Lcobucci\JWT\Signer\Key\InMemory;
 use Lcobucci\JWT\Validation\RequiredConstraintsViolated;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class TokenServiceTest extends TestCase
@@ -17,11 +16,12 @@ class TokenServiceTest extends TestCase
     /**
      * Given the token is dependent on a random nonce, the time of creation and
      * the APP_KEY, to test, we shall create a token, and then verify it.
+     *
+     * @test
      */
-    #[Test]
     public function tokenserviceCreatesAndValidatesTokens(): void
     {
-        $tokenService = new TokenService;
+        $tokenService = new TokenService();
         $data = [
             'me' => 'https://example.org',
             'client_id' => 'https://quill.p3k.io',
@@ -37,7 +37,11 @@ class TokenServiceTest extends TestCase
         $this->assertSame($data, $validData);
     }
 
-    #[Test]
+    /**
+     * @test
+     *
+     * @throws \Exception
+     */
     public function tokensWithDifferentSigningKeyThrowsException(): void
     {
         $this->expectException(RequiredConstraintsViolated::class);
@@ -51,7 +55,7 @@ class TokenServiceTest extends TestCase
         $config = resolve(Configuration::class);
 
         $token = $config->builder()
-            ->issuedAt(new DateTimeImmutable)
+            ->issuedAt(new DateTimeImmutable())
             ->withClaim('client_id', $data['client_id'])
             ->withClaim('me', $data['me'])
             ->withClaim('scope', $data['scope'])
@@ -59,7 +63,7 @@ class TokenServiceTest extends TestCase
             ->getToken($config->signer(), InMemory::plainText(random_bytes(32)))
             ->toString();
 
-        $service = new TokenService;
+        $service = new TokenService();
         $service->validateToken($token);
     }
 }

tests/Feature/WebMentionsControllerTest.php

@@ -8,14 +8,13 @@ use App\Jobs\ProcessWebMention;
 use App\Models\Note;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Illuminate\Support\Facades\Queue;
-use PHPUnit\Framework\Attributes\Test;
 use Tests\TestCase;
 
 class WebMentionsControllerTest extends TestCase
 {
     use RefreshDatabase;
 
-    #[Test]
+    /** @test */
     public function webmentionEndpointCanServeBrowserRequest(): void
     {
         $response = $this->get('/webmention');
@@ -24,8 +23,9 @@ class WebMentionsControllerTest extends TestCase
 
     /**
      * Test webmentions without source and target are rejected.
+     *
+     * @test
      */
-    #[Test]
     public function webmentionsWithoutSourceAndTargetAreRejected(): void
     {
         $response = $this->call('POST', '/webmention', ['source' => 'https://example.org/post/123']);
@@ -36,8 +36,9 @@ class WebMentionsControllerTest extends TestCase
      * Test invalid target gives a 400 response.
      *
      * In this case an invalid target is a URL that doesn’t exist on our domain.
+     *
+     * @test
      */
-    #[Test]
     public function invalidTargetReturnsErrorResponse(): void
     {
         $response = $this->call('POST', '/webmention', [
@@ -49,8 +50,9 @@ class WebMentionsControllerTest extends TestCase
 
     /**
      * Test blog target gets a 501 response due to our not supporting it.
+     *
+     * @test
      */
-    #[Test]
     public function blogTargetReturns501Response(): void
     {
         $response = $this->call('POST', '/webmention', [
@@ -62,8 +64,9 @@ class WebMentionsControllerTest extends TestCase
 
     /**
      * Test that a non-existent note gives a 400 response.
+     *
+     * @test
      */
-    #[Test]
     public function nonexistentNoteReturnsErrorResponse(): void
     {
         $response = $this->call('POST', '/webmention', [
@@ -73,7 +76,7 @@ class WebMentionsControllerTest extends TestCase
         $response->assertStatus(400);
     }
 
-    #[Test]
+    /** @test */
     public function legitimateWebmentionTriggersProcessWebmentionJob(): void
     {
         Queue::fake();

tests/TestCase.php

@@ -7,13 +7,4 @@ use Illuminate\Foundation\Testing\TestCase as BaseTestCase;
 abstract class TestCase extends BaseTestCase
 {
     use CreatesApplication;
-
-    public function removeDirIfEmpty(string $dir): void
-    {
-        // scandir() will always return `.` and `..` so even an “empty”
-        // directory will have a count of 2
-        if (is_dir($dir) && count(scandir($dir)) === 2) {
-            rmdir($dir);
-        }
-    }
 }

tests/TestToken.php

@@ -12,7 +12,7 @@ trait TestToken
         $config = $this->app->make(Configuration::class);
 
         return $config->builder()
-            ->issuedAt(new DateTimeImmutable)
+            ->issuedAt(new DateTimeImmutable())
             ->withClaim('client_id', 'https://quill.p3k.io')
             ->withClaim('me', 'http://jonnybarnes.localhost')
             ->withClaim('scope', ['create', 'update'])
@@ -25,7 +25,7 @@ trait TestToken
         $config = $this->app->make(Configuration::class);
 
         return $config->builder()
-            ->issuedAt(new DateTimeImmutable)
+            ->issuedAt(new DateTimeImmutable())
             ->withClaim('client_id', 'https://quill.p3k.io')
             ->withClaim('me', 'https://jonnybarnes.localhost')
             ->withClaim('scope', 'view')
@@ -38,7 +38,7 @@ trait TestToken
         $config = $this->app->make(Configuration::class);
 
         return $config->builder()
-            ->issuedAt(new DateTimeImmutable)
+            ->issuedAt(new DateTimeImmutable())
             ->withClaim('client_id', 'https://quill.p3k.io')
             ->withClaim('me', 'https://jonnybarnes.localhost')
             ->getToken($config->signer(), $config->signingKey())