Skip to content

Commit 6cf85a7

Browse files
committed
style: beautify 01eea03
1 parent 01eea03 commit 6cf85a7

File tree

2 files changed

+30
-28
lines changed

2 files changed

+30
-28
lines changed

semantic_release/history/parser_angular.py

+10-17
Original file line numberDiff line numberDiff line change
@@ -21,12 +21,7 @@
2121
"perf": "performance",
2222
}
2323

24-
LEVEL_BUMPS = {
25-
'no-release': 0,
26-
'patch': 1,
27-
'minor': 2,
28-
'major': 3
29-
}
24+
LEVEL_BUMPS = {"no-release": 0, "patch": 1, "minor": 2, "major": 3}
3025

3126

3227
@LoggedFunction(logger)
@@ -40,9 +35,9 @@ def parse_commit_message(message: str) -> ParsedCommit:
4035
"""
4136

4237
# loading these are here to make it easier to mock in tests
43-
allowed_types = config.get('parser_angular_allowed_types').split(',')
44-
minor_types = config.get('parser_angular_minor_types').split(',')
45-
patch_types = config.get('parser_angular_patch_types').split(',')
38+
allowed_types = config.get("parser_angular_allowed_types").split(",")
39+
minor_types = config.get("parser_angular_minor_types").split(",")
40+
patch_types = config.get("parser_angular_patch_types").split(",")
4641
re_parser = re.compile(
4742
r"(?P<type>" + "|".join(allowed_types) + ")"
4843
r"(?:\((?P<scope>[^\n]+)\))?"
@@ -52,19 +47,17 @@ def parse_commit_message(message: str) -> ParsedCommit:
5247
re.DOTALL,
5348
)
5449

55-
5650
# Attempt to parse the commit message with a regular expression
5751
parsed = re_parser.match(message)
5852
if not parsed:
5953
raise UnknownCommitMessageStyleError(
6054
f"Unable to parse the given commit message: {message}"
6155
)
62-
parsed_break = parsed.group('break')
63-
parsed_scope = parsed.group('scope')
64-
parsed_subject = parsed.group('subject')
65-
parsed_text = parsed.group('text')
66-
parsed_type = parsed.group('type')
67-
56+
parsed_break = parsed.group("break")
57+
parsed_scope = parsed.group("scope")
58+
parsed_subject = parsed.group("subject")
59+
parsed_text = parsed.group("text")
60+
parsed_type = parsed.group("type")
6861

6962
if parsed_text:
7063
descriptions = parse_paragraphs(parsed_text)
@@ -80,7 +73,7 @@ def parse_commit_message(message: str) -> ParsedCommit:
8073
if match
8174
]
8275

83-
default_level_bump = config.get('parser_angular_default_level_bump').lower()
76+
default_level_bump = config.get("parser_angular_default_level_bump").lower()
8477
if default_level_bump not in LEVEL_BUMPS.keys():
8578
raise ImproperConfigurationError(
8679
f"{default_level_bump} is not a valid option for "

tests/parsers/test_angular.py

+20-11
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
from semantic_release.errors import UnknownCommitMessageStyleError
44
from semantic_release.history import angular_parser
55

6-
from .. import mock,wrapped_config_get
6+
from .. import mock, wrapped_config_get
77

88
text = (
99
"This is an long explanatory part of a commit message. It should give "
@@ -100,32 +100,41 @@ def test_parser_should_accept_message_without_scope():
100100
assert angular_parser("fix: superfix")[0] == 1
101101
assert angular_parser("fix: superfix")[3][0] == "superfix"
102102

103+
103104
##############################
104105
# test custom parser options #
105106
##############################
106-
@mock.patch("semantic_release.history.parser_angular.config.get",
107-
wrapped_config_get(parser_angular_default_level_bump='minor'))
107+
@mock.patch(
108+
"semantic_release.history.parser_angular.config.get",
109+
wrapped_config_get(parser_angular_default_level_bump="minor"),
110+
)
108111
def test_parser_custom_default_level():
109112
assert angular_parser("test(parser): Add a test for angular parser")[0] == 2
110113

111114

112-
@mock.patch("semantic_release.history.parser_angular.config.get",
113-
wrapped_config_get(
114-
parser_angular_allowed_types=
115-
'custom,build,chore,ci,docs,fix,perf,style,refactor,test'))
115+
@mock.patch(
116+
"semantic_release.history.parser_angular.config.get",
117+
wrapped_config_get(
118+
parser_angular_allowed_types="custom,build,chore,ci,docs,fix,perf,style,refactor,test"
119+
),
120+
)
116121
def test_parser_custom_allowed_types():
117122
assert angular_parser("custom: ...")[0] == 0
118123
assert angular_parser("custom(parser): ...")[1] == "custom"
119124
pytest.raises(UnknownCommitMessageStyleError, angular_parser, "feat(parser): ...")
120125

121126

122-
@mock.patch("semantic_release.history.parser_angular.config.get",
123-
wrapped_config_get(parser_angular_minor_types='docs'))
127+
@mock.patch(
128+
"semantic_release.history.parser_angular.config.get",
129+
wrapped_config_get(parser_angular_minor_types="docs"),
130+
)
124131
def test_parser_custom_minor_types():
125132
assert angular_parser("docs: write some docs")[0] == 2
126133

127134

128-
@mock.patch("semantic_release.history.parser_angular.config.get",
129-
wrapped_config_get(parser_angular_patch_types='test'))
135+
@mock.patch(
136+
"semantic_release.history.parser_angular.config.get",
137+
wrapped_config_get(parser_angular_patch_types="test"),
138+
)
130139
def test_parser_custom_patch_types():
131140
assert angular_parser("test(this): added a test")[0] == 1

0 commit comments

Comments
 (0)