Remove similar test-cases, add timeoutChanged() signal test.

Change-Id: I9a3ed5dd0de14f72bf52eb0a70cd352ab505d1d6
Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>

1 files changed, 4 insertions, 7 deletions

diff --git a/tests/auto/qmodbusclient/tst_qmodbusclient.cpp b/tests/auto/qmodbusclient/tst_qmodbusclient.cpp
index 061bbef..7abcc11 100644
--- a/tests/auto/qmodbusclient/tst_qmodbusclient.cpp
+++ b/tests/auto/qmodbusclient/tst_qmodbusclient.cpp
@@ -75,18 +75,15 @@ private slots:
         TestClient client;
         QCOMPARE(client.timeout(), 1000); //default value test
 
-        client.setTimeout(300);
-        QCOMPARE(client.timeout(), 300);
-
+        QSignalSpy spy(&client, SIGNAL(timeoutChanged(int)));
         client.setTimeout(50);
         QCOMPARE(client.timeout(), 50);
+        QCOMPARE(spy.isEmpty(), false); // we expect the signal
 
+        spy.clear();
         client.setTimeout(49); // everything below 50 fails
         QVERIFY(client.timeout() >= 50);
-
-        // test timer firing off
-        client.setTimeout(100);
-        QCOMPARE(client.timeout(), 100);
+        QCOMPARE(spy.isEmpty(), true); // and the signal should not fire
     }
 
     void testProcessReadWriteSingleMultipleCoilsResponse()