summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMaurice Kalinowski <maurice.kalinowski@qt.io>2018-07-23 13:42:14 +0200
committerMaurice Kalinowski <maurice.kalinowski@qt.io>2018-07-24 12:32:43 +0000
commit86b2366942ce167fd065e6ee31ea9e59cf115c10 (patch)
tree5e4eb609908ea70fc4031a3f48ed7ae3d9badce0
parentba94777b84a67d46039241c3228a79b1f749a102 (diff)
Enable server client id test
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>
-rw-r--r--tests/auto/conformance/tst_conformance.cpp3
1 files changed, 1 insertions, 2 deletions
diff --git a/tests/auto/conformance/tst_conformance.cpp b/tests/auto/conformance/tst_conformance.cpp
index f5512a9..d093637 100644
--- a/tests/auto/conformance/tst_conformance.cpp
+++ b/tests/auto/conformance/tst_conformance.cpp
@@ -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.");
}