Remove incidental md5() function uses from several tests
authorPeter Eisentraut <[email protected]>
Tue, 4 Jul 2023 12:31:57 +0000 (14:31 +0200)
committerPeter Eisentraut <[email protected]>
Tue, 4 Jul 2023 12:31:57 +0000 (14:31 +0200)
commit657f5f223e8fd9aa12d8d9fc473faecc267b9b7a
tree14e8d6ed4e9be2b368b584c007ce2defce953fc0
parent625d5b3ca0968c1d8c080d5210f7209184c0d134
Remove incidental md5() function uses from several tests

This removes md5() function calls from these test suites:

- bloom
- test_decoding
- isolation
- recovery
- subscription

This covers all remaining test suites where md5() calls were just used
to generate some random data and can be replaced by appropriately
adapted sha256() calls.  This will eventually allow these tests to
pass in OpenSSL FIPS mode (which does not allow MD5 use).  See also
208bf364a9.  Unlike for the main regression tests, I didn't write a
fipshash() wrapper here, because that would have been too repetitive
and wouldn't really save much here.  In some cases it was easier to
remove one layer of indirection by changing column types from text to
bytea.

Reviewed-by: Daniel Gustafsson <[email protected]>
Discussion: https://www.postgresql.org/message-id/flat/f9b480b5-e473-d2d1-223a-4b9db30a229a@eisentraut.org
17 files changed:
contrib/bloom/expected/bloom.out
contrib/bloom/sql/bloom.sql
contrib/bloom/t/001_wal.pl
contrib/test_decoding/expected/concurrent_stream.out
contrib/test_decoding/specs/concurrent_stream.spec
src/test/isolation/specs/insert-conflict-specconflict.spec
src/test/recovery/t/015_promotion_pages.pl
src/test/recovery/t/026_overwrite_contrecord.pl
src/test/subscription/t/008_diff_schema.pl
src/test/subscription/t/015_stream.pl
src/test/subscription/t/016_stream_subxact.pl
src/test/subscription/t/017_stream_ddl.pl
src/test/subscription/t/018_stream_subxact_abort.pl
src/test/subscription/t/019_stream_subxact_ddl_abort.pl
src/test/subscription/t/022_twophase_cascade.pl
src/test/subscription/t/023_twophase_stream.pl
src/test/subscription/t/029_on_error.pl