test: remove manual sys.settrace(None) cleanup calls from debugger tests

- Remove sys.settrace(None) cleanup from test_context_enter_sets_trace
- Wrap multiply() call in mock_settrace context in test_decorator_enabled_true_executes_function
- Remove sys.settrace(None) cleanup from test_init_with_debug_true_sets_trace

Author: KJonline

tests/unit/test_debugger.py

@@ -33,7 +33,6 @@ def test_sets_sys_trace(self):
         with patch.object(sys, "settrace") as mock_settrace:
             result = ctx.__enter__()
             mock_settrace.assert_called_once_with(ctx.trace_calls)
-        sys.settrace(None)
         assert result is ctx
 
     def test_returns_self(self):
@@ -170,8 +169,8 @@ def test_decorator_enabled_true_executes_function(self):
         def multiply(x, y):
             return x * y
 
-        result = multiply(3, 4)
-        sys.settrace(None)
+        with patch.object(sys, "settrace"):
+            result = multiply(3, 4)
         assert result == 12
 
     def test_decorator_enabled_false_executes_function(self):

tests/unit/test_hive_module.py

@@ -236,7 +236,6 @@ async def test_init_with_debug_list_sets_trace(self):
                     mock_settrace.assert_called_with(trace_debug)
         finally:
             hive_module.debug = original_debug
-            sys.settrace(None)
 
     async def test_init_with_empty_debug_does_not_set_trace(self):
         import apyhiveapi.hive as hive_module