Files
XC_VM/tests/Unit/ServerRepositorySimpleTest.php
T
obscuremind 190afa8dc5 fix(cluster): apply the review of the servers, node and crontab sections
A node's replica now owns the servers cache and the crontab's jobs only
once an authoritative apply has built them (a `replica_owned` record in
tmp/cache), not as soon as the agent has stored the sections. A refused
section, and CONFIG turned off, hand the caches back to MAIN's database.
cron:cache applies the replica from disk every minute while CONFIG is on,
and getAll() rebuilds a missing owned cache from disk and never writes
the database's rows over one. A database copy with frozen liveness can no
longer pass for the replica's, and an old crontab is not reused after
CONFIG was off.

- Both crontab readers write ReplicaApply::crontabText, which is tested.
- The crontab section leaves out rows a node would refuse and audits them
  once. cronJob() anchors with \z, so a trailing newline is refused.
- Migration 043 gives the plan's MAIN-only crons the role main (maxmind
  stays all), so the section sends no job the LB build strips.
- config.changed goes only to agents that say config_changed at hello,
  each node on its own; a clone quarantine announces it too.
- The ETag cache's generation is a random token, so racing bumps differ.
- cron:certbot reads its own certbot_ssl from its row; service runs the
  boot apply only on a CONFIG node, with a 15 s timeout.
- Tests for the review's surviving mutations, test isolation fixes, and
  the ADR, the plan and the agent contract brought in line.
2026-09-26 22:20:28 +00:00

67 lines
2.5 KiB
PHP

<?php
use XcVm\Domain\Server\ServerRepository;
use PHPUnit\Framework\TestCase;
/**
* ServerRepository::getStreamingSimple / getProxySimple — server lists for the
* admin/reseller UI, filtered by permissions.
*
* Regression: these run during admin-API boot on the login page, where the
* $rPermissions global is still null. A strict `array $rPermissions` hint made
* that a fatal TypeError at boot; the parameter must accept null (the methods
* only read it via isset()). Also covers the online filter and reseller name
* masking. Driven against the SQLite TestDb via setDb().
*/
final class ServerRepositorySimpleTest extends TestCase {
private TestDb $db;
protected function setUp(): void {
$this->db = new TestDb();
$now = time();
$this->db->exec(
'CREATE TABLE servers (id INTEGER PRIMARY KEY, server_type INTEGER, status INTEGER, last_check_ago INTEGER, is_main INTEGER, `order` INTEGER, server_name TEXT);'
);
$this->db->query(
'INSERT INTO servers (id, server_type, status, last_check_ago, is_main, `order`, server_name) VALUES
(1, 0, 1, ?, 0, 0, "Stream-1"),
(2, 0, 2, ?, 0, 0, "Stream-2"),
(3, 1, 1, ?, 0, 0, "Proxy-1");',
$now,
$now - 100000,
$now
);
ServerRepository::setDb($this->db);
}
protected function tearDown(): void {
// The injected connection would outlive this test and stand in for the next one's.
(new ReflectionProperty(ServerRepository::class, 'db'))->setValue(null, null);
}
public function testGetStreamingSimpleAcceptsNullPermissions(): void {
// Regression: null perms (login-page boot) must not throw.
$servers = ServerRepository::getStreamingSimple(null);
$this->assertSame([1], array_keys($servers), 'only the online streaming server');
$this->assertSame('Stream-1', $servers[1]['server_name'], 'name not masked without reseller perms');
}
public function testGetStreamingSimpleAllTypeReturnsOfflineToo(): void {
$servers = ServerRepository::getStreamingSimple(null, 'all');
$this->assertSame([1, 2], array_keys($servers), 'both streaming servers, proxy excluded');
}
public function testGetStreamingSimpleMasksNamesForResellers(): void {
$servers = ServerRepository::getStreamingSimple(['is_reseller' => 1], 'all');
$this->assertSame('Server #1', $servers[1]['server_name']);
$this->assertSame('Server #2', $servers[2]['server_name']);
}
public function testGetProxySimpleAcceptsNullPermissions(): void {
$proxies = ServerRepository::getProxySimple(null);
$this->assertSame([3], array_keys($proxies), 'only proxy-type servers');
}
}