diff --git a/.circleci/scripts/stop_integration_processes.py b/.circleci/scripts/stop_integration_processes.py index e5f02e3ee3a..8f11aaaddbd 100644 --- a/.circleci/scripts/stop_integration_processes.py +++ b/.circleci/scripts/stop_integration_processes.py @@ -12,8 +12,6 @@ def is_owned(process: psutil.Process, identity: str, owner_uid: int) -> bool: def owned_processes(identity: str, owner_uid: int) -> tuple[psutil.Process, ...]: - # The runner invokes this helper with sudo. Unreadable ownership remains a hard - # failure: skipping it would make a successful cleanup unverifiable. return tuple(process for process in psutil.process_iter() if is_owned(process, identity, owner_uid)) diff --git a/tests/integration/database/test_transaction_atomicity.py b/tests/integration/database/test_transaction_atomicity.py index 37b5d2289b7..431d7982482 100644 --- a/tests/integration/database/test_transaction_atomicity.py +++ b/tests/integration/database/test_transaction_atomicity.py @@ -43,8 +43,6 @@ def test_access_group_second_key_constraint_failure_rolls_back_all_writes(gatewa assert connection.execute(sql.SQL("SELECT is_called FROM {}").format(sql.Identifier(witness))).fetchone() == (False,) failed: Final = gateway.request("POST", "/v1/access_group", body) assert failed.status_code == 500, failed.text - # Sequence advancement survives rollback and proves the rejecting - # constraint actually evaluated the second key's nonempty grant. assert connection.execute(sql.SQL("SELECT is_called FROM {}").format(sql.Identifier(witness))).fetchone() == (True,) assert read_rows('SELECT access_group_id FROM "LiteLLM_AccessGroupTable" WHERE access_group_name=%s', (name,)) == [] assert read_rows('SELECT token, access_group_ids FROM "LiteLLM_VerificationToken" WHERE token=ANY(%s) ORDER BY token', (tokens,)) == before diff --git a/tests/integration/spend/test_cache_and_quota.py b/tests/integration/spend/test_cache_and_quota.py index 2e66a538161..05bf100f091 100644 --- a/tests/integration/spend/test_cache_and_quota.py +++ b/tests/integration/spend/test_cache_and_quota.py @@ -57,8 +57,6 @@ def test_generated_cache_sequences_preserve_content_usage_and_zero_hit_cost(gate observed: Final = self.upstream.get("/__observations").json()["requests"] expected_calls: Final = 0 if marker in self.seen else 1 assert len(observed) == expected_calls, observed - # The response can retain its original cost header on a cache hit. - # Per-request billed cost is checked against fresh spend rows below. if marker not in self.seen: assert float(response.headers["x-litellm-response-cost"]) == pytest.approx(0.06) if marker in self.identities: