[PATCH/RFD 4/3] t7300: Tests for git-clean using filenames with spaces/punctuation
From: Michael Spang <hidden>
Date: 2016-06-15 22:43:08
Subsystem:
the rest · Maintainer:
Linus Torvalds
Possibly related (same subject, not in this thread)
- 2016-06-15 · Re: [PATCH/RFD 4/3] t7300: Tests for git-clean using filenames with spaces/punctuation · Junio C Hamano <hidden>
- 2016-06-15 · Re: [PATCH/RFD 4/3] t7300: Tests for git-clean using filenames with spaces/punctuation · Michael Spang <hidden>
Signed-off-by: Michael Spang <redacted> --- This isn't meant for applying, at least until this flaw is fixed. These tests are failing because ls-files does escaping and clean does not unescape. Does anyone know of a solution to this? sed could be used to remove the double quotes and replace escaped characters, but doing many replacements happens in multiple passes and hence does not always work as desired. Is this even properly solvable without making clean a builtin or writing git-unescape? t/t7300-clean.sh | 23 +++++++++++++++++++++++ 1 files changed, 23 insertions(+), 0 deletions(-)
diff --git a/t/t7300-clean.sh b/t/t7300-clean.sh
index 1fb3850..3792221 100755
--- a/t/t7300-clean.sh
+++ b/t/t7300-clean.sh@@ -7,6 +7,8 @@ test_description='git-clean basic tests' . ./test-lib.sh +TAB=" " + test_expect_success \ 'setup' \ "mkdir -p src &&
@@ -123,6 +125,27 @@ test_expect_success \ test ! -e obj.o && test ! -e build" +test_expect_success \ + 'filenames with spaces' \ + 'touch abc\ def && + touch 123 123\ && + git-add 123 && + git-clean && + test ! -e abc\ def && + test ! -e 123\ && + test -e 123' + +test_expect_success \ + 'filenames with escaped characters' \ + 'touch "'"$TAB"'" " " \\ \" \\\\\" \\t && + git-clean && + test ! -e "'"$TAB"'" && + test ! -e " " && + test ! -e \\ && + test ! -e \" && + test ! -e \\\\\" && + test ! -e \\t' + test_expect_failure \ 'clean.requireForce' \ "mkdir -p build docs &&