Skip to content

test: add logger unit tests#598

Open
MD-Mushfiqur123 wants to merge 1 commit into
Agent-Field:mainfrom
MD-Mushfiqur123:tests/logger-coverage
Open

test: add logger unit tests#598
MD-Mushfiqur123 wants to merge 1 commit into
Agent-Field:mainfrom
MD-Mushfiqur123:tests/logger-coverage

Conversation

@MD-Mushfiqur123
Copy link
Copy Markdown

Closes #400

Adds test_logger_globals.py with 21 tests covering:

  • Global get_logger/set_log_level/set_cp_client functions
  • Level gating (heartbeat/track/fire suppressed by default, shown when enabled)
  • Structured output through execution context
  • Internal methods: _normalize_level, _now_iso, _merge_attributes, _build_execution_record
  • warning alias
  • set_level runtime
cd sdk/python && python3 -m pytest tests/test_logger_globals.py -v
21 passed

PR generated by AI agent.

@MD-Mushfiqur123 MD-Mushfiqur123 requested review from a team and AbirAbbas as code owners May 28, 2026 07:16
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Developer seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Python SDK] Expand logger.py test coverage

2 participants