monitor/api/tests/Unit/Sync/CodebergSyncServiceTest.php
fds2610-bot 29a13e89eb Codeberg-Sync ordnet offene Issues per Label der richtigen Spalte zu
Bisher landeten alle offenen Issues unabhaengig von ihren Labels in
Backlog. Jetzt bestimmt das Label die Spalte: wip -> in_arbeit,
testing -> testing, automate -> bereit, sonst backlog. Geschlossene
Issues bleiben in Done. Die "manuell verschoben"-Sperre im
TicketRepository schuetzt jetzt nur noch qa/fertig (zukuenftige
Labels) statt aller Nicht-Backlog/Done-Spalten, damit der Sync
Tickets weiterhin zwischen den automatisierten Spalten verschieben
kann, wenn sich Labels aendern.

Schliesst #14
2026-07-29 16:48:23 +02:00

166 lines
6 KiB
PHP

<?php
declare(strict_types=1);
namespace Monitor\Api\Tests\Sync;
use Monitor\Api\Db\Database;
use Monitor\Api\Repository\TicketRepository;
use Monitor\Api\Sync\CodebergClient;
use Monitor\Api\Sync\CodebergSyncService;
use PHPUnit\Framework\TestCase;
final class FakeCodebergClient implements CodebergClient
{
/** @param array<string, array<int, array{number:int,title:string,body:?string,labels:array}>> $issuesByState */
public function __construct(private readonly array $issuesByState)
{
}
public function listIssues(string $owner, string $repo, string $state): array
{
return $this->issuesByState[$state] ?? [];
}
}
final class CodebergSyncServiceTest extends TestCase
{
public function testSyncCreatesBacklogAndDoneTickets(): void
{
$tickets = new TicketRepository(Database::inMemory());
$client = new FakeCodebergClient([
'open' => [
['number' => 1, 'title' => 'Offenes Issue', 'body' => 'Beschreibung', 'labels' => [['name' => 'bug', 'color' => 'ff0000']]],
],
'closed' => [
['number' => 2, 'title' => 'Geschlossenes Issue', 'body' => null, 'labels' => []],
],
]);
(new CodebergSyncService($client, $tickets, 'fds2610', 'monitor'))->sync();
$backlog = $tickets->all('backlog');
$done = $tickets->all('done');
self::assertCount(1, $backlog);
self::assertSame('Offenes Issue', $backlog[0]['title']);
self::assertSame('bug', $backlog[0]['type']);
self::assertSame('#ff0000', $backlog[0]['color']);
self::assertSame('codeberg', $backlog[0]['source']);
self::assertCount(1, $done);
self::assertSame('Geschlossenes Issue', $done[0]['title']);
self::assertNull($done[0]['type']);
self::assertNull($done[0]['color']);
}
public function testSyncNormalizesLabelColorRegardlessOfLeadingHash(): void
{
$tickets = new TicketRepository(Database::inMemory());
$client = new FakeCodebergClient([
'open' => [
['number' => 1, 'title' => 'Mit Raute', 'body' => null, 'labels' => [['name' => 'bug', 'color' => '#00ff00']]],
],
'closed' => [],
]);
(new CodebergSyncService($client, $tickets, 'fds2610', 'monitor'))->sync();
self::assertSame('#00ff00', $tickets->all('backlog')[0]['color']);
}
public function testSyncDoesNotResetManuallyMovedTicket(): void
{
$tickets = new TicketRepository(Database::inMemory());
$client = new FakeCodebergClient([
'open' => [['number' => 1, 'title' => 'Issue', 'body' => null, 'labels' => []]],
'closed' => [],
]);
$service = new CodebergSyncService($client, $tickets, 'fds2610', 'monitor');
$service->sync();
$ticket = $tickets->findByExternalRef('1');
$tickets->update((int) $ticket['id'], ['column' => 'qa']);
$service->sync();
$stillQa = $tickets->findByExternalRef('1');
self::assertSame('qa', $stillQa['column_name']);
}
/** @return array<int, array{0: string, 1: string}> */
public static function labelColumnProvider(): array
{
return [
'automate -> bereit' => ['automate', 'bereit'],
'wip -> in_arbeit' => ['wip', 'in_arbeit'],
'testing -> testing' => ['testing', 'testing'],
];
}
/** @dataProvider labelColumnProvider */
public function testSyncMapsOpenIssueLabelToBoardColumn(string $label, string $expectedColumn): void
{
$tickets = new TicketRepository(Database::inMemory());
$client = new FakeCodebergClient([
'open' => [
['number' => 1, 'title' => 'Issue', 'body' => null, 'labels' => [['name' => $label, 'color' => 'ff0000']]],
],
'closed' => [],
]);
(new CodebergSyncService($client, $tickets, 'fds2610', 'monitor'))->sync();
$ticket = $tickets->findByExternalRef('1');
self::assertSame($expectedColumn, $ticket['column_name']);
}
public function testSyncMovesTicketBetweenAutomatedColumnsWhenLabelChanges(): void
{
$tickets = new TicketRepository(Database::inMemory());
$client = new FakeCodebergClient([
'open' => [
['number' => 1, 'title' => 'Issue', 'body' => null, 'labels' => [['name' => 'automate', 'color' => 'ff0000']]],
],
'closed' => [],
]);
$service = new CodebergSyncService($client, $tickets, 'fds2610', 'monitor');
$service->sync();
self::assertSame('bereit', $tickets->findByExternalRef('1')['column_name']);
$client = new FakeCodebergClient([
'open' => [
['number' => 1, 'title' => 'Issue', 'body' => null, 'labels' => [['name' => 'wip', 'color' => 'ff0000']]],
],
'closed' => [],
]);
(new CodebergSyncService($client, $tickets, 'fds2610', 'monitor'))->sync();
self::assertSame('in_arbeit', $tickets->findByExternalRef('1')['column_name']);
}
public function testSyncClosesTicketFromAnyAutomatedColumnIntoDone(): void
{
$tickets = new TicketRepository(Database::inMemory());
$client = new FakeCodebergClient([
'open' => [
['number' => 1, 'title' => 'Issue', 'body' => null, 'labels' => [['name' => 'testing', 'color' => 'ff0000']]],
],
'closed' => [],
]);
$service = new CodebergSyncService($client, $tickets, 'fds2610', 'monitor');
$service->sync();
self::assertSame('testing', $tickets->findByExternalRef('1')['column_name']);
$client = new FakeCodebergClient([
'open' => [],
'closed' => [
['number' => 1, 'title' => 'Issue', 'body' => null, 'labels' => [['name' => 'testing', 'color' => 'ff0000']]],
],
]);
(new CodebergSyncService($client, $tickets, 'fds2610', 'monitor'))->sync();
self::assertSame('done', $tickets->findByExternalRef('1')['column_name']);
}
}