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

Unified Diff: tests/test_xtm_translations_utils.py

Issue 29923555: Noissue - Switch to binary operator after line break standard following PEP8 and Flake8 (Closed) Base URL: https://hg.adblockplus.org/cms
Patch Set: Created Oct. 24, 2018, 2:02 p.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 | « tests/test_additional_paths.py ('k') | tox.ini » ('j') | tox.ini » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/test_xtm_translations_utils.py
===================================================================
--- a/tests/test_xtm_translations_utils.py
+++ b/tests/test_xtm_translations_utils.py
@@ -77,18 +77,17 @@
def test_sanitize_project_name_characters():
"""Test if invalid name characters are replaced as expected."""
test_in = '{0}{1}{0}'.format(
'test', ''.join(const.ProjectName.INVALID_CHARS),
)
exp_out = '{0}{1}{0}'.format(
'test',
- const.ProjectName.NAME_WILDCARD *
- len(const.ProjectName.INVALID_CHARS),
+ const.ProjectName.NAME_WILDCARD * len(const.ProjectName.INVALID_CHARS),
)
assert exp_out == utils.sanitize_project_name(test_in)
def test_sanitize_project_name_length():
"""Test if names that are too long are truncated as expected."""
test_in = 'a' * (const.ProjectName.MAX_LENGTH + 10)
« no previous file with comments | « tests/test_additional_paths.py ('k') | tox.ini » ('j') | tox.ini » ('J')

Powered by Google App Engine
This is Rietveld