Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(40)

Unified Diff: client/site_tests/desktopui_IBusTest/desktopui_IBusTest.py

Issue 1540001: Adding more tests for ibus-gconf daemon. (Closed)
Patch Set: fix Created 10 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « client/deps/ibusclient/src/ibusclient.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: client/site_tests/desktopui_IBusTest/desktopui_IBusTest.py
diff --git a/client/site_tests/desktopui_IBusTest/desktopui_IBusTest.py b/client/site_tests/desktopui_IBusTest/desktopui_IBusTest.py
index 7a4e2c8acb76c1102123a8556945cba46ad5d927..0f44f0ace0bc71e07117efa75786b13436c4cc2c 100644
--- a/client/site_tests/desktopui_IBusTest/desktopui_IBusTest.py
+++ b/client/site_tests/desktopui_IBusTest/desktopui_IBusTest.py
@@ -50,6 +50,16 @@ class desktopui_IBusTest(test.test):
def test_config(self, type_name):
+ wrong_type_name = 'string'
+ if type_name == 'string':
+ wrong_type_name = 'int'
+ # First, write a dummy value which is not |type_name| type to make sure
+ # the second set_config overwrites this |wrong_type_name| value.
+ out = self.run_ibusclient('set_config %s' % wrong_type_name)
+ if not 'OK' in out:
+ raise error.TestFail('Failed to set %s value to '
+ 'the ibus config service' % wrong_type_name)
+ # Then overwrite a value of |type_name| type.
out = self.run_ibusclient('set_config %s' % type_name)
if not 'OK' in out:
raise error.TestFail('Failed to set %s value to '
@@ -84,7 +94,9 @@ class desktopui_IBusTest(test.test):
'deps/ibusclient/ibusclient')
self.test_reachable()
self.test_supported_engines()
- for type_name in ['boolean', 'int', 'double', 'string']:
+ for type_name in ['boolean', 'int', 'double', 'string',
+ 'boolean_list', 'int_list', 'double_list',
+ 'string_list']:
self.test_config(type_name)
finally:
# If we started logged out, log back out.
« no previous file with comments | « client/deps/ibusclient/src/ibusclient.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698