From 8e72a728b75d7724e08fa72829d2446f35af1ec6 Mon Sep 17 00:00:00 2001 From: Prasanna Santhanam Date: Thu, 25 Jul 2013 22:59:21 +0530 Subject: [PATCH] Fix pep8 that broken marvin build 10 days ago :( Signed-off-by: Prasanna Santhanam --- tools/marvin/marvin/cloudstackTestCase.py | 1 + tools/marvin/marvin/cloudstackTestClient.py | 2 +- tools/marvin/marvin/codegenerator.py | 18 +++++++++--------- tools/marvin/marvin/deployDataCenter.py | 3 ++- tools/marvin/marvin/marvinPlugin.py | 2 -- 5 files changed, 13 insertions(+), 13 deletions(-) diff --git a/tools/marvin/marvin/cloudstackTestCase.py b/tools/marvin/marvin/cloudstackTestCase.py index 742363d4ba2..b04342be58c 100644 --- a/tools/marvin/marvin/cloudstackTestCase.py +++ b/tools/marvin/marvin/cloudstackTestCase.py @@ -17,6 +17,7 @@ import unittest + def user(Name, DomainName, AcctType): def wrapper(cls): orig_init = cls.__init__ diff --git a/tools/marvin/marvin/cloudstackTestClient.py b/tools/marvin/marvin/cloudstackTestClient.py index f8c13d4f321..36f7f8d8369 100644 --- a/tools/marvin/marvin/cloudstackTestClient.py +++ b/tools/marvin/marvin/cloudstackTestClient.py @@ -198,4 +198,4 @@ class cloudstackTestClient(object): if self.asyncJobMgr is None: self.asyncJobMgr = asyncJobMgr.asyncJobMgr(self.apiClient, self.dbConnection) - self.asyncJobMgr.submitJobs(jobs, nums_threads, interval) \ No newline at end of file + self.asyncJobMgr.submitJobs(jobs, nums_threads, interval) diff --git a/tools/marvin/marvin/codegenerator.py b/tools/marvin/marvin/codegenerator.py index e37dbcc81b1..96729f6bbfe 100644 --- a/tools/marvin/marvin/codegenerator.py +++ b/tools/marvin/marvin/codegenerator.py @@ -196,15 +196,15 @@ class codeGenerator(object): body += 'return CloudStackAPIClient(copy.copy(self.connection))\n' body += self.newline -# The `id` property will be used to link the test with the cloud resource being created # -# @property -# def id(self): -# return self._id -# -# @id.setter -# def id(self, identifier): -# self._id = identifier - + # The `id` property will be used to link the test with the cloud + # resource being created + # @property + # def id(self): + # return self._id + # + # @id.setter + # def id(self, identifier): + # self._id = identifier body += self.space + '@property' + self.newline body += self.space + 'def id(self):' + self.newline diff --git a/tools/marvin/marvin/deployDataCenter.py b/tools/marvin/marvin/deployDataCenter.py index 2472b2039b3..279d4ee5785 100644 --- a/tools/marvin/marvin/deployDataCenter.py +++ b/tools/marvin/marvin/deployDataCenter.py @@ -159,7 +159,8 @@ specify a valid config file" % cfgFile) secondarycmd.provider = secondary.provider secondarycmd.details = [] - if secondarycmd.provider == 'S3' or secondarycmd.provider == "Swift": + if secondarycmd.provider == 'S3' \ + or secondarycmd.provider == "Swift": for key, value in vars(secondary.details).iteritems(): secondarycmd.details.append({ 'key': key, diff --git a/tools/marvin/marvin/marvinPlugin.py b/tools/marvin/marvin/marvinPlugin.py index 8a548cd63de..29491094db0 100644 --- a/tools/marvin/marvin/marvinPlugin.py +++ b/tools/marvin/marvin/marvinPlugin.py @@ -137,8 +137,6 @@ class MarvinPlugin(Plugin): def _injectClients(self, test): testcaselogger = logging.getLogger("testclient.testcase.%s" % test.__name__) - - self.debug_stream. \ setFormatter(logging. Formatter("%(asctime)s - %(levelname)s - %(name)s" +