hero123457/qtmqtt
1
0
Fork 0
mirror of https://github.com/qt/qtmqtt.git synced 2026-10-25 16:16:26 +00:00
Code Issues Projects Releases Wiki Activity

Enable server client id test

Browse Source 
When a client connects without a client id, the server might decide to
assign one to the client. The test was disabled so far, but
QMqttServerConnectionProperties handles this properly now.

Change-Id: If013ba67aa1f08c8842b872a6125e7d2bfa6430d
Reviewed-by: Karsten Heimrich <karsten.heimrich@qt.io>
Reviewed-by: hjk <hjk@qt.io>
This commit is contained in:
Maurice Kalinowski 2018-07-23 13:42:14 +02:00
parent ba94777b84
commit 86b2366942
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
tests/auto/conformance/tst_conformance.cpp
View File

@ -281,8 +281,7 @@ void Tst_MqttConformance::zero_length_clientid_test()
if (client.protocolVersion() == QMqttClient::MQTT_5_0) {
// For MQTT 5 the broker creates an ID and returns it in CONNACK
QTRY_VERIFY2(client.state() == QMqttClient::Connected, "Could not connect to broker.");
// ### TODO: Enable this check:
//QVERIFY(!client.clientId().isEmpty())
QVERIFY(!client.clientId().isEmpty());
} else {
QTRY_VERIFY2(client.state() == QMqttClient::Disconnected, "Sessions with empty client should not be allowed.");
}