/webkit-efl/Tools/Scripts/webkitpy/common/watchlist/watchlistparser_unittest.py
https://review.tizen.org/git/ · Python · 259 lines · 196 code · 35 blank · 28 comment · 0 complexity · 6e87e79de88f630155fa1c8d0b26f514 MD5 · raw file
- # Copyright (C) 2011 Google Inc. All rights reserved.
- #
- # Redistribution and use in source and binary forms, with or without
- # modification, are permitted provided that the following conditions are
- # met:
- #
- # * Redistributions of source code must retain the above copyright
- # notice, this list of conditions and the following disclaimer.
- # * Redistributions in binary form must reproduce the above
- # copyright notice, this list of conditions and the following disclaimer
- # in the documentation and/or other materials provided with the
- # distribution.
- # * Neither the name of Google Inc. nor the names of its
- # contributors may be used to endorse or promote products derived from
- # this software without specific prior written permission.
- #
- # THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- # "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- # LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- # A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- # OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- # SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- # LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- # DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- # THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- '''Unit tests for watchlistparser.py.'''
- import logging
- import sys
- from webkitpy.common import webkitunittest
- from webkitpy.common.system.outputcapture import OutputCapture
- from webkitpy.common.watchlist.watchlistparser import WatchListParser
- class WatchListParserTest(webkitunittest.TestCase):
- def setUp(self):
- webkitunittest.TestCase.setUp(self)
- self._watch_list_parser = WatchListParser()
- def test_bad_section(self):
- watch_list = ('{"FOO": {}}')
- OutputCapture().assert_outputs(self, self._watch_list_parser.parse, args=[watch_list],
- expected_logs='Unknown section "FOO" in watch list.\n')
- def test_section_typo(self):
- watch_list = ('{"DEFINTIONS": {}}')
- OutputCapture().assert_outputs(self, self._watch_list_parser.parse, args=[watch_list],
- expected_logs='Unknown section "DEFINTIONS" in watch list.'
- + '\n\nPerhaps it should be DEFINITIONS.\n')
- def test_bad_definition(self):
- watch_list = (
- '{'
- ' "DEFINITIONS": {'
- ' "WatchList1|A": {'
- ' "filename": r".*\\MyFileName\\.cpp",'
- ' },'
- ' },'
- '}')
- OutputCapture().assert_outputs(self, self._watch_list_parser.parse, args=[watch_list],
- expected_logs='Invalid character "|" in definition "WatchList1|A".\n')
- def test_bad_filename_regex(self):
- watch_list = (
- '{'
- ' "DEFINITIONS": {'
- ' "WatchList1": {'
- ' "filename": r"*",'
- ' "more": r"RefCounted",'
- ' },'
- ' },'
- ' "CC_RULES": {'
- ' "WatchList1": ["levin@chromium.org"],'
- ' },'
- '}')
- OutputCapture().assert_outputs(self, self._watch_list_parser.parse, args=[watch_list],
- expected_logs='The regex "*" is invalid due to "nothing to repeat".\n')
- def test_bad_more_regex(self):
- watch_list = (
- '{'
- ' "DEFINITIONS": {'
- ' "WatchList1": {'
- ' "filename": r"aFileName\\.cpp",'
- ' "more": r"*",'
- ' },'
- ' },'
- ' "CC_RULES": {'
- ' "WatchList1": ["levin@chromium.org"],'
- ' },'
- '}')
- OutputCapture().assert_outputs(self, self._watch_list_parser.parse, args=[watch_list],
- expected_logs='The regex "*" is invalid due to "nothing to repeat".\n')
- def test_bad_match_type(self):
- watch_list = (
- '{'
- ' "DEFINITIONS": {'
- ' "WatchList1": {'
- ' "nothing_matches_this": r".*\\MyFileName\\.cpp",'
- ' "filename": r".*\\MyFileName\\.cpp",'
- ' },'
- ' },'
- ' "CC_RULES": {'
- ' "WatchList1": ["levin@chromium.org"],'
- ' },'
- '}')
- OutputCapture().assert_outputs(self, self._watch_list_parser.parse, args=[watch_list],
- expected_logs='Unknown pattern type "nothing_matches_this" in definition "WatchList1".\n')
- def test_match_type_typo(self):
- watch_list = (
- '{'
- ' "DEFINITIONS": {'
- ' "WatchList1": {'
- ' "iflename": r".*\\MyFileName\\.cpp",'
- ' "more": r"RefCounted",'
- ' },'
- ' },'
- ' "CC_RULES": {'
- ' "WatchList1": ["levin@chromium.org"],'
- ' },'
- '}')
- OutputCapture().assert_outputs(self, self._watch_list_parser.parse, args=[watch_list],
- expected_logs='Unknown pattern type "iflename" in definition "WatchList1".'
- + '\n\nPerhaps it should be filename.\n')
- def test_empty_definition(self):
- watch_list = (
- '{'
- ' "DEFINITIONS": {'
- ' "WatchList1": {'
- ' },'
- ' },'
- ' "CC_RULES": {'
- ' "WatchList1": ["levin@chromium.org"],'
- ' },'
- '}')
- OutputCapture().assert_outputs(self, self._watch_list_parser.parse, args=[watch_list],
- expected_logs='The definition "WatchList1" has no patterns, so it should be deleted.\n')
- def test_empty_cc_rule(self):
- watch_list = (
- '{'
- ' "DEFINITIONS": {'
- ' "WatchList1": {'
- ' "filename": r".*\\MyFileName\\.cpp",'
- ' },'
- ' },'
- ' "CC_RULES": {'
- ' "WatchList1": [],'
- ' },'
- '}')
- OutputCapture().assert_outputs(self, self._watch_list_parser.parse, args=[watch_list],
- expected_logs='A rule for definition "WatchList1" is empty, so it should be deleted.\n'
- + 'The following definitions are not used and should be removed: WatchList1\n')
- def test_cc_rule_with_invalid_email(self):
- watch_list = (
- '{'
- ' "DEFINITIONS": {'
- ' "WatchList1": {'
- ' "filename": r".*\\MyFileName\\.cpp",'
- ' },'
- ' },'
- ' "CC_RULES": {'
- ' "WatchList1": ["levin+bad+email@chromium.org"],'
- ' },'
- '}')
- OutputCapture().assert_outputs(self, self._watch_list_parser.parse, args=[watch_list],
- expected_logs='The email alias levin+bad+email@chromium.org which is'
- + ' in the watchlist is not listed as a contributor in committers.py\n')
- def test_empty_message_rule(self):
- watch_list = (
- '{'
- ' "DEFINITIONS": {'
- ' "WatchList1": {'
- ' "filename": r".*\\MyFileName\\.cpp",'
- ' },'
- ' },'
- ' "MESSAGE_RULES": {'
- ' "WatchList1": ['
- ' ],'
- ' },'
- '}')
- OutputCapture().assert_outputs(self, self._watch_list_parser.parse, args=[watch_list],
- expected_logs='A rule for definition "WatchList1" is empty, so it should be deleted.\n'
- + 'The following definitions are not used and should be removed: WatchList1\n')
- def test_unused_defintion(self):
- watch_list = (
- '{'
- ' "DEFINITIONS": {'
- ' "WatchList1": {'
- ' "filename": r".*\\MyFileName\\.cpp",'
- ' },'
- ' },'
- '}')
- OutputCapture().assert_outputs(self, self._watch_list_parser.parse, args=[watch_list],
- expected_logs='The following definitions are not used and should be removed: WatchList1\n')
- def test_cc_rule_with_undefined_defintion(self):
- watch_list = (
- '{'
- ' "CC_RULES": {'
- ' "WatchList1": ["levin@chromium.org"]'
- ' },'
- '}')
- OutputCapture().assert_outputs(self, self._watch_list_parser.parse, args=[watch_list],
- expected_logs='In section "CC_RULES", the following definitions are not used and should be removed: WatchList1\n')
- def test_message_rule_with_undefined_defintion(self):
- watch_list = (
- '{'
- ' "MESSAGE_RULES": {'
- ' "WatchList1": ["The message."]'
- ' },'
- '}')
- OutputCapture().assert_outputs(self, self._watch_list_parser.parse, args=[watch_list],
- expected_logs='In section "MESSAGE_RULES", the following definitions are not used and should be removed: WatchList1\n')
- def test_cc_rule_with_undefined_defintion_with_suggestion(self):
- watch_list = (
- '{'
- ' "DEFINITIONS": {'
- ' "WatchList1": {'
- ' "filename": r".*\\MyFileName\\.cpp",'
- ' },'
- ' },'
- ' "CC_RULES": {'
- ' "WatchList": ["levin@chromium.org"]'
- ' },'
- ' "MESSAGE_RULES": {'
- ' "WatchList1": ["levin@chromium.org"]'
- ' },'
- '}')
- OutputCapture().assert_outputs(self, self._watch_list_parser.parse, args=[watch_list],
- expected_logs='In section "CC_RULES", the following definitions are not used and should be removed: WatchList'
- + '\n\nPerhaps it should be WatchList1.\n')