PolicyEngine / PolicyEngine/policyengine-us

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

Open
#5,587 0 comments 0 reactions 0 assignees View on GitHub
code-health
Dominant language
Python
Stars
163
Forks
213
Avg merge
3d 3h
Merged PRs (30d)
96

Description

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 https://github.com/PolicyEngine/policyengine-us/pull/5582. Let's rename these to indicate they're integration tests.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.