Skip to content

Commit

Permalink
Merge pull request #1160 from pipecat-ai/aleix/fix-unit-test-logging
Browse files Browse the repository at this point in the history
tests: remove logger from tests.utils
  • Loading branch information
aconchillo authored Feb 6, 2025
2 parents 3d4d613 + a96d929 commit 962fc27
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 6 deletions.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

### Fixed

- Fixed an issue that was causing `tests.utils` import to fail because of
logging setup.

- Fixed a `SentryMetrics` issue that was preventing any metrics to be sent to
Sentry and also was preventing from metrics frames to be pushed to the pipeline.

Expand Down
6 changes: 0 additions & 6 deletions src/pipecat/tests/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,8 @@
#

import asyncio
import sys
from typing import Any, Awaitable, Callable, Dict, Sequence, Tuple

from loguru import logger

from pipecat.frames.frames import (
EndFrame,
Frame,
Expand All @@ -22,9 +19,6 @@
from pipecat.pipeline.task import PipelineParams, PipelineTask
from pipecat.processors.frame_processor import FrameDirection, FrameProcessor

logger.remove(0)
logger.add(sys.stderr, level="TRACE")


class HeartbeatsObserver(BaseObserver):
def __init__(
Expand Down

0 comments on commit 962fc27

Please sign in to comment.