🧪 Merge NOTSET and TRACE into the same test

Author: webknjaz

tests/integration/logging_test.py

@@ -107,28 +107,19 @@ def test_session_log_level_no_log(
     assert not caplog.records
 
 
+@pytest.mark.parametrize(
+    'log_level',
+    (ANSIBLE_PYLIBSSH_TRACE, logging.NOTSET),
+    ids=('TRACE', 'NOTSET'),
+)
 def test_session_log_level_trace(
     caplog: pytest.LogCaptureFixture,
     free_port_num: int,
+    log_level: int,
 ) -> None:
-    """Test setting the log level to TRACE should provide all of the logs."""
+    """Test setting the most detailed log level provides all the logs."""
     ssh_session = Session()
-    ssh_session.set_log_level(ANSIBLE_PYLIBSSH_TRACE)
-
-    with _ctx.suppress(LibsshSessionException):
-        ssh_session.connect(host=LOCALHOST, port=free_port_num)
-
-    expected_poll_message = 'ssh_socket_pollcallback: Poll callback on socket'
-    assert expected_poll_message in caplog.text
-
-
-def test_session_log_level_not_set(
-    caplog: pytest.LogCaptureFixture,
-    free_port_num: int,
-) -> None:
-    """Test setting the log level to NOTSET = all messages should be logged too."""
-    ssh_session = Session()
-    ssh_session.set_log_level(logging.NOTSET)
+    ssh_session.set_log_level(log_level)
 
     with _ctx.suppress(LibsshSessionException):
         ssh_session.connect(host=LOCALHOST, port=free_port_num)