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

[PATCH v4] tests: modernize the test script t0010-racy-git.sh

From
Aryan Gupta via GitGitGadget <gitgitgadget@gmail.com>
Date
Mar 6, 2024, 09:14 UTC
Message-ID
<pull.1675.v4.git.1709716446874.gitgitgadget@gmail.com>
In-Reply-To
<pull.1675.v3.git.1709676557639.gitgitgadget@gmail.com>
From: Aryan Gupta <garyan447@gmail.com>

Modernize the formatting of the test script to align with current standards and improve its overall readability.

Signed-off-by: Aryan Gupta <garyan447@gmail.com>
---
    [GSOC][PATCH] Modernize a test script
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1675%2Faryangupta701%2Ftest-modernize-v4
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1675/aryangupta701/test-modernize-v4
Pull-Request: https://github.com/gitgitgadget/git/pull/1675
Range-diff vs v3:
 1:  05ee9e8a458 = 1:  14c7137baea tests: modernize the test script t0010-racy-git.sh
 t/t0010-racy-git.sh | 31 +++++++++++++++----------------
 1 file changed, 15 insertions(+), 16 deletions(-)
diff --git a/t/t0010-racy-git.sh b/t/t0010-racy-git.sh
index 837c8b7228b..84172a37390 100755
--- a/t/t0010-racy-git.sh
+++ b/t/t0010-racy-git.sh
@@ -10,25 +10,24 @@ TEST_PASSES_SANITIZE_LEAK=true
 
 for trial in 0 1 2 3 4
 do
-	rm -f .git/index
-	echo frotz >infocom
-	git update-index --add infocom
-	echo xyzzy >infocom
-
-	files=$(git diff-files -p)
-	test_expect_success \
-	"Racy GIT trial #$trial part A" \
-	'test "" != "$files"'
-
+	test_expect_success "Racy git trial #$trial part A" '
+		rm -f .git/index &&
+		echo frotz >infocom &&
+		git update-index --add infocom &&
+		echo xyzzy >infocom &&
+
+		git diff-files -p >out &&
+		test_file_not_empty out
+	'
 	sleep 1
-	echo xyzzy >cornerstone
-	git update-index --add cornerstone
 
-	files=$(git diff-files -p)
-	test_expect_success \
-	"Racy GIT trial #$trial part B" \
-	'test "" != "$files"'
+	test_expect_success "Racy git trial #$trial part B" '
+		echo xyzzy >cornerstone &&
+		git update-index --add cornerstone &&
 
+		git diff-files -p >out &&
+		test_file_not_empty out
+	'
 done
 
 test_done

base-commit: 3c2a3fdc388747b9eaf4a4a4f2035c1c9ddb26d0
-- 
gitgitgadget
Previous: Junio C HamanoNext: Christian Couder
Message 13 of 18 in “tests: modernize the test script t0010-racy-git.sh”
  1. tests: modernize the test script t0010-racy-git.shAryan Gupta via GitGitGadget, Feb 29, 2024
  2. Eric SunshineFeb 29, 2024
  3. tests: modernize the test script t0010-racy-git.shAryan Gupta via GitGitGadget, Feb 29, 2024
  4. Eric SunshineFeb 29, 2024
  5. Junio C HamanoFeb 29, 2024
  6. Eric SunshineFeb 29, 2024
  7. Junio C HamanoFeb 29, 2024
  8. Eric SunshineFeb 29, 2024
  9. Eric SunshineMar 1, 2024
  10. Junio C HamanoMar 1, 2024
  11. tests: modernize the test script t0010-racy-git.shAryan Gupta via GitGitGadget, Mar 5, 2024
  12. Junio C HamanoMar 5, 2024
  13. tests: modernize the test script t0010-racy-git.shAryan Gupta via GitGitGadget, Mar 6, 2024
  14. Christian CouderMar 7, 2024
  15. Aryan GuptaMar 7, 2024
  16. Junio C HamanoMar 7, 2024
  17. Junio C HamanoMar 7, 2024
  18. Aryan GuptaMar 7, 2024

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.