From cfb395ff1a9ef9d19d59ddece304ad2278a2e5b6 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Fri, 30 Jan 2026 16:36:27 +0000
Subject: [PATCH] fix(tests): update expected relay health state in unit test

This update aligns `tests/services/relay-health-service.test.mjs` with the actual state object structure used in `RelayHealthService`, which now includes `consecutiveFailures` and `lastUserLogAt` properties. This fixes a CI failure in the `agent-review` job.

The previous fix for `tests/app-batch-fetch-profiles.test.mjs` (duplicate variable) is also included in this branch history.

Co-authored-by: PR0M3TH3AN <53631862+PR0M3TH3AN@users.noreply.github.com>
---
 tests/services/relay-health-service.test.mjs | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tests/services/relay-health-service.test.mjs b/tests/services/relay-health-service.test.mjs
index 0d2ac973..1943423c 100644
--- a/tests/services/relay-health-service.test.mjs
+++ b/tests/services/relay-health-service.test.mjs
@@ -57,8 +57,10 @@ test("RelayHealthService: ensureRelayState creates default state", () => {
     connected: false,
     lastLatencyMs: null,
     errorCount: 0,
+    consecutiveFailures: 0,
     lastCheckedAt: null,
     lastErrorAt: null,
+    lastUserLogAt: null,
   });
 });