From: André Almeida Date: Wed, 24 Mar 2021 22:48:17 +0000 (-0300) Subject: generic/556: Fix rename test X-Git-Tag: v2022.05.01~490 X-Git-Url: http://git.apps.os.sepia.ceph.com/?p=xfstests-dev.git;a=commitdiff_plain;h=1fc1737959c95888d2085a9e97872c6734412a2c generic/556: Fix rename test Casefold rename test should check if renaming a file to an equivalent name fails as expect (e.g. renaming from "file.txt" to "FILE.TXT") and `mv` correctly identifies that those names refers to the same file. Currently, the test doesn't do what is expected given that it doesn't have the file to be renamed, and `mv` returns "No such file or directory". Fix that by creating test files and checking the correct output. Fixes: 12b7dddbc265fcb ("generic: Add tests for filename casefolding feature") Signed-off-by: André Almeida Reviewed-by: Gabriel Krisman Bertazi Signed-off-by: Eryu Guan --- diff --git a/tests/generic/556 b/tests/generic/556 index 3efd0510..3145188c 100755 --- a/tests/generic/556 +++ b/tests/generic/556 @@ -341,8 +341,10 @@ test_file_rename() mkdir -p ${basedir} _casefold_set_attr ${basedir} + touch ${basedir}/rename + # Move to an equivalent name should not work - mv ${basedir}/rename ${basedir}/rename 2>&1 | \ + mv ${basedir}/rename ${basedir}/RENAME 2>&1 | \ _filter_scratch _casefold_check_exact_name ${basedir} "rename" || \ diff --git a/tests/generic/556.out b/tests/generic/556.out index 47656065..f9dd9542 100644 --- a/tests/generic/556.out +++ b/tests/generic/556.out @@ -5,7 +5,7 @@ SCRATCH_MNT/casefold_flag_removal Casefold SCRATCH_MNT/casefold_flag_removal Casefold SCRATCH_MNT/flag_inheritance/d1/d2/d3 Casefold SCRATCH_MNT/symlink/ind1/TARGET -mv: cannot stat 'SCRATCH_MNT/rename/rename': No such file or directory +mv: 'SCRATCH_MNT/rename/rename' and 'SCRATCH_MNT/rename/RENAME' are the same file # file: SCRATCH_MNT/xattrs/x user.foo="bar"