Skip to content

Commit 3a6d9f5

Browse files
the-mikedavismergify[bot]
authored andcommitted
vhost_SUITE: Clean up default limit and policy changes in cases
These changes to the defaults can affect other test cases. In particular it will affect the time to live and deletion events in the vhost deletion idempotency case. (cherry picked from commit f093831)
1 parent 1b3ce51 commit 3a6d9f5

File tree

1 file changed

+18
-6
lines changed

1 file changed

+18
-6
lines changed

deps/rabbit/test/vhost_SUITE.erl

Lines changed: 18 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -506,9 +506,15 @@ vhost_is_created_with_default_limits(Config) ->
506506
Env = [{vhosts, [{<<"id">>, Limits++Pattern}]}],
507507
?assertEqual(ok, rabbit_ct_broker_helpers:rpc(Config, 0,
508508
application, set_env, [rabbit, default_limits, Env])),
509-
?assertEqual(ok, rabbit_ct_broker_helpers:add_vhost(Config, VHost)),
510-
?assertEqual(Limits, rabbit_ct_broker_helpers:rpc(Config, 0,
511-
rabbit_vhost_limit, list, [VHost])).
509+
try
510+
?assertEqual(ok, rabbit_ct_broker_helpers:add_vhost(Config, VHost)),
511+
?assertEqual(Limits, rabbit_ct_broker_helpers:rpc(Config, 0,
512+
rabbit_vhost_limit, list, [VHost]))
513+
after
514+
rabbit_ct_broker_helpers:rpc(
515+
Config, 0,
516+
application, unset_env, [rabbit, default_limits])
517+
end.
512518

513519
vhost_is_created_with_operator_policies(Config) ->
514520
VHost = <<"vhost1">>,
@@ -517,9 +523,15 @@ vhost_is_created_with_operator_policies(Config) ->
517523
Env = [{operator, [{PolicyName, Definition}]}],
518524
?assertEqual(ok, rabbit_ct_broker_helpers:rpc(Config, 0,
519525
application, set_env, [rabbit, default_policies, Env])),
520-
?assertEqual(ok, rabbit_ct_broker_helpers:add_vhost(Config, VHost)),
521-
?assertNotEqual(not_found, rabbit_ct_broker_helpers:rpc(Config, 0,
522-
rabbit_policy, lookup_op, [VHost, PolicyName])).
526+
try
527+
?assertEqual(ok, rabbit_ct_broker_helpers:add_vhost(Config, VHost)),
528+
?assertNotEqual(not_found, rabbit_ct_broker_helpers:rpc(Config, 0,
529+
rabbit_policy, lookup_op, [VHost, PolicyName]))
530+
after
531+
rabbit_ct_broker_helpers:rpc(
532+
Config, 0,
533+
application, unset_env, [rabbit, default_policies])
534+
end.
523535

524536
vhost_is_created_with_default_user(Config) ->
525537
VHost = <<"vhost1">>,

0 commit comments

Comments
 (0)