vacuumlo: add test to vacuumlo for test coverage
Hackorum builds and tests every patch posted to the lists, not only commitfest submissions. This is Hackorum's own CI rather than the PostgreSQL project's, and it is still under testing - please report anything that looks wrong.
You can run a PostgreSQL built from this patch straight from Docker, with no checkout and no build:
docker run --rm -p 5432:5432 ghcr.io/hackorum-dev/postgres-patch:t46547psql -h localhost -U postgresBuilt from patchset v1 (message #1), September 20, 2026 at 05:44 AM.
Every patchset is also pushed to a branch of our PostgreSQL fork, so you can check out the same tree CI built. Without a PostgreSQL checkout:
git clone --branch t46547_1 https://github.com/hackorum-dev/postgres.gitIn a checkout you already have, add the fork once:
git remote add hackorum https://github.com/hackorum-dev/postgres.gitthen, for this patchset and every later one:
git fetch hackorum t46547_1 && git checkout t46547_1Patchset v1 (message #1) is on t46547_1
On 3 Sep 2022, at 10:27, Dong Wook Lee <sh95119@gmail.com> wrote:
I write a tiny patch about vacuumlo to improve test coverage.
If we are paying for setting up a cluster we might as well test more scenarios
than just the one. Perhaps some other low-hanging fruit like calling vacuumlo
on a non-existing databsase, on one where no LO have been made etc?
One thing about the patch:
+IPC::Run::run [ 'vacuumlo', '-v', '-n', '-p', $port, 'postgres' ], '>', \$stdout;
This should use run_command() which provides facilities for running commands
and capturing STDOUT. With this the test can be rewritten something like:
my ($out, $err) = run_command(['vacuumlo', .. ]);
like($out, ..);
run_command() is defined in PostgreSQL::Test::Utils.
--
Daniel Gustafsson https://vmware.com/
2022年9月3日(土) 17:28 Dong Wook Lee <sh95119@gmail.com>:
Hi hackers,
I write a tiny patch about vacuumlo to improve test coverage.
I hope my work is meaningful.
Hi
While reviewing the patch backlog, we have determined that this patch adds
one or more TAP tests but has not added the test to the "meson.build" file.
To do this, locate the relevant "meson.build" file for each test and add it
in the 'tests' dictionary, which will look something like this:
'tap': {
'tests': [
't/001_basic.pl',
],
},
For some additional details please see this Wiki article:
https://wiki.postgresql.org/wiki/Meson_for_patch_authors
For more information on the meson build system for PostgreSQL see:
https://wiki.postgresql.org/wiki/Meson
Regards
Ian Barwick
On Wed, 16 Nov 2022 at 10:18, Ian Lawrence Barwick <barwick@gmail.com> wrote:
2022年9月3日(土) 17:28 Dong Wook Lee <sh95119@gmail.com>:
Hi hackers,
I write a tiny patch about vacuumlo to improve test coverage.
I hope my work is meaningful.Hi
While reviewing the patch backlog, we have determined that this patch adds
one or more TAP tests but has not added the test to the "meson.build" file.To do this, locate the relevant "meson.build" file for each test and add it
in the 'tests' dictionary, which will look something like this:'tap': {
'tests': [
't/001_basic.pl',
],
},For some additional details please see this Wiki article:
https://wiki.postgresql.org/wiki/Meson_for_patch_authors
For more information on the meson build system for PostgreSQL see:
Hi DongWook Lee,
Please plan to work on the comment and provide a patch. As CommitFest
2023-01 is currently underway, this would be an excellent time to
update the patch and get the patch in a better shape.
Regards,
Vignesh
On Tue, 17 Jan 2023 at 17:10, vignesh C <vignesh21@gmail.com> wrote:
On Wed, 16 Nov 2022 at 10:18, Ian Lawrence Barwick <barwick@gmail.com> wrote:
2022年9月3日(土) 17:28 Dong Wook Lee <sh95119@gmail.com>:
Hi hackers,
I write a tiny patch about vacuumlo to improve test coverage.
I hope my work is meaningful.Hi
While reviewing the patch backlog, we have determined that this patch adds
one or more TAP tests but has not added the test to the "meson.build" file.To do this, locate the relevant "meson.build" file for each test and add it
in the 'tests' dictionary, which will look something like this:'tap': {
'tests': [
't/001_basic.pl',
],
},For some additional details please see this Wiki article:
https://wiki.postgresql.org/wiki/Meson_for_patch_authors
For more information on the meson build system for PostgreSQL see:
Hi DongWook Lee,
Please plan to work on the comment and provide a patch. As CommitFest
2023-01 is currently underway, this would be an excellent time to
update the patch and get the patch in a better shape.
There has been no updates on this thread for some time, so this has
been switched as Returned with Feedback. Feel free to open it in the
next commitfest if you plan to continue on this.
Regards,
Vignesh