From 6bdffefaa89eacd306465c8e3bf1ca894ca25a98 Mon Sep 17 00:00:00 2001 From: Andreas Buhr Date: Fri, 29 Jan 2021 10:32:07 +0100 Subject: Fix QTimer::setInterval to remove existing bindings A recent change ( a7ca8b1a2864e47cacd530f6b10d2b415c5193f7 ) led to failure of binding removal in setInterval(). This was fixed by introducing setterScope. This patch add unit tests for this regression. Change-Id: Ic8da1f2d82ad6c8ccd81c9b1eff72d42cf75f28a Reviewed-by: Fabian Kosmale --- tests/auto/corelib/kernel/qtimer/tst_qtimer.cpp | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'tests') diff --git a/tests/auto/corelib/kernel/qtimer/tst_qtimer.cpp b/tests/auto/corelib/kernel/qtimer/tst_qtimer.cpp index cd0a0d03d9..221d818c6b 100644 --- a/tests/auto/corelib/kernel/qtimer/tst_qtimer.cpp +++ b/tests/auto/corelib/kernel/qtimer/tst_qtimer.cpp @@ -1158,6 +1158,10 @@ void tst_QTimer::bindTimer() QCOMPARE(timer.interval(), 10); interval = 100; QCOMPARE(timer.interval(), 100); + timer.setInterval(50); + QCOMPARE(timer.interval(), 50); + interval = 30; + QCOMPARE(timer.interval(), 50); // timerType property QCOMPARE(timer.timerType(), Qt::CoarseTimer); -- cgit v1.2.3