Merge lp:~elopio/ubuntu-ui-toolkit/fix1274240-gallery-textfield_helper into lp:ubuntu-ui-toolkit

[Leo Arias]

Use the autopilot text field helper in the gallery test cases

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:929
http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-ci/1648/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/2717/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/2483/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-amd64-ci/596/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-ci/596/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2719/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/2484/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-ui-toolkit-ci/1648/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:931
http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-ci/1649/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/2723
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/2489
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-amd64-ci/597
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-ci/597
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-ci/597/artifact/work/output/*zip*/output.zip
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/2384
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2725
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2725/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/2490
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/2490/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/4926
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/3455

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-ui-toolkit-ci/1649/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:932
http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-ci/1650/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/2725
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/2491/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-amd64-ci/598
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-ci/598
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-ci/598/artifact/work/output/*zip*/output.zip
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/2386
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2727
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2727/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/2492
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/2492/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/4927/console
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/3456

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-ui-toolkit-ci/1650/rebuild

review: Needs Fixing (continuous-integration)

[Leo Arias (elopio)]

Test is failing because of https://code.launchpad.net/~elopio/ubuntu-ui-toolkit/fix1274240-gallery-textfield_helper/+merge/203831

[Leo Arias (elopio)]

Ahh, wrong link. https://bugs.launchpad.net/ubuntu-ci-services-itself/+bug/1274309

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:932
http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-ci/1652/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/2738
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/2502/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-amd64-ci/600
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-ci/600
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-ci/600/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/2397
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2740
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2740/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/2503
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/2503/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/4937/console
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/3470

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-ui-toolkit-ci/1652/rebuild

review: Needs Fixing (continuous-integration)

[Francis Ginther (fginther)]

A rerun of just the mako test passed:
http://s-jenkins.ubuntu-ci:8080/job/generic-mediumtests-runner-mako/4942/

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:933
http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-ci/1659/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/2763
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/2524/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-amd64-ci/607
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-ci/607
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-ci/607/artifact/work/output/*zip*/output.zip
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/2414
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2765
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2765/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/2525
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/2525/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/4953/console
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/3489

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-ui-toolkit-ci/1659/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:934
http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-ci/1663/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/2821
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/2573/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-amd64-ci/611
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-ci/611
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-ci/611/artifact/work/output/*zip*/output.zip
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/2471
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2823
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2823/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/2574
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/2574/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/4999/console
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/3540

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-ui-toolkit-ci/1663/rebuild

review: Needs Fixing (continuous-integration)

[Cris Dywan (kalikiana)]

This fails on mako in ERROR: ubuntuuitoolkit.tests.test_emulators.TextFieldTestCase.test_write_without_clear

review: Needs Fixing

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:935
http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-ci/1673/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/2878
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/2623/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-amd64-ci/621
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-ci/621
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-ci/621/artifact/work/output/*zip*/output.zip
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/2524
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2880
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2880/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/2624
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/2624/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/5049/console
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/3601

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-ui-toolkit-ci/1673/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:936
http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-ci/1675/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/2888
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/2632
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-amd64-ci/623
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-ci/623
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-ci/623/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/2534
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2890
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2890/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/2633
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/2633/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/5058
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/3614

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-ui-toolkit-ci/1675/rebuild

review: Approve (continuous-integration)

[Cris Dywan (kalikiana)]

So why doesn't write work here? Is that a bug?

review: Needs Information

[Leo Arias (elopio)]

Because I added a focus.wait_for(True) on the write method, so it will only work on enabled textfields.
I thought that's ok, as we should never write on disabled textfields, and if we do it should fail anyway.

So, for this test on the disabled text field, we need to go one level lower than write, and call type so we avoid the focus.wait_for(True)

Let me know what you think about it.
Thanks for reviewing.

[Cris Dywan (kalikiana)]

It wasn't clear from the code, but the reasoning is sound.

review: Approve

[PS Jenkins bot (ps-jenkins)]

FAILED: Autolanding.
More details in the following jenkins job:
http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-autolanding/588/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/2921
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/2661/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-amd64-autolanding/191
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-autolanding/191
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-autolanding/191/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/2563
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2923
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2923/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/2662
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/2662/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/5086/console
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/3651

review: Needs Fixing (continuous-integration)

[Leo Arias (elopio)]

Thanks for the review!
All tests failed again in jenkins, I wonder what's going on there. Still, not related to this branch.

Now, after this branch lands, please let me know if you see any text fields misbehaving. The logs should provide more useful information and clues to what we can try next.

[PS Jenkins bot (ps-jenkins)]

review: Approve (continuous-integration)

=== modified file 'tests/autopilot/ubuntuuitoolkit/emulators.py'
--- tests/autopilot/ubuntuuitoolkit/emulators.py	2014-01-13 23:38:20 +0000
+++ tests/autopilot/ubuntuuitoolkit/emulators.py	2014-02-04 17:25:32 +0000
@@ -472,6 +472,7 @@
 
         """
         with self.keyboard.focused_type(self):
+            self.focus.wait_for(True)
             if clear:
                 self.clear()
             else:

=== modified file 'tests/autopilot/ubuntuuitoolkit/tests/gallery/test_textinput.py'
--- tests/autopilot/ubuntuuitoolkit/tests/gallery/test_textinput.py	2013-12-12 09:56:16 +0000
+++ tests/autopilot/ubuntuuitoolkit/tests/gallery/test_textinput.py	2014-02-04 17:25:32 +0000
@@ -1,6 +1,6 @@
 # -*- Mode: Python; coding: utf-8; indent-tabs-mode: nil; tab-width: 4 -*-
 #
-# Copyright (C) 2012, 2013 Canonical Ltd.
+# Copyright (C) 2012, 2013, 2014 Canonical Ltd.
 #
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU Lesser General Public License as published by
@@ -16,59 +16,59 @@
 
 """Tests for the Ubuntu UI Toolkit Gallery - TextInput components"""
 
-from autopilot.matchers import Eventually
-from testtools.matchers import Equals
+from ubuntuuitoolkit import emulators
 from ubuntuuitoolkit.tests.gallery import GalleryTestCase
 
 
-class TextInputTests(GalleryTestCase):
-    """Generic tests for the Gallery"""
-
-    def test_textfield_standard(self):
-        item = "Text Field"
-        self.loadItem(item)
-        self.checkPageHeader(item)
-        textfield_standard = self.getObject('textfield_standard')
-        self.pointing_device.click_object(textfield_standard)
-        self.assertThat(textfield_standard.focus, Eventually(Equals(True)))
-        self.type_string('Hello World')
-        self.assertThat(textfield_standard.text,
-                        Eventually(Equals('Hello World')))
-
-    def test_textfield_password(self):
-        item = "Text Field"
-        self.loadItem(item)
-        self.checkPageHeader(item)
-        textfield_password = self.getObject('textfield_password')
-        self.pointing_device.click_object(textfield_password)
-        self.assertThat(textfield_password.text, Equals('password'))
-
-        self.tap_clearButton('textfield_password')
-        self.assertThat(textfield_password.text, Eventually(Equals('')))
-
-        self.type_string('abcdefgh123')
-        self.assertThat(textfield_password.text,
-                        Eventually(Equals('abcdefgh123')))
-
-    def test_textfield_numbers(self):
-        item = "Text Field"
-        self.loadItem(item)
-        self.checkPageHeader(item)
-        textfield_numbers = self.getObject('textfield_numbers')
-        self.assertThat(textfield_numbers.text, Eventually(Equals('123')))
-
-        self.tap_clearButton('textfield_numbers')
-        self.assertThat(textfield_numbers.text, Eventually(Equals('')))
-        #try typing decimal value when text filed is int only.
-        self.type_string('-100.123')
-        self.assertThat(textfield_numbers.text, Eventually(Equals('-100123')))
+class WriteAndClearTextInputTestCase(GalleryTestCase):
+
+    scenarios = [
+        ('standard textfield', dict(
+            objectName='textfield_standard', text_to_write='Hello World',
+            expected_text='Hello World')),
+        ('password textfield', dict(
+            objectName='textfield_password', text_to_write='Test password',
+            expected_text='Test password')),
+        ('only integers textfield', dict(
+            objectName='textfield_numbers', text_to_write='-100.123',
+            expected_text='-100123'))
+    ]
+
+    def setUp(self):
+        super(WriteAndClearTextInputTestCase, self).setUp()
+        item = 'Text Field'
+        self.loadItem(item)
+        self.checkPageHeader(item)
+
+    def test_write_on_textfield_must_update_text(self):
+        textfield = self.main_view.select_single(
+            emulators.TextField, objectName=self.objectName)
+
+        textfield.write(self.text_to_write)
+        self.assertEqual(self.expected_text, textfield.text)
+
+    def test_clear_textfield_must_remove_text(self):
+        textfield = self.main_view.select_single(
+            emulators.TextField, objectName=self.objectName)
+        textfield.write(self.text_to_write)
+
+        textfield.clear()
+        self.assertEqual('', textfield.text)
+
+
+class DisabledTextInputTestCase(GalleryTestCase):
+
+    def setUp(self):
+        super(DisabledTextInputTestCase, self).setUp()
+        item = 'Text Field'
+        self.loadItem(item)
+        self.checkPageHeader(item)
 
     def test_textfield_disabled(self):
-        item = "Text Field"
-        self.loadItem(item)
-        self.checkPageHeader(item)
-        textfield_disabled = self.getObject('textfield_disabled')
+        textfield_disabled = self.main_view.select_single(
+            emulators.TextField, objectName='textfield_disabled')
         self.assertFalse(textfield_disabled.enabled)
-        #try tapping a disabled field and verify that focus is false.
+
         self.pointing_device.click_object(textfield_disabled)
-        self.assertFalse(textfield_disabled.focus)
+        textfield_disabled.keyboard.type('This should not be writen')
+        self.assertEqual('', textfield_disabled.text)

=== modified file 'tests/unit_x11/tst_components/tst_textfield.qml'
--- tests/unit_x11/tst_components/tst_textfield.qml	2013-10-22 17:15:24 +0000
+++ tests/unit_x11/tst_components/tst_textfield.qml	2014-02-04 17:25:32 +0000
@@ -64,6 +64,16 @@
         }
     }
 
+    TextField {
+        id: enabledTextField
+        enabled: true
+    }
+
+    TextField {
+        id: disabledTextField
+        enabled: false
+    }
+
     TestCase {
         name: "TextFieldAPI"
         when: windowShown
@@ -451,5 +461,25 @@
             id: regExpValidator
             regExp: /[a-z]*/
         }
+
+        function test_click_enabled_textfield_must_give_focus() {
+            textField.forceActiveFocus();
+            compare(
+                enabledTextField.focus, false,
+                'enabledTextField is not focused');
+            mouseClick(
+                enabledTextField, enabledTextField.width/2,
+                enabledTextField.height/2)
+            compare(
+                enabledTextField.focus, true, 'enabledTextField is focused')
+        }
+
+        function test_click_disabled_textfield_must_not_give_focus() {
+            mouseClick(
+                disabledTextField, disabledTextField.width/2,
+                disabledTextField.height/2)
+            compare(
+                textField.focus, false, 'disabledTextField is not focused');
+        }
     }
 }