diff options
| author |   SerenaFeng <feng.xiaowei@zte.com.cn> | 2017-02-20 14:32:29 +0800 |
|---|---|---|
| committer | Serena Feng <feng.xiaowei@zte.com.cn> | 2017-02-21 03:09:59 +0000 |
| commit | fd9becfdd2ac98ad55541987a2104ab46ca8fc99 (patch) | |
| tree | 90c37c9a854b1e60c1d3234a7483810f7bf4048b /testapi/opnfv_testapi/common/config.py | |
| parent | 5b0a40fe23993b7a799279290dabe767c2702bd6 (diff) | |
fix violation of import rule in TestAPI
TestAPI violate import rule of OpenStack Style a lot, like
Do not import objects, only modules
Do not import more than one module per line
Alphabetically order your imports by the full module path
This patch is mean to fix them.
Change-Id: Ia05b944a74b3f443c9101a9840f613f9a6f5ae49
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>
Diffstat (limited to 'testapi/opnfv_testapi/common/config.py')
| -rw-r--r-- | testapi/opnfv_testapi/common/config.py | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/testapi/opnfv_testapi/common/config.py b/testapi/opnfv_testapi/common/config.py index ecab88a..82d9c4d 100644 --- a/testapi/opnfv_testapi/common/config.py +++ b/testapi/opnfv_testapi/common/config.py @@ -7,9 +7,7 @@ # http://www.apache.org/licenses/LICENSE-2.0 # feng.xiaowei@zte.com.cn remove prepare_put_request 5-30-2016 ############################################################################## - - -from ConfigParser import SafeConfigParser, NoOptionError +import ConfigParser class ParseError(Exception): @@ -42,7 +40,7 @@ class APIConfig: def _get_parameter(self, section, param): try: return self._parser.get(section, param) - except NoOptionError: + except ConfigParser.NoOptionError: raise ParseError("[%s.%s] parameter not found" % (section, param)) def _get_int_parameter(self, section, param): @@ -68,7 +66,7 @@ class APIConfig: if config_location is None: config_location = obj._default_config_location - obj._parser = SafeConfigParser() + obj._parser = ConfigParser.SafeConfigParser() obj._parser.read(config_location) if not obj._parser: raise ParseError("%s not found" % config_location) |