from __future__ import annotations from textwrap import dedent from typing import TYPE_CHECKING, Iterable, Sequence import pytest from semantic_release.commit_parser.emoji import EmojiCommitParser, EmojiParserOptions from semantic_release.commit_parser.token import ParsedCommit, ParseError from semantic_release.enums import LevelBump from tests.const import SUPPORTED_ISSUE_CLOSURE_PREFIXES if TYPE_CHECKING: from tests.conftest import MakeCommitObjFn @pytest.mark.parametrize( "commit_message, bump, type_, descriptions, breaking_descriptions", [ # Major bump ( ":boom: Breaking changes\n\nMore description\n\nEven more description", LevelBump.MAJOR, ":boom:", [":boom: Breaking changes"], ["More description", "Even more description"], ), # Minor bump ( ":sparkles: Add a new feature\n\nSome description of the feature", LevelBump.MINOR, ":sparkles:", [":sparkles: Add a new feature", "Some description of the feature"], [], ), # Patch bump ( ":bug: Fixing a bug\n\nThe bug is finally gone!", LevelBump.PATCH, ":bug:", [":bug: Fixing a bug", "The bug is finally gone!"], [], ), # No release ( ":pencil: Documentation changes", LevelBump.NO_RELEASE, "Other", [":pencil: Documentation changes"], [], ), # Multiple emojis ( ":sparkles::pencil: Add a feature and document it", LevelBump.MINOR, ":sparkles:", [":sparkles::pencil: Add a feature and document it"], [], ), # Emoji in description ( ":sparkles: Add a new feature\n\n:boom: should not be detected", LevelBump.MINOR, ":sparkles:", [":sparkles: Add a new feature"], [], ), ], ) def test_default_emoji_parser( default_emoji_parser: EmojiCommitParser, commit_message: str, bump: LevelBump, type_: str, descriptions: list[str], breaking_descriptions: list[str], make_commit_obj: MakeCommitObjFn, ): commit = make_commit_obj(commit_message) parsed_results = default_emoji_parser.parse(commit) assert isinstance(parsed_results, Iterable) result = next(iter(parsed_results)) assert isinstance(result, ParsedCommit) assert bump is result.bump assert type_ == result.type assert descriptions == result.descriptions assert breaking_descriptions == result.breaking_descriptions @pytest.mark.parametrize( "message, subject, merge_request_number", [ # GitHub, Gitea style ( ":sparkles: add new feature (#123)", ":sparkles: add new feature", "#123", ), # GitLab style ( ":bug: fix regex in parser (!456)", ":bug: fix regex in parser", "!456", ), # BitBucket style ( ":sparkles: add new feature (pull request #123)", ":sparkles: add new feature", "#123", ), # Both a linked merge request and an issue footer (should return the linked merge request) (":bug: superfix (#123)\n\nCloses: #400", ":bug: superfix", "#123"), # None (":bug: superfix", ":bug: superfix", ""), # None but includes an issue footer it should not be considered a linked merge request (":bug: superfix\n\nCloses: #400", ":bug: superfix", ""), ], ) def test_parser_return_linked_merge_request_from_commit_message( default_emoji_parser: EmojiCommitParser, message: str, subject: str, merge_request_number: str, make_commit_obj: MakeCommitObjFn, ): parsed_results = default_emoji_parser.parse(make_commit_obj(message)) assert isinstance(parsed_results, Iterable) result = next(iter(parsed_results)) assert isinstance(result, ParsedCommit) assert merge_request_number == result.linked_merge_request assert subject == result.descriptions[0] @pytest.mark.parametrize( "message, linked_issues", [ *[ # GitHub, Gitea, GitLab style ( f":sparkles: (parser) add magic parser\n\n{footer}", linked_issues, ) for footer_prefix in SUPPORTED_ISSUE_CLOSURE_PREFIXES for footer, linked_issues in [ # Single issue ( f"{footer_prefix.capitalize()}: #555", ["#555"], ), # Git Footer style (capitalized) (f"{footer_prefix.lower()}: #555", ["#555"]), # lowercase prefix (f"{footer_prefix.upper()}: #555", ["#555"]), # uppercase prefix # Mulitple issues (variant 1: list with one prefix, not supported by GitHub) ( f"{footer_prefix}: #555,#444", ["#444", "#555"], ), # Comma separated (w/o space) ( f"{footer_prefix}: #555, #444", ["#444", "#555"], ), # Comma separated (w/ space) ( f"{footer_prefix}: #555 , #444", ["#444", "#555"], ), # Comma separated (w/ extra space) (f"{footer_prefix}: #555 #444", ["#444", "#555"]), # Space separated ( f"{footer_prefix}: #555;#444", ["#444", "#555"], ), # semicolon separated (w/o space) ( f"{footer_prefix}: #555; #444", ["#444", "#555"], ), # semicolon separated (w/ space) ( f"{footer_prefix}: #555 ; #444", ["#444", "#555"], ), # semicolon separated (w/ extra space) ( f"{footer_prefix}: #555/#444", ["#444", "#555"], ), # slash separated (w/o space) ( f"{footer_prefix}: #555/ #444", ["#444", "#555"], ), # slash separated (w/ space) ( f"{footer_prefix}: #555 / #444", ["#444", "#555"], ), # slash separated (w/ extra space) ( f"{footer_prefix}: #555Ƽ", ["#444", "#555"], ), # ampersand separated (w/o space) ( f"{footer_prefix}: #555& #444", ["#444", "#555"], ), # ampersand separated (w/ space) ( f"{footer_prefix}: #555 & #444", ["#444", "#555"], ), # ampersand separated (w/ extra space) (f"{footer_prefix}: #555 and #444", ["#444", "#555"]), # and separated ( f"{footer_prefix}: #555, #444, and #333", ["#333", "#444", "#555"], ), # and separated # Mulitple issues (variant 2: multiple footers, supported by GitHub) (f"{footer_prefix}: #555\n{footer_prefix}: #444", ["#444", "#555"]), # More than 2 issues ( f"{footer_prefix}: #555, #444, #333", ["#333", "#444", "#555"], ), # More than 2 issues (force numerical sort) ( f"{footer_prefix}: #555, #3333, #444", ["#444", "#555", "#3333"], ), # Single issue listed multiple times (f"{footer_prefix}: #555, #555", ["#555"]), # Multiple footers with the same issue (f"{footer_prefix}: #555\n{footer_prefix}: #555", ["#555"]), # Multiple issues via multiple inline git footers (f"{footer_prefix}: #555, {footer_prefix}: #444", ["#444", "#555"]), # Multiple valid footers ( str.join( "\n", [ f"{footer_prefix}: #555", "Signed-off-by: johndoe ", f"{footer_prefix}: #444", ], ), ["#444", "#555"], ), # ----------------------------------------- Invalid Sets ----------------------------------------- # # Must have colon because it is a git footer, these will not return a linked issue (f"{footer_prefix} #666", []), (f"{footer_prefix} #666, #777", []), # Invalid Multiple issues (although it is supported by GitHub, it is not supported by the parser) (f"{footer_prefix} #666, {footer_prefix} #777", []), # Invalid 'and' separation (f"{footer_prefix}: #666and#777", ["#666and#777"]), # Invalid prefix ("ref: #666", []), # body mentions an issue and has a different git footer ( "In #666, the devils in the details...\n\nSigned-off-by: johndoe ", [], ), ] ], *[ # JIRA style ( f":sparkles: (parser) add magic parser\n\n{footer}", linked_issues, ) for footer_prefix in SUPPORTED_ISSUE_CLOSURE_PREFIXES for footer, linked_issues in [ # Single issue ( f"{footer_prefix.capitalize()}: ABC-555", ["ABC-555"], ), # Git Footer style (capitalized) (f"{footer_prefix.lower()}: ABC-555", ["ABC-555"]), # lowercase prefix (f"{footer_prefix.upper()}: ABC-555", ["ABC-555"]), # uppercase prefix # Mulitple issues (variant 1: list with one prefix, not supported by GitHub) ( f"{footer_prefix}: ABC-555,ABC-444", ["ABC-444", "ABC-555"], ), # Comma separated (w/o space) ( f"{footer_prefix}: ABC-555, ABC-444", ["ABC-444", "ABC-555"], ), # Comma separated (w/ space) ( f"{footer_prefix}: ABC-555 , ABC-444", ["ABC-444", "ABC-555"], ), # Comma separated (w/ extra space) ( f"{footer_prefix}: ABC-555 ABC-444", ["ABC-444", "ABC-555"], ), # Space separated ( f"{footer_prefix}: ABC-555;ABC-444", ["ABC-444", "ABC-555"], ), # semicolon separated (w/o space) ( f"{footer_prefix}: ABC-555; ABC-444", ["ABC-444", "ABC-555"], ), # semicolon separated (w/ space) ( f"{footer_prefix}: ABC-555 ; ABC-444", ["ABC-444", "ABC-555"], ), # semicolon separated (w/ extra space) ( f"{footer_prefix}: ABC-555/ABC-444", ["ABC-444", "ABC-555"], ), # slash separated (w/o space) ( f"{footer_prefix}: ABC-555/ ABC-444", ["ABC-444", "ABC-555"], ), # slash separated (w/ space) ( f"{footer_prefix}: ABC-555 / ABC-444", ["ABC-444", "ABC-555"], ), # slash separated (w/ extra space) ( f"{footer_prefix}: ABC-555&ABC-444", ["ABC-444", "ABC-555"], ), # ampersand separated (w/o space) ( f"{footer_prefix}: ABC-555& ABC-444", ["ABC-444", "ABC-555"], ), # ampersand separated (w/ space) ( f"{footer_prefix}: ABC-555 & ABC-444", ["ABC-444", "ABC-555"], ), # ampersand separated (w/ extra space) ( f"{footer_prefix}: ABC-555 and ABC-444", ["ABC-444", "ABC-555"], ), # and separated ( f"{footer_prefix}: ABC-555, ABC-444, and ABC-333", ["ABC-333", "ABC-444", "ABC-555"], ), # and separated # Mulitple issues (variant 2: multiple footers, supported by GitHub) ( f"{footer_prefix}: ABC-555\n{footer_prefix}: ABC-444", ["ABC-444", "ABC-555"], ), # More than 2 issues ( f"{footer_prefix}: ABC-555, ABC-444, ABC-333", ["ABC-333", "ABC-444", "ABC-555"], ), # More than 2 issues (force numerical sort) ( f"{footer_prefix}: ABC-555, ABC-3333, ABC-444", ["ABC-444", "ABC-555", "ABC-3333"], ), # Single issue listed multiple times (f"{footer_prefix}: ABC-555, ABC-555", ["ABC-555"]), # Multiple footers with the same issue (f"{footer_prefix}: ABC-555\n{footer_prefix}: ABC-555", ["ABC-555"]), # Multiple issues via multiple inline git footers ( f"{footer_prefix}: ABC-666, {footer_prefix}: ABC-777", ["ABC-666", "ABC-777"], ), # Multiple valid footers ( str.join( "\n", [ f"{footer_prefix}: ABC-555", "Signed-off-by: johndoe ", f"{footer_prefix}: ABC-444", ], ), ["ABC-444", "ABC-555"], ), # ----------------------------------------- Invalid Sets ----------------------------------------- # # Must have colon because it is a git footer, these will not return a linked issue (f"{footer_prefix} ABC-666", []), (f"{footer_prefix} ABC-666, ABC-777", []), # Invalid Multiple issues (although it is supported by GitHub, it is not supported by the parser) (f"{footer_prefix} ABC-666, {footer_prefix} ABC-777", []), # Invalid 'and' separation (f"{footer_prefix}: ABC-666andABC-777", ["ABC-666andABC-777"]), # Invalid prefix ("ref: ABC-666", []), # body mentions an issue and has a different git footer ( "In ABC-666, the devils in the details...\n\nSigned-off-by: johndoe ", [], ), ] ], *[ ( f":sparkles: (parser) add magic parser\n\n{footer}", linked_issues, ) for footer, linked_issues in [ # Multiple footers with the same issue but different prefixes ("Resolves: #555\nfix: #444", ["#444", "#555"]), # Whitespace padded footer (" Resolves: #555\n", ["#555"]), ] ], ( # Only grabs the issue reference when there is a GitHub PR reference in the subject ":sparkles: (parser) add magic parser (#123)\n\nCloses: #555", ["#555"], ), # Does not grab an issue when there is only a GitHub PR reference in the subject (":sparkles: (parser) add magic parser (#123)", []), # Does not grab an issue when there is only a Bitbucket PR reference in the subject (":sparkles: (parser) add magic parser (pull request #123)", []), ], ) def test_parser_return_linked_issues_from_commit_message( default_emoji_parser: EmojiCommitParser, message: str, linked_issues: Sequence[str], make_commit_obj: MakeCommitObjFn, ): # Setup: Enable parsing of linked issues parser = EmojiCommitParser( options=EmojiParserOptions( **{ **default_emoji_parser.options.__dict__, "parse_linked_issues": True, } ) ) # Action parsed_results = parser.parse(make_commit_obj(message)) assert isinstance(parsed_results, Iterable) assert len(parsed_results) == 1 # Evaluate (expected -> actual) result = next(iter(parsed_results)) assert isinstance(result, ParsedCommit) assert tuple(linked_issues) == result.linked_issues @pytest.mark.parametrize( "message, notices", [ pytest.param( message, notices, id=test_id, ) for test_id, message, notices in [ ( "single notice", dedent( """\ :bug:(parser): fix regex in emoji parser NOTICE: This is a notice """ ), ["This is a notice"], ), ( "multiline notice", dedent( """\ :bug:(parser): fix regex in emoji parser NOTICE: This is a notice that is longer than other notices """ ), ["This is a notice that is longer than other notices"], ), ( "multiple notices", dedent( """\ :bug:(parser): fix regex in emoji parser NOTICE: This is a notice NOTICE: This is a second notice """ ), ["This is a notice", "This is a second notice"], ), ( "notice with other footer", dedent( """\ :bug:(parser): fix regex in emoji parser BREAKING CHANGE: This is a breaking change NOTICE: This is a notice """ ), ["This is a notice"], ), ] ], ) def test_parser_return_release_notices_from_commit_message( default_emoji_parser: EmojiCommitParser, message: str, notices: Sequence[str], make_commit_obj: MakeCommitObjFn, ): parsed_results = default_emoji_parser.parse(make_commit_obj(message)) assert isinstance(parsed_results, Iterable) assert len(parsed_results) == 1 result = next(iter(parsed_results)) assert isinstance(result, ParsedCommit) assert tuple(notices) == result.release_notices full_description = str.join("\n\n", result.descriptions) full_notice = str.join("\n\n", result.release_notices) assert full_notice not in full_description @pytest.mark.parametrize( "commit_message, expected_commit_details", [ pytest.param( commit_message, expected_commit_details, id=test_id, ) for test_id, commit_message, expected_commit_details in [ ( "Single commit squashed via BitBucket PR resolution", dedent( """\ Merged in feat/my-awesome-stuff (pull request #10) :bug:(release-config): some commit subject An additional description Second paragraph with multiple lines that will be condensed Resolves: #12 Signed-off-by: author """ ), [ { "bump": LevelBump.NO_RELEASE, "type": "Other", "descriptions": ["Merged in feat/my-awesome-stuff"], "linked_merge_request": "#10", }, { "bump": LevelBump.PATCH, "type": ":bug:", "scope": "release-config", "descriptions": [ ":bug:(release-config): some commit subject", "An additional description", "Second paragraph with multiple lines that will be condensed", "Signed-off-by: author ", ], "linked_issues": ("#12",), "linked_merge_request": "#10", }, ], ), ( "Multiple commits squashed via BitBucket PR resolution", dedent( """\ Merged in feat/my-awesome-stuff (pull request #10) :bug:(release-config): some commit subject An additional description Second paragraph with multiple lines that will be condensed Resolves: #12 Signed-off-by: author :sparkles: implemented searching gizmos by keyword :memo:(parser): add new parser pattern :boom::bug: changed option name A breaking change description Closes: #555 invalid non-conventional formatted commit """ ), [ { "bump": LevelBump.NO_RELEASE, "type": "Other", "descriptions": ["Merged in feat/my-awesome-stuff"], "linked_merge_request": "#10", }, { "bump": LevelBump.PATCH, "type": ":bug:", "scope": "release-config", "descriptions": [ ":bug:(release-config): some commit subject", "An additional description", "Second paragraph with multiple lines that will be condensed", "Signed-off-by: author ", ], "linked_issues": ("#12",), "linked_merge_request": "#10", }, { "bump": LevelBump.MINOR, "type": ":sparkles:", "descriptions": [ ":sparkles: implemented searching gizmos by keyword" ], "linked_merge_request": "#10", }, { "bump": LevelBump.NO_RELEASE, "type": ":memo:", "scope": "parser", "descriptions": [ ":memo:(parser): add new parser pattern", ], "linked_merge_request": "#10", }, { "bump": LevelBump.MAJOR, "type": ":boom:", "scope": "", "descriptions": [ ":boom::bug: changed option name", ], "breaking_descriptions": [ "A breaking change description", # This is a bit unusual but its because there is no identifier that will # identify this as a separate commit so it gets included in the previous commit "invalid non-conventional formatted commit", ], "linked_issues": ("#555",), "linked_merge_request": "#10", }, ], ), ] ], ) def test_parser_squashed_commit_bitbucket_squash_style( default_emoji_parser: EmojiCommitParser, make_commit_obj: MakeCommitObjFn, commit_message: str, expected_commit_details: Sequence[dict | None], ): # Setup: Enable squash commit parsing parser = EmojiCommitParser( options=EmojiParserOptions( **{ **default_emoji_parser.options.__dict__, "parse_squash_commits": True, "parse_linked_issues": True, } ) ) # Build the commit object and parse it the_commit = make_commit_obj(commit_message) parsed_results = parser.parse(the_commit) # Validate the results assert isinstance(parsed_results, Iterable) assert ( len(expected_commit_details) == len(parsed_results) ), f"Expected {len(expected_commit_details)} parsed results, but got {len(parsed_results)}" for result, expected in zip(parsed_results, expected_commit_details): if expected is None: assert isinstance(result, ParseError) continue assert isinstance(result, ParsedCommit) # Required assert expected["bump"] == result.bump assert expected["type"] == result.type # Optional assert expected.get("scope", "") == result.scope # TODO: v11 change to tuples assert expected.get("descriptions", []) == result.descriptions assert expected.get("breaking_descriptions", []) == result.breaking_descriptions assert expected.get("linked_issues", ()) == result.linked_issues assert expected.get("linked_merge_request", "") == result.linked_merge_request @pytest.mark.parametrize( "commit_message, expected_commit_details", [ pytest.param( commit_message, expected_commit_details, id=test_id, ) for test_id, commit_message, expected_commit_details in [ ( "Single commit squashed via manual Git squash merge", dedent( """\ Squashed commit of the following: commit 63ec09b9e844e616dcaa7bae35a0b66671b59fbb Author: author Date: Sun Jan 19 12:05:23 2025 +0000 :bug:(release-config): some commit subject An additional description Second paragraph with multiple lines that will be condensed Resolves: #12 Signed-off-by: author """ ), [ { "bump": LevelBump.PATCH, "type": ":bug:", "scope": "release-config", "descriptions": [ ":bug:(release-config): some commit subject", "An additional description", "Second paragraph with multiple lines that will be condensed", "Signed-off-by: author ", ], "linked_issues": ("#12",), } ], ), ( "Multiple commits squashed via manual Git squash merge", dedent( """\ Squashed commit of the following: commit 63ec09b9e844e616dcaa7bae35a0b66671b59fbb Author: author Date: Sun Jan 19 12:05:23 2025 +0000 :bug:(release-config): some commit subject An additional description Second paragraph with multiple lines that will be condensed Resolves: #12 Signed-off-by: author commit 1f34769bf8352131ad6f4879b8c47becf3c7aa69 Author: author Date: Sat Jan 18 10:13:53 2025 +0000 :sparkles: implemented searching gizmos by keyword commit b2334a64a11ef745a17a2a4034f651e08e8c45a6 Author: author Date: Sat Jan 18 10:13:53 2025 +0000 :memo:(parser): add new parser pattern commit 5f0292fb5a88c3a46e4a02bec35b85f5228e8e51 Author: author Date: Sat Jan 18 10:13:53 2025 +0000 :boom::bug: changed option name A breaking change description Closes: #555 commit 2f314e7924be161cfbf220d3b6e2a6189a3b5609 Author: author Date: Sat Jan 18 10:13:53 2025 +0000 invalid non-conventional formatted commit """ ), [ { "bump": LevelBump.PATCH, "type": ":bug:", "scope": "release-config", "descriptions": [ ":bug:(release-config): some commit subject", "An additional description", "Second paragraph with multiple lines that will be condensed", "Signed-off-by: author ", ], "linked_issues": ("#12",), }, { "bump": LevelBump.MINOR, "type": ":sparkles:", "descriptions": [ ":sparkles: implemented searching gizmos by keyword" ], }, { "bump": LevelBump.NO_RELEASE, "type": ":memo:", "scope": "parser", "descriptions": [ ":memo:(parser): add new parser pattern", ], }, { "bump": LevelBump.MAJOR, "type": ":boom:", "descriptions": [ ":boom::bug: changed option name", ], "breaking_descriptions": [ "A breaking change description", ], "linked_issues": ("#555",), }, { "bump": LevelBump.NO_RELEASE, "type": "Other", "descriptions": ["invalid non-conventional formatted commit"], }, ], ), ] ], ) def test_parser_squashed_commit_git_squash_style( default_emoji_parser: EmojiCommitParser, make_commit_obj: MakeCommitObjFn, commit_message: str, expected_commit_details: Sequence[dict | None], ): # Setup: Enable squash commit parsing parser = EmojiCommitParser( options=EmojiParserOptions( **{ **default_emoji_parser.options.__dict__, "parse_squash_commits": True, "parse_linked_issues": True, } ) ) # Build the commit object and parse it the_commit = make_commit_obj(commit_message) parsed_results = parser.parse(the_commit) # Validate the results assert isinstance(parsed_results, Iterable) assert ( len(expected_commit_details) == len(parsed_results) ), f"Expected {len(expected_commit_details)} parsed results, but got {len(parsed_results)}" for result, expected in zip(parsed_results, expected_commit_details): if expected is None: assert isinstance(result, ParseError) continue assert isinstance(result, ParsedCommit) # Required assert expected["bump"] == result.bump assert expected["type"] == result.type # Optional assert expected.get("scope", "") == result.scope # TODO: v11 change to tuples assert expected.get("descriptions", []) == result.descriptions assert expected.get("breaking_descriptions", []) == result.breaking_descriptions assert expected.get("linked_issues", ()) == result.linked_issues assert expected.get("linked_merge_request", "") == result.linked_merge_request @pytest.mark.parametrize( "commit_message, expected_commit_details", [ pytest.param( commit_message, expected_commit_details, id=test_id, ) for test_id, commit_message, expected_commit_details in [ ( "Single commit squashed via GitHub PR resolution", dedent( """\ :bug:(release-config): some commit subject (#10) An additional description Second paragraph with multiple lines that will be condensed Resolves: #12 Signed-off-by: author """ ), [ { "bump": LevelBump.PATCH, "type": ":bug:", "scope": "release-config", "descriptions": [ ":bug:(release-config): some commit subject", "An additional description", "Second paragraph with multiple lines that will be condensed", "Signed-off-by: author ", ], "linked_issues": ("#12",), "linked_merge_request": "#10", }, ], ), ( "Multiple commits squashed via GitHub PR resolution", dedent( """\ :bug:(release-config): some commit subject (#10) An additional description Second paragraph with multiple lines that will be condensed Resolves: #12 Signed-off-by: author * :sparkles: implemented searching gizmos by keyword * :memo:(parser): add new parser pattern * :boom::bug: changed option name A breaking change description Closes: #555 * invalid non-conventional formatted commit """ ), [ { "bump": LevelBump.PATCH, "type": ":bug:", "scope": "release-config", "descriptions": [ ":bug:(release-config): some commit subject", "An additional description", "Second paragraph with multiple lines that will be condensed", "Signed-off-by: author ", ], "linked_issues": ("#12",), "linked_merge_request": "#10", }, { "bump": LevelBump.MINOR, "type": ":sparkles:", "descriptions": [ ":sparkles: implemented searching gizmos by keyword" ], "linked_merge_request": "#10", }, { "bump": LevelBump.NO_RELEASE, "type": ":memo:", "scope": "parser", "descriptions": [ ":memo:(parser): add new parser pattern", ], "linked_merge_request": "#10", }, { "bump": LevelBump.MAJOR, "type": ":boom:", "scope": "", "descriptions": [ ":boom::bug: changed option name", ], "breaking_descriptions": [ "A breaking change description", # This is a bit unusual but its because there is no identifier that will # identify this as a separate commit so it gets included in the previous commit "* invalid non-conventional formatted commit", ], "linked_issues": ("#555",), "linked_merge_request": "#10", }, ], ), ] ], ) def test_parser_squashed_commit_github_squash_style( default_emoji_parser: EmojiCommitParser, make_commit_obj: MakeCommitObjFn, commit_message: str, expected_commit_details: Sequence[dict | None], ): # Setup: Enable squash commit parsing parser = EmojiCommitParser( options=EmojiParserOptions( **{ **default_emoji_parser.options.__dict__, "parse_squash_commits": True, "parse_linked_issues": True, } ) ) # Build the commit object and parse it the_commit = make_commit_obj(commit_message) parsed_results = parser.parse(the_commit) # Validate the results assert isinstance(parsed_results, Iterable) assert ( len(expected_commit_details) == len(parsed_results) ), f"Expected {len(expected_commit_details)} parsed results, but got {len(parsed_results)}" for result, expected in zip(parsed_results, expected_commit_details): if expected is None: assert isinstance(result, ParseError) continue assert isinstance(result, ParsedCommit) # Required assert expected["bump"] == result.bump assert expected["type"] == result.type # Optional assert expected.get("scope", "") == result.scope # TODO: v11 change to tuples assert expected.get("descriptions", []) == result.descriptions assert expected.get("breaking_descriptions", []) == result.breaking_descriptions assert expected.get("linked_issues", ()) == result.linked_issues assert expected.get("linked_merge_request", "") == result.linked_merge_request def test_parser_ignore_merge_commit( default_emoji_parser: EmojiCommitParser, make_commit_obj: MakeCommitObjFn, ): # Setup: Enable parsing of linked issues parser = EmojiCommitParser( options=EmojiParserOptions( **{ **default_emoji_parser.options.__dict__, "ignore_merge_commits": True, } ) ) base_commit = make_commit_obj("Merge branch 'fix/fix-feature' into 'main'") incomming_commit = make_commit_obj("feat: add a new feature") # Setup: Create a merge commit merge_commit = make_commit_obj("Merge branch 'feat/add-new-feature' into 'main'") merge_commit.parents = [base_commit, incomming_commit] # Action parsed_result = parser.parse(merge_commit) assert isinstance(parsed_result, ParseError) assert "Ignoring merge commit" in parsed_result.error