]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Replace flaky CIC/RI isolation tests with a TAP test
authorÁlvaro Herrera <alvherre@kurilemu.de>
Wed, 7 Jan 2026 22:44:57 +0000 (19:44 -0300)
committerÁlvaro Herrera <alvherre@kurilemu.de>
Wed, 7 Jan 2026 22:44:57 +0000 (19:44 -0300)
commite1c971945d62d3150ecfe353ce7bb4263916b489
tree6257fab07bb3b7e84f03602fc46ba526d118452c
parenta516b3f00d7469cbd1885a2c5903fbd62430a2ac
Replace flaky CIC/RI isolation tests with a TAP test

The isolation tests for INSERT ON CONFLICT behavior during CREATE INDEX
CONCURRENTLY and REINDEX CONCURRENTLY (added by bc32a12e0db2,
2bc7e886fc1b, and 90eae926abbb) were disabled in 77038d6d0b49 due to
persistent CI flakiness, after several attempts at stabilization.

This commit removes them and introduces a TAP test in test_misc module
(010_index_concurrently_upsert.pl) that covers the same scenarios.  This
new test should hopefully be more stable while providing assurance that
the fixes in all those commits (plus 81f72115cf18) continue to work.

Author: Mihail Nikalayeu <mihailnikalayeu@gmail.com>
Reported-by: Andres Freund <andres@anarazel.de>
Reviewed-by: Álvaro Herrera <alvherre@kurilemu.de>
Discussion: https://postgr.es/m/ccssrhafzbp3a3beju3ptyc56a7gbfimj4vwkbokoldofckrc7@bso37rxskjtf
Discussion: https://postgr.es/m/CANtu0ogv+6wqRzPK241jik4U95s1pW3MCZ3rX5ZqbFdUysz7Qw@mail.gmail.com
Discussion: https://postgr.es/m/202512112014.icpomgc37zx4@alvherre.pgsql
17 files changed:
src/test/modules/injection_points/Makefile
src/test/modules/injection_points/expected/index-concurrently-upsert-predicate.out [deleted file]
src/test/modules/injection_points/expected/index-concurrently-upsert-predicate_1.out [deleted file]
src/test/modules/injection_points/expected/index-concurrently-upsert.out [deleted file]
src/test/modules/injection_points/expected/index-concurrently-upsert_1.out [deleted file]
src/test/modules/injection_points/expected/reindex-concurrently-upsert-on-constraint.out [deleted file]
src/test/modules/injection_points/expected/reindex-concurrently-upsert-partitioned.out [deleted file]
src/test/modules/injection_points/expected/reindex-concurrently-upsert.out [deleted file]
src/test/modules/injection_points/meson.build
src/test/modules/injection_points/specs/index-concurrently-upsert-predicate.spec [deleted file]
src/test/modules/injection_points/specs/index-concurrently-upsert.spec [deleted file]
src/test/modules/injection_points/specs/reindex-concurrently-upsert-on-constraint.spec [deleted file]
src/test/modules/injection_points/specs/reindex-concurrently-upsert-partitioned.spec [deleted file]
src/test/modules/injection_points/specs/reindex-concurrently-upsert.spec [deleted file]
src/test/modules/test_misc/Makefile
src/test/modules/test_misc/meson.build
src/test/modules/test_misc/t/010_index_concurrently_upsert.pl [new file with mode: 0644]