Comment 4 for bug 1773763

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

This branch fixes the test: https://code.launchpad.net/~ahasenack/ubuntu/+source/skytools3/+git/skytools3/+ref/fix-dep8-new-postgresql

I also tried with the original xenial postgresql version, tests also pass there, i.e., this modification doesn't break them.

The diff in the test output is this:
--- londiste_create_part.out 2018-05-28 11:46:40.251216169 -0300
+++ londiste_create_part_1.out 2018-05-28 15:09:47.287071961 -0300
@@ -69,9 +69,9 @@
 (1 row)

 select tgenabled, pg_get_triggerdef(oid) from pg_trigger where tgrelid = 'events_2011_01'::regclass::oid;
- tgenabled | pg_get_triggerdef
------------+------------------------------------------------------------------------------------------------------
- A | CREATE TRIGGER "Fooza" AFTER DELETE ON events_2011_01 FOR EACH ROW EXECUTE PROCEDURE "NullTrigger"()
+ tgenabled | pg_get_triggerdef
+-----------+-------------------------------------------------------------------------------------------------------------
+ A | CREATE TRIGGER "Fooza" AFTER DELETE ON public.events_2011_01 FOR EACH ROW EXECUTE PROCEDURE "NullTrigger"()
 (1 row)

It's adding the "public." prefix to the "events_2011-01" table name, and compensating for the extra length in the output (whitespace and --- chars).