fix(ha): drop, not raise, a not-an-outage error at runtime

Re-raising HA_NOT_OUTAGE_ERRORS from every call put back main's behaviour
of one bad runtime call (a NaN attribute in set_state, a proxy redirect
loop) unwinding the whole update cycle. Only initialize() needs to fail
fast, so re-raise only for its untracked calls; at runtime log the error
and return None without touching the outage timer.

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

Author: chalfontchubby

apps/predbat/ha.py

@@ -1311,9 +1311,14 @@ def api_call(self, endpoint, data_in=None, post=False, core=True, silent=False,
             if track:
                 self.ha_unavailable("connection error from {}".format(endpoint))
             data = None
-        except HA_NOT_OUTAGE_ERRORS:
-            # Not an outage and waiting will not fix it, so let it fail loudly as before
-            raise
+        except HA_NOT_OUTAGE_ERRORS as e:
+            # Not an outage and waiting will not fix it: initialize() lets it fail at once naming the real problem,
+            # while at runtime one bad call is logged and dropped rather than aborting the whole update cycle
+            if not track_outage:
+                raise
+            if not silent:
+                self.log("Warn: Request error from {}: {}".format(url, e))
+            data = None
         except requests.exceptions.RequestException as e:
             # Anything else requests raises while Home Assistant is going down, e.g. a connection reset part way through the body
             if not silent:

apps/predbat/tests/test_hainterface_api.py

@@ -691,7 +691,7 @@ def test_hainterface_api_call_auth_rejected_repeatedly_is_fatal(my_predbat=None)
 
 
 def test_hainterface_api_call_config_error_not_outage(my_predbat=None):
-    """Test api_call() lets errors that are not Home Assistant being down (bad ha_url/ha_key, redirect loop, unencodable body) escape rather than recording an outage"""
+    """Test errors that are not Home Assistant being down (bad ha_url/ha_key, redirect loop, unencodable body) are never an outage: dropped at runtime, raised for initialize()"""
     print("\n=== Testing HAInterface api_call() errors that are not an outage ===")
     failed = 0
 
@@ -701,21 +701,30 @@ def test_hainterface_api_call_config_error_not_outage(my_predbat=None):
         ("post", requests.exceptions.InvalidJSONError("Out of range float values are not JSON compliant")),
     ]
     for method, error in cases:
-        mock_base = MockBase()
-        ha_interface = create_ha_interface(mock_base, ha_key="test_key", db_enable=False, db_mirror_ha=False, db_primary=False)
-        with patch("ha.requests." + method) as mock_call:
-            mock_call.side_effect = error
-            raised = False
-            try:
-                ha_interface.api_call("/api/states/sensor.test", data_in={"state": 1} if method == "post" else None, post=method == "post")
-            except type(error):
-                raised = True
-
-        if not raised or ha_interface.outage.since is not None:
-            print(f"ERROR: expected {type(error).__name__} to escape without an outage, got raised={raised} / {ha_interface.outage.since}")
-            failed += 1
-        else:
-            print(f"✓ {type(error).__name__} escaped and was not an outage")
+        name = type(error).__name__
+        for track_outage in (True, False):
+            mock_base = MockBase()
+            ha_interface = create_ha_interface(mock_base, ha_key="test_key", db_enable=False, db_mirror_ha=False, db_primary=False)
+            with patch("ha.requests." + method) as mock_call:
+                mock_call.side_effect = error
+                raised = False
+                result = "not set"
+                try:
+                    result = ha_interface.api_call("/api/states/sensor.test", data_in={"state": 1} if method == "post" else None, post=method == "post", track_outage=track_outage)
+                except type(error):
+                    raised = True
+
+            if ha_interface.outage.since is not None:
+                print(f"ERROR: {name} should not be recorded as an outage")
+                failed += 1
+            elif track_outage and (raised or result is not None):
+                print(f"ERROR: at runtime {name} should be logged and dropped, got raised={raised} result={result}")
+                failed += 1
+            elif not track_outage and not raised:
+                print(f"ERROR: an untracked call (initialize) should let {name} escape")
+                failed += 1
+            else:
+                print(f"✓ {name} with track_outage={track_outage} handled and not an outage")
 
     return failed