Skip to content

Commit fc0503b

Browse files
Martin Ågrengitster
authored andcommitted
t1400: drop debug echo to actually execute test
Instead of running `test "foo" = "$(bar)"`, we prefix the whole thing with `echo`. Comparing to nearby tests makes it clear that this is just debug leftover. This line has actually been modified four times since it was introduced in e522904 (General ref log reading improvements., 2006-05-19) and the `echo` has always survived. Let's finally drop it. This script could need some more cleanups. This is just an immediate fix so that we actually test what we intend to. All other hits for `git grep "\<echo test " -- t/` seem fine. They want to create some input or expected output data. Signed-off-by: Martin Ågren <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 1d4361b commit fc0503b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

t/t1400-update-ref.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -366,7 +366,7 @@ test_expect_success 'Query master@{2005-05-25} (before history)' '
366366
test_when_finished "rm -f o e" &&
367367
git rev-parse --verify master@{2005-05-25} >o 2>e &&
368368
test $C = $(cat o) &&
369-
echo test "warning: Log for '\''master'\'' only goes back to $ed." = "$(cat e)"
369+
test "warning: Log for '\''master'\'' only goes back to $ed." = "$(cat e)"
370370
'
371371
test_expect_success 'Query "master@{May 26 2005 23:31:59}" (1 second before history)' '
372372
test_when_finished "rm -f o e" &&

0 commit comments

Comments
 (0)