summaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorLincoln Ramsay <lincoln.ramsay@nokia.com>2012-06-04 12:14:59 +1000
committerQt by Nokia <qt-info@nokia.com>2012-06-04 06:16:48 +0200
commitf233e4266e20ea5ac06d51865b4da040b8e12fa9 (patch)
treed5423554b42d56ff0637c1a7be3bedf05c811bdd /tests
parent030a56186e1697c0c31a858752f17ca18323dc95 (diff)
downloadqtsensors-f233e4266e20ea5ac06d51865b4da040b8e12fa9.tar.gz
setDataRates clobbers user-set data rate
Under normal conditions, dataRate is set/checked like so. QSensor::setDataRate records the data rate. It can't be evaluated because we have no backend. Upon getting a backend, we are notified of the available data rates. Re-evaluate the previously-set dataRate after connecting to a backend. The problem is that connectToBackend did not save the dataRate value until after setDataRates was called and setDataRates was clobbering the dataRate property. Two fixes will be made. connectToBackend will save the dataRate before creating the backend and setDataRates will stop clobbering dataRate. Also, setDataRates is now banned from being called outside of the constructor because otherwise an available data rate may disappear during use. Add a test case to verify that a previously-set data rate is not clobbered. Change-Id: I9632b2c8ffcc6d48d9bc91f263b99028ee997ff9 Reviewed-by: Lorn Potter <lorn.potter@nokia.com>
Diffstat (limited to 'tests')
-rw-r--r--tests/auto/qsensor/tst_qsensor.cpp22
1 files changed, 22 insertions, 0 deletions
diff --git a/tests/auto/qsensor/tst_qsensor.cpp b/tests/auto/qsensor/tst_qsensor.cpp
index 92cb281..8025ab2 100644
--- a/tests/auto/qsensor/tst_qsensor.cpp
+++ b/tests/auto/qsensor/tst_qsensor.cpp
@@ -353,6 +353,28 @@ private slots:
QCOMPARE(actual, expected);
}
+ // Test that a previously-set, valid data rate is retained
+ {
+ TestSensor sensor;
+ sensor.setDataRate(100);
+ sensor.setProperty("doThis", "rates");
+ sensor.connectToBackend();
+ int actual = sensor.dataRate();
+ int expected = 100;
+ QCOMPARE(actual, expected);
+ }
+
+ // Test that a previously-set, invalid data rate is reset to 0
+ {
+ TestSensor sensor;
+ sensor.setDataRate(50);
+ sensor.setProperty("doThis", "rates");
+ sensor.connectToBackend();
+ int actual = sensor.dataRate();
+ int expected = 0;
+ QCOMPARE(actual, expected);
+ }
+
{
TestSensor sensor;
sensor.setProperty("doThis", "rates(0)");