MAINT-10 review: warn when HA did not answer and TZ is UTC; test the 3 s budget

HA can take minutes to start (2026-10-05, 502 from the proxy). Then TZ holds,
which the Supervisor sets from HA's zone, so the hour stays right; only a TZ
of UTC there now logs a WARNING asking for a restart. A test pins the 3 s
budget on the zone request (a mutant without it survived).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

goodwe_controller/test_maint10.py

@@ -316,6 +316,7 @@ class _Session:
 
     def get(self, url, **kw):
         self.asked.append(url)
+        self.timeouts = getattr(self, "timeouts", []) + [kw.get("timeout")]
         if url.endswith("/core/api/config"):
             return _Resp(self.zone_body)
         return _Resp({"data": {}})
@@ -325,6 +326,9 @@ s = _Session({"time_zone": "Europe/Brussels"})
 check("ha_time_zone() reads /core/api/config's time_zone",
       asyncio.run(M.ha_time_zone(s, "tok")) == ("Europe/Brussels", "")
       and s.asked == [M.CORE_API + "/config"])
+# Review: HA can take minutes to start (2026-10-05); the ask must not wait on it.
+check("...with a time budget of at most 3 s, so a slow HA cannot hold up start",
+      s.timeouts == [M.DOCS_LINK_TIMEOUT_S] and 0 < M.DOCS_LINK_TIMEOUT_S <= 3)
 check("...and a failure is None plus the reason, never a raise",
       asyncio.run(M.ha_time_zone(_Session(OSError("down")), "tok"))
       == (None, "OSError: down")
@@ -408,6 +412,17 @@ try:
     check("amain(), neither: UTC, with a WARNING line",
           tz is UTC and any(lvl == "WARNING" and "uses UTC" in m
                             for lvl, m in logs))
+    # Review: HA still starting (502) and the Supervisor's TZ is UTC.
+    tz, lines, logs = amain_zone(OSError("502"), "UTC")
+    check("amain(), Home Assistant unavailable and TZ is UTC: a WARNING "
+          "that says to restart", tz is UTC
+          and [(lvl, m) for lvl, m in logs if "Maintenance start" in m]
+          == [("WARNING", "Maintenance start hour uses UTC, because Home "
+               "Assistant did not answer. Restart the add-on when it is up.")])
+    tz, lines, logs = amain_zone(OSError("502"), "America/New_York")
+    check("...but a real TZ is used without a warning",
+          not any(lvl == "WARNING" for lvl, m in logs
+                  if "Maintenance start" in m))
 finally:
     MT._load_zone = _real_load