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

Re: Git Test Coverage Report (Friday, Nov 2)

From
Michał Górny <mgorny@gentoo.org>
Date
Nov 3, 2018, 16:22 UTC
Message-ID
<1541262152.1028.20.camel@gentoo.org>
In-Reply-To
<xmqqr2g2mqaq.fsf@gitster-ct.c.googlers.com>
On Sat, 2018-11-03 at 19:03 +0900, Junio C Hamano wrote:
Show 20 quoted lines
> Michał Górny <mgorny@gentoo.org> writes:
> 
> > As for how involved... we'd just have to use a key that has split
> > signing subkey.  Would it be fine to add the subkey to the existing key?
> >  It would imply updating keyids/fingerprints everywhere.
> 
> Yes, that "everywhere" is exactly what I meant by "how involved",
> and your suggestion answers "very much involved".
> 
> If we can easily add _another_ key with a subkey that is not the
> primary one we use for other tests, without touching the existing
> key and the existing tests that use it (including the one I touched
> below--- we'd want to see a sig with a key that is not split is
> shown with the same %GF and %GP), while adding a handful of new
> tests that create signed objects under the new & split key and 
> view them with %GF and %GP, then the end result would be that we
> managed to add a new test case where %GF/%GP are different without
> making very much involved changes.  I guess that was what I was
> getting at.
> 
I've just did a little research and came to the following results:
1. modifying the 'C. O. Mitter' key would require changes to 4 tests,
2. modifying the 'Eris Discordia' key would require changes to 2 tests
   (both in 7510).

Do you think 2. would be an acceptable option? I think changing 2 tests would be preferable to proliferating a third key for one test case. Also, given that both failing tests are specifically format string tests, one of them would serve additional purpose of testing %GP!=%GF.

-- 
Best regards,
Michał Górny
Previous: Junio C HamanoNext: Junio C Hamano
Message 5 of 7 in “Git Test Coverage Report (Friday, Nov 2)”
  1. Derrick StoleeNov 3, 2018
  2. Junio C HamanoNov 3, 2018
  3. Michał GórnyNov 3, 2018
  4. Junio C HamanoNov 3, 2018
  5. Michał GórnyNov 3, 2018
  6. Junio C HamanoNov 4, 2018
  7. SZEDER GáborNov 3, 2018

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.