fix: Use debug level for disconnect callback as it happens often when

battery is low
sopelj · Oct 26, 2023 · f796c83 · f796c83
1 parent 938c042
commit f796c83
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,10 @@
 
 ## [0.8.1]
 
+### Changed
+
+* Use debug level for disconnect callback instead of warning
+
 ### Fixed
 
 * Don't require adapter keyword for other backends

diff --git a/ember_mug/mug.py b/ember_mug/mug.py
@@ -168,7 +168,7 @@ def _disconnect_callback(self, client: BleakClient) -> None:
         if self._expected_disconnect:
             logger.debug("Disconnect callback called")
         else:
-            logger.warning("Unexpectedly disconnected")
+            logger.debug("Unexpectedly disconnected")
 
     def _fire_callbacks(self) -> None:
         """Fire the callbacks."""

diff --git a/tests/test_connection.py b/tests/test_connection.py
@@ -143,7 +143,7 @@ def test_disconnect_callback(mock_logger: Mock, ember_mug: AsyncMock) -> None:
 
     ember_mug._expected_disconnect = False
     ember_mug._disconnect_callback(AsyncMock())
-    mock_logger.warning.assert_called_with("Unexpectedly disconnected")
+    mock_logger.debug.assert_called_with("Unexpectedly disconnected")
 
 
 @patch("ember_mug.mug.logger")