Re: [PATCH 13/20] t: refactor tests depending on Perl for textconv scripts

> On Thu, Mar 20, 2025 at 03:37:08PM -0400, Eric Sunshine wrote:
> > On Thu, Mar 20, 2025 at 5:37 AM Patrick Steinhardt <ps@pks.im> wrote:
> > > -test_expect_success PERL_TEST_HELPERS 'rewrite diff respects textconv' '
> > > +test_expect_success 'rewrite diff respects textconv' '
> > >         git diff -B >diff &&
> > > -       grep "dissimilarity index" diff &&
> > > -       grep "^-61" diff &&
> > > -       grep "^-0" diff
> > > +       test_grep "dissimilarity index" diff &&
> > > +       test_grep "^-3d 0a 00" diff &&
> > > +       test_grep "^+3d 0a 01" diff
> > >  '
> >
> > This change seems unrelated to the stated purpose (`textconv`) of this patch(?).
>
> Not quite. The test previously didn't run because it depends on the
> Perl-based textconv script. Now that this textconv script was adapted
> to use shell scripting instead it can run, but as explained in the
> commit message the output of the textconv script changed. We don't
> really care for the exact output at all, we only care that textconv did
> its thing. But we do have to adapt the test accordingly.