fix: scope now allows usage of dots

Author: ddluke

conventional_pre_commit/format.py

@@ -128,7 +128,7 @@ def r_scope(self):
         """Regex str for an optional (scope)."""
         if self.scopes:
             scopes = self._r_or(self.scopes)
-            escaped_delimiters = list(map(re.escape, [":", ",", "-", "/"]))  # type: ignore
+            escaped_delimiters = list(map(re.escape, [":", ",", "-", "/", "."]))  # type: ignore
             delimiters_pattern = self._r_or(escaped_delimiters)
             scope_pattern = rf"\(\s*(?:(?i:{scopes}))(?:\s*(?:{delimiters_pattern})\s*(?:(?i:{scopes})))*\s*\)"
 
@@ -138,9 +138,9 @@ def r_scope(self):
                 return scope_pattern
 
         if self.scope_optional:
-            return r"(\([\w \/:,-]+\))?"
+            return r"(\([\w \/:,-\.]+\))?"
         else:
-            return r"(\([\w \/:,-]+\))"
+            return r"(\([\w \/:,-\.]+\))"
 
     @property
     def r_delim(self):

tests/conftest.py

@@ -34,6 +34,11 @@ def conventional_utf8_commit_path():
     return get_message_path("conventional_commit_utf-8")
 
 
+@pytest.fixture
+def conventional_commit_with_dots_path():
+    return get_message_path("conventional_commit_with_dots")
+
+
 @pytest.fixture
 def conventional_gbk_commit_path():
     return get_message_path("conventional_commit_gbk")

tests/messages/conventional_commit_with_dots

@@ -0,0 +1 @@
+feat(customer.registration): adds support for oauth2

tests/test_format.py

@@ -443,6 +443,7 @@ def test_r_scope__special_chars(conventional_commit_scope_required):
     assert regex.match("(some thing)")
     assert regex.match("(some:thing)")
     assert regex.match("(some,thing)")
+    assert regex.match("(some.thing)")
 
 
 def test_r_scope__scopes(conventional_commit_scope_required):
@@ -455,6 +456,7 @@ def test_r_scope__scopes(conventional_commit_scope_required):
     assert regex.match("(api: client)")
     assert regex.match("(api/client)")
     assert regex.match("(api-client)")
+    assert regex.match("(api.client)")
     assert not regex.match("(test)")
     assert not regex.match("(api; client)")
 
@@ -469,6 +471,7 @@ def test_r_scope__scopes_uppercase(conventional_commit_scope_required):
     assert regex.match("(API: CLIENT)")
     assert regex.match("(API/CLIENT)")
     assert regex.match("(API-CLIENT)")
+    assert regex.match("(API.CLIENT)")
     assert not regex.match("(TEST)")
     assert not regex.match("(API; CLIENT)")
 

tests/test_hook.py

@@ -54,6 +54,12 @@ def test_main_success__conventional_utf8(conventional_utf8_commit_path):
     assert result == RESULT_SUCCESS
 
 
+def test_main_success__conventional_commit_with_dots_path(conventional_commit_with_dots_path):
+    result = main([conventional_commit_with_dots_path])
+
+    assert result == RESULT_SUCCESS
+
+
 def test_main_fail__conventional_gbk(conventional_gbk_commit_path):
     result = main([conventional_gbk_commit_path])