diff options
author | Gabriel de Dietrich <gabriel.dedietrich@theqtcompany.com> | 2015-04-30 16:51:49 +0200 |
---|---|---|
committer | Gabriel de Dietrich <gabriel.dedietrich@theqtcompany.com> | 2015-05-04 10:49:01 +0000 |
commit | 029d119f4c0e13afbcbd5a0724af7d158acaf8e2 (patch) | |
tree | 2f9b8fd13b61c4a462dca8422807bb021a2e5bc4 /tests/auto/controls/data/tst_button.qml | |
parent | 0d3c6767c1117980f48edbd3f4737a8f37471a8f (diff) |
ControlSpy: Accept a list of signals to spy on
This API is cleaner than having to connect to each signal
handler namually.
We solve the 'pressed' signal problem by connecting to the
other signal accessor, namely 'onPressed'. And we do so for
all signals.
We also lay foundation to capture the signal parameters as
each signal is emitted. What we may do with those values is
not clear yet.
Auto-tests updated accordingly.
Change-Id: Ieda96a43072ca7dff2b183e07f6f67aa49c8cf5d
Reviewed-by: Mitch Curtis <mitch.curtis@digia.com>
Diffstat (limited to 'tests/auto/controls/data/tst_button.qml')
-rw-r--r-- | tests/auto/controls/data/tst_button.qml | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/tests/auto/controls/data/tst_button.qml b/tests/auto/controls/data/tst_button.qml index 865b495b..ce3ae6c5 100644 --- a/tests/auto/controls/data/tst_button.qml +++ b/tests/auto/controls/data/tst_button.qml @@ -56,15 +56,9 @@ TestCase { id: control property ControlSpy spy: ControlSpy { - id: spy target: control + signals: ["pressed", "released", "canceled", "clicked", "pressedChanged"] } - - onPressed: spy.checkSignal("pressed") - onReleased: spy.checkSignal("released") - onCanceled: spy.checkSignal("canceled") - onClicked: spy.checkSignal("clicked") - onPressedChanged: spy.checkSignal("pressedChanged") } } @@ -102,7 +96,9 @@ TestCase { compare(control.pressed, true) verify(control.spy.success) - control.spy.expectedSequence = [["pressedChanged", { "pressed": false }], "released", "clicked"] + control.spy.expectedSequence = [["pressedChanged", { "pressed": false }], + "released", + "clicked"] mouseRelease(control, control.width / 2, control.height / 2, Qt.LeftButton) compare(control.pressed, false) verify(control.spy.success) @@ -142,8 +138,11 @@ TestCase { verify(control.activeFocus) // click - control.spy.expectedSequence = [["pressedChanged", { "pressed": true }], "pressed", - ["pressedChanged", { "pressed": false }], "released", "clicked"] + control.spy.expectedSequence = [["pressedChanged", { "pressed": true }], + "pressed", + ["pressedChanged", { "pressed": false }], + "released", + "clicked"] keyClick(Qt.Key_Space) verify(control.spy.success) @@ -151,6 +150,7 @@ TestCase { control.spy.expectedSequence = [] var keys = [Qt.Key_Enter, Qt.Key_Return, Qt.Key_Escape, Qt.Key_Tab] for (var i = 0; i < keys.length; ++i) { + control.spy.reset() keyClick(keys[i]) verify(control.spy.success) } |