Reorganize tests into a pytest package (and fix bugs it exposed)

[derek73]

Summary

Splits the single 97KB tests.py (344 tests in one file, unittest) into a tests/ pytest package — one file per concern — and runs everything under pytest. Along the way, the migration exposed and fixed two real defects.

Test reorganization

• 13 test classes → 13 tests/test_*.py files (one per concern); tests/base.py holds a plain HumanNameTestBase with the m() assert helper + thin assert* shims, so test bodies moved over verbatim.
• tests/conftest.py reproduces the old dual run (every test under empty_attribute_default = '' and None) as a clean parametrized autouse fixture — reported counts are doubled (692 results from 346 methods).
• @unittest.expectedFailure → @pytest.mark.xfail (10 methods), @unittest.skipUnless(dill,…) → @pytest.mark.skipif, self.assertRaises → pytest.raises.
• ConstantsCustomization → ConstantsCustomizationTests so it matches python_classes = ["*Tests","*TestCase"].
• Debug CLI moved out of tests.py into nameparser/__main__.py: python -m nameparser "Some Name".
• Docs + CI (AGENTS.md, CONTRIBUTING.md, GitHub Actions, MANIFEST.in) updated to pytest.

Bug fixes the migration surfaced

• initials() rendered the literal "None" for empty name parts when empty_attribute_default=None (e.g. "J. None D."). Now empty parts render as '' and a fully-empty result falls back to empty_attribute_default, consistent with the first/title accessors. (Behavior change for empty_attribute_default=None users.) Pinned by dedicated regression tests in tests/test_initials.py (they fail against the old parser.py).
• Test-isolation leak: several tests mutate global CONSTANTS without restoring it; the old suite only passed because unittest runs methods alphabetically. pytest runs in definition order, which exposed it. The fixture now snapshots/restores both the scalar and collection config per test (including the global SetManager/TupleManager constants and the lazy _pst cache), so even a test that mutates a global collection and fails before its own cleanup can't leak — tests are order-independent.

Review follow-ups

• Extended the conftest snapshot to cover collection constants (above), removing the last manual, non-exception-safe cleanup.
• Added explicit regression tests pinning the initials()/None fix.
• nameparser/__main__.py: dropped the inert encoding=sys.stdout.encoding argument and switched the Initials: line to comma-style printing (no TypeError when initials() returns None).
• Enabled xfail_strict so an xfail that starts passing fails the suite instead of silently becoming an xpass.
• Fixed the stale python -m nameparser repr example in CONTRIBUTING.md (lowercase field labels + nickname).

Test Plan

• pytest → 672 passed, 20 xfailed, zero failures
• ruff check clean; mypy clean
• Order-independence verified (reversed file order, randomized orderings)
• sphinx-build and sdist packaging unaffected
• python -m nameparser "Dr. Juan Q. Xavier de la Vega III" parses correctly

🤖 Generated with Claude Code