Fix H404/405 violations for unit tests
There is a lot of H404/405 violations in Tempest now, and that leads difficult to migrate the code to tempest-lib or the other projects' repos. This patch fixes these violations for unit tests. Change-Id: Ib2418300b2679bee477c84cd0943382d031ff4a9
This commit is contained in:
parent
cb67d2dd46
commit
c864f59913
|
@ -26,8 +26,7 @@ class TestCase(base.BaseTestCase):
|
||||||
self.stubs = mox_fixture.stubs
|
self.stubs = mox_fixture.stubs
|
||||||
|
|
||||||
def patch(self, target, **kwargs):
|
def patch(self, target, **kwargs):
|
||||||
"""
|
"""Returns a started `mock.patch` object for the supplied target.
|
||||||
Returns a started `mock.patch` object for the supplied target.
|
|
||||||
|
|
||||||
The caller may then call the returned patcher to create a mock object.
|
The caller may then call the returned patcher to create a mock object.
|
||||||
|
|
||||||
|
|
|
@ -50,7 +50,8 @@ class fake_httplib2(object):
|
||||||
class fake_httplib(object):
|
class fake_httplib(object):
|
||||||
def __init__(self, headers, body=None,
|
def __init__(self, headers, body=None,
|
||||||
version=1.0, status=200, reason="Ok"):
|
version=1.0, status=200, reason="Ok"):
|
||||||
"""
|
"""Initialization of fake httplib
|
||||||
|
|
||||||
:param headers: dict representing HTTP response headers
|
:param headers: dict representing HTTP response headers
|
||||||
:param body: file-like object
|
:param body: file-like object
|
||||||
:param version: HTTP Version
|
:param version: HTTP Version
|
||||||
|
|
|
@ -25,8 +25,7 @@ LOG = logging.getLogger(__name__)
|
||||||
|
|
||||||
|
|
||||||
class StressFrameworkTest(base.TestCase):
|
class StressFrameworkTest(base.TestCase):
|
||||||
"""Basic test for the stress test framework.
|
"""Basic test for the stress test framework."""
|
||||||
"""
|
|
||||||
|
|
||||||
def _cmd(self, cmd, param):
|
def _cmd(self, cmd, param):
|
||||||
"""Executes specified command."""
|
"""Executes specified command."""
|
||||||
|
|
|
@ -17,7 +17,8 @@ from tempest.tests import base
|
||||||
|
|
||||||
|
|
||||||
class HackingTestCase(base.TestCase):
|
class HackingTestCase(base.TestCase):
|
||||||
"""
|
"""Test class for hacking rule
|
||||||
|
|
||||||
This class tests the hacking checks in tempest.hacking.checks by passing
|
This class tests the hacking checks in tempest.hacking.checks by passing
|
||||||
strings to the check methods like the pep8/flake8 parser would. The parser
|
strings to the check methods like the pep8/flake8 parser would. The parser
|
||||||
loops over each line in the file and then passes the parameters to the
|
loops over each line in the file and then passes the parameters to the
|
||||||
|
|
Loading…
Reference in New Issue