monitor/api/tests/Unit/Repository/SystemRepositoryTest.php
fds2610-bot 55d5c34ca6 Ueberwachte Hosts ueber ein Zahnrad am System-Monitor pflegen (Issue #70)
Wer einen Server aus der Ping-Ueberwachung nehmen wollte, musste sich bisher auf den
Rechner mit check_fds.sh einloggen und die hosts.conf editieren - und die Kachel blieb
danach trotzdem im System-Monitor stehen, weil ein einmal gemeldetes System nirgends
wieder geloescht werden konnte.

Die Liste steht jetzt in der Datenbank und ist ueber ein Zahnrad neben der Ueberschrift
"Systeme" pflegbar, analog zum Zahnrad am PO-Platz (Issue #27). Jeder Host gehoert dem
OPs-Agenten (Issue #66), der ihn anpingt: API und Ping-Check laufen nicht zwangslaeufig
auf demselben Rechner, die API kann die hosts.conf also nicht selbst schreiben.
check_fds.sh holt sie stattdessen bei jedem Lauf mit seinem Agenten-Key ab und schreibt
daraus die lokale Datei; ist die API nicht erreichbar, pingt es unveraendert nach der
zuletzt geschriebenen Liste weiter.

Beim ersten Lauf laedt der Check seine bestehende hosts.conf hoch, damit die bereits
ueberwachten Server nicht von Hand nachgetragen werden muessen. Dass die API das nur
einmal annimmt (Marker in host_config_imports) und "noch nie konfiguriert" per 204 von
"im Frontend geleert" per 200 unterscheidet, ist noetig, damit ein Ping-Rechner mit alter
lokaler Datei die dort geloeschten Hosts nicht wieder anlegt.

Loeschen raeumt mit auf: Ein entfernter Host nimmt seine Kachel mit - beide Ids, denn der
status.txt-Import benennt ihn nach dem Hostnamen, die REST-Meldung nach dessen Slug.
Kacheln ohne zugehoerigen Host (Selbstmelder oder Reste laengst entfernter Hosts) stehen
im selben Dialog unter "Weitere Kacheln im System-Monitor" und sind dort einzeln
loeschbar - dafuer gibt es neu DELETE /api/v1/systems/{id}.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015vNjbfr8M3HSfWMee92GTE
2026-09-01 13:14:11 +02:00

108 lines
3.7 KiB
PHP

<?php
declare(strict_types=1);
namespace Monitor\Api\Tests\Repository;
use InvalidArgumentException;
use Monitor\Api\Db\Database;
use Monitor\Api\Repository\SystemRepository;
use PHPUnit\Framework\TestCase;
final class SystemRepositoryTest extends TestCase
{
private SystemRepository $repository;
protected function setUp(): void
{
$this->repository = new SystemRepository(Database::inMemory());
}
public function testReportStatusInsertsSystem(): void
{
$system = $this->repository->reportStatus('srv1', ['status' => 'ok', 'name' => 'Server 1']);
self::assertSame('ok', $system['status']);
self::assertSame('api', $system['source']);
}
public function testReportStatusRejectsInvalidStatus(): void
{
$this->expectException(InvalidArgumentException::class);
$this->repository->reportStatus('srv1', ['status' => 'unknown']);
}
public function testReportStatusUpdatesExistingSystem(): void
{
$this->repository->reportStatus('srv1', ['status' => 'ok']);
$updated = $this->repository->reportStatus('srv1', ['status' => 'error', 'detail' => 'Timeout']);
self::assertSame('error', $updated['status']);
self::assertSame('Timeout', $updated['detail']);
}
public function testImportLegacyRowsDoesNotOverwriteApiSourcedSystem(): void
{
$this->repository->reportStatus('srv1', ['status' => 'ok']);
$this->repository->importLegacyRows([
['id' => 'srv1', 'name' => 'srv1', 'status' => 'error', 'detail' => '1.2.3.4', 'last_check_at' => '2026-01-01 00:00'],
]);
$system = $this->repository->find('srv1');
self::assertSame('ok', $system['status']);
self::assertSame('api', $system['source']);
}
public function testImportLegacyRowsCreatesNewLegacySystem(): void
{
$this->repository->importLegacyRows([
['id' => 'srv2', 'name' => 'srv2', 'status' => 'ok', 'detail' => '1.2.3.5', 'last_check_at' => '2026-01-01 00:00'],
]);
$system = $this->repository->find('srv2');
self::assertNotNull($system);
self::assertSame('legacy', $system['source']);
}
public function testAllOrdersErrorsFirstAndLimitsToNineTiles(): void
{
for ($i = 0; $i < 12; $i++) {
$this->repository->reportStatus('srv' . $i, ['status' => 'ok']);
}
$this->repository->reportStatus('broken', ['status' => 'error']);
$all = $this->repository->all();
self::assertCount(SystemRepository::MAX_TILES, $all);
self::assertSame('error', $all[0]['status']);
}
public function testFingerprintChangesWhenStatusIsReported(): void
{
$before = $this->repository->fingerprint();
$this->repository->reportStatus('srv1', ['status' => 'ok']);
self::assertNotSame($before, $this->repository->fingerprint());
}
public function testDeleteRemovesSystems(): void
{
$this->repository->reportStatus('srv1', ['status' => 'ok']);
$this->repository->reportStatus('srv2', ['status' => 'ok']);
$deleted = $this->repository->delete(['srv1', 'gibt-es-nicht']);
self::assertSame(1, $deleted);
self::assertNull($this->repository->find('srv1'));
self::assertNotNull($this->repository->find('srv2'));
}
public function testIdsForHostnameCoversBothReportingPaths(): void
{
self::assertSame(['Nextcloud(lokal)', 'nextcloud-lokal'], SystemRepository::idsForHostname('Nextcloud(lokal)'));
// Ein Name ohne Sonderzeichen ergibt nur eine Id - der Slug ist dann identisch.
self::assertSame(['monitor'], SystemRepository::idsForHostname('monitor'));
}
}