Skip to content

Commit 46dbb1f

Browse files
committed
feature_flags_SUITE: Fix style
[Why] Several lines were crossing the 80-columns boundary, plus messages without a capital first letter.
1 parent 6acc2b1 commit 46dbb1f

File tree

1 file changed

+21
-14
lines changed

1 file changed

+21
-14
lines changed

deps/rabbit/test/feature_flags_SUITE.erl

Lines changed: 21 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,8 @@
22
%% License, v. 2.0. If a copy of the MPL was not distributed with this
33
%% file, You can obtain one at https://mozilla.org/MPL/2.0/.
44
%%
5-
%% Copyright (c) 2007-2025 Broadcom. All Rights Reserved. The term “Broadcom” refers to Broadcom Inc. and/or its subsidiaries. All rights reserved.
5+
%% Copyright (c) 2019-2025 Broadcom. All Rights Reserved. The term “Broadcom”
6+
%% refers to Broadcom Inc. and/or its subsidiaries. All rights reserved.
67
%%
78

89
-module(feature_flags_SUITE).
@@ -197,14 +198,15 @@ init_per_group(clustering, Config) ->
197198
{rmq_nodes_clustered, false},
198199
{start_rmq_with_plugins_disabled, true}]),
199200
Config2 = rabbit_ct_helpers:merge_app_env(
200-
Config1, {rabbit, [{forced_feature_flags_on_init, [
201-
restart_streams,
202-
stream_sac_coordinator_unblock_group,
203-
stream_update_config_command,
204-
stream_filtering,
205-
message_containers,
206-
quorum_queue_non_voters
207-
]}]}),
201+
Config1, {rabbit, [{forced_feature_flags_on_init,
202+
[
203+
restart_streams,
204+
stream_sac_coordinator_unblock_group,
205+
stream_update_config_command,
206+
stream_filtering,
207+
message_containers,
208+
quorum_queue_non_voters
209+
]}]}),
208210
rabbit_ct_helpers:run_setup_steps(Config2, [fun prepare_my_plugin/1]);
209211
init_per_group(activating_plugin, Config) ->
210212
Config1 = rabbit_ct_helpers:set_config(
@@ -219,7 +221,8 @@ init_per_group(_, Config) ->
219221
end_per_group(_, Config) ->
220222
Config.
221223

222-
init_per_testcase(enable_feature_flag_when_ff_file_is_unwritable = Testcase, Config) ->
224+
init_per_testcase(
225+
enable_feature_flag_when_ff_file_is_unwritable = Testcase, Config) ->
223226
case erlang:system_info(otp_release) of
224227
"26" ->
225228
{skip, "Hits a crash in Mnesia fairly frequently"};
@@ -1284,11 +1287,13 @@ activating_plugin_with_new_ff_enabled(Config) ->
12841287
ok.
12851288

12861289
enable_plugin_feature_flag_after_deactivating_plugin(Config) ->
1287-
case rabbit_ct_broker_helpers:is_feature_flag_enabled(Config, 'rabbitmq_4.0.0') of
1290+
RabbitMQ40Enabled = rabbit_ct_broker_helpers:is_feature_flag_enabled(
1291+
Config, 'rabbitmq_4.0.0'),
1292+
case RabbitMQ40Enabled of
12881293
true ->
12891294
ok;
12901295
false ->
1291-
throw({skip, "this test triggers a bug present in 3.13"})
1296+
throw({skip, "This test triggers a bug present in 3.13"})
12921297
end,
12931298

12941299
FFSubsysOk = is_feature_flag_subsystem_available(Config),
@@ -1321,11 +1326,13 @@ enable_plugin_feature_flag_after_deactivating_plugin(Config) ->
13211326
ok.
13221327

13231328
restart_node_with_unknown_enabled_feature_flag(Config) ->
1324-
case rabbit_ct_broker_helpers:is_feature_flag_enabled(Config, 'rabbitmq_4.0.0') of
1329+
RabbitMQ40Enabled = rabbit_ct_broker_helpers:is_feature_flag_enabled(
1330+
Config, 'rabbitmq_4.0.0'),
1331+
case RabbitMQ40Enabled of
13251332
true ->
13261333
ok;
13271334
false ->
1328-
throw({skip, "this test triggers a bug present in 3.13"})
1335+
throw({skip, "This test triggers a bug present in 3.13"})
13291336
end,
13301337

13311338
FFSubsysOk = is_feature_flag_subsystem_available(Config),

0 commit comments

Comments
 (0)