Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: tests/test_translations.py

Issue 29569623: Noissue - Added flake8-docstrings (cms) (Closed)
Patch Set: Created Oct. 9, 2017, 1:30 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | tox.ini » ('j') | tox.ini » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/test_translations.py
===================================================================
--- a/tests/test_translations.py
+++ b/tests/test_translations.py
@@ -46,8 +46,7 @@
def test_sync(temp_site, make_intercept, make_crowdin_zip, expect_requests):
translate.crowdin_sync(temp_site, 'test_key')
- for (url, data), (expect_url, expect_data) in zip(
- app.request_log,
+ for (url, data), (expect_url, expect_data) in zip(app.request_log,
Sebastian Noack 2017/10/09 01:59:12 For some reason flake8 didn't detect the redundant
expect_requests):
assert expect_url in url
assert expect_data in data
« no previous file with comments | « no previous file | tox.ini » ('j') | tox.ini » ('J')

Powered by Google App Engine
This is Rietveld