diff options
author | Cédric Ollivier <cedric.ollivier@orange.com> | 2018-06-19 07:14:42 +0200 |
---|---|---|
committer | Cédric Ollivier <cedric.ollivier@orange.com> | 2018-06-19 07:16:33 +0200 |
commit | a1a85f7cc121b1c1176f2b79b22f4e4bca43177a (patch) | |
tree | b602db75f9577576b4b8fb84dc01e33b24c3dcdf /functest/opnfv_tests/openstack/rally | |
parent | f1353fff60344b01c588d9724b3b1de819cbe661 (diff) |
Fix pylint errors
They are highlighted by the last pylint version.
Change-Id: I473212a413013492716993fb536903b13bff66ed
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
Diffstat (limited to 'functest/opnfv_tests/openstack/rally')
-rw-r--r-- | functest/opnfv_tests/openstack/rally/rally.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/functest/opnfv_tests/openstack/rally/rally.py b/functest/opnfv_tests/openstack/rally/rally.py index cca198853..c3c53fbca 100644 --- a/functest/opnfv_tests/openstack/rally/rally.py +++ b/functest/opnfv_tests/openstack/rally/rally.py @@ -286,7 +286,7 @@ class RallyBase(testcase.TestCase): self.excl_scenario())) if black_tests: - LOGGER.debug("Blacklisted tests: " + str(black_tests)) + LOGGER.debug("Blacklisted tests: %s", str(black_tests)) include = True for cases_line in cases_file: @@ -358,9 +358,9 @@ class RallyBase(testcase.TestCase): # parse JSON operation result if self.task_succeed(json_results): - LOGGER.info('Test scenario: "{}" OK.'.format(test_name) + "\n") + LOGGER.info('Test scenario: "%s" OK.', test_name) else: - LOGGER.info('Test scenario: "{}" Failed.'.format(test_name) + "\n") + LOGGER.info('Test scenario: "%s" Failed.', test_name) def _run_task(self, test_name): """Run a task.""" @@ -369,7 +369,7 @@ class RallyBase(testcase.TestCase): task_file = os.path.join(self.RALLY_DIR, 'task.yaml') if not os.path.exists(task_file): LOGGER.error("Task file '%s' does not exist.", task_file) - raise Exception("Task file '%s' does not exist.", task_file) + raise Exception("Task file '{}' does not exist.".format(task_file)) file_name = self._prepare_test_list(test_name) if self.file_is_empty(file_name): |