Ignore ibus preferences that get set at an indeterminate time

client/site_tests/desktopui_IBusTest/desktopui_IBusTest.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 logging, os, re, string, time
 from autotest_lib.client.bin import site_ui_test, test
 from autotest_lib.client.common_lib import error, site_ui, utils
 
 class desktopui_IBusTest(site_ui_test.UITest):
     version = 1
@@ skipping 153 matching lines @@
                                   'engine/PinyinFuzzyPinyin_S_SH',
                                   'engine/PinyinFuzzyPinyin_UANG_UAN',
                                   'engine/PinyinFuzzyPinyin_UAN_UANG',
                                   'engine/PinyinFuzzyPinyin_ZH_Z',
                                   'engine/PinyinFuzzyPinyin_Z_ZH',
                                   'engine/PinyinCorrectPinyin_UEI_UI',
                                   'engine/PinyinIncompletePinyin',
                                   'engine/PinyinLookupTableOrientation',
                                   'engine/PinyinSpecialPhrases',
 
-                                  # These preferences are actually read, but
-                                  # ibus-daemon reads them before chrome
-                                  # connects,  so they show up as a false
-                                  # failure.
-                                  'general/hotkeynext_engine_in_menu',
-                                  'general/hotkeyprevious_engine',
-                                  'generalglobal_engine',
-
                                   # We don't set these prefernces.
                                   'general/hotkeytrigger',
                                   'generalembed_preedit_text',
                                   'generalenable_by_default',
                                   'generalpreload_engines',
                                   'generaluse_global_engine',
                                   'generaluse_system_keyboard_layout'])
 
+        # These preferences are actually written, but due to a race condition
+        # on startup, they can be read by ibus-daemon before chrome connects,
+        # and so sometimes they show up as a false failure.
+        ignored_unwritten = set(['general/hotkeynext_engine_in_menu',
+                                 'general/hotkeyprevious_engine',
+                                 'generalglobal_engine'])
+
         self.preload_engines(engine_list)
 
         # ibus takes some time to preload the engines, and they can't be
         # activated until they are done loading.  Since we don't get notified
         # when they are ready, we have to wait here to give them time.
         time.sleep(2)
 
         # Send a ctrl+l to enter a text field.
         ax = self.get_autox()
         ax.send_hotkey('Ctrl-l')
 
         for engine_name in engine_list:
             self.activate_engine(engine_name)
 
         out = self.run_ibusclient('get_unused')
         match = re.match(r"Unread:(.*)Unwritten:(.*)", out, re.DOTALL)
         if not match:
             raise error.TestFail('Could not read unused values from ibus')
 
         actual_unread = set(re.split('\n', match.group(1).strip()))
         actual_unwritten = set(re.split('\n', match.group(2).strip()))
 
+        # Filter out any preferences we're ignoring
+        actual_unwritten.difference_update(ignored_unwritten)
+
         new_unread = actual_unread.difference(expected_unread)
         now_read = expected_unread.difference(actual_unread)
         new_unwritten = actual_unwritten.difference(expected_unwritten)
         now_written = expected_unwritten.difference(actual_unwritten)
 
         if new_unread or now_read or new_unwritten or now_written:
             message = ['iBus config has changed:']
             if new_unread:
                 message.append('New unread values:')
                 for key in new_unread:
@@ skipping 52 matching lines @@
             "/usr/bin/ibus-daemon --panel=disable --cache=none --restart")
         time.sleep(2)
         self._candidate_window_job = utils.BgJob(
             "su chronos -c '/opt/google/chrome/candidate_window'")
         start_time = time.time()
         while time.time() - start_time < timeout:
             if os.system('pgrep ^ibus-daemon$') == 0:
                 return
             time.sleep(1)
         raise error.TestFail('ibus-daemon is not running')