Skip to content

Commit a6f9151

Browse files
author
Rayan Chikhi
committed
only pep8 changes
1 parent ec6bdc6 commit a6f9151

3 files changed

Lines changed: 120 additions & 100 deletions

File tree

codespeed/api.py

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -224,17 +224,23 @@ def _populate_obj_by_data(self):
224224
"""
225225
def populate(key):
226226
return {'project': lambda: ProjectResource().get_via_uri(
227-
self.data['project'], request=self.request),
227+
self.data['project'], request=self.request),
228+
228229
'executable': lambda: ExecutableResource().get_via_uri(
229-
self.data['executable'], request=self.request),
230+
self.data['executable'], request=self.request),
231+
230232
'benchmark': lambda: BenchmarkResource().get_via_uri(
231-
self.data['benchmark'], request=self.request),
233+
self.data['benchmark'], request=self.request),
234+
232235
'environment': lambda: EnvironmentResource().get_via_uri(
233-
self.data['environment'], request=self.request),
236+
self.data['environment'], request=self.request),
237+
234238
'branch': lambda: BranchResource().get_via_uri(
235-
self.data['branch'], request=self.request),
239+
self.data['branch'], request=self.request),
240+
236241
'revision': lambda: RevisionResource().get_via_uri(
237-
self.data['commitid'], request=self.request)}.get(key, None)()
242+
self.data['commitid'],
243+
request=self.request)}.get(key, None)()
238244

239245
try:
240246
self.obj.value = float(self.data['result_value'])
@@ -440,7 +446,8 @@ def post_list(self, request, **kwargs):
440446
format=request.META.get('CONTENT_TYPE', 'application/json')
441447
)
442448
deserialized = self.alter_deserialized_list_data(request, deserialized)
443-
bundle = ResultBundle(request=request, **dict_strip_unicode_keys(deserialized))
449+
bundle = ResultBundle(request=request,
450+
**dict_strip_unicode_keys(deserialized))
444451
self.is_valid(bundle)
445452
updated_bundle = self.obj_create(bundle, request=request)
446453
return HttpCreated(location=self.get_resource_uri(updated_bundle))

codespeed/tests/tests.py

Lines changed: 40 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -26,23 +26,23 @@ def setUp(self):
2626
self.cdate = datetime(
2727
temp.year, temp.month, temp.day, temp.hour, temp.minute, temp.second)
2828
self.data = {
29-
'commitid': '23',
30-
'branch': 'default',
31-
'project': 'MyProject',
32-
'executable': 'myexe O3 64bits',
33-
'benchmark': 'float',
34-
'environment': 'Dual Core',
35-
'result_value': 456,
29+
'commitid': '23',
30+
'branch': 'default',
31+
'project': 'MyProject',
32+
'executable': 'myexe O3 64bits',
33+
'benchmark': 'float',
34+
'environment': 'Dual Core',
35+
'result_value': 456,
3636
}
3737

3838
def test_add_correct_result(self):
3939
"""Add correct result data"""
4040
response = self.client.post(self.path, self.data)
41-
41+
4242
# Check that we get a success response
4343
self.assertEquals(response.status_code, 202)
4444
self.assertEquals(response.content, "Result data saved successfully")
45-
45+
4646
# Check that the data was correctly saved
4747
e = Environment.objects.get(name='Dual Core')
4848
b = Benchmark.objects.get(name='float')
@@ -171,36 +171,36 @@ def setUp(self):
171171
temp = datetime.today()
172172
self.cdate = datetime(
173173
temp.year, temp.month, temp.day, temp.hour, temp.minute, temp.second)
174-
174+
175175
self.data = [
176176
{'commitid': '123',
177-
'project': 'pypy',
178-
'branch': 'default',
179-
'executable': 'pypy-c',
180-
'benchmark': 'Richards',
181-
'environment': 'bigdog',
182-
'result_value': 456},
177+
'project': 'pypy',
178+
'branch': 'default',
179+
'executable': 'pypy-c',
180+
'benchmark': 'Richards',
181+
'environment': 'bigdog',
182+
'result_value': 456},
183183
{'commitid': '456',
184-
'project': 'pypy',
185-
'branch': 'default',
186-
'executable': 'pypy-c',
187-
'benchmark': 'Richards',
188-
'environment': 'bigdog',
189-
'result_value': 457},
184+
'project': 'pypy',
185+
'branch': 'default',
186+
'executable': 'pypy-c',
187+
'benchmark': 'Richards',
188+
'environment': 'bigdog',
189+
'result_value': 457},
190190
{'commitid': '456',
191-
'project': 'pypy',
192-
'branch': 'default',
193-
'executable': 'pypy-c',
194-
'benchmark': 'Richards2',
195-
'environment': 'bigdog',
196-
'result_value': 34},
191+
'project': 'pypy',
192+
'branch': 'default',
193+
'executable': 'pypy-c',
194+
'benchmark': 'Richards2',
195+
'environment': 'bigdog',
196+
'result_value': 34},
197197
{'commitid': '789',
198-
'project': 'pypy',
199-
'branch': 'default',
200-
'executable': 'pypy-c',
201-
'benchmark': 'Richards',
202-
'environment': 'bigdog',
203-
'result_value': 458},
198+
'project': 'pypy',
199+
'branch': 'default',
200+
'executable': 'pypy-c',
201+
'benchmark': 'Richards',
202+
'environment': 'bigdog',
203+
'result_value': 458},
204204
]
205205

206206
def test_add_correct_results(self):
@@ -390,7 +390,8 @@ class TestViewHelpers(TestCase):
390390

391391
def setUp(self):
392392
self.project = Project.objects.create(name='Test')
393-
self.executable = Executable.objects.create(name='TestExecutable', project=self.project)
393+
self.executable = Executable.objects.create(
394+
name='TestExecutable', project=self.project)
394395
self.branch = Branch.objects.create(name='master', project=self.project)
395396

396397
def test_get_baseline_executables(self):
@@ -421,7 +422,8 @@ class ProjectTest(TestCase):
421422
"""Test project model"""
422423

423424
def setUp(self):
424-
self.github_project = Project(repo_type='H', repo_path='https://github.com/tobami/codespeed.git')
425+
self.github_project = Project(repo_type='H',
426+
repo_path='https://github.com/tobami/codespeed.git')
425427
self.git_project = Project(repo_type='G', repo_path='/home/foo/codespeed')
426428

427429
def test_repo_name(self):
@@ -435,7 +437,8 @@ def test_working_copy(self):
435437
"""Test that only projects with local repositories have a working_copy attribute
436438
"""
437439
self.assertEqual(self.git_project.working_copy,
438-
os.path.join(settings.REPOSITORY_BASE_PATH, self.git_project.repo_name))
440+
os.path.join(settings.REPOSITORY_BASE_PATH,
441+
self.git_project.repo_name))
439442

440443
self.assertRaises(
441444
AttributeError, getattr, self.github_project, 'working_copy')

0 commit comments

Comments
 (0)