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

test: add test for F1099Info #668

Merged
merged 2 commits into from
Oct 24, 2021
Merged

test: add test for F1099Info #668

merged 2 commits into from
Oct 24, 2021

Conversation

aserputov
Copy link
Contributor

What kind of change does this PR introduce? (delete not applicable)

  • Other, please describe: Add test for F1099Info

Fixes: #650
Received Help from : @arisaokasaka

@zakpatterson
Copy link
Collaborator

Looks like your new test is passing but #663 might have been merged prematurely, I reopened #654.

@zakpatterson
Copy link
Collaborator

Might take a look at npm run test -- --coverage F1099Info. Maybe there are some gaps here, will wait for @thegrims to take a look

@thegrims
Copy link
Collaborator

@aserputov Thanks for your contribution!

@thegrims thegrims merged commit de21805 into ustaxes:master Oct 24, 2021
zakpatterson added a commit to zakpatterson/UsTaxes that referenced this pull request Oct 29, 2021
Co-authored-by: Zak Patterson <pattersonzak@gmail.com>
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.

Write unit tests for F1099Info
3 participants