From af0feaf9c81b8b9f443e58fbe7d23f1ff7ba63d2 Mon Sep 17 00:00:00 2001 From: Walter Boring Date: Tue, 7 Oct 2025 14:18:50 -0400 Subject: [PATCH] Fixed some unit tests Fixed unit tests related to the updated static method signatures of the client and drivers. --- aprsd/client/drivers/tcpkiss.py | 2 +- tests/client/drivers/test_aprsis_driver.py | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/aprsd/client/drivers/tcpkiss.py b/aprsd/client/drivers/tcpkiss.py index 3f28f76..79ada2f 100644 --- a/aprsd/client/drivers/tcpkiss.py +++ b/aprsd/client/drivers/tcpkiss.py @@ -80,7 +80,7 @@ class TCPKISSDriver: return client.TRANSPORT_TCPKISS @staticmethod - def is_enabled(cls) -> bool: + def is_enabled() -> bool: """Check if KISS is enabled in configuration. Returns: diff --git a/tests/client/drivers/test_aprsis_driver.py b/tests/client/drivers/test_aprsis_driver.py index 4ab1d6c..ab52eeb 100644 --- a/tests/client/drivers/test_aprsis_driver.py +++ b/tests/client/drivers/test_aprsis_driver.py @@ -353,6 +353,7 @@ class TestAPRSISDriver(unittest.TestCase): def test_consumer_success(self, mock_log): """Test consumer forwards callback to client.""" self.driver._client = self.mock_client + self.driver.connected = True mock_callback = mock.MagicMock() self.driver.consumer(mock_callback, raw=True) @@ -365,6 +366,7 @@ class TestAPRSISDriver(unittest.TestCase): def test_consumer_exception(self, mock_log): """Test consumer handles exceptions.""" self.driver._client = self.mock_client + self.driver.connected = True mock_callback = mock.MagicMock() test_error = Exception('Test error') self.mock_client.consumer.side_effect = test_error @@ -381,8 +383,6 @@ class TestAPRSISDriver(unittest.TestCase): mock_callback = mock.MagicMock() self.driver.consumer(mock_callback) - - mock_log.warning.assert_called_once() self.assertFalse(self.driver.connected) def test_stats_configured_with_client(self):