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

Unified Diff: sitescripts/stats/test/common.py

Issue 29345242: Noissue - Adapt quotes for compliance with our coding style in sitescripts (Closed)
Patch Set: Fixed raw string Created May 30, 2016, 8:47 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
Index: sitescripts/stats/test/common.py
===================================================================
--- a/sitescripts/stats/test/common.py
+++ b/sitescripts/stats/test/common.py
@@ -24,20 +24,20 @@
def test_fileencoding(self):
tests = [
- ("foo_bar", True),
- ("1234.txt", True),
- ("xYz.DAT", True),
- ("foo/bar.txt", False),
- ("foo/bar+bas-xyz.txt", False),
- (u"foo\u1234-bar\u4321", False),
- ("foobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobar", u"foobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobar\u2026")
+ ('foo_bar', True),
+ ('1234.txt', True),
+ ('xYz.DAT', True),
+ ('foo/bar.txt', False),
+ ('foo/bar+bas-xyz.txt', False),
+ (u'foo\u1234-bar\u4321', False),
+ ('foobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobar', u'foobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobarfoobar\u2026')
]
for name, expect_identical in tests:
path = common.filename_encode(name)
if expect_identical is True:
self.assertEqual(path, name, "Encoding '%s' shouldn't change string" % name)
else:
- self.assertTrue(re.match(r"^[\w\.\-]*$", path), "Encoding '%s' should replace all special characters" % name)
+ self.assertTrue(re.match(r'^[\w\.\-]*$', path), "Encoding '%s' should replace all special characters" % name)
if isinstance(expect_identical, basestring):
self.assertEqual(common.filename_decode(path), expect_identical, "Encoding and decoding '%s' should produce a truncated string as result" % name)

Powered by Google App Engine
This is Rietveld