Skip to content

test: cover utility helper edge cases#10926

Open
Yijian6 wants to merge 1 commit into
python-poetry:mainfrom
Yijian6:test-utils-helpers-coverage
Open

test: cover utility helper edge cases#10926
Yijian6 wants to merge 1 commit into
python-poetry:mainfrom
Yijian6:test-utils-helpers-coverage

Conversation

@Yijian6
Copy link
Copy Markdown
Contributor

@Yijian6 Yijian6 commented May 26, 2026

Summary

  • cover downloads when a response includes an invalid Content-Length header
  • cover directory() restoring the original working directory after an error
  • cover nested merge_dicts() behavior used by configuration merging

Context

Refs #3155.

This is a test-only follow-up for helper utilities with behavior that is used by downloads, command working directory handling, and config merging. The added tests focus on existing edge-case behavior without changing production code.

Tests

  • PYTHONPATH=src python -m pytest tests/utils/test_helpers.py -q
  • python -m ruff check tests/utils/test_helpers.py
  • PYTHONPATH=src python -m mypy tests/utils/test_helpers.py

Copy link
Copy Markdown

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@Yijian6 Yijian6 force-pushed the test-utils-helpers-coverage branch from ff922a1 to 14597ac Compare May 26, 2026 13:05
@Yijian6 Yijian6 force-pushed the test-utils-helpers-coverage branch from 14597ac to 7bdb4a5 Compare May 26, 2026 13:15
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.

1 participant