Skip to content

Commit 7ee981e

Browse files
devin-ai-integration[bot]bot_apk
andcommitted
fix: regenerate poetry.lock and address review bot comments
- Regenerate poetry.lock after adding datadog optional dependency - Remove unused imports (CGROUP_V1_MEMORY_LIMIT, etc.) from test_memory.py - Fix duplicate import style in test_metrics_client.py Co-Authored-By: bot_apk <apk@cognition.ai>
1 parent 3c98bd4 commit 7ee981e

3 files changed

Lines changed: 19 additions & 7 deletions

File tree

poetry.lock

Lines changed: 18 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

unit_tests/metrics/test_memory.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,10 +10,6 @@
1010
import pytest
1111

1212
from airbyte_cdk.metrics.memory import (
13-
CGROUP_V1_MEMORY_LIMIT,
14-
CGROUP_V1_MEMORY_USAGE,
15-
CGROUP_V2_MEMORY_CURRENT,
16-
CGROUP_V2_MEMORY_MAX,
1713
MemoryInfo,
1814
_read_cgroup_v1_memory,
1915
_read_cgroup_v2_memory,

unit_tests/metrics/test_metrics_client.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
sys.modules["datadog"] = _mock_datadog
2626
sys.modules["datadog.dogstatsd"] = _mock_dogstatsd_mod
2727

28+
import airbyte_cdk.metrics as metrics_module # noqa: E402
2829
from airbyte_cdk.metrics import MetricsClient, get_metrics_client # noqa: E402
2930

3031

@@ -204,8 +205,6 @@ def test_noop_when_disabled(self) -> None:
204205

205206
class TestGetMetricsClient:
206207
def test_returns_singleton(self) -> None:
207-
import airbyte_cdk.metrics as metrics_module
208-
209208
# Reset the singleton for test isolation
210209
metrics_module._metrics_client = None
211210
try:

0 commit comments

Comments
 (0)