Skip to content

Commit 5fad38e

Browse files
author
Daniil Fedotov
committed
Update clustering management test to support new CLI output.
Already joined node message. Changing disk node to disk should not fail
1 parent 2fda08f commit 5fad38e

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

test/clustering_management_SUITE.erl

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,8 @@ join_and_part_cluster(Config) ->
135135

136136
%% Allow clustering with already clustered node
137137
ok = stop_app(Rabbit),
138-
ok = join_cluster(Rabbit, Hare),
138+
{ok, <<"The node is already a member of this cluster">>} =
139+
join_cluster(Rabbit, Hare),
139140
ok = start_app(Rabbit),
140141

141142
stop_reset_start(Rabbit),
@@ -388,10 +389,9 @@ force_boot(Config) ->
388389
change_cluster_node_type(Config) ->
389390
[Rabbit, Hare, _Bunny] = cluster_members(Config),
390391

391-
%% Trying to change the ram node when not clustered should always fail
392+
%% Trying to change the node to the ram type when not clustered should always fail
392393
ok = stop_app(Rabbit),
393394
assert_failure(fun () -> change_cluster_node_type(Rabbit, ram) end),
394-
assert_failure(fun () -> change_cluster_node_type(Rabbit, disc) end),
395395
ok = start_app(Rabbit),
396396

397397
ok = stop_app(Rabbit),
@@ -643,7 +643,7 @@ wait_for_pid_file_to_contain_running_process_pid(PidFile, Attempts, Timeout) ->
643643
Pid = pid_from_file(PidFile),
644644
case rabbit_misc:is_os_process_alive(Pid) of
645645
true -> ok;
646-
false ->
646+
false ->
647647
ct:sleep(Timeout),
648648
wait_for_pid_file_to_contain_running_process_pid(PidFile, Attempts - 1, Timeout)
649649
end.

0 commit comments

Comments
 (0)