From 314f4da1804078911b635f60e68430a3aa1ad5ea Mon Sep 17 00:00:00 2001 From: Walter Boring Date: Fri, 27 Mar 2026 10:18:07 -0400 Subject: [PATCH] fix: Restore max_delta after custom stale_timeout test The singleton's max_delta was being modified by test_init_custom_stale_timeout and not restored, causing test_is_stale_connection_false to fail because it expected 2 minutes but got 60 seconds. --- tests/client/drivers/test_aprsis_driver.py | 20 ++++++++++++++------ 1 file changed, 14 insertions(+), 6 deletions(-) diff --git a/tests/client/drivers/test_aprsis_driver.py b/tests/client/drivers/test_aprsis_driver.py index 63eec8f..bf5b420 100644 --- a/tests/client/drivers/test_aprsis_driver.py +++ b/tests/client/drivers/test_aprsis_driver.py @@ -80,14 +80,22 @@ class TestAPRSISDriver(unittest.TestCase): def test_init_custom_stale_timeout(self): """Test initialization with custom stale_timeout.""" - # Set a custom stale_timeout - self.mock_conf.aprs_network.stale_timeout = 60 # 1 minute + # Save original max_delta + original_max_delta = self.driver.max_delta - # Re-initialize the existing driver with new config - # (singleton pattern means we can't create a new instance) - self.driver.__init__() + try: + # Set a custom stale_timeout + self.mock_conf.aprs_network.stale_timeout = 60 # 1 minute - self.assertEqual(self.driver.max_delta, datetime.timedelta(seconds=60)) + # Re-initialize the existing driver with new config + # (singleton pattern means we can't create a new instance) + self.driver.__init__() + + self.assertEqual(self.driver.max_delta, datetime.timedelta(seconds=60)) + finally: + # Restore original max_delta so other tests aren't affected + self.driver.max_delta = original_max_delta + self.mock_conf.aprs_network.stale_timeout = 120 def test_is_enabled_true(self): """Test is_enabled returns True when APRS-IS is enabled."""