perf(orders): fix audits table being queried for each order in overview

src/Pdk/Audit/Repository/WcPdkAuditRepository.php

@@ -13,7 +13,10 @@
 use MyParcelNL\Pdk\Storage\Contract\StorageInterface;
 use MyParcelNL\WooCommerce\Database\Contract\WpDatabaseServiceInterface;
 
-final class WcPdkAuditRepository extends Repository implements PdkAuditRepositoryInterface
+/**
+ * @final
+ */
+class WcPdkAuditRepository extends Repository implements PdkAuditRepositoryInterface
 {
     /**
      * @var \MyParcelNL\WooCommerce\Database\Contract\WpDatabaseServiceInterface
@@ -36,19 +39,21 @@ public function __construct(StorageInterface $storage, WpDatabaseServiceInterfac
      */
     public function all(): AuditCollection
     {
-        $audits = $this->wpDatabaseService->getAll(Pdk::get('tableNameAudits'));
+        return $this->retrieve('audits', function () {
+            $audits = $this->wpDatabaseService->getAll(Pdk::get('tableNameAudits'));
 
-        return new AuditCollection($audits->map(static function (array $audit): Audit {
-            return new Audit([
-                'id'              => $audit['auditId'],
-                'arguments'       => json_decode($audit['arguments'], true),
-                'action'          => $audit['action'],
-                'model'           => $audit['model'],
-                'modelIdentifier' => $audit['modelIdentifier'],
-                'created'         => new DateTime($audit['created']),
-                'type'            => $audit['type'],
-            ]);
-        }));
+            return new AuditCollection($audits->map(static function (array $audit): Audit {
+                return new Audit([
+                    'id'              => $audit['auditId'],
+                    'arguments'       => json_decode($audit['arguments'], true),
+                    'action'          => $audit['action'],
+                    'model'           => $audit['model'],
+                    'modelIdentifier' => $audit['modelIdentifier'],
+                    'created'         => new DateTime($audit['created']),
+                    'type'            => $audit['type'],
+                ]);
+            }));
+        });
     }
 
     /**

tests/Mock/MockWcPdkAuditRepository.php

@@ -0,0 +1,16 @@
+<?php
+
+declare(strict_types=1);
+
+namespace MyParcelNL\WooCommerce\Tests\Mock;
+
+use MyParcelNL\WooCommerce\Pdk\Audit\Repository\WcPdkAuditRepository;
+use Symfony\Contracts\Service\ResetInterface;
+
+final class MockWcPdkAuditRepository extends WcPdkAuditRepository implements ResetInterface
+{
+    public function reset(): void
+    {
+        $this->storage->delete('audits');
+    }
+}

tests/Mock/MockWcPdkBootstrapper.php

@@ -6,6 +6,7 @@
 
 use MyParcelNL\Pdk\Base\Concern\PdkInterface;
 use MyParcelNL\WooCommerce\Database\Contract\WpDatabaseServiceInterface;
+use MyParcelNL\WooCommerce\Pdk\Audit\Repository\WcPdkAuditRepository;
 use MyParcelNL\WooCommerce\Pdk\WcPdkBootstrapper;
 use function DI\get;
 
@@ -98,6 +99,7 @@ protected function getAdditionalConfig(
             self::$config,
             [
                 WpDatabaseServiceInterface::class => get(MockWpDatabaseService::class),
+                WcPdkAuditRepository::class       => get(MockWcPdkAuditRepository::class),
             ]
         );
     }

tests/Unit/Pdk/Plugin/Repository/WcPdkAuditRepositoryTest.php

@@ -37,9 +37,12 @@
         'orderIds' => [$wcOrder->get_id()],
     ]);
 
-    /** @var \MyParcelNL\Pdk\Audit\Contract\PdkAuditRepositoryInterface $auditRepository */
+    /** @var \MyParcelNL\WooCommerce\Tests\Mock\MockWcPdkAuditRepository $auditRepository */
     $auditRepository = Pdk::get(PdkAuditRepositoryInterface::class);
 
+    // Clear the cache of retrieved audits
+    $auditRepository->reset();
+
     $audit = $auditRepository->all()
         ->first();