From: Glen Choo <chooglen@google.com>
Date: Mon, 06 Feb 2023 08:31:13 GMT
Subject: Re: [PATCH v4 1/9] config tests: cover blind spots in git_die_config() tests
Message-ID: <kl6l357ji4a6.fsf@chooglen-macbookpro.roam.corp.google.com>
In-Reply-To: <patch-v4-1.9-4ae56cab7c7-20230202T131155Z-avarab@gmail.com>

Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:

> We need tests for this because a subsequent commit will alter the
> return value of git_config_get_value_multi(), which is used to get the
> config values in the git_die_config() function. This test coverage
> helps to build confidence in that subsequent change.

In v3, I recall having to read ahead quite far ahead in the series to
understand why we needed this patch. In comparison, this is a lot
clearer :)