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>

1 files changed, 11 insertions, 11 deletions

diff --git a/tests/auto/controls/data/tst_scrollindicator.qml b/tests/auto/controls/data/tst_scrollindicator.qml
index a5cccba8..77d162b6 100644
--- a/tests/auto/controls/data/tst_scrollindicator.qml
+++ b/tests/auto/controls/data/tst_scrollindicator.qml
@@ -67,23 +67,18 @@ TestCase {
         }
     }
 
-    function test_defaults() {
-        var control = scrollIndicator.createObject(testCase)
-        verify(control)
-        verify(control.indicator)
-        compare(control.size, 0.0)
-        compare(control.position, 0.0)
-        compare(control.active, false)
-        compare(control.orientation, Qt.Vertical)
-        control.destroy()
-    }
-
     function test_attach() {
         var container = flickable.createObject(testCase)
+        verify(container)
         waitForRendering(container)
 
         var vertical = scrollIndicator.createObject()
         verify(!vertical.parent)
+        compare(vertical.size, 0.0)
+        compare(vertical.position, 0.0)
+        compare(vertical.active, false)
+        compare(vertical.orientation, Qt.Vertical)
+
         container.AbstractScrollIndicator.vertical = vertical
         compare(vertical.parent, container)
         compare(vertical.orientation, Qt.Vertical)
@@ -92,6 +87,11 @@ TestCase {
 
         var horizontal = scrollIndicator.createObject()
         verify(!horizontal.parent)
+        compare(horizontal.size, 0.0)
+        compare(horizontal.position, 0.0)
+        compare(horizontal.active, false)
+        compare(horizontal.orientation, Qt.Vertical)
+
         container.AbstractScrollIndicator.horizontal = horizontal
         compare(horizontal.parent, container)
         compare(horizontal.orientation, Qt.Horizontal)