Re: [PATCH] progress.c tests: fix breakage with COLUMNS != 80
- From
Jeff King <peff@peff.net>
- Date
- Jun 24, 2021, 05:40 UTC
- Message-ID
- <YNQawgw0EzG86k8S@coredump.intra.peff.net>
- In-Reply-To
- <20210624051253.GG6312@szeder.dev>
On Thu, Jun 24, 2021 at 07:12:53AM +0200, SZEDER Gábor wrote:
Show 11 quoted lines
> > I admit this a _bit_ of a nitpick (since as far as we know none of the > > other scripts care about the terminal width), so I'm OK with this as-is > > if you feel strongly the other way. > > I remember one commit-graph test that does check the number of lines > in the progress output, assuming one progress line per commit-graph > layer, which breaks when we break the progress line in a too narrow > terminal. Running './t5324-split-commit-graph.sh -v -i' in a 46 > column wide terminal fails for me, but succeeds with 47 columns. > > I do suggest setting COLUMNS=80 in 'test-lib.sh'.
Thanks for providing a concrete example. I agree we should just set it for all scripts via test-lib.sh, then.
-Peff