Use admin as fallback string for actor when transferring packages in bulk

Author: stevenrombauts

src/Entity/AuditRecord.php

@@ -75,7 +75,7 @@ public static function packageTransferred(Package $package, ?User $actor, array
         $previous = array_map($callback, $previousMaintainers);
         $current = array_map($callback, $currentMaintainers);
 
-        return new self(AuditRecordType::PackageTransferred, ['name' => $package->getName(), 'actor' => self::getUserData($actor), 'previous_maintainers' => $previous, 'current_maintainers' => $current], $actor?->getId(), $package->getVendor(), $package->getId());
+        return new self(AuditRecordType::PackageTransferred, ['name' => $package->getName(), 'actor' => self::getUserData($actor, 'admin'), 'previous_maintainers' => $previous, 'current_maintainers' => $current], $actor?->getId(), $package->getVendor(), $package->getId());
     }
 
     public static function versionDeleted(Version $version, ?User $actor): self

tests/Command/TransferOwnershipCommandTest.php

@@ -141,7 +141,7 @@ private function assertAuditLogWasCreated(Package $package, array $oldMaintainer
         ]);
 
         $this->assertNotNull($record);
-        $this->assertSame('unknown', $record->attributes['actor']);
+        $this->assertSame('admin', $record->attributes['actor']);
         $this->assertSame($package->getId(), $record->packageId);
 
         $callable = fn (array $user) => $user['username'];