From e28a4376cd29c7385e958616f643d508fe75abce Mon Sep 17 00:00:00 2001 From: Sateesh Kumar Date: Sat, 7 Dec 2013 22:25:53 +0530 Subject: [PATCH] Fix pylint warnings --- jenkinsapi/jenkins.py | 8 ++++---- jenkinsapi_tests/unittests/test_jenkins.py | 8 ++++---- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/jenkinsapi/jenkins.py b/jenkinsapi/jenkins.py index c5f06ba..11c65d8 100644 --- a/jenkinsapi/jenkins.py +++ b/jenkinsapi/jenkins.py @@ -318,14 +318,14 @@ class Jenkins(JenkinsBase): def get_executors(self, nodename): url = '%s/computer/%s' % (self.baseurl, nodename) return Executors(url, nodename, self) - + def get_version(self): """ Return version number of Jenkins """ - r = self.requester.get_and_confirm_status(self.baseurl) + response = self.requester.get_and_confirm_status(self.baseurl) try: - return r.headers['X-jenkins'] + return response.headers['X-jenkins'] except KeyError: - msg = '%s is not a valid Jenkins URL.' %(self.baseurl) + msg = '%s is not a valid Jenkins URL.' % (self.baseurl) raise BadURL(msg) diff --git a/jenkinsapi_tests/unittests/test_jenkins.py b/jenkinsapi_tests/unittests/test_jenkins.py index 371167a..0ff7ab9 100644 --- a/jenkinsapi_tests/unittests/test_jenkins.py +++ b/jenkinsapi_tests/unittests/test_jenkins.py @@ -305,7 +305,7 @@ class TestJenkins(unittest.TestCase): @mock.patch.object(JenkinsBase, '_poll') @mock.patch.object(Jenkins, '_poll') - def test_get_version(self, _base_poll,_poll): + def test_get_version(self, _base_poll, _poll): class MockResponse(object): def __init__(self): self.headers = {} @@ -319,7 +319,7 @@ class TestJenkins(unittest.TestCase): @mock.patch.object(JenkinsBase, '_poll') @mock.patch.object(Jenkins, '_poll') - def test_get_version_exception(self, _base_poll,_poll): + def test_get_version_exception(self, _base_poll, _poll): class MockResponse(object): def __init__(self): self.headers = {} @@ -331,8 +331,8 @@ class TestJenkins(unittest.TestCase): requester=mock_requester) with self.assertRaises(BadURL) as context: J.get_version() - msg = '%s is not a valid Jenkins URL.' %(base_url) - self.assertEqual(context.exception.message,msg) + msg = '%s is not a valid Jenkins URL.' % (base_url) + self.assertEqual(context.exception.message, msg) class TestJenkinsURLs(unittest.TestCase): -- 2.7.4