From c82e9ba5977424e6f03ddbbfcbd816634ef667ce Mon Sep 17 00:00:00 2001 From: Hemna Date: Thu, 21 Nov 2024 09:01:19 -0500 Subject: [PATCH] Fixed broken unit tests This patch fixes some broken unit tests with the aprsis client. --- aprsd/client/aprsis.py | 2 +- tests/client/test_aprsis.py | 13 ++++++++++--- 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/aprsd/client/aprsis.py b/aprsd/client/aprsis.py index 1cd544d..d9b79bd 100644 --- a/aprsd/client/aprsis.py +++ b/aprsd/client/aprsis.py @@ -37,8 +37,8 @@ class APRSISClient(base.APRSClient): stats = { "connected": self.is_connected, "filter": self.filter, - "keepalive": keepalive, "login_status": self.login_status, + "server_keepalive": keepalive, "server_string": server_string, "transport": self.transport(), } diff --git a/tests/client/test_aprsis.py b/tests/client/test_aprsis.py index e1c7bad..33a4fda 100644 --- a/tests/client/test_aprsis.py +++ b/tests/client/test_aprsis.py @@ -53,12 +53,19 @@ class TestAPRSISClient(unittest.TestCase): with mock.patch.object(APRSISClient, "is_configured", return_value=True): stats = self.client.stats() + from rich.console import Console + c = Console() + c.print(stats) self.assertEqual( { - "server_string": mock_client.server_string, - "sever_keepalive": mock_client.aprsd_keepalive, + "connected": True, "filter": "m/50", - }, stats, + "login_status": {"message": mock.ANY, "success": True}, + "server_keepalive": mock_client.aprsd_keepalive, + "server_string": mock_client.server_string, + "transport": "aprsis", + }, + stats, ) def test_is_configured_missing_login(self):