diff options
author | J-P Nurmi <jpnurmi@theqtcompany.com> | 2015-06-29 19:14:55 +0200 |
---|---|---|
committer | J-P Nurmi <jpnurmi@theqtcompany.com> | 2015-06-29 17:56:58 +0000 |
commit | e7869a71faf9d3b2c120e734c7323df0bf41dff6 (patch) | |
tree | f7aef5868b2379d80bf726407375a64990dcd279 /tests/auto/controls/data/tst_exclusivegroup.qml | |
parent | 68e45b4852c2fa3a7722619470d35fb506d44a54 (diff) |
Cleanup & improve auto tests
- always verify component creation to catch errors early
- remove empty init() and cleanup() functions
- remove redundant "defaults" checks
Change-Id: I39f16404fa2e3d3e38348742afd8f51ea36b2f05
Reviewed-by: J-P Nurmi <jpnurmi@theqtcompany.com>
Diffstat (limited to 'tests/auto/controls/data/tst_exclusivegroup.qml')
-rw-r--r-- | tests/auto/controls/data/tst_exclusivegroup.qml | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/tests/auto/controls/data/tst_exclusivegroup.qml b/tests/auto/controls/data/tst_exclusivegroup.qml index d8ae6103..ba840d26 100644 --- a/tests/auto/controls/data/tst_exclusivegroup.qml +++ b/tests/auto/controls/data/tst_exclusivegroup.qml @@ -70,15 +70,9 @@ TestCase { currentSpy.clear() } - function test_defaults() { - var group = exclusiveGroup.createObject(testCase) - verify(group) - verify(!group.current) - group.destroy() - } - function test_null() { var group = exclusiveGroup.createObject(testCase) + verify(group) group.addCheckable(null) group.removeCheckable(null) @@ -93,9 +87,11 @@ TestCase { function test_current() { var group = exclusiveGroup.createObject(testCase) + verify(group) currentSpy.target = group verify(currentSpy.valid) + verify(!group.current) var checkable1 = checkable.createObject(testCase, {checked: true}) var checkable2 = checkable.createObject(testCase, {checked: false}) @@ -211,6 +207,7 @@ TestCase { function test_controls(data) { var container = data.component.createObject(testCase) + verify(container) verify(!container.group.current) |