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

Re: [PATCH 0/2] Improve test code coverage on jt/partial-clone-missing-ref-delta-base

From
Derrick Stolee <stolee@gmail.com>
Date
Jun 12, 2019, 02:39 UTC
Message-ID
<464f2cb0-68a5-ec2b-e526-98874c27a8b4@gmail.com>
In-Reply-To
<cover.1560286956.git.jonathantanmy@google.com>
On 6/11/2019 5:06 PM, Jonathan Tan wrote:
Show 9 quoted lines
> This series is on jt/partial-clone-missing-ref-delta-base.
> 
> Patch 1 is a cleanup, and patch 2 improves test code coverage. Thanks to
> Stolee for the code coverage report.
> 
> I have checked this by running `make coverage` then `make cover_db_html`
> on both "next" and this series, and comparing the output - the
> relevant line indeed shows up as uncovered on "next", and covered on
> this.

Thanks for the additional tests, Jonathan! The update in PATCH 2 looks like it took a bit of work, so thanks for your efforts there!

LGTM.
-Stolee
Previous: Jonathan Tan
Message 4 of 4 in “Improve test code coverage on jt/partial-clone-missing-ref-delta-base”
  1. 0/2 Improve test code coverage on jt/partial-clone-missing-ref-delta-baseJonathan Tan, Jun 11, 2019
  2. 1/2 t5616: use correct flag to check object is missingJonathan Tan, Jun 11, 2019
  3. 2/2 t5616: cover case of client having delta baseJonathan Tan, Jun 11, 2019
  4. Derrick StoleeJun 12, 2019

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.