[1.x] Test cache logging via the event, not the cache-clear controller

The /api/cache controller also runs assets:publish, which returns a
non-zero exit code (409) in CI where asset publishing isn't available.
Dispatch the ClearingCache event directly and assert the log entry, which
tests the listener without depending on that environment-specific step.

Author: imorland

extensions/audit/tests/integration/CoreCacheTest.php

@@ -9,15 +9,26 @@
 
 namespace Flarum\Audit\Tests\integration;
 
+use Flarum\Audit\AuditLog;
+use Flarum\Foundation\Event\ClearingCache;
+use Illuminate\Contracts\Events\Dispatcher;
+
 class CoreCacheTest extends TestCase
 {
     /**
      * @test
      */
-    public function web_cache_clear()
+    public function cache_clear_is_logged()
     {
-        $this->sendSuccessfulRequest('DELETE', '/api/cache', [], 204);
+        // Dispatch the event our listener hooks directly, rather than driving the /api/cache
+        // controller, which also runs assets:publish — a step that isn't reliably available in CI.
+        $this->app()->getContainer()->make(Dispatcher::class)->dispatch(new ClearingCache());
+
+        $log = AuditLog::query()->where('action', 'cache_cleared')->first();
 
-        $this->assertLogExists('cache_cleared');
+        $this->assertNotNull($log, 'A cache_cleared entry should be logged');
+        // Dispatched outside an HTTP request, so there is no actor or IP context.
+        $this->assertNull($log->actor_id);
+        $this->assertNull($log->payload);
     }
 }

extensions/audit/tests/integration/TestCase.php

@@ -34,7 +34,6 @@ public function setUp(): void
                 ->exemptRoute('confirmEmail.submit')
                 ->exemptRoute('savePassword')
                 ->exemptRoute('forgot')
-                ->exemptRoute('cache.clear')
         );
 
         $this->prepareDatabase([