Syntax branchname@{upstream} should interpret its argument as a name of
a branch. Add the test to check that it doesn't try to interpret it as a
refname if the branch in question does not exist.

Signed-off-by: Kacper Kornet <>

Maybe I'm too cautious adding this test. But just in case here it is.

 t/ | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/t/ b/t/
index d6e5761..b27a720 100755
--- a/t/
+++ b/t/
@@ -54,6 +54,10 @@ test_expect_success 'my-side@{upstream} resolves to correct 
full name' '
        test refs/remotes/origin/side = "$(full_name my-side@{u})"
+test_expect_success 'refs/heads/my-side@{upstream} does not resolve to 
my-side{upstream}' '
+       test_must_fail full_name refs/heads/my-side@{upstream}
 test_expect_success 'my-side@{u} resolves to correct commit' '
        git checkout side &&
        test_commit 5 &&

To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to
More majordomo info at

Reply via email to