diff options
author | Dylan Baker <dylan.c.baker@intel.com> | 2021-08-04 11:32:19 -0700 |
---|---|---|
committer | Marge Bot <eric+marge@anholt.net> | 2021-08-04 23:03:57 +0000 |
commit | 30f7b55e475965d31585bbfb0bd03f234cda16b0 (patch) | |
tree | 2f9bd26512f13be7902bec22bb0d03a1419beefd /bin/gen_release_notes_test.py | |
parent | 9dc3672b00f4f04c57b46a6fa675ecd2c03bad07 (diff) | |
download | mesa-30f7b55e475965d31585bbfb0bd03f234cda16b0.tar.gz mesa-30f7b55e475965d31585bbfb0bd03f234cda16b0.tar.bz2 mesa-30f7b55e475965d31585bbfb0bd03f234cda16b0.zip |
bin/gen_release_notes: Don't consider issues for other projects
We have enough commits in mesa that have external dependencies that we
need to be sure that a Closes: https://... is actually for mesa and not
for another project.
Reviewed-by: Eric Engestrom <eric@engestrom.ch>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/12201>
Diffstat (limited to 'bin/gen_release_notes_test.py')
-rw-r--r-- | bin/gen_release_notes_test.py | 22 |
1 files changed, 22 insertions, 0 deletions
diff --git a/bin/gen_release_notes_test.py b/bin/gen_release_notes_test.py index eca0c4b8af4..194fbc745ac 100644 --- a/bin/gen_release_notes_test.py +++ b/bin/gen_release_notes_test.py @@ -77,6 +77,8 @@ async def test_gather_commits(): [ # It is important to have the title on a new line, as # textwrap.dedent wont work otherwise. + + # Test the `Closes: #N` syntax ( '''\ A commit @@ -87,6 +89,8 @@ async def test_gather_commits(): ''', ['1'], ), + + # Test the Full url ( '''\ A commit with no body @@ -95,6 +99,24 @@ async def test_gather_commits(): ''', ['3456'], ), + + # Test projects that are not mesa + ( + '''\ + A commit for libdrm + + Closes: https://gitlab.freedesktop.org/mesa/drm/-/3456 + ''', + [], + ), + ( + '''\ + A commit for for something else completely + + Closes: https://github.com/Organiztion/project/1234 + ''', + [], + ), ]) async def test_parse_issues(content: str, bugs: typing.List[str]) -> None: mock_com = mock.AsyncMock(return_value=(textwrap.dedent(content).encode(), '')) |