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

[PATCH 4/5] bisect: add tests for the --no-checkout option.

From
Jon Seymour <jon.seymour@gmail.com>
Date
Jul 30, 2011, 08:28 UTC
Message-ID
<1312014511-7157-5-git-send-email-jon.seymour@gmail.com>
In-Reply-To
<1312014511-7157-1-git-send-email-jon.seymour@gmail.com>
We expect to git-bisect --no-checkout to tolerate various kinds of repository damage.
Signed-off-by: Jon Seymour <jon.seymour@gmail.com>
---
 t/t6030-bisect-porcelain.sh |   84 +++++++++++++++++++++++++++++++++++++++++++
 1 files changed, 84 insertions(+), 0 deletions(-)
diff --git a/t/t6030-bisect-porcelain.sh b/t/t6030-bisect-porcelain.sh
index c28da2d..a9e86d2 100755
--- a/t/t6030-bisect-porcelain.sh
+++ b/t/t6030-bisect-porcelain.sh
@@ -608,6 +608,14 @@ cat > expected.missing-tree.default <<EOF
 fatal: unable to read tree 39f7e61a724187ab767d2e08442d9b6b9dab587d
 EOF
 
+check_same()
+{
+    echo "Checking $1 is the same as $2" &&
+    git rev-parse "$1" > expected.same &&
+    git rev-parse "$2" > expected.actual &&
+    test_cmp expected.same expected.actual
+}
+
 test_expect_success 'bisect fails if tree is broken on start commit' '
 	git bisect reset &&
 	test_must_fail git bisect start BROKEN_HASH7 BROKEN_HASH4 2>error.txt &&
@@ -622,4 +630,80 @@ test_expect_success 'bisect fails if tree is broken on trial commit' '
 	test_cmp expected.missing-tree.default error.txt
 '
 
+test_expect_success 'bisect: --no-checkout - start commit bad' '
+	git bisect reset &&
+	git bisect start BROKEN_HASH7 BROKEN_HASH4 --no-checkout &&
+	check_same BROKEN_HASH6 HEAD &&
+	git bisect reset
+'
+
+test_expect_success 'bisect: --no-checkout - trial commit bad' '
+	git bisect reset &&
+	git bisect start broken BROKEN_HASH4 --no-checkout &&
+	check_same BROKEN_HASH6 HEAD &&
+	git bisect reset
+'
+
+test_expect_success 'bisect: --no-checkout - target before breakage' '
+	git bisect reset &&
+	git bisect start broken BROKEN_HASH4 --no-checkout &&
+	check_same BROKEN_HASH6 HEAD &&
+	git bisect bad HEAD &&
+	check_same BROKEN_HASH5 HEAD &&
+	git bisect bad HEAD &&
+	check_same BROKEN_HASH5 bisect/bad &&
+	git bisect reset
+'
+
+test_expect_success 'bisect: --no-checkout - target in breakage' '
+	git bisect reset &&
+	git bisect start broken BROKEN_HASH4 --no-checkout &&
+	check_same BROKEN_HASH6 HEAD &&
+	git bisect bad HEAD &&
+	check_same BROKEN_HASH5 HEAD &&
+	git bisect good HEAD &&
+	check_same BROKEN_HASH6 bisect/bad &&
+	git bisect reset
+'
+
+test_expect_success 'bisect: --no-checkout - target after breakage' '
+	git bisect reset &&
+	git bisect start broken BROKEN_HASH4 --no-checkout &&
+	check_same BROKEN_HASH6 HEAD &&
+	git bisect good HEAD &&
+	check_same BROKEN_HASH8 HEAD &&
+	git bisect good HEAD &&
+	check_same BROKEN_HASH9 bisect/bad &&
+	git bisect reset
+'
+
+test_expect_success 'bisect: --no-checkout=CURSOR - check HEAD is unmodified' '
+	git bisect reset &&
+	git checkout broken &&
+	BROKEN=$(git rev-parse broken) &&
+	git bisect start broken BROKEN_HASH4 --no-checkout=CURSOR &&
+	check_same CURSOR BROKEN_HASH6 &&
+	test "refs/heads/broken" = "$(git rev-parse --symbolic-full-name HEAD)" &&
+	git bisect bad CURSOR &&
+	test "refs/heads/broken" = "$(git rev-parse --symbolic-full-name HEAD)" &&
+	check_same CURSOR BROKEN_HASH5 &&
+	test "refs/heads/broken" = "$(git rev-parse --symbolic-full-name HEAD)" &&
+	git bisect good CURSOR &&
+	check_same BROKEN_HASH6 bisect/bad &&
+	test "refs/heads/broken" = "$(git rev-parse --symbolic-full-name HEAD)" &&
+	git bisect reset &&
+	test "refs/heads/broken" = "$(git rev-parse --symbolic-full-name HEAD)" &&
+	check_same HEAD broken &&
+	check_same broken $BROKEN
+'
+
+test_expect_success 'bisect: demonstrate identification of damage boundary' '
+	git bisect reset &&
+	git checkout broken &&
+	git bisect start broken master --no-checkout &&
+	git bisect run eval "git rev-list --objects HEAD && git rev-list --objects HEAD | git pack-objects --stdout >/dev/null || false" &&
+	check_same BROKEN_HASH6 bisect/bad &&
+	git bisect reset
+'
+
 test_done
-- 
1.7.6.387.g991c2
Previous: Christian CouderNext: Jon Seymour
Message 11 of 26 in “bisect: Add support for a --no-checkout option.”
  1. 0/5 bisect: Add support for a --no-checkout option.Jon Seymour, Jul 30, 2011
  2. 1/5 bisect: add tests to document expected behaviour in presence of broken trees.Jon Seymour, Jul 30, 2011
  3. 2/5 bisect: introduce support for --no-checkout=<ref> option.Jon Seymour, Jul 30, 2011
  4. Christian CouderJul 30, 2011
  5. 3/5 bisect: introduce --no-checkout[=<ref>] support into porcelain.Jon Seymour, Jul 30, 2011
  6. Christian CouderJul 30, 2011
  7. Jon SeymourJul 30, 2011
  8. Christian CouderJul 31, 2011
  9. Junio C HamanoAug 1, 2011
  10. Christian CouderAug 1, 2011
  11. 4/5 bisect: add tests for the --no-checkout option.Jon Seymour, Jul 30, 2011
  12. 5/5 bisect: add documentation for --no-checkout[=<ref>] option.Jon Seymour, Jul 30, 2011
  13. Jon SeymourJul 30, 2011
  14. Christian CouderJul 30, 2011
  15. Jon SeymourJul 30, 2011
  16. Christian CouderJul 30, 2011
  17. Junio C HamanoAug 1, 2011
  18. Christian CouderAug 1, 2011
  19. Jon SeymourAug 1, 2011
  20. Junio C HamanoAug 1, 2011
  21. Jon SeymourAug 1, 2011
  22. Junio C HamanoAug 1, 2011
  23. Jon SeymourAug 2, 2011
  24. Jon SeymourAug 2, 2011
  25. Junio C HamanoAug 2, 2011
  26. Jon SeymourAug 2, 2011

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.