feat: enhance template tests and address review feedback

[jrepp]

Summary

This PR includes enhancements to template tests and addresses code review feedback from previous PRs.

Changes

Test Enhancements (PR #38)

• Expanded test_required_fields_basic to test all document types (adr, rfc, memo, prd, None)
• Improved test coverage for ensure_required_fields() function
• Better validation of required field behavior across document types

Review Feedback Fixes

• Removed unused doc_type parameter from ensure_required_fields()
• Removed hardcoded test count badge from README
• Addressed all Copilot review feedback

Test Status

✅ All tests passing
✅ Lint checks passing

Related

• Follow-up to PR feat: comprehensively enhance all document templates #34 (template enhancements - merged)
• Includes changes from PR fix: address PR #34 code review feedback #36 (review feedback - merged)
• Includes changes from PR Address feedback from PR #34 code review #38 (test improvements - merged to this branch)

🤖 Generated with Claude Code

[Copilot]

Pull Request Overview

This PR enhances template tests and addresses code review feedback by improving test coverage and removing unnecessary code elements.

• Expanded test_required_fields_basic to validate required fields across all document types
• Removed unused doc_type parameter from ensure_required_fields() function
• Removed hardcoded test count badge from README

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.65%. Comparing base (e5e6b47) to head (413dc92).
⚠️ Report is 15 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main      #39   +/-   ##
=======================================
  Coverage   61.65%   61.65%           
=======================================
  Files          23       23           
  Lines        2929     2929           
  Branches      541      541           
=======================================
  Hits         1806     1806           
  Misses       1062     1062           
  Partials       61       61           

Flag	Coverage Δ
unittests	61.65% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e5e6b47...413dc92. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[jrepp]

Status Update ✅

All CI checks passing:

• ✅ lint (13s)
• ✅ build (9s)
• ✅ actionlint (8s)
• ✅ All tests passing on Python 3.9, 3.10, 3.11, 3.12, 3.13

Coverage: 61.65% (maintained, no regression)

Copilot Review: No issues raised - clean review summary provided

Changes Included:

1. Expanded test_required_fields_basic to test all document types
2. Removed unused doc_type parameter from ensure_required_fields()
3. Removed hardcoded test count badge from README

All modified and coverable lines are covered by tests. Ready for merge!

🤖 Generated with Claude Code