Re: [PATCHv4 1/1] git-p4: add unshelve command

Subsystems: the rest

2 messages, 2 authors, 2018-05-21 · open the first message on its own page

Re: [PATCHv4 1/1] git-p4: add unshelve command

From: Junio C Hamano <hidden>
Date: 2018-05-21 07:07:12

SZEDER Gábor [off-list ref] writes:
quoted
diff --git a/t/t9832-unshelve.sh b/t/t9832-unshelve.sh
new file mode 100755
index 0000000000..cca2dec536
... in short, I'd queue a fix-up on top like this to be later
squashed after getting an ack?

 t/t9832-unshelve.sh | 23 ++++-------------------
 1 file changed, 4 insertions(+), 19 deletions(-)
diff --git a/t/t9832-unshelve.sh b/t/t9832-unshelve.sh
index cca2dec536..3513abd21a 100755
--- a/t/t9832-unshelve.sh
+++ b/t/t9832-unshelve.sh
@@ -1,6 +1,6 @@
 #!/bin/sh
 
-last_shelved_change() {
+last_shelved_change () {
 	p4 changes -s shelved -m1 | cut -d " " -f 2
 }
 
@@ -17,7 +17,7 @@ test_expect_success 'init depot' '
 		cd "$cli" &&
 		echo file1 >file1 &&
 		p4 add file1 &&
-		p4 submit -d "change 1"
+		p4 submit -d "change 1" &&
 		: >file_to_delete &&
 		p4 add file_to_delete &&
 		p4 submit -d "file to delete"
@@ -120,29 +120,14 @@ EOF
 	)
 '
 
-diff_adds_line() {
-	text="$1" &&
-	file="$2" &&
-	grep -q "^+$text" $file || (echo "expected \"text\" $text not found in $file" && exit 1)
-}
-
-diff_excludes_line() {
-	text="$1" &&
-	file="$2" &&
-	if grep -q "^+$text" $file; then
-		echo "unexpected text \"$text\" found in $file" &&
-		exit 1
-	fi
-}
-
 # Now try to unshelve it. git-p4 should refuse to do so.
 test_expect_success 'try to unshelve the change' '
 	test_when_finished cleanup_git &&
 	(
 		change=$(last_shelved_change) &&
 		cd "$git" &&
-		! git p4 unshelve $change >out.txt 2>&1 &&
-		grep -q "cannot unshelve" out.txt
+		test_must_fail git p4 unshelve $change 2>err.txt &&
+		grep -q "cannot unshelve" err.txt
 	)
 '
 

Re: [PATCHv4 1/1] git-p4: add unshelve command

From: Luke Diamand <hidden>
Date: 2018-05-21 14:59:05

On 21 May 2018 at 08:07, Junio C Hamano [off-list ref] wrote:
SZEDER Gábor [off-list ref] writes:
quoted
quoted
diff --git a/t/t9832-unshelve.sh b/t/t9832-unshelve.sh
new file mode 100755
index 0000000000..cca2dec536
... in short, I'd queue a fix-up on top like this to be later
squashed after getting an ack?
That looks good to me, thanks! Ack.

Luke
quoted hunk
 t/t9832-unshelve.sh | 23 ++++-------------------
 1 file changed, 4 insertions(+), 19 deletions(-)
diff --git a/t/t9832-unshelve.sh b/t/t9832-unshelve.sh
index cca2dec536..3513abd21a 100755
--- a/t/t9832-unshelve.sh
+++ b/t/t9832-unshelve.sh
@@ -1,6 +1,6 @@
 #!/bin/sh

-last_shelved_change() {
+last_shelved_change () {
        p4 changes -s shelved -m1 | cut -d " " -f 2
 }
@@ -17,7 +17,7 @@ test_expect_success 'init depot' '
                cd "$cli" &&
                echo file1 >file1 &&
                p4 add file1 &&
-               p4 submit -d "change 1"
+               p4 submit -d "change 1" &&
                : >file_to_delete &&
                p4 add file_to_delete &&
                p4 submit -d "file to delete"
@@ -120,29 +120,14 @@ EOF
        )
 '

-diff_adds_line() {
-       text="$1" &&
-       file="$2" &&
-       grep -q "^+$text" $file || (echo "expected \"text\" $text not found in $file" && exit 1)
-}
-
-diff_excludes_line() {
-       text="$1" &&
-       file="$2" &&
-       if grep -q "^+$text" $file; then
-               echo "unexpected text \"$text\" found in $file" &&
-               exit 1
-       fi
-}
-
 # Now try to unshelve it. git-p4 should refuse to do so.
 test_expect_success 'try to unshelve the change' '
        test_when_finished cleanup_git &&
        (
                change=$(last_shelved_change) &&
                cd "$git" &&
-               ! git p4 unshelve $change >out.txt 2>&1 &&
-               grep -q "cannot unshelve" out.txt
+               test_must_fail git p4 unshelve $change 2>err.txt &&
+               grep -q "cannot unshelve" err.txt
        )
 '
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help