Added IME tests for web forms

client/site_tests/desktopui_ImeTest/desktopui_ImeTest.py

 # Copyright (c) 2010 The Chromium OS Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import os, string, time, gtk
 from autotest_lib.client.bin import site_ui_test, test
-from autotest_lib.client.common_lib import error, site_ui, utils
+from autotest_lib.client.common_lib import error, site_ui, utils, site_httpd
 
 
 class desktopui_ImeTest(site_ui_test.UITest):
     version = 1
     preserve_srcdir = True
 
     def setup(self):
         # TODO: We shouldn't use ibusclient, we should talk to Chrome directly
         self.job.setup_dep(['ibusclient'])
 
+    def initialize(self, creds='$default'):
+        self._test_url = 'http://localhost:8000/interaction_form.html'
+        self._testServer = site_httpd.HTTPListener(8000, docroot=self.bindir)

[satorux1, 2010/11/17 04:35:38]

_test_server per our python style guide.

[timothe, 2010/11/17 05:04:12]

Done.

+        self._testServer.run()
+
+        site_ui_test.UITest.initialize(self, creds)
+
+
+    def cleanup(self):
+        self._testServer.stop()
+        site_ui_test.UITest.cleanup(self)
+
 
     def log_error(self, test_name, message):
         self.job.record('ERROR', None, test_name, message)
         self._failed.append(test_name)
 
 
     # TODO(zork) We should share this with platform_ProcessPrivleges.
     # See: crosbug.com/7453
     def check_process(self, process, user=None):
         """Check if the process is running as the specified user / root.
@@ skipping 41 matching lines @@
                 if 'root' in ps:
                     self.log_error('check_process %s' % process,
                                    'Process %s running as root' % process)
                     return
 
 
     # TODO: Get rid of this function.
     def run_ibusclient(self, options):
         cmd = site_ui.xcommand_as('%s %s' % (self.exefile, options), 'chronos')
         return utils.system_output(cmd, retain_output=True)
 

[Zachary Kuznia, 2010/11/16 10:22:58]

This blank line should go back in.

[timothe, 2010/11/17 01:47:05]

Done.

-
     # TODO: Make this function talk to chrome directly
     def preload_engines(self, engine_list):
         engine_names = string.join(engine_list, " ")
         out = self.run_ibusclient('preload_engines %s' % engine_names)
         if not 'OK' in out:
             self.log_error('preload_engines %s' % engine_names,
                            'Failed to preload engines: %s' % engine_names)
 
 
     # TODO: Make this function talk to chrome directly
     def activate_engine(self, engine_name):
         start_time = time.time()
         while time.time() - start_time < 10:
             out = self.run_ibusclient('activate_engine %s' % engine_name)
             if 'OK' in out and self.get_active_engine() == engine_name:
                 return
             time.sleep(1)
         self.log_error('activate_engine',
                        'Failed to activate engine: %s' % engine_name)
 
 
     def get_active_engine(self):
         out = self.run_ibusclient('get_active_engine')
         return out.strip()
 
 
+    # TODO: Make this function set the config value directly, instead of
+    # attempting to navigate the UI.

[Zachary Kuznia, 2010/11/16 10:22:58]

This comment shouldn't be re-added.

[timothe, 2010/11/17 01:47:05]

Done.

     def toggle_ime_process(self):
         ax = self.get_autox()
 
         # Open the config dialog.
         ax.send_hotkey('Ctrl+t')
         time.sleep(1)
         ax.send_hotkey('Ctrl+l')
         time.sleep(1)
         # Navigate to the "Languages and Input" menu.
         ax.send_text('chrome://settings/language#lang=%s,focus=%s\n' %
@@ skipping 34 matching lines @@
         ax.send_hotkey('Ctrl+t')
         time.sleep(1)
         ax.send_hotkey('Ctrl+l')
         time.sleep(1)
         ax.send_text('chrome://settings/language#lang=%s,focus=%s\n' %
                      (language, engine))
         time.sleep(3)
 
         # Toggle the checkbox.
         ax.send_text(' ')
-        time.sleep(1)
+        time.sleep(2)

[Zachary Kuznia, 2010/11/16 10:22:58]

This value shouldn't need to change.  Was it failing at 1?

[timothe, 2010/11/17 01:47:05]

on my PC sometimes yes, the new language would not be activated with a 1 sec
delay.
On 2010/11/16 10:22:58, Zachary Kuznia wrote:

[Zachary Kuznia, 2010/11/17 03:17:46]

Ok.  Add a comment to note this.

[timothe, 2010/11/17 04:06:57]

Done.

 
         # Close the window.
         ax.send_hotkey('Ctrl+w')
         time.sleep(1)
 
 
     def stop_ime_language(self, language):
         """
         Remove a language from Chrome's preferred list and disable all its IMEs.
 
@@ skipping 72 matching lines @@
         expected_initial_engine = 'xkb:us::eng'
         expected_other_engine = 'xkb:us:altgr-intl:eng'
 
         current_engine = self.get_active_engine()
         if current_engine != expected_initial_engine:
             self.log_error('test_keyboard_shortcut',
                            'Initial engine is %s, expected %s' %
                            (current_engine, expected_initial_engine))
         ax = self.get_autox()
         ax.send_hotkey('Ctrl-l')
+        # If we don't sleep here sometimes the following keys are not received
+        time.sleep(1)
         ax.send_hotkey('Ctrl-space')
         start_time = time.time()
         while time.time() - start_time < 10:
             current_engine = self.get_active_engine()
             if current_engine == expected_other_engine:
                 ax.send_hotkey('Ctrl-space')
                 return
             time.sleep(1)
         self.log_error('test_keyboard_shortcut',
                        'Current engine is %s, expected %s' %
                        (current_engine, expected_other_engine))
 
 
     def test_engine(self, language, engine_name, input_string, expected_string):
         self.start_ime_engine(language, engine_name)
         self.activate_engine(engine_name)
+        self.test_engine_omnibox(language, engine_name, input_string,
+                                 expected_string)
+        self.test_engine_form(language, engine_name, input_string,
+                              expected_string)
+        self.activate_engine('xkb:us::eng')
+        self.stop_ime_language(language)
 
+
+    def test_engine_omnibox(self, language, engine_name, input_string, expected_string):

[Zachary Kuznia, 2010/11/16 10:22:58]

nit: 80 characters.

[timothe, 2010/11/17 01:47:05]

Done.

         ax = self.get_autox()
 
         # Focus on the omnibox so that we can enter text.
         ax.send_hotkey('Ctrl-l')
 
         # Sometimes there is a slight delay before input can be received in the
         # omnibox.
         time.sleep(1)
 
         ax.send_text(input_string)
 
         text = self.get_current_text()
         if text != expected_string:
             self.log_error(
-                'test_engine %s' % engine_name,
-                'Engine %s failed: Got %s, expected %s' % (engine_name, text,
-                                                           expected_string))
-        self.activate_engine('xkb:us::eng')
-        self.stop_ime_language(language)
+                'test_engine %s in omnibox' % engine_name,
+                'Engine %s failed : Got %s, expected %s' % (
+                    engine_name, text, expected_string))
+        #clear the omnibox for future tests

[Zachary Kuznia, 2010/11/16 10:22:58]

Nit: Space after #, capitalize Clear, add a period.

[timothe, 2010/11/17 01:47:05]

Done.

+        ax.send_hotkey('BackSpace')
+
+
+    def test_engine_form(self, language, engine_name, input_string, expected_string):

[Zachary Kuznia, 2010/11/16 10:22:58]

Nit: 80 characters

[timothe, 2010/11/17 01:47:05]

Done.

+        ax = self.get_autox()
+        # Refresh the page so that the focus gets on the text input widget
+        ax.send_hotkey('Ctrl-r')

[Zachary Kuznia, 2010/11/16 10:22:58]

Navigate to the page here, so that it relies less on current state.

[timothe, 2010/11/17 01:47:05]

Done.

+        time.sleep(1)
+        ax.send_text(input_string)
+        text = self.get_current_text()
+        if text != expected_string:
+            self.log_error(
+                'test_engine %s in form' % engine_name,
+                'Engine %s failed : Got %s, expected %s' % (
+                    engine_name, text, expected_string))
 
 
     def run_once(self):
         self._failed = []
         dep = 'ibusclient'
         dep_dir = os.path.join(self.autodir, 'deps', dep)
         self.job.install_pkg(dep, 'dep', dep_dir)
 
         self.exefile = os.path.join(self.autodir,
                                     'deps/ibusclient/ibusclient')
 
         # Before we try to activate the options menu, we need to wait for
         # previous actions to complete.  Most notable is that keystrokes
         # immediately after login get lost.
         time.sleep(5)
 
+        # Navigate to the webpage containing the form

[Zachary Kuznia, 2010/11/16 10:22:58]

See above.

[timothe, 2010/11/17 01:47:05]

Done.

+        ax = self.get_autox()
+        ax.send_hotkey('Ctrl+l')
+        time.sleep(1)
+        ax.send_text("%s \n"%self._test_url)
         self.test_ibus_start_process()
 
         self.check_process('candidate_window', user='chronos')
         self.check_process('ibus-daemon', user='chronos')
         self.check_process('ibus-memconf', user='chronos')
 
         self.test_keyboard_shortcut()
         self.test_engine('ja', 'mozc', 'nihongo \n',
-                         '\xE6\x97\xA5\xE6\x9C\xAC\xE8\xAA\x9E')
+                        '\xE6\x97\xA5\xE6\x9C\xAC\xE8\xAA\x9E')
         self.test_engine('zh-TW', 'chewing', 'hol \n', '\xE6\x93\x8D')
+        self.test_engine('zh-TW', 'm17n:zh:quick', 'aa ', '\xE9\x96\x93')

[Zachary Kuznia, 2010/11/16 10:22:58]

Leave these in the original order to minimize your diff.

[timothe, 2010/11/17 01:47:05]

Done.

         self.test_engine('ko', 'hangul', 'wl ', '\xEC\xA7\x80 ')
-        self.test_engine('zh-CN', 'pinyin', 'nihao ',
-                         '\xE4\xBD\xA0\xE5\xA5\xBD')
-        self.test_engine('zh-TW', 'm17n:zh:quick', 'aa', '\xE9\x96\x93')
+        self.test_engine('zh-CN', 'pinyin', 'nihao ', '\xE4\xBD\xA0\xE5\xA5\xBD')

[Zachary Kuznia, 2010/11/16 10:22:58]

nit: 80 characters.

[timothe, 2010/11/17 01:47:05]

Done.

 
         self.test_ibus_stop_process()
+
         if len(self._failed) != 0:
             raise error.TestFail(
                 'Failed: %s' % ','.join(self._failed))