]> git.ipfire.org Git - thirdparty/postgresql.git/commitdiff
regress: fix test instability
authorAlvaro Herrera <alvherre@alvh.no-ip.org>
Fri, 5 Aug 2022 09:55:52 +0000 (11:55 +0200)
committerAlvaro Herrera <alvherre@alvh.no-ip.org>
Fri, 5 Aug 2022 09:55:52 +0000 (11:55 +0200)
Having additional triggers in a test table made the ORDER BY clauses in
old queries underspecified.  Add another column there for stability.

Per sporadic buildfarm pink.

src/test/regress/expected/triggers.out
src/test/regress/sql/triggers.sql

index 49c5ccfbd6e0dd89f4be36c6b882eae37d8aac08..799f58dac811e4b68a4969836a6882d4ddf696eb 100644 (file)
@@ -2536,7 +2536,7 @@ create trigger tg_stmt after insert on parent
   for statement execute procedure trig_nothing();
 select tgrelid::regclass, tgname, tgenabled from pg_trigger
   where tgrelid in ('parent'::regclass, 'child1'::regclass)
-  order by tgrelid::regclass::text;
+  order by tgrelid::regclass::text, tgname;
  tgrelid | tgname  | tgenabled 
 ---------+---------+-----------
  child1  | tg      | O
@@ -2548,7 +2548,7 @@ alter table only parent enable always trigger tg; -- no recursion because ONLY
 alter table parent enable always trigger tg_stmt;      -- no recursion because statement trigger
 select tgrelid::regclass, tgname, tgenabled from pg_trigger
   where tgrelid in ('parent'::regclass, 'child1'::regclass)
-  order by tgrelid::regclass::text;
+  order by tgrelid::regclass::text, tgname;
  tgrelid | tgname  | tgenabled 
 ---------+---------+-----------
  child1  | tg      | O
@@ -2561,7 +2561,7 @@ select tgrelid::regclass, tgname, tgenabled from pg_trigger
 alter table parent enable always trigger tg;
 select tgrelid::regclass, tgname, tgenabled from pg_trigger
   where tgrelid in ('parent'::regclass, 'child1'::regclass)
-  order by tgrelid::regclass::text;
+  order by tgrelid::regclass::text, tgname;
  tgrelid | tgname  | tgenabled 
 ---------+---------+-----------
  child1  | tg      | A
index fa0a2455524e47faebf53b0df97bb01b82bd53df..98911b989a239023ea70edfbfb143e833c9eedaf 100644 (file)
@@ -1754,18 +1754,18 @@ create trigger tg_stmt after insert on parent
   for statement execute procedure trig_nothing();
 select tgrelid::regclass, tgname, tgenabled from pg_trigger
   where tgrelid in ('parent'::regclass, 'child1'::regclass)
-  order by tgrelid::regclass::text;
+  order by tgrelid::regclass::text, tgname;
 alter table only parent enable always trigger tg;      -- no recursion because ONLY
 alter table parent enable always trigger tg_stmt;      -- no recursion because statement trigger
 select tgrelid::regclass, tgname, tgenabled from pg_trigger
   where tgrelid in ('parent'::regclass, 'child1'::regclass)
-  order by tgrelid::regclass::text;
+  order by tgrelid::regclass::text, tgname;
 -- The following is a no-op for the parent trigger but not so
 -- for the child trigger, so recursion should be applied.
 alter table parent enable always trigger tg;
 select tgrelid::regclass, tgname, tgenabled from pg_trigger
   where tgrelid in ('parent'::regclass, 'child1'::regclass)
-  order by tgrelid::regclass::text;
+  order by tgrelid::regclass::text, tgname;
 drop table parent, child1;
 
 -- Verify that firing state propagates correctly on creation, too