Skip to content

Commit 94ff257

Browse files
Resolve linting messages in readthedocs.rtd_tests.*
1 parent dc6a2fe commit 94ff257

File tree

5 files changed

+12
-9
lines changed

5 files changed

+12
-9
lines changed

readthedocs/rtd_tests/base.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,9 @@ def post_step(self, step, **data):
6161
self.assertIsNotNone(resp)
6262
return resp
6363

64-
def assertWizardResponse(self, response, step=None):
64+
# We use camelCase on purpose here to conform with unittest's naming
65+
# conventions.
66+
def assertWizardResponse(self, response, step=None): # noqa
6567
'''Assert successful wizard response'''
6668
# Is is the last form
6769
if step is None:
@@ -84,7 +86,9 @@ def assertWizardResponse(self, response, step=None):
8486
self.assertIn('{0}-current_step'.format(self.wizard_class_slug),
8587
response.content)
8688

87-
def assertWizardFailure(self, response, field, match=None):
89+
# We use camelCase on purpose here to conform with unittest's naming
90+
# conventions.
91+
def assertWizardFailure(self, response, field, match=None): # noqa
8892
'''Assert field threw a validation error
8993
9094
response

readthedocs/rtd_tests/factories/comments_factories.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,12 @@ class DocumentNodeFactory(factory.DjangoModelFactory):
2323
page = "page-about-nothing"
2424

2525
@classmethod
26-
def _create(self, *args, **kwargs):
26+
def _create(cls, *args, **kwargs):
2727
if not kwargs.get('hash'):
2828
kwargs['hash'] = random.getrandbits(128)
2929
if not kwargs.get('commit'):
3030
kwargs['commit'] = random.getrandbits(128)
31-
return super(DocumentNodeFactory, self)._create(*args, **kwargs)
31+
return super(DocumentNodeFactory, cls)._create(*args, **kwargs)
3232

3333

3434
class DocumentCommentFactory(factory.DjangoModelFactory):

readthedocs/rtd_tests/factories/general_factories.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ class UserFactory(factory.Factory):
77

88
username = factory.Sequence(lambda n: 'joe %s' % n)
99

10-
1110
@classmethod
1211
def _prepare(cls, create, password="password_sentinel", **kwargs):
1312
'''
Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +0,0 @@
1-
#.

readthedocs/rtd_tests/mocks/mock_api.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,13 @@
44

55
# Mock tastypi API.
66

7+
78
class ProjectData(object):
89
def get(self):
910
return dict()
1011

1112

12-
def mockVersion(repo):
13+
def mock_version(repo):
1314
class MockVersion(object):
1415
def __init__(self, x=None):
1516
pass
@@ -65,13 +66,13 @@ def get(self, **kwargs):
6566
if 'slug' in kwargs:
6667
return {'objects': [version], 'project': project}
6768
else:
68-
return version
69+
return version
6970
return MockVersion
7071

7172

7273
class MockApi(object):
7374
def __init__(self, repo):
74-
self.version = mockVersion(repo)
75+
self.version = mock_version(repo)
7576

7677
def project(self, x):
7778
return ProjectData()

0 commit comments

Comments
 (0)