test: add coverage for suffix_not_acronyms members in suffix-comma format

Add tests for 'I' (same single-letter ambiguity as 'V'), a mixed
suffix-comma case (V MD), and two suffix_not_acronyms members together
(V Jr.). Also tighten the are_suffixes_after_comma docstring to describe
the short-circuit mechanism accurately and enumerate the full scope of
affected suffix_not_acronyms entries.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: derek73

nameparser/parser.py

@@ -488,12 +488,14 @@ def are_suffixes(self, pieces: Iterable[str]) -> bool:
         return True
 
     def are_suffixes_after_comma(self, pieces: Iterable[str]) -> bool:
-        """Like are_suffixes but suffix_not_acronyms members are always
-        recognized, even when they look like single-letter initials.
+        """Like are_suffixes, but pieces found in suffix_not_acronyms are
+        accepted unconditionally without passing through is_suffix().
 
         Used when detecting suffix-comma format (e.g. "John Ingram, V") where
-        the post-comma position is unambiguous: a single uppercase roman
-        numeral like 'V' is definitionally a suffix there, not an initial.
+        the post-comma position is unambiguous. This covers all
+        suffix_not_acronyms members (i, ii, iii, iv, v, jr, sr, etc.),
+        case-insensitively, including single-letter entries that is_suffix()
+        would otherwise reject via is_an_initial().
         """
         for piece in pieces:
             if lc(piece) in self.C.suffix_not_acronyms:

tests/test_suffixes.py

@@ -35,14 +35,34 @@ def test_two_suffixes_lastname_comma_format(self) -> None:
         self.m(hn.suffix, "Jr., MD", hn)
 
     def test_roman_numeral_v_suffix_comma_format(self) -> None:
-        # 'V' is a single uppercase letter, so it was incorrectly matched as an
-        # initial and not recognized as a suffix (issue #136)
+        # suffix-comma position is unambiguous: 'V' must be a suffix, not a single-letter initial
         hn = HumanName("John W. Ingram, V")
         self.m(hn.first, "John", hn)
         self.m(hn.middle, "W.", hn)
         self.m(hn.last, "Ingram", hn)
         self.m(hn.suffix, "V", hn)
 
+    def test_roman_numeral_i_suffix_comma_format(self) -> None:
+        # 'I' has the same single-letter ambiguity as 'V'
+        hn = HumanName("John W. Smith, I")
+        self.m(hn.first, "John", hn)
+        self.m(hn.middle, "W.", hn)
+        self.m(hn.last, "Smith", hn)
+        self.m(hn.suffix, "I", hn)
+
+    def test_suffix_not_acronym_then_acronym_suffix_comma_format(self) -> None:
+        # single-letter suffix_not_acronyms entry followed by an acronym suffix
+        hn = HumanName("John Smith, V MD")
+        self.m(hn.first, "John", hn)
+        self.m(hn.last, "Smith", hn)
+        self.m(hn.suffix, "V MD", hn)
+
+    def test_two_suffix_not_acronyms_suffix_comma_format(self) -> None:
+        hn = HumanName("John Smith, V Jr.")
+        self.m(hn.first, "John", hn)
+        self.m(hn.last, "Smith", hn)
+        self.m(hn.suffix, "V Jr.", hn)
+
     def test_two_suffixes_suffix_comma_format(self) -> None:
         hn = HumanName("Franklin Washington, Jr. MD")
         self.m(hn.first, "Franklin", hn)