Get rid of the TRACE log level and just use DEBUG

Author: puddly

tests/test_utils.py

@@ -1,11 +1,3 @@
-import logging
-
-from .utils import TraceLogger
-
-if not hasattr(logging.getLoggerClass(), "trace"):
-    logging.setLoggerClass(TraceLogger)
-
-
 MAJOR_VERSION = 0
 MINOR_VERSION = 0
 PATCH_VERSION = 9

zigpy_znp/__init__.py

@@ -206,17 +206,17 @@ def close(self) -> None:
         self._cancel_all_listeners()
 
     def _remove_listener(self, listener: BaseResponseListener) -> None:
-        LOGGER.trace("Removing listener %s", listener)
+        LOGGER.debug("Removing listener %s", listener)
 
         for header in listener.matching_headers():
             self._response_listeners[header].remove(listener)
 
             if not self._response_listeners[header]:
-                LOGGER.trace("Cleaning up empty listener list for header %s", header)
+                LOGGER.debug("Cleaning up empty listener list for header %s", header)
                 del self._response_listeners[header]
 
         total_listeners = sum(map(len, self._response_listeners.values()))
-        LOGGER.trace("There are %d listeners remaining", total_listeners)
+        LOGGER.debug("There are %d listeners remaining", total_listeners)
 
     def frame_received(self, frame: GeneralFrame) -> None:
         """
@@ -233,19 +233,19 @@ def frame_received(self, frame: GeneralFrame) -> None:
 
         for listener in self._response_listeners[command.header]:
             if not listener.resolve(command):
-                LOGGER.trace("%s does not match %s", command, listener)
+                LOGGER.debug("%s does not match %s", command, listener)
                 continue
 
             matched = True
-            LOGGER.trace("%s matches %s", command, listener)
+            LOGGER.debug("%s matches %s", command, listener)
 
         if not matched:
             LOGGER.warning("Received an unhandled command: %s", command)
 
     def callback_for_responses(self, responses, callback) -> None:
         listener = CallbackResponseListener(responses, callback=callback)
 
-        LOGGER.trace("Creating callback %s", listener)
+        LOGGER.debug("Creating callback %s", listener)
 
         for header in listener.matching_headers():
             self._response_listeners[header].append(listener)
@@ -256,7 +256,7 @@ def callback_for_response(self, response, callback) -> None:
     def wait_for_responses(self, responses) -> asyncio.Future:
         listener = OneShotResponseListener(responses)
 
-        LOGGER.trace("Creating one-shot listener %s", listener)
+        LOGGER.debug("Creating one-shot listener %s", listener)
 
         for header in listener.matching_headers():
             self._response_listeners[header].append(listener)
@@ -296,7 +296,7 @@ async def request(self, request, **response_params):
 
         # If our request has no response, we cannot wait for one
         if not request.Rsp:
-            LOGGER.trace("Request has no response, not waiting for one.")
+            LOGGER.debug("Request has no response, not waiting for one.")
             self._uart.send(request.to_frame())
             return
 

zigpy_znp/api.py

@@ -63,10 +63,10 @@ def data_received(self, data: bytes) -> None:
         """Callback when data is received."""
         self._buffer += data
 
-        LOGGER.trace("Received data: %s", Bytes.__repr__(data))
+        LOGGER.debug("Received data: %s", Bytes.__repr__(data))
 
         for frame in self._extract_frames():
-            LOGGER.trace("Parsed frame: %s", frame)
+            LOGGER.debug("Parsed frame: %s", frame)
 
             try:
                 self._api.frame_received(frame.payload)
@@ -80,7 +80,7 @@ def send(self, payload: frames.GeneralFrame) -> None:
         self._transport_write(frames.TransportFrame(payload).serialize())
 
     def _transport_write(self, data: bytes) -> None:
-        LOGGER.trace("Sending data: %s", Bytes.__repr__(data))
+        LOGGER.debug("Sending data: %s", Bytes.__repr__(data))
         self.transport.write(data)
 
     def _extract_frames(self) -> typing.Iterator[frames.TransportFrame]:

zigpy_znp/uart.py

@@ -159,7 +159,7 @@ def _receive_zdo_message(
         zdo_args = [t(zdo_kwargs[n]) for n, t in zip(field_names, field_types)]
         message = t.serialize_list([t.uint8_t(tsn)] + zdo_args)
 
-        LOGGER.trace("Pretending we received a ZDO message: %s", message)
+        LOGGER.debug("Pretending we received a ZDO message: %s", message)
         self.handle_message(
             sender=sender,
             profile=zigpy.profiles.zha.PROFILE_ID,
@@ -575,7 +575,7 @@ async def _send_zdo_request(
         This method translates the ZDO_* MT response into one zigpy can handle.
         """
 
-        LOGGER.trace(
+        LOGGER.debug(
             "Intercepted a ZDO request: dst_addr=%s, dst_ep=%s, src_ep=%s, "
             "cluster=%s, sequence=%s, options=%s, radius=%s, data=%s",
             dst_addr,