refactor(cdk): make DEFAULT_CHECK_INTERVAL private and drop circular tests

Co-Authored-By: bot_apk <apk@cognition.ai>

Author: devin-ai-integration[bot]

airbyte_cdk/utils/memory_monitor.py

@@ -26,7 +26,7 @@
 _DEFAULT_CRITICAL_THRESHOLD = 0.95
 
 # Check interval (every N messages)
-DEFAULT_CHECK_INTERVAL = 1000
+_DEFAULT_CHECK_INTERVAL = 1000
 
 
 class MemoryLimitExceeded(AirbyteTracedException):
@@ -47,7 +47,7 @@ def __init__(
         self,
         warning_threshold: float = _DEFAULT_WARNING_THRESHOLD,
         critical_threshold: float = _DEFAULT_CRITICAL_THRESHOLD,
-        check_interval: int = DEFAULT_CHECK_INTERVAL,
+        check_interval: int = _DEFAULT_CHECK_INTERVAL,
     ) -> None:
         self._warning_threshold = warning_threshold
         self._critical_threshold = critical_threshold

unit_tests/utils/test_memory_monitor.py

@@ -13,7 +13,6 @@
     _CGROUP_V1_USAGE,
     _CGROUP_V2_CURRENT,
     _CGROUP_V2_MAX,
-    DEFAULT_CHECK_INTERVAL,
     MemoryLimitExceeded,
     MemoryMonitor,
 )
@@ -389,13 +388,3 @@ def test_default_attributes(self) -> None:
         assert exc.failure_type == FailureType.system_error
         assert exc.message == "Source exceeded memory limit."
         assert exc.internal_message == "Memory at 96%"
-
-
-class TestDefaultCheckInterval:
-    """Tests for the DEFAULT_CHECK_INTERVAL constant."""
-
-    def test_check_interval_is_positive(self) -> None:
-        assert DEFAULT_CHECK_INTERVAL > 0
-
-    def test_check_interval_value(self) -> None:
-        assert DEFAULT_CHECK_INTERVAL == 1000