Skip to content

Commit 94b664f

Browse files
devin-ai-integration[bot]bot_apk
andcommitted
docs: fix stale test docstrings to reference 90% RssAnon threshold
Co-Authored-By: bot_apk <apk@cognition.ai>
1 parent 10fc815 commit 94b664f

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

unit_tests/utils/test_memory_monitor.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -323,7 +323,7 @@ def mock_read_text(self: Path) -> str:
323323

324324

325325
def test_raises_when_both_cgroup_and_anon_rss_above_thresholds() -> None:
326-
"""Fail-fast raises AirbyteTracedException when both cgroup >= 98% and RssAnon >= 80%."""
326+
"""Fail-fast raises AirbyteTracedException when both cgroup >= 98% and RssAnon >= 90%."""
327327
monitor = MemoryMonitor(check_interval=1, fail_fast=True)
328328
with (
329329
patch.object(Path, "exists", _v2_exists),
@@ -338,7 +338,7 @@ def test_raises_when_both_cgroup_and_anon_rss_above_thresholds() -> None:
338338

339339

340340
def test_no_raise_when_cgroup_high_but_anon_rss_low(caplog: pytest.LogCaptureFixture) -> None:
341-
"""No exception when cgroup >= 98% but RssAnon < 80% (file-backed pages scenario).
341+
"""No exception when cgroup >= 98% but RssAnon < 90% (file-backed pages scenario).
342342
343343
This test also proves the metric choice matters: VmRSS is 90% (high) but
344344
RssAnon is only 50% (low), so the pressure is from file-backed pages.

0 commit comments

Comments
 (0)