Re: [PATCH] Add tests for Bitmapset

On Sep 16 2025, at 8:02 am, Robert Haas <robertmhaas@gmail.com> wrote:

> On Tue, Sep 16, 2025 at 2:04 AM Michael Paquier <michael@paquier.xyz> wrote:
>> one SQL function mapping to each C function we are testing?
>
> Yes, I think we should do this, if possible.

Michael, Robert,

Thanks for your time reviewing the proposed code and for providing
feedback. This patch started life as simply copy of the test_radixtree
module which uses a single test function triggered in SQL, hence my
approach.  I guess I should have copied the test_slru module instead! :)

I see the value in the idea of splitting it up into separate functions
and I'll give that a try.

Michael, apologies for the white space issues.  I'll clean that up, and
thank you for the `git diff --check` tip, I'll add that to my toolbox/routine.

> --
> Robert Haas
> EDB: http://www.enterprisedb.com

best.

-greg