Chromium Code Reviews| Index: PRESUBMIT_test.py |
| diff --git a/PRESUBMIT_test.py b/PRESUBMIT_test.py |
| index 2aed5dfc1b4e074b5efa131b74da0000f4c891fc..11b14d190af8d912be4c5d7851c6d825b439c9fd 100755 |
| --- a/PRESUBMIT_test.py |
| +++ b/PRESUBMIT_test.py |
| @@ -413,6 +413,58 @@ class InvalidOSMacroNamesTest(unittest.TestCase): |
| self.assertEqual(0, len(errors)) |
| +class InvalidOverideAndFinalTest(unittest.TestCase): |
| + def testValidOverrideConstructs(self): |
| + mock_input_api = MockInputApi() |
| + lines = ['foo1() override;', |
| + 'foo2() final;', |
| + '#DEFINE OVERRIDE_METHOD_OVERLOAD', |
| + '#DEFINE FINAL_METHOD', |
|
Mike West
2014/10/15 13:10:03
Nit: For completeness, please add test cases for "
MRV
2014/10/15 13:34:46
Done.
|
| + '#endif // FINAL_METHOD', |
| + '#endif // OVERRIDE_METHOD_OVERLOAD'] |
| + mock_file_h = MockFile('something.h', lines) |
| + mock_input_api.files = [mock_file_h] |
| + errors = PRESUBMIT._CheckForOverrideAndFinalRules(mock_input_api, |
| + MockOutputApi()) |
| + self.assertEqual(0, len(errors)) |
| + |
| + def testInvalidOverrideConstructsInHeaders(self): |
| + mock_input_api = MockInputApi() |
| + lines = ['foo1() OVERRIDE;'] |
| + mock_file_h = MockFile('something.h', lines) |
| + mock_input_api.files = [mock_file_h] |
| + errors = PRESUBMIT._CheckForOverrideAndFinalRules(mock_input_api, |
| + MockOutputApi()) |
| + self.assertEqual(1, len(errors)) |
| + |
| + def testInvalidOverrideConstructsInCpp(self): |
| + mock_input_api = MockInputApi() |
| + lines = ['foo2() FINAL;'] |
| + mock_file_cpp= MockFile('something.cpp', lines) |
| + mock_input_api.files = [mock_file_cpp] |
| + errors = PRESUBMIT._CheckForOverrideAndFinalRules(mock_input_api, |
| + MockOutputApi()) |
| + self.assertEqual(1, len(errors)) |
| + |
| + def testInvalidOverrideConstructsInCc(self): |
| + mock_input_api = MockInputApi() |
| + lines = ['foo3() override FINAL;'] |
| + mock_file_cc = MockFile('something.cc', lines) |
| + mock_input_api.files = [mock_file_cc] |
| + errors = PRESUBMIT._CheckForOverrideAndFinalRules(mock_input_api, |
| + MockOutputApi()) |
| + self.assertEqual(1, len(errors)) |
| + |
| + def testInvalidOverrideConstructsInMm(self): |
| + mock_input_api = MockInputApi() |
| + lines = ['foo4() OVERRIDE final;'] |
| + mock_file_mm = MockFile('something.mm', lines) |
| + mock_input_api.files = [mock_file_mm] |
| + errors = PRESUBMIT._CheckForOverrideAndFinalRules(mock_input_api, |
| + MockOutputApi()) |
| + self.assertEqual(1, len(errors)) |
| + |
| + |
| class InvalidIfDefinedMacroNamesTest(unittest.TestCase): |
| def testInvalidIfDefinedMacroNames(self): |
| lines = ['#if defined(TARGET_IPHONE_SIMULATOR)', |