Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 8e75372

Browse files
tglsfdctglsfdc
committed
Suppress less info in regression tests using DROP CASCADE.
DROP CASCADE doesn't currently promise to visit dependent objects in a fixed order, so when the regression tests use it, we typically need to suppress the details of which objects get dropped in order to have predictable test output. Traditionally we've done that by setting client_min_messages higher than NOTICE, but there's a better way: we can "\set VERBOSITY terse" in psql. That suppresses the DETAIL message with the object list, but we still get the basic notice telling how many objects were dropped. So at least the test case can verify that the expected number of objects were dropped. The VERBOSITY method was already in use in a few places, but run around and use it wherever it makes sense. Discussion: https://postgr.es/m/[email protected]
1 parent 1e165d0 commit 8e75372

18 files changed

+52
-56
lines changed

‎src/test/regress/expected/alter_generic.out

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -673,13 +673,17 @@ SELECT nspname, prsname
673673
---
674674
--- Cleanup resources
675675
---
676-
set client_min_messages to warning; -- suppress cascade notices
676+
\set VERBOSITY terse \\ -- suppress cascade details
677677
DROP FOREIGN DATA WRAPPER alt_fdw2 CASCADE;
678+
NOTICE: drop cascades to server alt_fserv2
678679
DROP FOREIGN DATA WRAPPER alt_fdw3 CASCADE;
680+
NOTICE: drop cascades to server alt_fserv3
679681
DROP LANGUAGE alt_lang2 CASCADE;
680682
DROP LANGUAGE alt_lang3 CASCADE;
681683
DROP SCHEMA alt_nsp1 CASCADE;
684+
NOTICE: drop cascades to 28 other objects
682685
DROP SCHEMA alt_nsp2 CASCADE;
686+
NOTICE: drop cascades to 9 other objects
683687
DROP USER regress_alter_user1;
684688
DROP USER regress_alter_user2;
685689
DROP USER regress_alter_user3;

‎src/test/regress/expected/create_index.out

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3064,6 +3064,6 @@ ERROR: must be owner of schema schema_to_reindex
30643064
-- Clean up
30653065
RESET ROLE;
30663066
DROP ROLE regress_reindexuser;
3067-
SET client_min_messages TO 'warning';
3067+
\set VERBOSITY terse \\ -- suppress cascade details
30683068
DROP SCHEMA schema_to_reindex CASCADE;
3069-
RESET client_min_messages;
3069+
NOTICE: drop cascades to 6 other objects

‎src/test/regress/expected/create_view.out

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1707,6 +1707,8 @@ select pg_get_ruledef(oid, true) from pg_rewrite
17071707
(1 row)
17081708

17091709
-- clean up all the random objects we made above
1710-
set client_min_messages = warning;
1710+
\set VERBOSITY terse \\ -- suppress cascade details
17111711
DROP SCHEMA temp_view_test CASCADE;
1712+
NOTICE: drop cascades to 27 other objects
17121713
DROP SCHEMA testviewschm2 CASCADE;
1714+
NOTICE: drop cascades to 62 other objects

‎src/test/regress/expected/object_address.out

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -474,10 +474,12 @@ SELECT (pg_identify_object(addr1.classid, addr1.objid, addr1.objsubid)).*,
474474
---
475475
--- Cleanup resources
476476
---
477-
SET client_min_messages TO 'warning';
477+
\set VERBOSITY terse \\ -- suppress cascade details
478478
DROP FOREIGN DATA WRAPPER addr_fdw CASCADE;
479+
NOTICE: drop cascades to 4 other objects
479480
DROP PUBLICATION addr_pub;
480481
DROP SUBSCRIPTION addr_sub;
481482
DROP SCHEMA addr_nsp CASCADE;
483+
NOTICE: drop cascades to 12 other objects
482484
DROP OWNED BY regress_addr_user;
483485
DROP USER regress_addr_user;

‎src/test/regress/expected/privileges.out

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1620,9 +1620,10 @@ SELECT has_function_privilege('regress_user1', 'testns.testfunc(int)', 'EXECUTE'
16201620
f
16211621
(1 row)
16221622

1623-
SET client_min_messages TO 'warning';
1623+
\set VERBOSITY terse \\ -- suppress cascade details
16241624
DROP SCHEMA testns CASCADE;
1625-
RESET client_min_messages;
1625+
NOTICE: drop cascades to 3 other objects
1626+
\set VERBOSITY default
16261627
-- Change owner of the schema & and rename of new schema owner
16271628
\c -
16281629
CREATE ROLE regress_schemauser1 superuser login;
@@ -1644,9 +1645,9 @@ SELECT nspname, rolname FROM pg_namespace, pg_roles WHERE pg_namespace.nspname =
16441645
(1 row)
16451646

16461647
set session role regress_schemauser_renamed;
1647-
SET client_min_messages TO 'warning';
1648+
\set VERBOSITY terse \\ -- suppress cascade details
16481649
DROP SCHEMA testns CASCADE;
1649-
RESET client_min_messages;
1650+
\set VERBOSITY default
16501651
-- clean up
16511652
\c -
16521653
DROP ROLE regress_schemauser1;

‎src/test/regress/expected/rowsecurity.out

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1393,10 +1393,10 @@ ERROR: infinite recursion detected in policy for relation "rec1"
13931393
-- Mutual recursion via .s.b views
13941394
--
13951395
SET SESSION AUTHORIZATION regress_rls_bob;
1396-
-- Suppress NOTICE messages when doing a cascaded drop.
1397-
SET client_min_messages TO 'warning';
1396+
\set VERBOSITY terse \\ -- suppress cascade details
13981397
DROP VIEW rec1v, rec2v CASCADE;
1399-
RESET client_min_messages;
1398+
NOTICE: drop cascades to 2 other objects
1399+
\set VERBOSITY default
14001400
CREATE VIEW rec1v WITH (security_barrier) AS SELECT * FROM rec1;
14011401
CREATE VIEW rec2v WITH (security_barrier) AS SELECT * FROM rec2;
14021402
SET SESSION AUTHORIZATION regress_rls_alice;
@@ -2775,10 +2775,10 @@ DROP TABLE test_qual_pushdown;
27752775
-- Plancache invalidate on user change.
27762776
--
27772777
RESET SESSION AUTHORIZATION;
2778-
-- Suppress NOTICE messages when doing a cascaded drop.
2779-
SET client_min_messages TO 'warning';
2778+
\set VERBOSITY terse \\ -- suppress cascade details
27802779
DROP TABLE t1 CASCADE;
2781-
RESET client_min_messages;
2780+
NOTICE: drop cascades to 2 other objects
2781+
\set VERBOSITY default
27822782
CREATE TABLE t1 (a integer);
27832783
GRANT SELECT ON t1 TO regress_rls_bob, regress_rls_carol;
27842784
CREATE POLICY p1 ON t1 TO regress_rls_bob USING ((a % 2) = 0);
@@ -3902,10 +3902,10 @@ DROP USER regress_rls_dob_role2;
39023902
-- Clean up objects
39033903
--
39043904
RESET SESSION AUTHORIZATION;
3905-
-- Suppress NOTICE messages when doing a cascaded drop.
3906-
SET client_min_messages TO 'warning';
3905+
\set VERBOSITY terse \\ -- suppress cascade details
39073906
DROP SCHEMA regress_rls_schema CASCADE;
3908-
RESET client_min_messages;
3907+
NOTICE: drop cascades to 29 other objects
3908+
\set VERBOSITY default
39093909
DROP USER regress_rls_alice;
39103910
DROP USER regress_rls_bob;
39113911
DROP USER regress_rls_carol;

‎src/test/regress/expected/rules.out

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2622,9 +2622,10 @@ select * from id_ordered;
26222622
6 | Test 6
26232623
(6 rows)
26242624

2625-
set client_min_messages to warning; -- suppress cascade notices
2625+
\set VERBOSITY terse \\ -- suppress cascade details
26262626
drop table id cascade;
2627-
reset client_min_messages;
2627+
NOTICE: drop cascades to 4 other objects
2628+
\set VERBOSITY default
26282629
--
26292630
-- check corner case where an entirely-dummy subplan is created by
26302631
-- constraint exclusion

‎src/test/regress/expected/stats_ext.out

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -122,10 +122,12 @@ EXCEPTION WHEN wrong_object_type THEN
122122
END;
123123
$$;
124124
NOTICE: stats on toast table not created
125-
SET client_min_messages TO warning;
125+
\set VERBOSITY terse \\ -- suppress cascade details
126126
DROP SCHEMA tststats CASCADE;
127+
NOTICE: drop cascades to 7 other objects
127128
DROP FOREIGN DATA WRAPPER extstats_dummy_fdw CASCADE;
128-
RESET client_min_messages;
129+
NOTICE: drop cascades to server extstats_dummy_srv
130+
\set VERBOSITY default
129131
-- n-distinct tests
130132
CREATE TABLE ndistinct (
131133
filler1 TEXT,

‎src/test/regress/expected/typed_table.out

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,3 @@
1-
-- Clean up in case a prior regression run failed
2-
SET client_min_messages TO 'warning';
3-
DROP TYPE IF EXISTS person_type CASCADE;
4-
RESET client_min_messages;
51
CREATE TABLE ttable1 OF nothing;
62
ERROR: type "nothing" does not exist
73
CREATE TYPE person_type AS (id int, name text);

‎src/test/regress/sql/alter_generic.sql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -573,7 +573,7 @@ SELECT nspname, prsname
573573
---
574574
--- Cleanup resources
575575
---
576-
set client_min_messages to warning; -- suppress cascade notices
576+
\set VERBOSITY terse \\ -- suppress cascade details
577577

578578
DROP FOREIGN DATA WRAPPER alt_fdw2 CASCADE;
579579
DROP FOREIGN DATA WRAPPER alt_fdw3 CASCADE;

0 commit comments

Comments
 (0)