diff options
author | Maurice Kalinowski <maurice.kalinowski@qt.io> | 2018-07-23 13:42:14 +0200 |
---|---|---|
committer | Maurice Kalinowski <maurice.kalinowski@qt.io> | 2018-07-24 12:32:43 +0000 |
commit | 86b2366942ce167fd065e6ee31ea9e59cf115c10 (patch) | |
tree | 5e4eb609908ea70fc4031a3f48ed7ae3d9badce0 | |
parent | ba94777b84a67d46039241c3228a79b1f749a102 (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.cpp | 3 |
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."); } |