git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH v2 7/8] t5523-push-upstream: test progress messages

From
Tay Ray Chuan <rctay89@gmail.com>
Date
Oct 16, 2010, 18:37 UTC
Message-ID
<1287254223-4496-8-git-send-email-rctay89@gmail.com>
In-Reply-To
<1287254223-4496-7-git-send-email-rctay89@gmail.com>
Reported-by: Chase Brammer <cbrammer@gmail.com>
Signed-off-by: Tay Ray Chuan <rctay89@gmail.com>
---
  Squashed in Jeff's additional tests, as well as added (missing) TTY
  pre-requisites pointed out by Johnathan.
 t/t5523-push-upstream.sh |   38 ++++++++++++++++++++++++++++++++++++++
 1 files changed, 38 insertions(+), 0 deletions(-)
diff --git a/t/t5523-push-upstream.sh b/t/t5523-push-upstream.sh
index 5a18533..f43d760 100755
--- a/t/t5523-push-upstream.sh
+++ b/t/t5523-push-upstream.sh
@@ -2,6 +2,7 @@
 
 test_description='push with --set-upstream'
 . ./test-lib.sh
+. "$TEST_DIRECTORY"/lib-terminal.sh
 
 ensure_fresh_upstream() {
 	rm -rf parent && git init --bare parent
@@ -70,4 +71,41 @@ test_expect_success 'push -u HEAD' '
 	check_config headbranch upstream refs/heads/headbranch
 '
 
+test_expect_success TTY 'progress messages go to tty' '
+	ensure_fresh_upstream &&
+
+	test_terminal git push -u upstream master >out 2>err &&
+	grep "Writing objects" err
+'
+
+test_expect_failure 'progress messages do not go to non-tty' '
+	ensure_fresh_upstream &&
+
+	# skip progress messages, since stderr is non-tty
+	git push -u upstream master >out 2>err &&
+	! grep "Writing objects" err
+'
+
+test_expect_failure 'progress messages go to non-tty (forced)' '
+	ensure_fresh_upstream &&
+
+	# force progress messages to stderr, even though it is non-tty
+	git push -u --progress upstream master >out 2>err &&
+	grep "Writing objects" err
+'
+
+test_expect_success TTY 'push -q suppresses progress' '
+	ensure_fresh_upstream &&
+
+	test_terminal git push -u -q upstream master >out 2>err &&
+	! grep "Writing objects" err
+'
+
+test_expect_failure TTY 'push --no-progress suppresses progress' '
+	ensure_fresh_upstream &&
+
+	test_terminal git push -u --no-progress upstream master >out 2>err &&
+	! grep "Writing objects" err
+'
+
 test_done
-- 
1.7.2.2.513.ge1ef3
Previous: Tay Ray ChuanNext: Tay Ray Chuan
Message 21 of 41 in “Fix to push --progress. The --progress flag was not being passed into tranport.c from send-pack.h, making the --progress flag unusable”
  1. Fix to push --progress. The --progress flag was not being passed into tranport.c from send-pack.h, making the --progress flag unusableChase Brammer, Oct 12, 2010
  2. Jonathan NiederOct 12, 2010
  3. Junio C HamanoOct 13, 2010
  4. Jeff KingOct 13, 2010
  5. Tay Ray ChuanOct 13, 2010
  6. 0/3 fix push --progress over file://, git://, etc.Tay Ray Chuan, Oct 13, 2010
  7. 1/3 t5523-push-upstream: add function to ensure fresh upstream repoTay Ray Chuan, Oct 13, 2010
  8. Jonathan NiederOct 13, 2010
  9. 2/3 t5523-push-upstream: test progress messagesTay Ray Chuan, Oct 13, 2010
  10. 3/3 push: pass --progress down to git-pack-objectsTay Ray Chuan, Oct 13, 2010
  11. Tay Ray ChuanOct 14, 2010
  12. Jeff KingOct 14, 2010
  13. Tay Ray ChuanOct 13, 2010
  14. 0/8 fix push --progress over file://, git://, etc.Tay Ray Chuan, Oct 16, 2010
  15. 1/8 tests: factor out terminal handling from t7006Tay Ray Chuan, Oct 16, 2010
  16. 2/8 tests: test terminal output to both stdout and stderrTay Ray Chuan, Oct 16, 2010
  17. 3/8 test-lib: allow test code to check the list of declared prerequisitesTay Ray Chuan, Oct 16, 2010
  18. 4/8 test_terminal: catch use without TTY prerequisiteTay Ray Chuan, Oct 16, 2010
  19. 5/8 test_terminal: give priority to test-terminal.perl usageTay Ray Chuan, Oct 16, 2010
  20. 6/8 t5523-push-upstream: add function to ensure fresh upstream repoTay Ray Chuan, Oct 16, 2010
  21. 7/8 t5523-push-upstream: test progress messagesTay Ray Chuan, Oct 16, 2010
  22. 8/8 push: pass --progress down to git-pack-objectsTay Ray Chuan, Oct 16, 2010
  23. Jonathan NiederOct 17, 2010
  24. Jonathan NiederOct 17, 2010
  25. Jeff KingOct 22, 2010
  26. Jonathan NiederOct 17, 2010
  27. 0/3 more push progress testsJeff King, Oct 14, 2010
  28. 1/3 tests: factor out terminal handling from t7006Jeff King, Oct 14, 2010
  29. Jonathan NiederOct 14, 2010
  30. 2/3 tests: test terminal output to both stdout and stderrJeff King, Oct 14, 2010
  31. Jonathan NiederOct 14, 2010
  32. 3/3 t5523: test push progress output to ttyJeff King, Oct 14, 2010
  33. Jonathan NiederOct 14, 2010
  34. Jeff KingOct 14, 2010
  35. 0/2 test_terminal: check that TTY prerequisite is declaredJonathan Nieder, Oct 14, 2010
  36. 1/2 test-lib: allow test code to check the list of declared prerequisitesJonathan Nieder, Oct 14, 2010
  37. Ævar Arnfjörð BjarmasonOct 15, 2010
  38. Jonathan NiederOct 15, 2010
  39. 2/2 test_terminal: catch use without TTY prerequisiteJonathan Nieder, Oct 14, 2010
  40. Jeff KingOct 15, 2010
  41. Tay Ray ChuanOct 15, 2010

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.