| # Copyright (C) 2009 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. |
| |
| from __future__ import with_statement |
| |
| import codecs |
| import os |
| import tempfile |
| import unittest |
| |
| from StringIO import StringIO |
| |
| from webkitpy.common.checkout.changelog import * |
| |
| |
| class ChangeLogTest(unittest.TestCase): |
| |
| _example_entry = u'''2009-08-17 Peter Kasting <pkasting@google.com> |
| |
| Reviewed by Tor Arne Vestb\xf8. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=27323 |
| Only add Cygwin to the path when it isn't already there. This avoids |
| causing problems for people who purposefully have non-Cygwin versions of |
| executables like svn in front of the Cygwin ones in their paths. |
| |
| * DumpRenderTree/win/DumpRenderTree.vcproj: |
| * DumpRenderTree/win/ImageDiff.vcproj: |
| * DumpRenderTree/win/TestNetscapePlugin/TestNetscapePlugin.vcproj: |
| ''' |
| |
| _rolled_over_footer = '== Rolled over to ChangeLog-2009-06-16 ==' |
| |
| # More example text than we need. Eventually we need to support parsing this all and write tests for the parsing. |
| _example_changelog = u"""2009-08-17 Tor Arne Vestb\xf8 <vestbo@webkit.org> |
| |
| <http://webkit.org/b/28393> check-webkit-style: add check for use of std::max()/std::min() instead of MAX()/MIN() |
| |
| Reviewed by David Levin. |
| |
| * Scripts/modules/cpp_style.py: |
| (_ERROR_CATEGORIES): Added 'runtime/max_min_macros'. |
| (check_max_min_macros): Added. Returns level 4 error when MAX() |
| and MIN() macros are used in header files and C++ source files. |
| (check_style): Added call to check_max_min_macros(). |
| * Scripts/modules/cpp_style_unittest.py: Added unit tests. |
| (test_max_macro): Added. |
| (test_min_macro): Added. |
| |
| 2009-08-16 David Kilzer <ddkilzer@apple.com> |
| |
| Backed out r47343 which was mistakenly committed |
| |
| * Scripts/bugzilla-tool: |
| * Scripts/modules/scm.py: |
| |
| 2009-06-18 Darin Adler <darin@apple.com> |
| |
| Rubber stamped by Mark Rowe. |
| |
| * DumpRenderTree/mac/DumpRenderTreeWindow.mm: |
| (-[DumpRenderTreeWindow close]): Resolved crashes seen during regression |
| tests. The close method can be called on a window that's already closed |
| so we can't assert here. |
| |
| == Rolled over to ChangeLog-2009-06-16 == |
| """ |
| |
| def test_latest_entry_parse(self): |
| changelog_contents = u"%s\n%s" % (self._example_entry, self._example_changelog) |
| changelog_file = StringIO(changelog_contents) |
| latest_entry = ChangeLog.parse_latest_entry_from_file(changelog_file) |
| self.assertEquals(latest_entry.contents(), self._example_entry) |
| self.assertEquals(latest_entry.author_name(), "Peter Kasting") |
| self.assertEquals(latest_entry.author_email(), "pkasting@google.com") |
| self.assertEquals(latest_entry.reviewer_text(), u"Tor Arne Vestb\xf8") |
| self.assertTrue(latest_entry.reviewer()) # Make sure that our UTF8-based lookup of Tor works. |
| |
| def test_latest_entry_parse_single_entry(self): |
| changelog_contents = u"%s\n%s" % (self._example_entry, self._rolled_over_footer) |
| changelog_file = StringIO(changelog_contents) |
| latest_entry = ChangeLog.parse_latest_entry_from_file(changelog_file) |
| self.assertEquals(latest_entry.contents(), self._example_entry) |
| self.assertEquals(latest_entry.author_name(), "Peter Kasting") |
| |
| @staticmethod |
| def _write_tmp_file_with_contents(byte_array): |
| assert(isinstance(byte_array, str)) |
| (file_descriptor, file_path) = tempfile.mkstemp() # NamedTemporaryFile always deletes the file on close in python < 2.6 |
| with os.fdopen(file_descriptor, "w") as file: |
| file.write(byte_array) |
| return file_path |
| |
| @staticmethod |
| def _read_file_contents(file_path, encoding): |
| with codecs.open(file_path, "r", encoding) as file: |
| return file.read() |
| |
| _new_entry_boilerplate = '''2009-08-19 Eric Seidel <eric@webkit.org> |
| |
| Reviewed by NOBODY (OOPS!). |
| |
| Need a short description and bug URL (OOPS!) |
| |
| * Scripts/bugzilla-tool: |
| ''' |
| |
| def test_set_reviewer(self): |
| changelog_contents = u"%s\n%s" % (self._new_entry_boilerplate, self._example_changelog) |
| changelog_path = self._write_tmp_file_with_contents(changelog_contents.encode("utf-8")) |
| reviewer_name = 'Test Reviewer' |
| ChangeLog(changelog_path).set_reviewer(reviewer_name) |
| actual_contents = self._read_file_contents(changelog_path, "utf-8") |
| expected_contents = changelog_contents.replace('NOBODY (OOPS!)', reviewer_name) |
| os.remove(changelog_path) |
| self.assertEquals(actual_contents, expected_contents) |
| |
| def test_set_short_description_and_bug_url(self): |
| changelog_contents = u"%s\n%s" % (self._new_entry_boilerplate, self._example_changelog) |
| changelog_path = self._write_tmp_file_with_contents(changelog_contents.encode("utf-8")) |
| short_description = "A short description" |
| bug_url = "http://example.com/b/2344" |
| ChangeLog(changelog_path).set_short_description_and_bug_url(short_description, bug_url) |
| actual_contents = self._read_file_contents(changelog_path, "utf-8") |
| expected_message = "%s\n %s" % (short_description, bug_url) |
| expected_contents = changelog_contents.replace("Need a short description and bug URL (OOPS!)", expected_message) |
| os.remove(changelog_path) |
| self.assertEquals(actual_contents, expected_contents) |