On 01/05/2023 23:52, Junio C Hamano wrote:
Andrei Rybak <rybak.a.v@xxxxxxxxx> writes:
Creation of files from redirecting output of Git commands in tests has been
removed for files which aren't being used for assertions. CC'ed are authors of
the affected tests.
v1 cover letter:
https://lore.kernel.org/git/20230401212858.266508-1-rybak.a.v@xxxxxxxxx/
v2 cover letter:
https://lore.kernel.org/git/20230403223338.468025-1-rybak.a.v@xxxxxxxxx/
This round has not seen any further comments; shall we consider it
pretty much done and ready to move to 'next' by now?
In general, I'm OK with the series as is.
While answering Ævar's questions to some of the patches in v2, I
went quite deep in trying to investigate what is and isn't important
to validate/assert in particular tests, but I haven't come up with
a good way to include this information in commit messages for this
series.
Notes per patch:
- 1/6 for t0300 is just an explanation about why one out of three
cases in one test does not check stdout (and doesn't need to).
https://lore.kernel.org/git/db2de983-9b1f-5efb-0fdc-cc704e6b875b@xxxxxxxxx/
- 3/6 for t1300 lead to a separate series
https://lore.kernel.org/git/20230423134649.431783-1-rybak.a.v@xxxxxxxxx/
- 4/6 for t1450 had an idea for a test 'fresh repository has no
dangling objects'. I'm doubtful about usefulness of such a test,
so hasn't sent it as a patch yet.
https://lore.kernel.org/git/35bc2dc5-d5cb-3492-ff94-41b93b7563d4@xxxxxxxxx/
- 6/6 for t2019 -- a dive into how output of "git checkout" is tested
https://lore.kernel.org/git/4ef5464b-31dd-3c3e-05be-9891162e4f05@xxxxxxxxx/#t
Patches 2/6 and 5/6 are different from others, because they fix
more obvious issues.
Thanks.
