]> git.ipfire.org Git - thirdparty/git.git/commitdiff
reset: fix validation in sparse index test
authorVictoria Dye <vdye@github.com>
Tue, 11 Jan 2022 18:04:58 +0000 (18:04 +0000)
committerJunio C Hamano <gitster@pobox.com>
Thu, 13 Jan 2022 21:49:44 +0000 (13:49 -0800)
Update t1092 test 'reset with pathspecs outside sparse definition' to verify
index contents. The use of `rev-parse` verifies the contents of HEAD, not
the index, providing no real validation of the reset results. Conversely,
`ls-files` reports the contents of the index (OIDs, flags, filenames), which
are then compared across checkouts to ensure compatible index states.

Fixes 741a2c9ffa (reset: expand test coverage for sparse checkouts,
2021-09-27).

Signed-off-by: Victoria Dye <vdye@github.com>
Reviewed-by: Elijah Newren <newren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t1092-sparse-checkout-compatibility.sh

index 49f70a656927d0a6893a78ac079a8ac76c53c429..d5167e7ed691f57edb39c28d0555841142d1d116 100755 (executable)
@@ -596,13 +596,11 @@ test_expect_success 'reset with pathspecs outside sparse definition' '
 
        test_sparse_match git reset update-folder1 -- folder1 &&
        git -C full-checkout reset update-folder1 -- folder1 &&
-       test_sparse_match git status --porcelain=v2 &&
-       test_all_match git rev-parse HEAD:folder1 &&
+       test_all_match git ls-files -s -- folder1 &&
 
        test_sparse_match git reset update-folder2 -- folder2/a &&
        git -C full-checkout reset update-folder2 -- folder2/a &&
-       test_sparse_match git status --porcelain=v2 &&
-       test_all_match git rev-parse HEAD:folder2/a
+       test_all_match git ls-files -s -- folder2/a
 '
 
 test_expect_success 'reset with wildcard pathspec' '