Change the key/val delimiter to | by default. Make it an option.
Signed-off-by: Jean-Marc Eurin <jmeurin@google.com>
git-svn-id: http://test.kernel.org/svn/autotest/trunk@2783 592f7852-d20e-0410-864c-8624ca9c26a4
diff --git a/cli/topic_common_unittest.py b/cli/topic_common_unittest.py
index 30882fc..99b516d 100755
--- a/cli/topic_common_unittest.py
+++ b/cli/topic_common_unittest.py
@@ -427,6 +427,7 @@
self.assertEqual({'mlist': flist,
'web_server': 'fooweb',
'parse': True,
+ 'parse_delim': '|',
'kill_on_failure': True,
'verbose': False,
'debug': False}, options)
@@ -449,6 +450,7 @@
self.assertEqual({'mlist': flist,
'web_server': 'fooweb',
'parse': True,
+ 'parse_delim': '|',
'kill_on_failure': True,
'verbose': False,
'debug': True}, options)
@@ -484,6 +486,7 @@
'user': 'myuser,youruser',
'web_server': None,
'parse': True,
+ 'parse_delim': '|',
'kill_on_failure': True,
'verbose': False,
'debug': True}, options)
@@ -519,6 +522,7 @@
'user': 'myuser,youruser',
'web_server': None,
'parse': True,
+ 'parse_delim': '|',
'kill_on_failure': True,
'verbose': False,
'debug': True}, options)
@@ -554,6 +558,7 @@
'user': 'myuser,youruser',
'web_server': None,
'parse': True,
+ 'parse_delim': '|',
'kill_on_failure': True,
'verbose': False,
'debug': True}, options)
@@ -587,6 +592,7 @@
'user': None,
'web_server': None,
'parse': False,
+ 'parse_delim': '|',
'kill_on_failure': True,
'verbose': False,
'debug': False}, options)
@@ -618,6 +624,7 @@
'user': 'myuser,youruser',
'web_server': None,
'parse': True,
+ 'parse_delim': '|',
'kill_on_failure': True,
'verbose': False,
'debug': True}, options)
@@ -647,6 +654,7 @@
'acl': 'acl1,acl2',
'web_server': None,
'parse': False,
+ 'parse_delim': '|',
'kill_on_failure': True,
'verbose': False,
'debug': False}, options)
@@ -667,6 +675,7 @@
self.assertEqual({'user': 'myuser,youruser',
'web_server': None,
'parse': False,
+ 'parse_delim': '|',
'kill_on_failure': False,
'verbose': False,
'debug': False}, options)
@@ -701,10 +710,12 @@
def test_parse_all_set(self):
sys.argv = ['atest', '--web', 'fooweb', '--parse', '--debug',
- '--kill-on-failure', '--verbose', 'left1', 'left2']
+ '--kill-on-failure', '--verbose', 'left1', 'left2',
+ '--parse-delim', '?']
(options, leftover) = self.atest.parse()
self.assertEqual({'web_server': 'fooweb',
'parse': True,
+ 'parse_delim': '?',
'kill_on_failure': True,
'verbose': True,
'debug': True}, options)
@@ -813,9 +824,9 @@
def test_print_fields_parse_all_fields(self):
self.__test_print_fields_parse(['hostname', 'platform', 'locked'],
- 'Host=h0:Platform=p0:'
+ 'Host=h0|Platform=p0|'
'Locked=True\n'
- 'Host=h1:Platform=p1:'
+ 'Host=h1|Platform=p1|'
'Locked=False\n')
@@ -847,13 +858,22 @@
def test_print_table_parse_all_fields(self):
self.__test_print_table_parse(['hostname', 'platform',
'locked'],
- 'Host=h0:Platform=p0:Locked=True\n'
- 'Host=h1:Platform=p1:Locked=False\n')
+ 'Host=h0|Platform=p0|Locked=True\n'
+ 'Host=h1|Platform=p1|Locked=False\n')
+
+
+ def test_print_table_parse_all_fields(self):
+ self.atest.parse_delim = '?'
+ self.__test_print_table_parse(['hostname', 'platform',
+ 'locked'],
+ 'Host=h0?Platform=p0?Locked=True\n'
+ 'Host=h1?Platform=p1?Locked=False\n')
+
def test_print_table_parse_empty_fields(self):
self.__test_print_fields(self.atest.print_table_parse,
- 'Host=h0:Platform=p0\n'
- 'Host=h1:Platform=p1:Labels=l2, l3\n',
+ 'Host=h0|Platform=p0\n'
+ 'Host=h1|Platform=p1|Labels=l2, l3\n',
items=[{'hostname': 'h0',
'platform': 'p0',
'labels': [],
@@ -915,8 +935,8 @@
def test_print_by_ids_parse_all_fields(self):
- self.__test_print_by_ids_parse('Id=id0:Name=name0:'
- 'Id=id1:Name=name1\n')
+ self.__test_print_by_ids_parse('Id=id0|Name=name0|'
+ 'Id=id1|Name=name1\n')
if __name__ == '__main__':