> On 10 Feb 2022, at 01:58, Michael Paquier <michael@paquier.xyz> wrote:
>>> Daniel Gustafsson <daniel@yesql.se> writes:
>>>> The attached patch removes all Test::More planning and instead ensures that all
>>>> tests conclude with a done_testing() call.
Pushed to master now with a few more additional hunks fixing test changes that
happened between posting this and now.
> Could it be possible to backpatch that even if
> this could be qualified as only cosmetic? Each time a test is
> backpatched we need to tweak the number of tests planned, and that may
> change slightly depending on the branch dealt with.
I opted out of backpatching for now, to solicit more comments on that. It's
not a bugfix, but it's also not affecting the compiled bits that we ship, so I
think there's a case to be made both for and against a backpatch. Looking at
the oldest branch we support, it seems we've done roughly 25 changes to the
test plans in REL_10_STABLE over the years, so it's neither insignificant nor
an everyday activity. Personally I don't have strong opinions, what do others
think?
--
Daniel Gustafsson https://vmware.com/