Skip to content

Commit 43dfa4d

Browse files
committed
Fix MQTT test flakes
Tests session_reconnect and session_takeover were flaky, specifically when run under Khepri. The issue was in the test itself that the connect properties didn't apply. Therefore, prior to this commit an exclusive queue got created.
1 parent 7cd91a8 commit 43dfa4d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

deps/rabbitmq_mqtt/test/shared_SUITE.erl

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1404,14 +1404,14 @@ session_takeover(Config) ->
14041404
session_switch(Config, Disconnect) ->
14051405
Topic = ClientId = atom_to_binary(?FUNCTION_NAME),
14061406
%% Connect to old node in mixed version cluster.
1407-
C1 = connect(ClientId, Config, 1, [non_clean_sess_opts()]),
1407+
C1 = connect(ClientId, Config, 1, non_clean_sess_opts()),
14081408
{ok, _, [1]} = emqtt:subscribe(C1, Topic, qos1),
14091409
case Disconnect of
14101410
true -> ok = emqtt:disconnect(C1);
14111411
false -> unlink(C1)
14121412
end,
14131413
%% Connect to new node in mixed version cluster.
1414-
C2 = connect(ClientId, Config, 0, [non_clean_sess_opts()]),
1414+
C2 = connect(ClientId, Config, 0, non_clean_sess_opts()),
14151415
case Disconnect of
14161416
true -> ok;
14171417
false -> assert_v5_disconnect_reason_code(Config, ?RC_SESSION_TAKEN_OVER)

0 commit comments

Comments
 (0)