Skip to content

Commit c66e874

Browse files
committed
rabbit tests: Redirect logs to ct always
Doing it on a per test suite basis leads to issues if multiple suites try to configure it, and there's no cleanup performed anyway.
1 parent 7ad8e28 commit c66e874

File tree

6 files changed

+13
-12
lines changed

6 files changed

+13
-12
lines changed

deps/rabbit/Makefile

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -224,6 +224,8 @@ ct-fast:
224224
ct-slow:
225225
$(MAKE) ct CT_SUITES='$(SLOW_CT_SUITES)'
226226

227+
CT_OPTS += -ct_hooks rabbit_ct_hook []
228+
227229
# --------------------------------------------------------------------
228230
# Compilation.
229231
# --------------------------------------------------------------------

deps/rabbit/test/deprecated_features_SUITE.erl

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -85,9 +85,7 @@ groups() ->
8585
init_per_suite(Config) ->
8686
rabbit_ct_helpers:log_environment(),
8787
logger:set_primary_config(level, debug),
88-
rabbit_ct_helpers:run_setup_steps(
89-
Config,
90-
[fun rabbit_ct_helpers:redirect_logger_to_ct_logs/1]).
88+
rabbit_ct_helpers:run_setup_steps(Config, []).
9189

9290
end_per_suite(Config) ->
9391
Config.

deps/rabbit/test/feature_flags_SUITE.erl

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -122,9 +122,7 @@ end_per_suite(Config) ->
122122

123123
init_per_group(registry, Config) ->
124124
logger:set_primary_config(level, debug),
125-
rabbit_ct_helpers:run_steps(
126-
Config,
127-
[fun rabbit_ct_helpers:redirect_logger_to_ct_logs/1]);
125+
rabbit_ct_helpers:run_steps(Config, []);
128126
init_per_group(feature_flags_v2, Config) ->
129127
%% `feature_flags_v2' is now required and won't work in mixed-version
130128
%% clusters if the other version doesn't support it.

deps/rabbit/test/feature_flags_v2_SUITE.erl

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -114,9 +114,7 @@ groups() ->
114114
init_per_suite(Config) ->
115115
rabbit_ct_helpers:log_environment(),
116116
logger:set_primary_config(level, debug),
117-
rabbit_ct_helpers:run_steps(
118-
Config,
119-
[fun rabbit_ct_helpers:redirect_logger_to_ct_logs/1]).
117+
rabbit_ct_helpers:run_steps(Config, []).
120118

121119
end_per_suite(Config) ->
122120
Config.

deps/rabbit/test/rabbit_ct_hook.erl

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
-module(rabbit_ct_hook).
2+
3+
-export([init/2]).
4+
5+
init(_, _) ->
6+
_ = rabbit_ct_helpers:redirect_logger_to_ct_logs([]),
7+
{ok, undefined}.

deps/rabbit/test/rabbitmq_4_0_deprecations_SUITE.erl

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -75,9 +75,7 @@ groups() ->
7575
init_per_suite(Config) ->
7676
rabbit_ct_helpers:log_environment(),
7777
logger:set_primary_config(level, debug),
78-
rabbit_ct_helpers:run_setup_steps(
79-
Config,
80-
[fun rabbit_ct_helpers:redirect_logger_to_ct_logs/1]).
78+
rabbit_ct_helpers:run_setup_steps(Config, []).
8179

8280
end_per_suite(Config) ->
8381
Config.

0 commit comments

Comments
 (0)