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

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

Issue 29344600: Noissue - Got rid of E129, E704, E712, E731 and F821 errors (Closed)
Patch Set: Created May 26, 2016, 12:49 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 | « sitescripts/reports/utils.py ('k') | sitescripts/subscriptions/bin/updateStatusPage.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sitescripts/stats/test/common.py
===================================================================
--- a/sitescripts/stats/test/common.py
+++ b/sitescripts/stats/test/common.py
@@ -36,7 +36,7 @@
]
for name, expect_identical in tests:
path = common.filename_encode(name)
- if expect_identical == True:
+ 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)
« no previous file with comments | « sitescripts/reports/utils.py ('k') | sitescripts/subscriptions/bin/updateStatusPage.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld