From: Deveshi Dwivedi <deveshigurgaon@gmail.com>
To: git@vger.kernel.org
Cc: deveshigurgaon@gmail.com, sunshine@sunshineco.com,
pushkarkumarsingh1970@gmail.com
Subject: [PATCH v2 0/2] t5403: improve post-checkout hook testing
Date: Mon, 12 Jan 2026 06:52:59 +0000 [thread overview]
Message-ID: <20260112065301.1290-1-deveshigurgaon@gmail.com> (raw)
In-Reply-To: <20260111072950.9463-1-deveshigurgaon@gmail.com>
Following up on the recently merged patch that replaced 'test -f' with
test_path_is_file, this series continues the cleanup of the
post-checkout hook tests.
Patch 1/2 introduces a check_post_checkout helper function to replace
the repeated pattern of reading and validating hook arguments. This
refactoring does not change test behavior, but makes the code easier
to maintain and prepares it for further improvement.
Patch 2/2 updates the helper and hook output format to use test_cmp
instead of individual test commands. This provides clearer error
messages on failure, making it easier to see which argument (old ref,
new ref, or flag) did not match the expectation
Changes since v1:
- Updated the check_post_checkout helper comment to be more descriptive.
Deveshi Dwivedi (2):
t5403:introduce check_post_checkout helper function
t5403: use test_cmp for post-checkout argument checks
t/t5403-post-checkout-hook.sh | 53 ++++++++++++++++++++---------------
1 file changed, 30 insertions(+), 23 deletions(-)
--
2.52.0.230.gd8af7cadaa
next prev parent reply other threads:[~2026-01-12 6:53 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-11 7:29 [PATCH 0/2] t5403: improve post-checkout hook testing Deveshi Dwivedi
2026-01-11 7:29 ` [PATCH 1/2] t5403:introduce check_post_checkout helper function Deveshi Dwivedi
2026-01-11 7:53 ` Eric Sunshine
2026-01-11 11:01 ` Pushkar Singh
2026-01-12 6:44 ` Deveshi Dwivedi
2026-01-11 12:10 ` [PATCH] t5403: document check_post_checkout helper Pushkar Singh
2026-01-11 7:29 ` [PATCH 2/2] t5403: use test_cmp for post-checkout argument checks Deveshi Dwivedi
2026-01-12 6:52 ` Deveshi Dwivedi [this message]
2026-01-12 6:53 ` [PATCH v2 1/2] t5403:introduce check_post_checkout helper function Deveshi Dwivedi
2026-01-12 14:48 ` Junio C Hamano
2026-01-12 6:53 ` [PATCH v2 2/2] t5403: use test_cmp for post-checkout argument checks Deveshi Dwivedi
2026-01-12 16:36 ` [PATCH v3 0/2] t5403: improve post-checkout hook testing Deveshi Dwivedi
2026-01-12 16:36 ` [PATCH v3 1/2] t5403:introduce check_post_checkout helper function Deveshi Dwivedi
2026-01-12 16:36 ` [PATCH v3 2/2] t5403: use test_cmp for post-checkout argument checks Deveshi Dwivedi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260112065301.1290-1-deveshigurgaon@gmail.com \
--to=deveshigurgaon@gmail.com \
--cc=git@vger.kernel.org \
--cc=pushkarkumarsingh1970@gmail.com \
--cc=sunshine@sunshineco.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox