From 0b7610054b43f9d5799b4b10b7afe30695658151 Mon Sep 17 00:00:00 2001 From: openhands Date: Fri, 19 Dec 2025 16:22:03 +0000 Subject: [PATCH] fix(telemetry): Apply enterprise formatting fixes - Remove extra blank line in lifecycle.py import section - Remove unused MagicMock import in test_lifecycle.py - Reformat multi-line AsyncMock side_effect argument Co-authored-by: openhands --- enterprise/server/telemetry/lifecycle.py | 1 - enterprise/tests/unit/telemetry/test_lifecycle.py | 7 ++++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/enterprise/server/telemetry/lifecycle.py b/enterprise/server/telemetry/lifecycle.py index 359459d63d..cbd526986a 100644 --- a/enterprise/server/telemetry/lifecycle.py +++ b/enterprise/server/telemetry/lifecycle.py @@ -4,7 +4,6 @@ from contextlib import asynccontextmanager from typing import AsyncIterator from fastapi import FastAPI - from server.logger import logger from server.telemetry.service import telemetry_service diff --git a/enterprise/tests/unit/telemetry/test_lifecycle.py b/enterprise/tests/unit/telemetry/test_lifecycle.py index 09d6ab50a2..33d521eb04 100644 --- a/enterprise/tests/unit/telemetry/test_lifecycle.py +++ b/enterprise/tests/unit/telemetry/test_lifecycle.py @@ -1,10 +1,9 @@ """Unit tests for the telemetry lifespan integration.""" -from unittest.mock import AsyncMock, MagicMock, patch +from unittest.mock import AsyncMock, patch import pytest from fastapi import FastAPI - from server.telemetry.lifecycle import telemetry_lifespan @@ -59,7 +58,9 @@ class TestTelemetryLifespan: 'server.telemetry.lifecycle.telemetry_service' ) as mock_telemetry_service: mock_telemetry_service.start = AsyncMock() - mock_telemetry_service.stop = AsyncMock(side_effect=Exception('Stop failed')) + mock_telemetry_service.stop = AsyncMock( + side_effect=Exception('Stop failed') + ) # Should not raise exception async with telemetry_lifespan(mock_app):