Skip to content

Conversation

@MaxGhenis
Copy link
Contributor

Summary

  • Add YAML tests for tax_unit_is_filer variable to improve code coverage

Test cases added

  • High income - required to file
  • Low income with refund (EITC-eligible) - files anyway
  • Married filing separately - lower threshold applies
  • Low income but high unearned income - required to file
  • Senior with low income - files due to refund
  • Zero income - not required to file

Context

Part of effort to improve policyengine-us code coverage. These tests exercise the filing requirement logic in policyengine_us/variables/gov/irs/tax_unit_is_filer.py.

🤖 Generated with Claude Code

MaxGhenis and others added 3 commits January 31, 2026 09:06
Add YAML tests for tax_unit_is_filer variable covering:
- High income filing requirement
- Low income with refund (EITC-eligible)
- Married filing separately (lower threshold)
- Unearned income over threshold
- Senior with low income
- Zero income (not required to file)

Part of effort to improve code coverage toward 100%.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Revert version bump - let CI handle it.
Add changelog_entry.yaml for CI to process.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@MaxGhenis MaxGhenis merged commit 50554f7 into main Jan 31, 2026
7 checks passed
@MaxGhenis MaxGhenis deleted the coverage-100 branch January 31, 2026 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants