Applied, with a minor update below:
On 1 June 2015 at 17:30, Peter Maydell <[email protected]> wrote:
> --- a/t/t3300-edit.sh
> +++ b/t/t3300-edit.sh
> @@ -177,6 +177,12 @@ test_expect_success 'Acknowledge a patch' '
> test "$(msg HEAD^)" = "$m//Acked-by: C O Mitter <[email protected]>"
> '
>
> +test_expect_success 'Review a patch' '
> + m=$(msg HEAD^) &&
> + stg edit --review p1 &&
> + test "$(msg HEAD^)" = "$m//Reviewed-by: C O Mitter
> <[email protected]>"
> +'
The test failed for me since Reviewed-by doesn't have two \n after Acked-by.
diff --git a/t/t3300-edit.sh b/t/t3300-edit.sh
index f29fdf6db52b..42152ddc0ab9 100755
--- a/t/t3300-edit.sh
+++ b/t/t3300-edit.sh
@@ -180,7 +180,7 @@ test_expect_success 'Acknowledge a patch' '
test_expect_success 'Review a patch' '
m=$(msg HEAD^) &&
stg edit --review p1 &&
- test "$(msg HEAD^)" = "$m//Reviewed-by: C O Mitter <[email protected]>"
+ test "$(msg HEAD^)" = "$m/Reviewed-by: C O Mitter <[email protected]>"
'
test_expect_success 'Set author' '
Catalin
_______________________________________________
stgit-users mailing list
[email protected]
https://mail.gna.org/listinfo/stgit-users
