Restore upgrade testing to CI and fix upgrades

Fix upgrades from 1.1.0 to 1.2.0, from 1.3.0 to 1.3.1, and from 1.3.1 to
1.3.2. Restore upgrade testing to the test workflow.

Author: theory

.github/workflows/test.yml

@@ -42,7 +42,7 @@ jobs:
       - run: pg-build-test
 
       # Test update.
-      - run: 'if [ -d "$UPDATE_FROM" ]; then make uninstall clean updatecheck; fi'
+      - run: 'if [ -n "$UPDATE_FROM" ]; then make uninstall clean updatecheck; fi'
 
       # Test all, install, test, test-serial, and test-parallel, both from clean
       # repo and repeated with existing build, with and without PARALLEL_CONN=1.

Changes

@@ -15,6 +15,8 @@ Revision history for pgTAP
   `dynamic_library_path`.
 * Tested on PostgreSQL 9.1–18 and fixed a test failure on PostgreSQL 18.
 * Improved Markdown formatting in `README.md`.
+* Fixed upgrades from 1.1.0 to 1.2.0, from 1.3.0 to 1.3.1, and from 1.3.1 to
+  1.3.2 and restored upgrade testing to the test workflow.
 
 1.3.3 2024-04-08T13:44:11Z
 --------------------------
@@ -148,6 +150,18 @@ Revision history for pgTAP
   serially (#279).
 * Fixed an issue where tests might not run in parallel even when the server
   supported it (#279).
+* Fixed failing tests on Postgres 8.1.
+* Added function type testing functions to complement `is_aggregate()` and
+  `isnt_aggregate()`:
+    + `is_normal_function()`
+    + `isnt_normal_function()`
+    + `is_window()`
+    + `isnt_window()`
+    + `is_procedure()`
+    + `isnt_procedure()`
+* Made the diagnostic output of `is_aggregate()` and `isnt_aggregate()`
+  consistent. Previously, when the function did not exist, some instances would
+  generate diagnostic output and some would not.
 
 1.1.0 2019-11-25T19:05:38Z
 --------------------------
@@ -258,18 +272,6 @@ Revision history for pgTAP
     + `partitions_are()`
 * Added the materialized view-testing assertion functions to the v0.95.0 upgrade
   script; they were inadvertently omitted in the v0.95.0 release.
-* Fixed failing tests on Postgres 8.1.
-* Added function type testing functions to complement `is_aggregate()` and
-  `isnt_aggregate()`:
-    + `is_normal_function()`
-    + `isnt_normal_function()`
-    + `is_window()`
-    + `isnt_window()`
-    + `is_procedure()`
-    + `isnt_procedure()`
-* Made the diagnostic output of `is_aggregate()` and `isnt_aggregate()`
-  consistent. Previously, when the function did not exist, some instances would
-  generate diagnostic output and some would not.
 
 0.97.0 2016-11-28T22:18:29Z
 ---------------------------

sql/pgtap--1.1.0--1.2.0.sql

@@ -329,6 +329,26 @@ BEGIN
 END;
 $$ LANGUAGE plpgsql STABLE;
 
+DROP VIEW IF EXISTS tap_funky;
+CREATE VIEW tap_funky
+ AS SELECT p.oid         AS oid,
+           n.nspname     AS schema,
+           p.proname     AS name,
+           pg_catalog.pg_get_userbyid(p.proowner) AS owner,
+           array_to_string(p.proargtypes::regtype[], ',') AS args,
+           CASE p.proretset WHEN TRUE THEN 'setof ' ELSE '' END
+             || p.prorettype::regtype AS returns,
+           p.prolang     AS langoid,
+           p.proisstrict AS is_strict,
+           _prokind(p.oid) AS kind,
+           p.prosecdef   AS is_definer,
+           p.proretset   AS returns_set,
+           p.provolatile::char AS volatility,
+           pg_catalog.pg_function_is_visible(p.oid) AS is_visible
+      FROM pg_catalog.pg_proc p
+      JOIN pg_catalog.pg_namespace n ON p.pronamespace = n.oid
+;
+
 -- Returns true if the specified function exists and is the specified type,
 -- false if it exists and is not the specified type, and NULL if it does not
 -- exist. Types are f for a normal function, p for a procedure, a for an

sql/pgtap--1.3.0--1.3.1.sql

@@ -1,7 +1,17 @@
-CREATE FUNCTION parse_type(type text, OUT typid oid, OUT typmod int4)
-RETURNS RECORD
-AS 'pgtap'
-LANGUAGE C STABLE STRICT;
+-- Use DO to ignore failures when the shared library doesn't exist, because it
+-- only existed for 1.3.1, and if upgrades are running from a later release it
+-- won't be there.
+DO $$
+BEGIN
+    CREATE FUNCTION parse_type(type text, OUT typid oid, OUT typmod int4)
+    RETURNS RECORD
+    AS 'pgtap'
+    LANGUAGE C STABLE STRICT;
+EXCEPTION
+    WHEN undefined_file THEN
+        RAISE NOTICE 'No pgtap shared library; skipping C parse_type()';
+END;
+$$;
 
 CREATE OR REPLACE FUNCTION format_type_string ( TEXT )
 RETURNS TEXT AS $$
@@ -211,3 +221,9 @@ END;
 $$ LANGUAGE plpgsql;
 
 DROP FUNCTION _quote_ident_like(TEXT, TEXT);
+
+-- has_pk( schema, table )
+CREATE OR REPLACE FUNCTION has_pk ( NAME, NAME )
+RETURNS TEXT AS $$
+    SELECT has_pk( $1, $2, 'Table ' || quote_ident($1) || '.' || quote_ident($2) || ' should have a primary key' );
+$$ LANGUAGE sql;

sql/pgtap--1.3.1--1.3.2.sql

@@ -1,4 +1,4 @@
-DROP FUNCTION parse_type(type text, OUT typid oid, OUT typmod int4);
+DROP FUNCTION IF EXISTS parse_type(type text, OUT typid oid, OUT typmod int4);
 
 CREATE OR REPLACE FUNCTION format_type_string ( TEXT )
 RETURNS TEXT AS $$