pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Michael Paquier <michael@paquier.xyz>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: test_bitmapset: Expand more the test coverage
Date: Mon, 29 Sep 2025 06:18:46 +0000
Message-ID: <E1v37Dh-000a6w-30@gemulon.postgresql.org> (raw)

test_bitmapset: Expand more the test coverage

This commit expands the set of tests added by 00c3d87a5cab, to bring the
coverage of bitmapset.c close to 100% by addressing a lot of corner
cases (most of these relate to word counts and reallocations).

Some of the functions of this module also have their own idea of the
result to return depending on the input values given.  These are
specific to the module, still let's add more coverage for all of them.

Some comments are made more consistent in the tests, while on it.

Author: Greg Burd <greg@burd.me>
Reviewed-by: Michael Paquier <michael@paquier.xyz>
Discussion: https://postgr.es/m/aNR-gsGmLnMaNT5i@paquier.xyz

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/5668fff3c512a61b7f8a2b9bf04270578cf5665a

Modified Files
--------------
.../test_bitmapset/expected/test_bitmapset.out     | 1072 +++++++++++++++++---
.../modules/test_bitmapset/sql/test_bitmapset.sql  |  189 +++-
.../modules/test_bitmapset/test_bitmapset--1.0.sql |    4 +
src/test/modules/test_bitmapset/test_bitmapset.c   |   55 +-
4 files changed, 1157 insertions(+), 163 deletions(-)



Message-ID: <E1v37Dh-000a6w-30@gemulon.postgresql.org>
Permalink:  ../E1v37Dh-000a6w-30@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1v37Dh-000a6w-30@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: michael@paquier.xyz, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: test_bitmapset: Expand more the test coverage
  In-Reply-To: <E1v37Dh-000a6w-30@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox