x/deb-python-tempest-lib
Code Issues Proposed changes

Add a new exception for invalid structure

Browse Source 
This commit fixes too broad exception in output_parser. Otherwise, we
can't assert the Exception in unit tests.

Partially implements bp unit-tests

Change-Id: I9e45571dbf964a37dd9d0249a6f662216d5d046c
This commit is contained in:
Masayuki Igawa
2014-03-17 17:55:23 +09:00
parent f052944f46
commit 38355f253d
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
tempest/cli/output_parser.py
View File

@@ -17,6 +17,7 @@
import re import re
from tempest import exceptions
from tempest.openstack.common import log as logging from tempest.openstack.common import log as logging
@@ -37,7 +38,7 @@ def details_multiple(output_lines, with_label=False):
for table_ in tables_: for table_ in tables_:
if 'Property' not in table_['headers'] \ if 'Property' not in table_['headers'] \
or 'Value' not in table_['headers']: or 'Value' not in table_['headers']:
raise Exception('Invalid structure of table with details') raise exceptions.InvalidStructure()
item = {} item = {}
for value in table_['values']: for value in table_['values']:
item[value[0]] = value[1] item[value[0]] = value[1]