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

[RFC/PATCH] t5525: test the tagopt variable and that it can be overridden

From
Daniel Johnson <computerdruid@gmail.com>
Date
Aug 13, 2010, 20:13 UTC
Message-ID
<1281730410-20009-1-git-send-email-ComputerDruid@gmail.com>
In-Reply-To
<7v8w4bmhmc.fsf@alter.siamese.dyndns.org>
---
Show 6 quoted lines
> The current behaviour seems to me a bug introduced while git-fetch was
> rewritten in C (the original found in contrib/examples reads from the
> config only when no --tags/--no-tags option is given from the command
> line).
>
> Is this something we can protect with a test script from future breakages?

This should test that behavior. I'd appreciate feedback on how to improve this test. I'm not sure if this is the right name/number either.

 t/t5525-fetch-tagopt.sh |   44 ++++++++++++++++++++++++++++++++++++++++++++
 1 files changed, 44 insertions(+), 0 deletions(-)
 create mode 100755 t/t5525-fetch-tagopt.sh
diff --git a/t/t5525-fetch-tagopt.sh b/t/t5525-fetch-tagopt.sh
new file mode 100755
index 0000000..17bd407
--- /dev/null
+++ b/t/t5525-fetch-tagopt.sh
@@ -0,0 +1,44 @@
+
+#!/bin/sh
+
+test_description='tagopt variable affects "git fetch" and is overridden by commandline.'
+
+. ./test-lib.sh
+
+setup_clone () {
+	(git clone --mirror . $1 &&
+	git remote add remote_$1 $1 &&
+	cd $1 &&
+	git tag tag_$1)
+}
+
+test_expect_success setup '
+	echo >file original &&
+	git add file &&
+	git commit -a -m original &&
+	setup_clone one &&
+	git config remote.remote_one.tagopt --no-tags &&
+	setup_clone two &&
+	git config remote.remote_two.tagopt --tags
+	'
+
+test_expect_success "fetch with tagopt=--no-tags does not get tag" '
+	git fetch remote_one &&
+	! (git show-ref tag_one)
+	'
+
+test_expect_success "fetch --tags with tagopt=--no-tags gets tag" '
+	git fetch --tags remote_one &&
+	(git show-ref tag_one)
+	'
+
+test_expect_success "fetch --no-tags with tagopt=--tags does not get tag" '
+	git fetch --no-tags remote_two &&
+	! (git show-ref tag_two)
+	'
+
+test_expect_success "fetch with tagopt=--tags gets tag" '
+	git fetch remote_two &&
+	(git show-ref tag_two)
+	'
+test_done
-- 
1.7.2
Previous: Junio C HamanoNext: Ævar Arnfjörð Bjarmason
Message 7 of 12 in “fetch: allow command line --tags to override config”
  1. fetch: allow command line --tags to override configDaniel Johnson, Aug 4, 2010
  2. Tay Ray ChuanAug 5, 2010
  3. Documentation: changes in the behavior of tagoptDaniel Johnson, Aug 6, 2010
  4. Tay Ray ChuanAug 8, 2010
  5. [RFC/PATCHv2] fetch: allow command line --tags to override configDaniel Johnson, Aug 11, 2010
  6. Junio C HamanoAug 13, 2010
  7. t5525: test the tagopt variable and that it can be overriddenDaniel Johnson, Aug 13, 2010
  8. Ævar Arnfjörð BjarmasonAug 13, 2010
  9. [RFC/PATCHv2] t5525: test the tagopt variable and that it can be overriddenDaniel Johnson, Aug 13, 2010
  10. Ævar Arnfjörð BjarmasonAug 13, 2010
  11. Daniel JohnsonAug 14, 2010
  12. Ævar Arnfjörð BjarmasonAug 14, 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.