From 0615c784ac91072817d94936e64e25884843150a Mon Sep 17 00:00:00 2001 From: Andras Bacsai <5845193+andrasbacsai@users.noreply.github.com> Date: Tue, 8 Sep 2026 12:53:57 +0200 Subject: [PATCH] fix(server): remove duplicate Disabled badge from metrics empty state The metrics-disabled empty state already titles itself "Metrics are disabled", so the extra status badge was redundant. Cover the Sentinel-enabled branch so that badge does not return. --- resources/views/livewire/server/charts.blade.php | 4 ---- tests/Feature/ServerMetricsEmptyStateTest.php | 12 ++++++++++++ 2 files changed, 12 insertions(+), 4 deletions(-) diff --git a/resources/views/livewire/server/charts.blade.php b/resources/views/livewire/server/charts.blade.php index da992ded2d..5fabd36782 100644 --- a/resources/views/livewire/server/charts.blade.php +++ b/resources/views/livewire/server/charts.blade.php @@ -262,10 +262,6 @@ @elseif ($server->isSentinelEnabled()) - - - - diff --git a/tests/Feature/ServerMetricsEmptyStateTest.php b/tests/Feature/ServerMetricsEmptyStateTest.php index e3c34065bf..b96dc4b4b5 100644 --- a/tests/Feature/ServerMetricsEmptyStateTest.php +++ b/tests/Feature/ServerMetricsEmptyStateTest.php @@ -8,3 +8,15 @@ test('sentinel-required metrics state does not repeat an unavailable badge', fun ->toContain('title="Sentinel is required"') ->not->toContain('status="Unavailable"'); }); + +test('disabled metrics state does not repeat a disabled badge', function () { + $view = file_get_contents(resource_path('views/livewire/server/charts.blade.php')); + $disabledState = str($view) + ->after('@elseif ($server->isSentinelEnabled())') + ->before('@else') + ->toString(); + + expect($disabledState) + ->toContain('title="Metrics are disabled"') + ->not->toContain('status="Disabled"'); +});