Re: Use \if/\endif to remove non-libxml2 expected output in regression tests

From: Daniel Gustafsson <daniel(at)yesql(dot)se>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Use \if/\endif to remove non-libxml2 expected output in regression tests
Date: 2026-06-12 08:18:44
Message-ID: 32CC16E4-DA1F-46E1-815D-106A8075ADD1@yesql.se
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> On 12 Jun 2026, at 09:49, Michael Paquier <michael(at)paquier(dot)xyz> wrote:

> I'd like to backpatch that down to v14, to ease future maintenance.
> Any comments and/or objections?

Skipping tests when the underlying library/platform support is missing is in
line with how the TAP tests already do it. We would lose coverage of invoking
XML functions without libxml support, which risk hiding bugs in the error-
handling (which is already the case with the TAP tests etc). For libxml that
might be acceptable since the error paths are quite small (and by now
welltested) and the XML code isn't really moving all that much.

I think there needs to be a big comment explaining the skip_test clause though.

--
Daniel Gustafsson

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Kyotaro Horiguchi 2026-06-12 08:33:30 Re: [BUG] Take a long time to reach consistent after pg_rewind
Previous Message Michael Paquier 2026-06-12 07:49:29 Use \if/\endif to remove non-libxml2 expected output in regression tests