AdjustUpgrade.pm should zap test_ext_cine, too.
authorTom Lane <[email protected]>
Tue, 17 Jan 2023 21:00:39 +0000 (16:00 -0500)
committerTom Lane <[email protected]>
Tue, 17 Jan 2023 21:00:39 +0000 (16:00 -0500)
test_extensions' test_ext_cine extension has the same upgrade hazard
as test_ext7: the regression test leaves it in an updated state
from which no downgrade path to default is provided.  This causes
the update_extensions.sql script helpfully provided by pg_upgrade
to fail.  So drop it in cross-version-upgrade testing.

Not entirely sure how come I didn't hit this in testing yesterday;
possibly I'd built the upgrade reference databases with
testmodules-install-check disabled.

Back to v10 where this module was introduced.

src/test/perl/PostgreSQL/Test/AdjustUpgrade.pm

index 7b4a19be2a5e2aede3b9089502a1e79edef9503c..7cf4ced392fddf495201eeff582e1417b000312b 100644 (file)
@@ -88,12 +88,13 @@ sub adjust_database_contents
        }
    }
 
-   # avoid version number issues with test_ext7
+   # avoid no-path-to-downgrade-extension-version issues
    if ($dbnames{contrib_regression_test_extensions})
    {
        _add_st(
            $result,
            'contrib_regression_test_extensions',
+           'drop extension if exists test_ext_cine',
            'drop extension if exists test_ext7');
    }