Fix plpgsql tests for debug_invalidate_system_caches_always.
authorTom Lane
Fri, 8 Jan 2021 23:12:07 +0000 (18:12 -0500)
committerTom Lane
Fri, 8 Jan 2021 23:12:07 +0000 (18:12 -0500)
commit39d4a153105f0693d93f593a23e5144e2bd031ef
tree411628d9a8cd4e8a966af7ba4cf012cfdbb04bd9
parentafcc8772edcec687d87b6f762ca6113229af7291
Fix plpgsql tests for debug_invalidate_system_caches_always.

Commit c9d529848 resulted in having a couple more places where
the error context stack for a failure varies depending on
debug_invalidate_system_caches_always (nee CLOBBER_CACHE_ALWAYS).
This is not very surprising, since we have to re-parse cached
plans if the plan cache is clobbered.  Stabilize the expected
test output by hiding the context stack in these places,
as we've done elsewhere in this test script.

(Another idea worth considering, now that we have
debug_invalidate_system_caches_always, is to force it to zero for
these test cases.  That seems like it'd risk reducing the coverage
of cache-clobber testing, which might or might not be worth being
able to verify that we get the expected error output in normal
cases.  For the moment I just stuck with the existing technique.)

In passing, update comments that referred to CLOBBER_CACHE_ALWAYS.

Per buildfarm member hyrax.
src/pl/plpgsql/src/expected/plpgsql_cache.out
src/pl/plpgsql/src/expected/plpgsql_cache_1.out
src/pl/plpgsql/src/expected/plpgsql_record.out
src/pl/plpgsql/src/sql/plpgsql_cache.sql
src/pl/plpgsql/src/sql/plpgsql_record.sql