Der Codeberg-Sync liest nun Issues aus mehreren konfigurierbaren Projekten (Config::codebergRepos(), Default: fds2610/monitor gelb, fds2610/Steuerkram gruen, fds2610/nc-gantt blau) statt nur aus einem einzigen Repo. Damit external_ref (Issue-Nummer) projektuebergreifend nicht kollidiert, ist es jetzt zusammen mit dem neuen Ticket-Feld "repo" eindeutig; Migration 002_multi_repo_tickets.sql ergaenzt die Spalte und den Index. Da Migrationen dadurch nicht mehr rein additiv sind, trackt Database::migrate() jetzt angewandte Dateien in schema_migrations, damit ALTER TABLE nicht bei jedem Prozessstart erneut laeuft. Das Frontend baut Codeberg-Links jetzt aus dem repo-Feld jedes Tickets statt aus einer global fixen owner/repo-Angabe. Schliesst #26
193 lines
7.6 KiB
PHP
193 lines
7.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<string, array<int, array{number:int,title:string,body:?string,labels:array}>>> $issuesByRepoAndState */
|
|
public function __construct(private readonly array $issuesByRepoAndState)
|
|
{
|
|
}
|
|
|
|
public function listIssues(string $owner, string $repo, string $state): array
|
|
{
|
|
return $this->issuesByRepoAndState[$owner . '/' . $repo][$state] ?? [];
|
|
}
|
|
}
|
|
|
|
final class CodebergSyncServiceTest extends TestCase
|
|
{
|
|
private const MONITOR_REPO = ['owner' => 'fds2610', 'repo' => 'monitor', 'color' => '#fbca04'];
|
|
private const MONITOR_SLUG = 'fds2610/monitor';
|
|
|
|
/** @param array<string, array<int, array{number:int,title:string,body:?string,labels:array}>> $issuesByState */
|
|
private static function clientForMonitor(array $issuesByState): FakeCodebergClient
|
|
{
|
|
return new FakeCodebergClient([self::MONITOR_SLUG => $issuesByState]);
|
|
}
|
|
|
|
public function testSyncCreatesBacklogAndDoneTicketsColoredByProject(): void
|
|
{
|
|
$tickets = new TicketRepository(Database::inMemory());
|
|
$client = self::clientForMonitor([
|
|
'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, [self::MONITOR_REPO]))->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('#fbca04', $backlog[0]['color']);
|
|
self::assertSame(self::MONITOR_SLUG, $backlog[0]['repo']);
|
|
self::assertSame('codeberg', $backlog[0]['source']);
|
|
|
|
self::assertCount(1, $done);
|
|
self::assertSame('Geschlossenes Issue', $done[0]['title']);
|
|
self::assertNull($done[0]['type']);
|
|
self::assertSame('#fbca04', $done[0]['color']);
|
|
}
|
|
|
|
public function testSyncKeepsIssuesFromDifferentProjectsSeparateAndColorsThemPerProject(): void
|
|
{
|
|
$tickets = new TicketRepository(Database::inMemory());
|
|
$client = new FakeCodebergClient([
|
|
'fds2610/monitor' => [
|
|
'open' => [['number' => 1, 'title' => 'Monitor-Issue 1', 'body' => null, 'labels' => []]],
|
|
'closed' => [],
|
|
],
|
|
'fds2610/Steuerkram' => [
|
|
'open' => [['number' => 1, 'title' => 'Steuerkram-Issue 1', 'body' => null, 'labels' => []]],
|
|
'closed' => [],
|
|
],
|
|
]);
|
|
|
|
(new CodebergSyncService($client, $tickets, [
|
|
self::MONITOR_REPO,
|
|
['owner' => 'fds2610', 'repo' => 'Steuerkram', 'color' => '#28a745'],
|
|
]))->sync();
|
|
|
|
$backlog = $tickets->all('backlog');
|
|
self::assertCount(2, $backlog, 'gleiche Issue-Nummer aus verschiedenen Projekten darf nicht zu einem Ticket verschmelzen');
|
|
|
|
$monitorTicket = $tickets->findByExternalRef('1', 'fds2610/monitor');
|
|
$steuerkramTicket = $tickets->findByExternalRef('1', 'fds2610/Steuerkram');
|
|
|
|
self::assertSame('Monitor-Issue 1', $monitorTicket['title']);
|
|
self::assertSame('#fbca04', $monitorTicket['color']);
|
|
|
|
self::assertSame('Steuerkram-Issue 1', $steuerkramTicket['title']);
|
|
self::assertSame('#28a745', $steuerkramTicket['color']);
|
|
}
|
|
|
|
public function testSyncDoesNotResetManuallyMovedTicket(): void
|
|
{
|
|
$tickets = new TicketRepository(Database::inMemory());
|
|
$client = self::clientForMonitor([
|
|
'open' => [['number' => 1, 'title' => 'Issue', 'body' => null, 'labels' => []]],
|
|
'closed' => [],
|
|
]);
|
|
$service = new CodebergSyncService($client, $tickets, [self::MONITOR_REPO]);
|
|
|
|
$service->sync();
|
|
$ticket = $tickets->findByExternalRef('1', self::MONITOR_SLUG);
|
|
$tickets->update((int) $ticket['id'], ['column' => 'qa']);
|
|
|
|
$service->sync();
|
|
|
|
$stillQa = $tickets->findByExternalRef('1', self::MONITOR_SLUG);
|
|
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 = self::clientForMonitor([
|
|
'open' => [
|
|
['number' => 1, 'title' => 'Issue', 'body' => null, 'labels' => [['name' => $label, 'color' => 'ff0000']]],
|
|
],
|
|
'closed' => [],
|
|
]);
|
|
|
|
(new CodebergSyncService($client, $tickets, [self::MONITOR_REPO]))->sync();
|
|
|
|
$ticket = $tickets->findByExternalRef('1', self::MONITOR_SLUG);
|
|
self::assertSame($expectedColumn, $ticket['column_name']);
|
|
}
|
|
|
|
public function testSyncMovesTicketBetweenAutomatedColumnsWhenLabelChanges(): void
|
|
{
|
|
$tickets = new TicketRepository(Database::inMemory());
|
|
$client = self::clientForMonitor([
|
|
'open' => [
|
|
['number' => 1, 'title' => 'Issue', 'body' => null, 'labels' => [['name' => 'automate', 'color' => 'ff0000']]],
|
|
],
|
|
'closed' => [],
|
|
]);
|
|
$service = new CodebergSyncService($client, $tickets, [self::MONITOR_REPO]);
|
|
|
|
$service->sync();
|
|
self::assertSame('bereit', $tickets->findByExternalRef('1', self::MONITOR_SLUG)['column_name']);
|
|
|
|
$client = self::clientForMonitor([
|
|
'open' => [
|
|
['number' => 1, 'title' => 'Issue', 'body' => null, 'labels' => [['name' => 'wip', 'color' => 'ff0000']]],
|
|
],
|
|
'closed' => [],
|
|
]);
|
|
(new CodebergSyncService($client, $tickets, [self::MONITOR_REPO]))->sync();
|
|
|
|
self::assertSame('in_arbeit', $tickets->findByExternalRef('1', self::MONITOR_SLUG)['column_name']);
|
|
}
|
|
|
|
public function testSyncClosesTicketFromAnyAutomatedColumnIntoDone(): void
|
|
{
|
|
$tickets = new TicketRepository(Database::inMemory());
|
|
$client = self::clientForMonitor([
|
|
'open' => [
|
|
['number' => 1, 'title' => 'Issue', 'body' => null, 'labels' => [['name' => 'testing', 'color' => 'ff0000']]],
|
|
],
|
|
'closed' => [],
|
|
]);
|
|
$service = new CodebergSyncService($client, $tickets, [self::MONITOR_REPO]);
|
|
$service->sync();
|
|
self::assertSame('testing', $tickets->findByExternalRef('1', self::MONITOR_SLUG)['column_name']);
|
|
|
|
$client = self::clientForMonitor([
|
|
'open' => [],
|
|
'closed' => [
|
|
['number' => 1, 'title' => 'Issue', 'body' => null, 'labels' => [['name' => 'testing', 'color' => 'ff0000']]],
|
|
],
|
|
]);
|
|
(new CodebergSyncService($client, $tickets, [self::MONITOR_REPO]))->sync();
|
|
|
|
self::assertSame('done', $tickets->findByExternalRef('1', self::MONITOR_SLUG)['column_name']);
|
|
}
|
|
}
|