Fix alter_table.sql test case to test what it claims to.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 10 Nov 2022 22:24:26 +0000 (17:24 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 10 Nov 2022 22:24:26 +0000 (17:24 -0500)
commitb158e0b1b1bfd2dfce6d66d67dba3c94449f9cac
tree95d7b5b0f3eb1265727e1bfdb0538535bf45efe2
parent36e545cd051ffe58064c1f38faeb64f4feef4aec
Fix alter_table.sql test case to test what it claims to.

The stanza "SET STORAGE may need to add a TOAST table" does not
test what it's supposed to, and hasn't done so since we added
the ability to store constant column default values as metadata.
We need to use a non-constant default to get the expected table
rewrite to actually happen.

Fix that, and add the missing checks that would have exposed the
problem to begin with.

Noted while reviewing a patch that made changes in this test case.
Back-patch to v11 where the problem came in.
src/test/regress/expected/alter_table.out
src/test/regress/sql/alter_table.sql