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

Re: [PATCH] t1517: more coverage for commands that work without repository

From
Junio C Hamano <gitster@pobox.com>
Date
May 31, 2024, 14:57 UTC
Message-ID
<xmqqplt2hvv8.fsf@gitster.g>
In-Reply-To
<Zlmz9Edyv_gehMVr@tanuki>
Patrick Steinhardt <ps@pks.im> writes:
Show 11 quoted lines
> On Thu, May 30, 2024 at 09:58:46AM -0700, Junio C Hamano wrote:
> [snip]
>> +test_expect_success 'stripspace outside repository' '
>> +	nongit git stripspace -s </dev/null
>> +'
>
> I was wondering whether we also want to verify that it does the correct
> thing here as it's comparatively easy:
> ...
> For the other tests it's a bit more involved. And given that these are
> all designed as smoke tests, I think it's fine overall.

Yup, as I said in the proposed log message, we may want to update them to actually do something useful later, but this would at least help us catch regressions by mistake.

Previous: Patrick Steinhardt
Message 4 of 4 in “t1517: more coverage for commands that work without repository”
  1. t1517: more coverage for commands that work without repositoryJunio C Hamano, May 30, 2024
  2. t1517: more coverage for commands that work without repositoryJunio C Hamano, May 31, 2024
  3. Patrick SteinhardtMay 31, 2024
  4. Junio C HamanoMay 31, 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.