Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename taxsim validation test files to indicate they're not unit tests #5587

Open
MaxGhenis opened this issue Feb 13, 2025 · 0 comments
Open

Comments

@MaxGhenis
Copy link
Contributor

Currently many test files named after variables (our convention for unit tests) have more inputs than those immediately used in the variable's formula, as they validate against taxsim and came from scripts that generated many test cases. For example, see the state-level tests modified in #5582. Let's rename these to indicate they're integration tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant