bin/post_version.py: Update the release calendar as well
authorDylan Baker <dylan@pnwbakers.com>
Thu, 24 Oct 2019 22:51:30 +0000 (15:51 -0700)
committerDylan Baker <dylan@pnwbakers.com>
Thu, 5 Mar 2020 23:14:56 +0000 (15:14 -0800)
Acked-by: Eric Engestrom <eric.engestrom@intel.com>
Reviewed-by: Juan A. Suarez <jasuarez@igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/2505>

bin/post_version.py
bin/post_version_test.py [new file with mode: 0644]

index d9b2a1de3245880b12c9746401fcaf0878fd5dd3..5bf68fe35d2646a26d0b4d0a27a7d5bcf5fb26bc 100755 (executable)
@@ -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 (file)
index 0000000..11b2806
--- /dev/null
@@ -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("""\
+        <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
+        <html lang="en">
+        <head>
+        <meta http-equiv="content-type" content="text/html; charset=utf-8">
+        <title>Release Calendar</title>
+        <link rel="stylesheet" type="text/css" href="mesa.css">
+        </head>
+        <body>
+        """)
+
+    TABLE = textwrap.dedent("""\
+        <table>
+        <tr>
+        <th>Branch</th>
+        <th>Expected date</th>
+        <th>Release</th>
+        <th>Release manager</th>
+        <th>Notes</th>
+        </tr>
+        """)
+
+    FOOT = "</body></html>"
+
+    TABLE_FOOT = "</table>"
+
+    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("""\
+            <tr>
+            <td rowspan="3">19.2</td>
+            <td>2019-11-06</td>
+            <td>19.2.3</td>
+            <td>Dylan Baker</td>
+            </tr>
+            <tr>
+            <td>2019-11-20</td>
+            <td>19.2.4</td>
+            <td>Dylan Baker</td>
+            </tr>
+            <tr>
+            <td>2019-12-04</td>
+            <td>19.2.5</td>
+            <td>Dylan Baker</td>
+            <td>Last planned 19.2.x release</td>
+            </tr>
+            """))
+
+        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("""\
+            <tr>
+            <td rowspan="1">19.1</td>
+            <td>2019-11-06</td>
+            <td>19.1.8</td>
+            <td>Not Dylan Baker</td>
+            </tr>
+            <tr>
+            <td rowspan="3">19.2</td>
+            <td>2019-11-06</td>
+            <td>19.2.3</td>
+            <td>Dylan Baker</td>
+            </tr>
+            <tr>
+            <td>2019-11-20</td>
+            <td>19.2.4</td>
+            <td>Dylan Baker</td>
+            </tr>
+            <tr>
+            <td>2019-12-04</td>
+            <td>19.2.5</td>
+            <td>Dylan Baker</td>
+            <td>Last planned 19.2.x release</td>
+            </tr>
+            """))
+
+        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"