Few cleanups and modernizations

This commit introduces few source compatible changes:
- Change obsoleted Q_ENUMS => Q_ENUM
- Mark QSensor::type property as CONSTANT as it cannot change
- Add change signal to QSensor::identifier and a testcase for it

Task-number: QTBUG-92505
Change-Id: I6e3a652929d15bc5211448be2ae9254ed6d21118
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

1 files changed, 28 insertions, 0 deletions

diff --git a/tests/auto/qsensor/tst_qsensor.cpp b/tests/auto/qsensor/tst_qsensor.cpp
index d4884da9..34f44013 100644
--- a/tests/auto/qsensor/tst_qsensor.cpp
+++ b/tests/auto/qsensor/tst_qsensor.cpp
@@ -939,6 +939,34 @@ private slots:
         QVERIFY(sensor2.isActive());
     }
 
+    void testIdenfifierChanged()
+    {
+      TestSensor sensor;
+      QSignalSpy spy(&sensor, SIGNAL(identifierChanged()));
+      QCOMPARE(sensor.identifier(), "");
+
+      // Change id and verify change
+      sensor.setIdentifier("a");
+      QCOMPARE(sensor.identifier(), "a");
+      QCOMPARE(spy.count(), 1);
+
+      // Set same id and verify that no changes
+      sensor.setIdentifier("a");
+      QCOMPARE(sensor.identifier(), "a");
+      QCOMPARE(spy.count(), 1);
+
+      // Change id and verify change
+      sensor.setIdentifier(testsensorimpl::id);
+      QCOMPARE(sensor.identifier(), testsensorimpl::id);
+      QCOMPARE(spy.count(), 2);
+
+      // Identifier cant be changed after connected to backend
+      QVERIFY(sensor.connectToBackend());
+      sensor.setIdentifier("c");
+      QCOMPARE(sensor.identifier(), testsensorimpl::id);
+      QCOMPARE(spy.count(), 2);
+    }
+
     void testSupportedFeatures()
     {
         TestSensor sensor;