Skip to content

Commit bb7ead2

Browse files
committed
Fix test case block_connack_timeout
The new client library version returns reason `{shutdown, connack_timeout}` in the monitor message. Delete the monitor since the test case already asserts on the `{error, connack_timeout}` return value from emqtt:connect/1.
1 parent bfa8f72 commit bb7ead2

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

deps/rabbitmq_mqtt/test/reader_SUITE.erl

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -112,11 +112,7 @@ block_connack_timeout(Config) ->
112112
{proto_ver, ?config(mqtt_version, Config)},
113113
{connect_timeout, 1}]),
114114
unlink(Client),
115-
ClientMRef = monitor(process, Client),
116-
{error, connack_timeout} = emqtt:connect(Client),
117-
receive {'DOWN', ClientMRef, process, Client, connack_timeout} -> ok
118-
after 30_000 -> ct:fail("missing connack_timeout in client")
119-
end,
115+
?assertEqual({error, connack_timeout}, emqtt:connect(Client)),
120116

121117
MqttReader = rpc(Config, ?MODULE, mqtt_connection_pid, [Ports]),
122118
MqttReaderMRef = monitor(process, MqttReader),

0 commit comments

Comments
 (0)