Thread (75 messages) 75 messages, 8 authors, 2022-01-27

Re: [PATCH 1/2] object-name tests: tighten up advise() output test

From: Eric Sunshine <hidden>
Date: 2021-10-04 02:53:09

On Sun, Oct 3, 2021 at 9:43 PM Ævar Arnfjörð Bjarmason [off-list ref] wrote:
Change tests added in 1ffa26c4614 (get_short_sha1: list ambiguous
objects on error, 2016-09-26) to only care about the OIDs that are
listed, which is what the test is trying to check for.

This isn't needed by the subsequent commit, which won't change any of
the output, but a mere tightening of the tests assertions to more
closely match what we really want to test for here.

Now if the advise() message itself were change the phrasing around the
s/were change/were to change/
list of OIDs we won't have this test break. We're assuming that such
output won't have a need to indent anything except the OIDs.

Signed-off-by: Ævar Arnfjörð Bjarmason <redacted>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help