kivy / kivy / 30009365486

60%

Build Type

push

github

Committed by web-flow

Commit Message

Accept pathlib.Path in path-accepting APIs (#9350)

Kivy's path-accepting APIs historically required a str. This lets them also
accept any os.PathLike (e.g. pathlib.Path) by coercing to str via os.fspath()
at a small set of central choke points, keeping all internals (string parsing,
cache keys, Cython .encode()) string-based.

- utils: add path_to_str() helper (coerces os.PathLike -> str; leaves str,
  bytes, None, file-like and URIs unchanged).
- properties: StringProperty.convert() now coerces os.PathLike -> str, so every
  path-bearing StringProperty (Image.source, font_name, App.kv_file, the
  background_*/atlas image sources, Window.icon, ...) accepts Path. Non-path,
  non-str values still raise ValueError.
- resource_find / resource_add_path / resource_remove_path: coerce up front so
  cache keys and path-list membership stay consistent.
- core loaders: Image/ImageLoader (load/save/filename), SoundLoader.load,
  VideoBase.filename/generate_thumbnail, LabelBase.resolve_font_name/register,
  SvgLoader.load, Lottie filename.
- lang/atlas/storage: Builder.load_file/unload_file/load_string, Atlas,
  JsonStore, DictStore.
- app/config: App.load_kv/get_application_config; Config.read now accepts
  os.PathLike while still rejecting multiple filenames.
- tests: new kivy/tests/test_pathlib_support.py, parametrized over str vs Path.

Coercion only fires for isinstance(x, os.PathLike), so str inputs (including
atlas://, http://, data: URIs) and BytesIO pass through unchanged. On Windows a
Path canonicalizes to native separators; coercion is consistent per argument.

Coverage Stats

3271 of 6511 branches covered (50.24%)

180 of 195 new or added lines in 15 files covered. (92.31%)

6 existing lines in 3 files now uncovered.

29418 of 49039 relevant lines covered (59.99%)

1.18 hits per line