Merge pull request #28 from m-chaturvedi/py3_support

Add python 3 support to cpplint_unittest

Author: jwnimmer-tri

cpplint/cpplint_unittest.py

@@ -3151,7 +3151,7 @@ def DoTest(self, raw_bytes, has_invalid_utf8):
       error_collector = ErrorCollector(self.assert_)
       cpplint.ProcessFileData(
           'foo.cc', 'cc',
-          unicode(raw_bytes, 'utf8', 'replace').split('\n'),
+          raw_bytes.decode('utf-8', errors='replace').split('\n'),
           error_collector)
       # The warning appears only once.
       self.assertEquals(
@@ -3161,12 +3161,12 @@ def DoTest(self, raw_bytes, has_invalid_utf8):
               ' (or Unicode replacement character).'
               '  [readability/utf8] [5]'))
 
-    DoTest(self, 'Hello world\n', False)
-    DoTest(self, '\xe9\x8e\xbd\n', False)
-    DoTest(self, '\xe9x\x8e\xbd\n', True)
+    DoTest(self, b'Hello world\n', False)
+    DoTest(self, b'\xe9\x8e\xbd\n', False)
+    DoTest(self, b'\xe9x\x8e\xbd\n', True)
     # This is the encoding of the replacement character itself (which
     # you can see by evaluating codecs.getencoder('utf8')(u'\ufffd')).
-    DoTest(self, '\xef\xbf\xbd\n', True)
+    DoTest(self, b'\xef\xbf\xbd\n', True)
 
   def testBadCharacters(self):
     # Test for NUL bytes only
@@ -3184,7 +3184,7 @@ def testBadCharacters(self):
     cpplint.ProcessFileData(
         'nul_utf8.cc', 'cc',
         ['// Copyright 2014 Your Company.',
-         unicode('\xe9x\0', 'utf8', 'replace'), ''],
+        b'\xe9x\0'.decode('utf8', errors='replace'), ''],
         error_collector)
     self.assertEquals(
         error_collector.Results(),
@@ -5810,8 +5810,9 @@ def _runCppLint(self, *args):
 
   def testNonQuietWithErrors(self):
     # This will fail: the test header is missing a copyright and header guard.
-    (return_code, output) = self._runCppLint()
+    (return_code, output_bytes) = self._runCppLint()
     self.assertEquals(1, return_code)
+    output = output_bytes.decode('utf-8')
     # Always-on behavior: Print error messages as they come up.
     self.assertIn("[legal/copyright]", output)
     self.assertIn("[build/header_guard]", output)
@@ -5821,7 +5822,8 @@ def testNonQuietWithErrors(self):
 
   def testQuietWithErrors(self):
     # When there are errors, behavior is identical to not passing --quiet.
-    (return_code, output) = self._runCppLint('--quiet')
+    (return_code, output_bytes) = self._runCppLint('--quiet')
+    output = output_bytes.decode('utf-8')
     self.assertEquals(1, return_code)
     self.assertIn("[legal/copyright]", output)
     self.assertIn("[build/header_guard]", output)
@@ -5831,9 +5833,10 @@ def testQuietWithErrors(self):
 
   def testNonQuietWithoutErrors(self):
     # This will succeed. We filtered out all the known errors for that file.
-    (return_code, output) = self._runCppLint('--filter=' +
-                                                '-legal/copyright,' +
-                                                '-build/header_guard')
+    (return_code, output_bytes) = self._runCppLint('--filter=' +
+                                                   '-legal/copyright,' +
+                                                   '-build/header_guard')
+    output = output_bytes.decode('utf-8')
     self.assertEquals(0, return_code, output)
     # No cpplint errors are printed since there were no errors.
     self.assertNotIn("[legal/copyright]", output)
@@ -5845,10 +5848,11 @@ def testNonQuietWithoutErrors(self):
 
   def testQuietWithoutErrors(self):
     # This will succeed. We filtered out all the known errors for that file.
-    (return_code, output) = self._runCppLint('--quiet',
-                                             '--filter=' +
-                                                 '-legal/copyright,' +
-                                                 '-build/header_guard')
+    (return_code, output_bytes) = self._runCppLint('--quiet',
+                                                   '--filter=' +
+                                                   '-legal/copyright,' +
+                                                   '-build/header_guard')
+    output = output_bytes.decode('utf-8')
     self.assertEquals(0, return_code, output)
     # No cpplint errors are printed since there were no errors.
     self.assertNotIn("[legal/copyright]", output)