3a8f52ce16
Set unreachable_count to 3 and unreachable_notification_sent to true on all team servers in subscriptionEnded(), so the existing cleanup command can pick them up after the 7-day grace period. Also adds feature tests for the subscription-ended cleanup flow and casts server IP to string in existing unreachable server tests to fix type comparison.
73 lines
2.1 KiB
PHP
73 lines
2.1 KiB
PHP
<?php
|
|
|
|
use App\Models\Server;
|
|
use App\Models\Team;
|
|
use Illuminate\Foundation\Testing\RefreshDatabase;
|
|
|
|
uses(RefreshDatabase::class);
|
|
|
|
it('cleans up servers with unreachable_count >= 3 after 7 days', function () {
|
|
$team = Team::factory()->create();
|
|
$server = Server::factory()->create([
|
|
'team_id' => $team->id,
|
|
'unreachable_count' => 50,
|
|
'unreachable_notification_sent' => true,
|
|
'updated_at' => now()->subDays(8),
|
|
]);
|
|
|
|
$this->artisan('cleanup:unreachable-servers')->assertSuccessful();
|
|
|
|
$server->refresh();
|
|
expect($server->ip)->toBe('1.2.3.4');
|
|
});
|
|
|
|
it('does not clean up servers with unreachable_count less than 3', function () {
|
|
$team = Team::factory()->create();
|
|
$server = Server::factory()->create([
|
|
'team_id' => $team->id,
|
|
'unreachable_count' => 2,
|
|
'unreachable_notification_sent' => true,
|
|
'updated_at' => now()->subDays(8),
|
|
]);
|
|
|
|
$originalIp = (string) $server->ip;
|
|
|
|
$this->artisan('cleanup:unreachable-servers')->assertSuccessful();
|
|
|
|
$server->refresh();
|
|
expect($server->ip)->toBe($originalIp);
|
|
});
|
|
|
|
it('does not clean up servers updated within 7 days', function () {
|
|
$team = Team::factory()->create();
|
|
$server = Server::factory()->create([
|
|
'team_id' => $team->id,
|
|
'unreachable_count' => 10,
|
|
'unreachable_notification_sent' => true,
|
|
'updated_at' => now()->subDays(3),
|
|
]);
|
|
|
|
$originalIp = (string) $server->ip;
|
|
|
|
$this->artisan('cleanup:unreachable-servers')->assertSuccessful();
|
|
|
|
$server->refresh();
|
|
expect($server->ip)->toBe($originalIp);
|
|
});
|
|
|
|
it('does not clean up servers without notification sent', function () {
|
|
$team = Team::factory()->create();
|
|
$server = Server::factory()->create([
|
|
'team_id' => $team->id,
|
|
'unreachable_count' => 10,
|
|
'unreachable_notification_sent' => false,
|
|
'updated_at' => now()->subDays(8),
|
|
]);
|
|
|
|
$originalIp = (string) $server->ip;
|
|
|
|
$this->artisan('cleanup:unreachable-servers')->assertSuccessful();
|
|
|
|
$server->refresh();
|
|
expect($server->ip)->toBe($originalIp);
|
|
});
|