Skip to content

Add edge-case tests for safe_filename() normalization #2

Description

@xiaoleiy

Good first issue (tests only, no risk). Strengthen coverage for core.safe_filename, which makes
downloaded filenames cloud-/filesystem-safe.

Add cases to tests/test_core.py::test_safe_filename for:

  • very long mixed CJK+ASCII titles (result stays ≤ 120 chars and non-empty),
  • titles that reduce to empty after stripping (→ "untitled"),
  • more Windows reserved names: lowercase com1, LPT9, CON with a trailing space,
  • titles made up entirely of emoji/symbols.

Where: logic in src/podpull/core.py; tests in tests/test_core.py.
Acceptance: new assertions pass. If you find a real gap in safe_filename, note it here before changing it.

Metadata

Metadata

Assignees

Labels

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions