From: Andrei Rybak Date: Tue, 02 May 2023 21:03:36 GMT Subject: Re: [PATCH v3 0/6] t: fix unused files, part 2 Message-ID: <73ead9f4-4e12-09c6-8abb-e338f71efceb@gmail.com> In-Reply-To: On 01/05/2023 23:52, Junio C Hamano wrote: > Andrei Rybak 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@gmail.com/ >> v2 cover letter: >> https://lore.kernel.org/git/20230403223338.468025-1-rybak.a.v@gmail.com/ > > 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@gmail.com/ - 3/6 for t1300 lead to a separate series https://lore.kernel.org/git/20230423134649.431783-1-rybak.a.v@gmail.com/ - 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@gmail.com/ - 6/6 for t2019 -- a dive into how output of "git checkout" is tested https://lore.kernel.org/git/4ef5464b-31dd-3c3e-05be-9891162e4f05@gmail.com/#t Patches 2/6 and 5/6 are different from others, because they fix more obvious issues. > Thanks.