Close initials_separator ordering gap; drop remaining redundant restores

test_initials_separator_default_on_constants now runs after the test
that mutates initials_separator, so it actually verifies the autouse
fixture restored the default rather than merely asserting an unrelated
value. Also removes the last manual CONSTANTS restore assignments in
test_output_format.py and test_constants.py, now redundant since the
autouse fixture in conftest.py covers all of these attributes.

Author: derek73

tests/test_constants.py

@@ -99,7 +99,6 @@ def test_clear_removes_all_entries(self) -> None:
 
     def test_empty_attribute_default(self) -> None:
         from nameparser.config import CONSTANTS
-        _orig = CONSTANTS.empty_attribute_default
         CONSTANTS.empty_attribute_default = None
         hn = HumanName("")
         self.m(hn.title, None, hn)
@@ -108,7 +107,6 @@ def test_empty_attribute_default(self) -> None:
         self.m(hn.last, None, hn)
         self.m(hn.suffix, None, hn)
         self.m(hn.nickname, None, hn)
-        CONSTANTS.empty_attribute_default = _orig
 
     def test_empty_attribute_on_instance(self) -> None:
         hn = HumanName("", None)

tests/test_initials.py

@@ -70,10 +70,6 @@ def test_initials_delimiter_constants(self) -> None:
         hn = HumanName("Doe, John A. Kenneth, Jr.")
         self.m(hn.initials(), "J; A; K; D;", hn)
 
-    def test_initials_separator_default_on_constants(self) -> None:
-        from nameparser.config import CONSTANTS
-        self.assertEqual(CONSTANTS.initials_separator, " ")
-
     def test_initials_list(self) -> None:
         hn = HumanName("Andrew Boris Petersen")
         self.m(hn.initials_list(), ["A", "B", "P"], hn)
@@ -196,3 +192,10 @@ def test_initials_separator_constants_multi_part_middle(self) -> None:
         CONSTANTS.initials_format = "{first}{middle}{last}"
         hn = HumanName("Doe, John A. Kenneth")
         self.m(hn.initials(), "JAKD", hn)
+
+    def test_initials_separator_default_on_constants(self) -> None:
+        # Runs after test_initials_separator_constants_multi_part_middle so that,
+        # in file/definition order, it verifies the autouse fixture restored
+        # CONSTANTS.initials_separator rather than leaking the "" set above.
+        from nameparser.config import CONSTANTS
+        self.assertEqual(CONSTANTS.initials_separator, " ")

tests/test_output_format.py

@@ -12,26 +12,22 @@ def test_formatting_init_argument(self) -> None:
 
     def test_formatting_constants_attribute(self) -> None:
         from nameparser.config import CONSTANTS
-        _orig = CONSTANTS.string_format
         CONSTANTS.string_format = "TEST2"
         hn = HumanName("Rev John A. Kenneth Doe III (Kenny)")
         self.assertEqual(str(hn), "TEST2")
-        CONSTANTS.string_format = _orig
 
     def test_capitalize_name_constants_attribute(self) -> None:
         from nameparser.config import CONSTANTS
         CONSTANTS.capitalize_name = True
         hn = HumanName("bob v. de la macdole-eisenhower phd")
         self.assertEqual(str(hn), "Bob V. de la MacDole-Eisenhower Ph.D.")
-        CONSTANTS.capitalize_name = False
 
     def test_force_mixed_case_capitalization_constants_attribute(self) -> None:
         from nameparser.config import CONSTANTS
         CONSTANTS.force_mixed_case_capitalization = True
         hn = HumanName('Shirley Maclaine')
         hn.capitalize()
         self.assertEqual(str(hn), "Shirley MacLaine")
-        CONSTANTS.force_mixed_case_capitalization = False
 
     def test_capitalize_name_and_force_mixed_case_capitalization_constants_attributes(self) -> None:
         from nameparser.config import CONSTANTS