diff options
| author | Christian Walther <cwalther@gmx.ch> | 2023-07-31 18:25:26 +0200 |
|---|---|---|
| committer | Christian Walther <cwalther@gmx.ch> | 2023-10-19 16:21:08 +0200 |
| commit | e3ba6f952bda153cfc3389eb48c7645d71b2b094 (patch) | |
| tree | 8dbbc2d610e3d7c2614388813c67420b2ae80698 /tests | |
| parent | 86c7b957a85d69ffb19bc6e4db719e902abd3fc1 (diff) | |
extmod/vfs_posix: Fix relative root path.
A VfsPosix created with a relative root path would get confused when
chdir() was called on it and become unable to properly resolve absolute
paths, because changing directories effectively shifted its root. The
simplest fix for that would be to say "don't do that", but since the
unit tests themselves do it, fix it by making a relative path absolute
before storing it.
Signed-off-by: Christian Walther <cwalther@gmx.ch>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/extmod/vfs_posix.py | 13 | ||||
| -rw-r--r-- | tests/extmod/vfs_posix.py.exp | 1 |
2 files changed, 14 insertions, 0 deletions
diff --git a/tests/extmod/vfs_posix.py b/tests/extmod/vfs_posix.py index 05dc0f537..ebbd08a4a 100644 --- a/tests/extmod/vfs_posix.py +++ b/tests/extmod/vfs_posix.py @@ -99,6 +99,19 @@ if hasattr(vfs, "statvfs"): print(type(list(vfs.ilistdir("."))[0][0])) print(type(list(vfs.ilistdir(b"."))[0][0])) +# chdir should not affect absolute paths (regression test) +vfs.mkdir("/subdir") +vfs.mkdir("/subdir/micropy_test_dir") +with vfs.open("/subdir/micropy_test_dir/test2", "w") as f: + f.write("wrong") +vfs.chdir("/subdir") +with vfs.open("/test2", "r") as f: + print(f.read()) +os.chdir(curdir) +vfs.remove("/subdir/micropy_test_dir/test2") +vfs.rmdir("/subdir/micropy_test_dir") +vfs.rmdir("/subdir") + # remove os.remove(temp_dir + "/test2") print(os.listdir(temp_dir)) diff --git a/tests/extmod/vfs_posix.py.exp b/tests/extmod/vfs_posix.py.exp index 99922e621..bd1ec7bad 100644 --- a/tests/extmod/vfs_posix.py.exp +++ b/tests/extmod/vfs_posix.py.exp @@ -10,6 +10,7 @@ next_file_no <= base_file_no True <class 'tuple'> <class 'str'> <class 'bytes'> +hello [] remove OSError False |