[FEATURE] Adds the unit-test for case "NOLINT, NOLINTNEXTLINE works for };".

piotr-semenov · piotr-semenov · commit a3c36d93d236 · 2016-06-30T11:45:37.000+03:00
diff --git a/cpplint/cpplint_unittest.py b/cpplint/cpplint_unittest.py
@@ -521,6 +521,21 @@ def testErrorSuppression(self):
                              ''],
                             error_collector)
     self.assertEquals('', error_collector.Results())
+    # NOLINT, NOLINTNEXTLINE silences the readability/braces warning for "};".
+    error_collector = ErrorCollector(self.assert_)
+    cpplint.ProcessFileData('test.cc', 'cc',
+                            ['// Copyright 2014 Your Company.',
+                             'for (int i = 0; i != 100; ++i) {',
+                             '\tstd::cout << i << std::endl;',
+                             '};  // NOLINT',
+                             'for (int i = 0; i != 100; ++i) {',
+                             '\tstd::cout << i << std::endl;',
+                             '// NOLINTNEXTLINE',
+                             '};',
+                             '//  LINT_KERNEL_FILE',
+                             ''],
+                            error_collector)
+    self.assertEquals('', error_collector.Results())
 
   # Test Variable Declarations.
   def testVariableDeclarations(self):
