Allow relative and home paths

Author: reindervdw-cmi

nibabel/filename_parser.py

@@ -10,6 +10,7 @@
 from __future__ import annotations
 
 import os
+import pathlib
 import typing as ty
 
 if ty.TYPE_CHECKING:  # pragma: no cover
@@ -37,9 +38,7 @@ def _stringify_path(filepath_or_buffer: FileSpec) -> str:
     Adapted from:
     https://github.com/pandas-dev/pandas/blob/325dd68/pandas/io/common.py#L131-L160
     """
-    if isinstance(filepath_or_buffer, os.PathLike):
-        return filepath_or_buffer.__fspath__()
-    return filepath_or_buffer
+    return str(pathlib.Path(filepath_or_buffer).expanduser().resolve())
 
 
 def types_filenames(

nibabel/tests/test_filename_parser.py

@@ -7,10 +7,11 @@
 #
 ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ##
 """Tests for filename container"""
+import pathlib
 
 import pytest
 
-from ..filename_parser import TypesFilenamesError, parse_filename, splitext_addext, types_filenames
+from ..filename_parser import TypesFilenamesError, parse_filename, splitext_addext, types_filenames, _stringify_path
 
 
 def test_filenames():
@@ -123,3 +124,26 @@ def test_splitext_addext():
     assert res == ('..', '', '')
     res = splitext_addext('...')
     assert res == ('...', '', '')
+
+
+def test__stringify_path():
+    current_directory = pathlib.Path.cwd()
+    res = _stringify_path('')
+    assert res == str(current_directory)
+    res = _stringify_path('fname.ext.gz')
+    assert res == str(current_directory / 'fname.ext.gz')
+    res = _stringify_path(pathlib.Path('fname.ext.gz'))
+    assert res == str(current_directory / 'fname.ext.gz')
+
+    home = pathlib.Path.home()
+    res = _stringify_path(pathlib.Path('~/fname.ext.gz'))
+    assert res == str(home) + '/fname.ext.gz'
+
+    res = _stringify_path(pathlib.Path('./fname.ext.gz'))
+    assert res == str(current_directory / 'fname.ext.gz')
+    res = _stringify_path(pathlib.Path('../fname.ext.gz'))
+    assert res == str(current_directory.parent / 'fname.ext.gz')
+    
+
+
+