author | Junio C Hamano <gitster@pobox.com> | |
Wed, 14 Nov 2007 22:25:46 +0000 (14:25 -0800) | ||
committer | Junio C Hamano <gitster@pobox.com> | |
Wed, 14 Nov 2007 22:25:46 +0000 (14:25 -0800) |
* bs/maint-commit-options:
git-commit: Add tests for invalid usage of -a/--interactive with paths
git-commit.sh: Fix usage checks regarding paths given when they do not make sense
git-commit: Add tests for invalid usage of -a/--interactive with paths
git-commit.sh: Fix usage checks regarding paths given when they do not make sense
git-commit.sh | patch | blob | history | |
t/t7501-commit.sh | patch | blob | history |
diff --git a/git-commit.sh b/git-commit.sh
index 6490045d3024034a947495f0c2e1fb94e41dfae4..959c4d68e7bbb501dee1778a33d45eaf09b4bf2d 100755 (executable)
--- a/git-commit.sh
+++ b/git-commit.sh
case "$all,$interactive,$also,$#" in
*t,*t,*)
die "Cannot use -a, --interactive or -i at the same time." ;;
-t,,[1-9]*)
+t,,,[1-9]*)
die "Paths with -a does not make sense." ;;
-,t,[1-9]*)
+,t,,[1-9]*)
die "Paths with --interactive does not make sense." ;;
,,t,0)
die "No paths with -i does not make sense." ;;
diff --git a/t/t7501-commit.sh b/t/t7501-commit.sh
index 4dc35bdf558750ef41f229fbbb5dbef0a05401c5..9dba104b1f8bf9125f1a947a2fd115b684a94095 100644 (file)
--- a/t/t7501-commit.sh
+++ b/t/t7501-commit.sh
"invalid options 2" \
"git-commit -C HEAD -m illegal"
+test_expect_failure \
+ "using paths with -a" \
+ "echo King of the bongo >file &&
+ git-commit -m foo -a file"
+
+test_expect_failure \
+ "using paths with --interactive" \
+ "echo bong-o-bong >file &&
+ echo 7 | git-commit -m foo --interactive file"
+
test_expect_failure \
"using invalid commit with -C" \
"git-commit -C bogus"