1 files changed, 44 insertions, 8 deletions
diff --git a/tests/auto/controls/data/tst_rangeslider.qml b/tests/auto/controls/data/tst_rangeslider.qml
index 24f8a207..42f5bad5 100644
--- a/tests/auto/controls/data/tst_rangeslider.qml
+++ b/tests/auto/controls/data/tst_rangeslider.qml
@@ -159,6 +159,43 @@ TestCase {
         compare(control.first.position, 0.5)
     }
 
+    function test_setToFromUpdatesHandles() {
+        var control = createTemporaryObject(sliderComponent, testCase, { from: 0, to: 100, "first.value": 50, "second.value": 75 })
+        verify(control)
+
+        let firstPos = control.first.position
+        let secondPos = control.second.position
+
+        var firstPosChangesSpy = signalSpy.createObject(control, {target: control.first, signalName: "positionChanged"})
+        verify(firstPosChangesSpy.valid)
+
+        var secondPosChangesSpy = signalSpy.createObject(control, {target: control.second, signalName: "positionChanged"})
+        verify(secondPosChangesSpy.valid)
+
+        // Increasing the 'to' value, so the positions of the handles should be
+        // moved to the left (become smaller)
+        control.to = 200;
+        compare(firstPosChangesSpy.count, 1)
+        compare(secondPosChangesSpy.count, 1)
+        verify(control.first.position < firstPos)
+        verify(control.second.position < secondPos)
+
+        // resetting the values
+        control.to = 100
+        firstPosChangesSpy.clear()
+        secondPosChangesSpy.clear()
+        firstPos = control.first.position
+        secondPos = control.second.position
+
+        // Decreasing the 'from' value, so the positions of the handles should
+        // be moved to the right (become larger)
+        control.from = -100
+        compare(firstPosChangesSpy.count, 1)
+        compare(secondPosChangesSpy.count, 1)
+        verify(control.first.position > firstPos)
+        verify(control.second.position > secondPos)
+    }
+
     function test_setValues() {
         var control = createTemporaryObject(sliderComponent, testCase)
         verify(control)
@@ -612,7 +649,7 @@ TestCase {
     }
 
     function test_overlappingHandles() {
-        var control = createTemporaryObject(sliderComponent, testCase, { orientation: data.orientation })
+        var control = createTemporaryObject(sliderComponent, testCase)
         verify(control)
 
         // By default, we force the second handle to be after the first in
@@ -1064,17 +1101,16 @@ TestCase {
 
     function test_valueAt_data() {
         return [
-            { tag: "0.0..1.0", from: 0.0, to: 1.0, values: [0.0, 0.2, 0.5, 1.0] },
-            { tag: "0..100", from: 0, to: 100, values: [0, 20, 50, 100] },
-            { tag: "100..-100", from: 100, to: -100, values: [100, 60, 0, -100] },
-            { tag: "-7..7", from: -7, to: 7, stepSize: 1.0, values: [-7.0, -4.0, 0.0, 7.0] },
-            { tag: "-3..7", from: -3, to: 7, stepSize: 5.0, values: [-3.0, -3.0, 2.0, 7.0] },
+            { tag: "0.0..1.0", properties: { from: 0.0, to: 1.0 }, values: [0.0, 0.2, 0.5, 1.0] },
+            { tag: "0..100", properties: { from: 0, to: 100 }, values: [0, 20, 50, 100] },
+            { tag: "100..-100", properties: { from: 100, to: -100 }, values: [100, 60, 0, -100] },
+            { tag: "-7..7", properties: { from: -7, to: 7, stepSize: 1.0 }, values: [-7.0, -4.0, 0.0, 7.0] },
+            { tag: "-3..7", properties: { from: -3, to: 7, stepSize: 5.0 }, values: [-3.0, -3.0, 2.0, 7.0] },
         ]
     }
 
     function test_valueAt(data) {
-        var control = createTemporaryObject(sliderComponent, testCase,
-            { from: data.from, to: data.to, stepSize: data.stepSize })
+        var control = createTemporaryObject(sliderComponent, testCase, data.properties)
         verify(control)
 
         compare(control.valueAt(0.0), data.values[0])