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

Unit testing of calcfunctions.py #2419

Open
jdebacker opened this issue Jun 17, 2020 · 4 comments
Open

Unit testing of calcfunctions.py #2419

jdebacker opened this issue Jun 17, 2020 · 4 comments
Labels
good first issue Good issues for a person new to this project help wanted

Comments

@jdebacker
Copy link
Member

While is full code coverage for tests of calcfunctions.py, the unit tests essentially only confirm that all function arguments are passed properly and that those passed are used. There ought to be tests that validate the logic of each function (i.e., that given certain values for inputs, the expected output values are produced).

@jdebacker
Copy link
Member Author

cc @MattHJensen This might be a good beginner issue to work on (at least for most of the functions in calcfunctions.py).

@MattHJensen
Copy link
Contributor

Evidence in favor: #2551

@jdebacker
Copy link
Member Author

jdebacker commented Sep 27, 2021

Checklist of functions in calcfunctions.py module that have unit tests written specifically for them (or in progress through open PRs) in test_calcfunctions.py:

  • BenefitPrograms
  • EI_PayrollTax
  • DependentCare
  • Adj
  • ALD_InvInc_ec_base
  • CapGains
  • SSBenefits
  • UBI
  • AGI
  • ItemDedCap
  • ItemDed
  • AdditionalMedicareTax
  • StdDed
  • TaxInc
  • SchXYZ
  • SchXYZTax
  • GainsTax
  • AGIsurtax
  • AMT
  • NetInvIncTax
  • F2441
  • EITCamount
  • EITC
  • RefundablePayrollTaxCredit
  • ChildDepTaxCredit
  • PersonalTaxCredit
  • AmOppCreditParts
  • SchR
  • EducationTaxCredit
  • CharityCredit
  • NonrefundableCredits
  • AdditionalCTC
  • C1040
  • CTC_new
  • IITAX
  • Taxes
  • ComputeBenefit
  • BenefitSurtax
  • BenefitLimitation
  • FairShareTax
  • LumpSumTax
  • ExpandIncome
  • AfterTaxIncome

@jdebacker
Copy link
Member Author

Writing unit test for a single one of these functions is a good place to start to learn more about Tax-Calculator. You can deal with the a single function without having to know the full model.

These are ideal issues for someone with good knowledge of the tax code (or an interest in learning more about it) and moderate experience with Python (and a willingness to learn more).

@jdebacker jdebacker added good first issue Good issues for a person new to this project help wanted Hacktoberfest Highlighting good issues for Hacktoberfest labels Sep 27, 2021
@jdebacker jdebacker removed the Hacktoberfest Highlighting good issues for Hacktoberfest label Nov 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good issues for a person new to this project help wanted
Projects
None yet
Development

No branches or pull requests

2 participants