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).
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: create_ part.out 2018-05-28 11:46:40.251216169 -0300 create_ part_1. out 2018-05-28 15:09:47.287071961 -0300
--- londiste_
+++ londiste_
@@ -69,9 +69,9 @@
(1 row)
select tgenabled, pg_get_ triggerdef( oid) from pg_trigger where tgrelid = 'events_ 2011_01' ::regclass: :oid; -----+- ------- ------- ------- ------- ------- ------- ------- ------- ------- ------- ------- ------- ------- ------- --- -----+- ------- ------- ------- ------- ------- ------- ------- ------- ------- ------- ------- ------- ------- ------- ------- --- events_ 2011_01 FOR EACH ROW EXECUTE PROCEDURE "NullTrigger"()
- 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.
(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).