From d7ada7d7e06359b01fcb6c25b295f58f10f50682 Mon Sep 17 00:00:00 2001 From: Dylan Baker Date: Thu, 24 Oct 2019 15:51:30 -0700 Subject: [PATCH] bin/post_version.py: Update the release calendar as well Acked-by: Eric Engestrom Reviewed-by: Juan A. Suarez Part-of: --- bin/post_version.py | 34 +++++++++++ bin/post_version_test.py | 156 +++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 190 insertions(+) create mode 100644 bin/post_version_test.py diff --git a/bin/post_version.py b/bin/post_version.py index d9b2a1d..5bf68fe 100755 --- a/bin/post_version.py +++ b/bin/post_version.py @@ -101,6 +101,39 @@ def update_release_notes(previous_version: str) -> None: tree.write(p.as_posix(), method='html') +def update_calendar(previous_version: str) -> None: + p = pathlib.Path(__file__).parent.parent / 'docs' / 'release-calendar.html' + with p.open('rt') as f: + tree = html.parse(f) + + base_version = previous_version[:-2] + + old = None + new = None + + for tr in tree.xpath('.//tr'): + if old is not None: + new = tr + break + + for td in tr.xpath('./td'): + if td.text == base_version: + old = tr + break + + assert old is not None + assert new is not None + old.getparent().remove(old) + + # rowspan is 1 based in html, but 0 based in lxml + rowspan = int(td.get("rowspan")) - 1 + if rowspan: + td.set("rowspan", str(rowspan)) + new.insert(0, td) + + tree.write(p.as_posix(), method='html') + + def main() -> None: parser = argparse.ArgumentParser() parser.add_argument('version', help="The released version.") @@ -111,6 +144,7 @@ def main() -> None: update_index(is_point, args.version, previous_version) update_release_notes(previous_version) + update_calendar(previous_version) if __name__ == "__main__": diff --git a/bin/post_version_test.py b/bin/post_version_test.py new file mode 100644 index 0000000..11b2806 --- /dev/null +++ b/bin/post_version_test.py @@ -0,0 +1,156 @@ +# Copyright © 2019 Intel Corporation + +# Permission is hereby granted, free of charge, to any person obtaining a copy +# of this software and associated documentation files (the "Software"), to deal +# in the Software without restriction, including without limitation the rights +# to use, copy, modify, merge, publish, distribute, sublicense, and/or sell +# copies of the Software, and to permit persons to whom the Software is +# furnished to do so, subject to the following conditions: + +# The above copyright notice and this permission notice shall be included in +# all copies or substantial portions of the Software. + +# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR +# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, +# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE +# AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER +# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, +# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE +# SOFTWARE. + +from unittest import mock +import textwrap + +from lxml import html +import pytest + +from . import post_version + + +class TestUpdateCalendar: + + HEAD = textwrap.dedent("""\ + + + + + Release Calendar + + + + """) + + TABLE = textwrap.dedent("""\ + + + + + + + + + """) + + FOOT = "" + + TABLE_FOOT = "
BranchExpected dateReleaseRelease managerNotes
" + + def wrap_table(self, table: str) -> str: + return self.HEAD + self.TABLE + table + self.TABLE_FOOT + self.FOOT + + def test_basic(self): + data = self.wrap_table(textwrap.dedent("""\ + + 19.2 + 2019-11-06 + 19.2.3 + Dylan Baker + + + 2019-11-20 + 19.2.4 + Dylan Baker + + + 2019-12-04 + 19.2.5 + Dylan Baker + Last planned 19.2.x release + + """)) + + parsed = html.fromstring(data) + parsed.write = mock.Mock() + + with mock.patch('bin.post_version.html.parse', + mock.Mock(return_value=parsed)): + post_version.update_calendar('19.2.3') + + assert len(parsed.findall('.//tr')) == 3 + # we need the second element becouse the first is the header + + tr = parsed.findall('.//tr')[1] + tds = tr.findall('.//td') + assert tds[0].get("rowspan") == "2" + assert tds[0].text == "19.2" + assert tds[1].text == "2019-11-20" + + @pytest.fixture + def two_releases(self) -> html.etree.ElementTree: + data = self.wrap_table(textwrap.dedent("""\ + + 19.1 + 2019-11-06 + 19.1.8 + Not Dylan Baker + + + 19.2 + 2019-11-06 + 19.2.3 + Dylan Baker + + + 2019-11-20 + 19.2.4 + Dylan Baker + + + 2019-12-04 + 19.2.5 + Dylan Baker + Last planned 19.2.x release + + """)) + + p = html.fromstring(data) + p.write = mock.Mock() + return p + + def test_two_releases(self, two_releases: html.etree.ElementTree): + with mock.patch('bin.post_version.html.parse', + mock.Mock(return_value=two_releases)): + post_version.update_calendar('19.2.3') + + assert len(two_releases.findall('.//tr')) == 4 + # we need the second element becouse the first is the header + + tr = two_releases.findall('.//tr')[2] + tds = tr.findall('.//td') + assert tds[0].get("rowspan") == "2" + assert tds[0].text == "19.2" + assert tds[1].text == "2019-11-20" + + def test_last_Release(self, two_releases: html.etree.ElementTree): + with mock.patch('bin.post_version.html.parse', + mock.Mock(return_value=two_releases)): + post_version.update_calendar('19.1.8') + + assert len(two_releases.findall('.//tr')) == 4 + # we need the second element becouse the first is the header + + tr = two_releases.findall('.//tr')[1] + tds = tr.findall('.//td') + assert tds[0].get("rowspan") == "3" + assert tds[0].text == "19.2" + assert tds[1].text == "2019-11-06" -- 2.7.4